Module:Labelled list hatnote: Difference between revisions

Fixed iteration-and-removal bug
m (1 revision imported)
(Fixed iteration-and-removal bug)
Line 10:
local mHatlist = require('Module:Hatnote list')
local mArguments --initialize lazily
local yesno --initialize lazily
local p = {}
 
Line 18 ⟶ 19:
prefixes = {'label', 'label ', 'l'},
template = 'Module:Labelled list hatnote'
}
 
-- Localizable message strings
local msg = {
errorSuffix = '#Errors',
noInputWarning = 'no page names specified',
noOutputWarning =
"'''[[%s]] — no output: none of the target pages exist.'''"
}
 
Line 26 ⟶ 35:
-- They each have numbers auto-appended, e.g. 'label1', 'label 1', & 'l1'
prefixes = prefixes or defaults.prefixes
local pagesindices = {}
local sparsePages = {}
for k, v in pairs(args) do
if type(k) == 'number' then
pagesindices[#pagesindices + 1] = pagek
local display
for i = 1, #prefixes do
Line 34 ⟶ 45:
if display then break end
end
local pagesparsePages[k] = display and
string.format('%s|%s', string.gsub(v, '|.*$', ''), display) or v
pages[#pages + 1] = page
end
end
table.sort(indices)
iflocal #pages == 0 then{}
for k, v in ipairs(indices) do pages[#pages + 1] = sparsePages[v] end
return pages
end
 
--Helper function to get a page target from a processed page string
--e.g. "Page|Label" → "Page" or "Target" → "Target"
local function getTarget(pagename)
local pipe = string.find(pagename, '|')
return string.sub(pagename, 0, pipe and pipe - 1 or nil)
end
 
Line 49 ⟶ 69:
function p.labelledList (frame)
mArguments = require('Module:Arguments')
yesno = require('Module:Yesno')
local labels = {frame.args[1] or defaults.label}
labels[2] = frame.args[2] or labels[1]
Line 57 ⟶ 78:
local pages = p.preprocessDisplays(args)
local options = {
category = yesno(args.category),
extraclasses = frame.args.extraclasses,
categoryifexists = yesno(frame.args.categoryifexists),
selfrefnamespace = frame.args.selfrefnamespace or args.selfrefnamespace,
selfref = yesno(frame.args.selfref or args.selfref),
template = template
}
Line 66 ⟶ 89:
 
function p._labelledList (pages, labels, options)
if options.ifexists then
labels = labels or {}
for k = #pages, 1, -1 do --iterate backwards to allow smooth removals
if #pages == 0 then
local v = pages[k]
return mHatnote.makeWikitextError(
local title = mw.title.new(getTarget(v), namespace)
'no page names specified',
if (v == '') or title == nil or not title.exists then
(options.template or defaults.template) .. '#Errors',
table.remove(pages, k)
options.category
) end
end
end
labels = labels or {}
label = (#pages == 1 and labels[1] or labels[2]) or defaults.label
for k, v in pairs(pages) do
Line 83 ⟶ 108:
(labels[4] or labels[2] or defaults.label)
) or defaults.label
end
end
if #pages == 0 then
if options.ifexists then
mw.addWarning(
string.format(
msg.noOutputWarning, options.template or defaults.template
)
)
return ''
else
return mHatnote.makeWikitextError(
msg.noInputWarning,
(options.template or defaults.template) .. '#Errors'msg.errorSuffix,
options.category
)
end
end
Anonymous user