Moduldiskussion:categorize

Senaste kommentaren: för 2 år sedan av Taylor 49 i ämnet Dela upp modulen

Vore det inte mer logiskt att ha språkkod som första parameter? Förresten, bra jobbat med STANDARDSORTERING! Nu återstår väl att ta bort alla nyckelanrop? 84.55.98.173 27 maj 2017 kl. 03.36 (CEST)Svara

Se även Malldiskussion:talesätt för ett eventuellt tillvägagångssätt för att möjliggöra enklare ändringar av argumentordningen. 84.55.98.173 27 maj 2017 kl. 04.43 (CEST)Svara

Namngivning av funktioner redigera

Funktionerna har lite konstiga namn, vad tycker ni om följande namn:

  • categorize.getMainEntryWikitext
  • categorize.getLanguageTemplateWikitext

möjliggör för att kunna kategorisera andra sorters uppslag med till exempel funktioner med namn som

  • categorize.getInflectionEntryWikitext

Vad tycker ni andra? –dMoberg 4 juni 2017 kl. 18.46 (CEST)Svara

Behövs "Language"? Kan eventuellt behov av language ges av argument istället? Vilka antra typer av mallkategoriseringar behöver vi ta hänsyn till när vi pratar om namngivning? Behövs "Entry" i namnet, är inte "Main" och "Inflection" tillräckligt, alltså "getMainWikitext()" respektive "getInflectionWikitext"? ~ Dodde (diskussion) 4 juni 2017 kl. 19.07 (CEST)Svara
Menar du att modulen även ska användas för kategorier som Kategori:Wiktionary:Mallar? Om det inte finns någon gemensam kod för kategorisering av uppslag respektive mallar kan det vara en idé att dela upp modulen i två (en för kategorisering av uppslag och en för kategorisering av mallar). Annars håller jag med om namnen. 84.55.98.173 4 juni 2017 kl. 23.38 (CEST)Svara
84.55.98.173, håller med om getMainEntryWikitext eller getMainWikitext respektive getInflectionEntryWikitext eller getInflectionWikitext? getLanguageTemplateWikitext eller getTemplateWikitext (dvs. har vi ens konstaterat om "Language" är nödvändigt, om inte nödvändigt, varför isåfall ha med "Language")? ~ Dodde (diskussion) 5 juni 2017 kl. 00.30 (CEST)Svara
Jag håller med er båda, d.v.s. att Mobergs förslag är bättre än de nuvarande namnen och att "Entry" kan utelämnas (men det betyder inte nödvändigtvis att "Entry" ska utelämnas). Det kan jämföras med valet mellan "getText" och "getWikitext". Jag anser det funkar både med och utan. När det gäller "Language" så är det nödvändigt att veta vilka kategorier modulen ska användas för. Om det handlar om alla mallkategorier som listas på Kategori:Wiktionary:Mallar så är det befogat att ha en funktion bara för språkkategorier, vilken rimligen kan heta "getLanguageTemplateWikitext", och en funktion för övriga mallkategorier (t.ex. användarmallar‎, grafiska mallar‎ och programteknikmallar‎). Det verkar i alla fall bli ganska många funktioner, varför det kan vara en idé att dela upp modulen i exempelvis Modul:entryCat och Modul:templateCat (speciellt om det inte finns några gemensamma funktioner i båda delarna). Modul:entryCat skulle då ha "getMainWikitext" och "getInflectionWikitext" medan Modul:templateCat skulle ha "getLanguageTemplateWikitext" och "getTemplateWikitext" (exempelnamn). 84.55.98.173 5 juni 2017 kl. 01.55 (CEST)Svara
Modulnamn verkar aldrig vara camelCase, så t.ex. "template-cat" är nog bättre. Jag tror att jag gillar Nummers förslag på att dela upp modulen, och skulle nog föreslå ett gemensamt prefix på modulnamnet: Module:categorize-entry och Modul:categorize-template. (Jag har ingen stark åsikt om funktionsnamnet.) //Skal 5 juni 2017 kl. 14.22 (CEST)Svara

Dela upp modulen redigera

Den här modulen bara växer och växer. Men funktionerna som tillkommer verkar vara oberoende av varandra. I sådana fall brukar det vara en bra idé att dela upp ett program i flera filer. Om det är något som ska göras, så bör det göras innan hundratals mallar börjar anropa denna modul. 84.55.98.173 8 juni 2017 kl. 20.21 (CEST)Svara

Jag tycker inte att modulen är stor. ~ Dodde (diskussion) 10 juni 2017 kl. 11.57 (CEST)Svara
Jag har tagit bort "getInflectionWikitext". Modul:bjform anropar direkt Modul:sort istället för att gå via "Modul:categorize". Modul:tagg har gjort på samma sätt sedan år 2017. Taylor 49 (diskussion) 7 augusti 2021 kl. 04.16 (CEST)Svara
Tillbaka till sidan "categorize".