Moduldiskussion:Ping

Kodkontroll

Larske, Innocent bystander, EnDumEn: Det här är till Mall:Ping. Den gamla mallkoden är komplicerad, innehåller upprepningar, och är dessutom begränsad till sju mottagare, vilket gjorde att jag, Julle, Lindansaren och Tegel inte fick någon ping i Special:Diff/40021420.

Den nya koden ska vara identisk med den gamla, förutom att den nya accepterar en parameter som heter prefix, som kan användas för att ändra eller ta bort @ i början. Det är inte livsnödvändigt, men det är en av de saker som har tagits upp på Malldiskussion:Ping. Den gamla mallkoden verkade också versalisera användarnamn (med BASEPAGENAME?). Jag vet inte om själva versalisering var en poäng i sig, eller om det var en sidoeffekt av något annat, men den nya modulkoden gör i alla fall ingen sådan versalisering.

Titta gärna att det ser ok ut. Jag vill inte paja något. Jag har diverse testcases i Special:PermanentLink/40031301. Mallen används på 2838 sidor. Nirmos (diskussion) 14 juni 2017 kl. 17.21 (CEST)

Jag vet inte vad det är som triggar meddelandesystemet, men det kanske är närvaron av klassen "template-ping" och den finns ju med även i den nya versionen. Den gamla mallen hanterar felfallet att ingen användare har angivits, se sidan med testfall. Har du medvetet tagit bort det från den nya? --Larske (diskussion) 14 juni 2017 kl. 18.14 (CEST)

Larske: Tack för att du snyggade till testfallen. Klassen template-ping gör inget. Det är bara ett handtag om man vill göra något med CSS eller JavaScript. Det som gör att en avisering skickas är att en länk till mottagarens användarsida och signatur från tilden läggs till i samma redigering. Du får väldigt gärna visa hur du skulle lägga till felmeddelande då inga användarnamn anges. Nirmos (diskussion) 14 juni 2017 kl. 18.43 (CEST)

Jag testade lite. Om man lägger in den här if-satsen direkt efter tilldelningen av args, blir det samma felutskrift som i den gamla mallen om man har utelämnat parametern för användare eller om den är tom.

      if not args[1] or args[1] == '' then
	   return mw.getCurrentFrame():preprocess("{{error|Fel i [[Mall:Ping]]: Användarnamn ej angivet.}}")
       end

--Larske (diskussion) 14 juni 2017 kl. 20.51 (CEST)

Det som alarmerar meddelandesystemet är tillägg av ny rad på sida tillsammans med en signatur. Rent tekniskt kan länken till användarnamnen döljas med css-magi, men länken måste ändå alltid finnas där, dold eller inte. Sedan finns det en övre gräns för hur många användare som kan pingas på en enda gång. Minns jag rätt så var den högre än 7, men en gräns fanns. (Kanske för att en arkivering av en diskussionssida eller en återställning av en sida inte av misstag ska pinga alla som har sin signatur på nämnda sida.) Men den gränsen tror jag går att justera med lite utvecklar-magi. Jag tror den är ganska hög på WD. -- Innocent bystander (diskussion) 14 juni 2017 kl. 21.20 (CEST)

Tack, Larske, gjorde så.

Innocent bystander: $wgEchoMaxMentionsCount är satt till 50. Jag vet inte om det finns någon vits med att ha med det i modulen, åtminstone inte på svwiki. När det har pingats som flest i ett och samma meddelande så är det en fjärdedel av alla administratörer i samband med omvalsperiod, och det kommer inte i närheten av maxgränsen. Nirmos (diskussion) 15 juni 2017 kl. 12.44 (CEST)

Pling:NirmosPlong Den siffran var betydligt lägre när den här funktionen initierades här! -- Innocent bystander (diskussion) 15 juni 2017 kl. 21.53 (CEST)

Avslutande kolon

Skulle någon som vet vad hen gör kunna ta bort det avslutande kolonet? Det är inte alltid man vill anropa någon med "@namn:". Jag skulle göra det själv, men jag är inte helt säker på att jag inte skulle förstöra något utan att märka det.

andejons (diskussion) 31 juli 2017 kl. 10.29 (CEST)

@Andejons Du kan själv ta bort det genom att välja hur du anropar mallen.

@Andejons§ Som sagt!

Andejons: Du kan även ta bort snabelaet

$Andejons: Eller välja något annat! -- Innocent bystander (diskussion) 31 juli 2017 kl. 10.40 (CEST)

Ja, fast det finns inte dokumenterat och det är dålig design. Att skriva "{{ping|andejons}}," är enklare och betydligt mer intuitivt. Jag förstår inte varför skiljetecknet överhuvudtaget skall vara en del av mallen.

andejons (diskussion) 31 juli 2017 kl. 10.51 (CEST)

"finns inte dokumenterat"??? Mall:Ping/dok har funnits sedan 2014 då $$$Ainali£££ skapade den! Modulen är sedan en förenkling av kodningen av denna mall. Det var också då parametrarna prefix och p tillkom. Mallen är sannolikt importerad från annat projekt. Jag tror vi i ett falls som detta inte ska skilja oss för mycket från andra projekt. Vill man slippa strula med parametrarna så finns alltid Andejons och Andejons som funnits sedan tidernas gryning här. -- Innocent bystander (diskussion) 31 juli 2017 kl. 11.03 (CEST)

OK då: dokumentationen finns, men den är dålig, eftersom man ögna genom en skärm text innan det finns det minsta tecken på att man kan ta bort kolonet, och då står det under en inte helt tydlig rubrik. Och i valet mellan "göra på ett dåligt, ogenomtänkt sätt" och "skilja sig från andra språkversioner" så tycker jag valet är lätt.

andejons (diskussion) 31 juli 2017 kl. 12.13 (CEST)

Det här är ingen mall som används i artiklar. Den används i diskussioner, där även användare från andra projekt dyker upp. Då ser jag det nog som att det är en fördel om de utifrån känner igen sig. Jag använder den gärna själv när jag kommer till ett nytt projekt. Då är det väldigt förvirrande om det inte funkar som jag är van. I artiklarna däremot, där är jag med dig fullständigt! Där är det absolut bättre att det blir bra än att det blir likadant. -- Innocent bystander (diskussion) 31 juli 2017 kl. 16.07 (CEST)

Om någon utifrån använder mallen och ser att det saknas ett kolon är det lätt att åtgärda: en eller två knapptryckningar på tangentbordet. Vill man däremot ha bort kolonet måste man gå till mallens hjälpsida, och läsa genom den noggrant. Jag kan inte se att kostnaden "extra knapptryckning" motsvarar "behöver lära sig en malls parametrar".

Det bör kanske också nämnas att den här mallen kallas "svar till" på de flesta andra språk. Om man nu tycker detta är så viktigt skulle man kunna använda {{svar_till}} med kolon och {{ping}} utan.

andejons (diskussion) 31 juli 2017 kl. 17.45 (CEST)