Moduldiskusjon:Official links

Flytting av element i lister

Ser at første element i lista nå flyttes til siste posisjon bak "final-*-combiner". Er det med overlegg? Burde ikke det være det siste elementet, altså slik?

local function buildList( pid, t  )
	if #t == 0 then
		return ''
	elseif #t == 1 then
		return t[1]
	end
	local last = table.remove(t) -- ta ut siste element
	local str = table.concat(t, mw.message.newRawMessage(conf:g('initial-' .. pid .. '-combiner', 'initial-default-combiner')):plain())
	return mw.message.newRawMessage(conf:g('final-' .. pid .. '-combiner', 'final-default-combiner'), str, last):plain()
end

Jeg er usikker på om claims blir returnert i følge rank fra mw.wikibase.getEntityObject(), men hvis de kommer i følge rank, vil jo dette sørge for at WD sin preferred alltid vil komme sist så lenge det er en lokal prioriteringsliste her i tillegg. Stigmj (diskusjon) 13. feb. 2016 kl. 18:25 (CET)

Hvilken ende som brukes som siste er uvesentlig iom at lista er uordnet. Strengt tatt kan hvilken som helst verdi brukes så lenge den samme verdien tas ut av lista. Endret koden fordi alternativet er noe raskere. Claims er ikke ordnet etter rang, ordningen er definert som «tilfeldig». — Jeblad 13. feb. 2016 kl. 19:49 (CET)