Moduldiskussion:categorize

Definition från Wiktionary, den fria ordlistan.
Hoppa till navigering Hoppa till sök

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]