Moduldiskussion:External links

Optimering

Modulet hentede entiteten på Wikidata for den aktuelle artikel 2 gange for hver external link det testede for. Det var ekstremt ineffektivt. Jeg lavede en hurtig ændring så den kun hente én gang i alt. Entiteten for Usain Bolt er meget stor med opremsning af alle hans resultater. Før gav siden Usain Bolt skriptfejl på grund af mere end 10 sekunder lua-køretid. Nu er Lua-køretiden for siden lidt under et halvt sekund. --Kartebolle (Dipsacus fullonum) (diskussion) 25. aug 2018, 22:45 (CEST)

Artslenker

Så lige på no.wiki no:mal:Artslenker, - Det var måske en idé at få den med på da.wiki? Nico (diskussion) 26. sep 2018, 10:49 (CEST)

URI-indkodning

Seneste indlæg: for 9 måneder siden2 indlæg2 personer i diskussionen

På Brugerdiskussion:Hejsa#Cyclocross24 rytter ID på Sportshenvisninger diskuteres at {{Sportshenvisninger}} ikke virker for Cyclocross24 rytter ID (P13724) fordi de indeholdte binstestreger i URL'erne bliver procent-indkodet. Som man kan se på en:Percent-encoding#Types of URI characters er bindestreg ikke et reserveret ASCII-tegn i URI'er, og bør derfor ikke indkodes.

For webservere som virker korrekt, bør det ikke gøre nogen forskel alligevel at procent-indkode bindestreger eller andre ikke-reservede tegn, men for nogle servere som i dette tilfælde https://cyclocross24.com får det URL'en til ikke at virke. en:Percent-encoding#Unreserved characters siger herom:

URIs that differ only by whether an unreserved character is percent-encoded or appears literally are equivalent by definition, but URI processors, in practice, may not always recognize this equivalence. For example, URI consumers should not treat %41 differently from A or %7E differently from ~, but some do. For maximal interoperability, URI producers are discouraged from percent-encoding unreserved characters.

Jeg foreslår at ændre

function string.urlEncode( str )
   if ( str ) then
      str = string.gsub( str, "\n", "\r\n" )
      str = string.gsub( str, "([^%w/ ])",
         function (c) return string.format( "%%%02X", string.byte(c) ) end )
      str = string.gsub( str, " ", "+" )
   end
   return str
end

til

function string.urlEncode( str )
   if ( str ) then
      str = string.gsub( str, "\n", "\r\n" )
      str = mw.uri.encode( str, "QUERY" )
   end
   return str
end

Der er ingen grund til selv at kode for at lave indkodningen når Scribunto har en indbygget til at gøre det. (Skrev Dipsacus fullonum (diskussion • bidrag) . Husk at signere dine indlæg.)