Module:Old moves - Revision history

Joy: per Talk, parse the first unnamed parameter as list, with a tracking category

2025-04-25T19:12:43Z

per Talk, parse the first unnamed parameter as list, with a tracking category

← Previous revision		Revision as of 19:12, 25 April 2025
Line 1:		Line 1:
			-- This module implements {{Old move}}.

	local p = {}		local p = {}

Line 106:		Line 108:
	local function manualList(args)		local function manualList(args)
	local manualListText = ""		local manualListText = ""
	if (args["list"]) then		if (args["list"] or args[1]) then
	if (args["result"] or args["result1"]) then		if (args["result"] or args["result1"]) then
	manualListText = mw.ustring.format("%s\n'''Other discussions: '''\n%s", manualListText, args["list"])		manualListText = mw.ustring.format("%s\n'''Other discussions: '''\n%s", manualListText, args["list"] or args[1])
	else		else
	manualListText = mw.ustring.format("%s\n%s", manualListText, args["list"])		manualListText = mw.ustring.format("%s\n%s", manualListText, args["list"] or args[1])
	end		end
	end		end
Line 145:		Line 147:
	local is_collapsed = yesno(args["collapse"]) or yesno(args["collapsed"])		local is_collapsed = yesno(args["collapse"]) or yesno(args["collapsed"])
	local has_log_args = args["title"] or args["title1"]		local has_log_args = args["title"] or args["title1"]
	local has_manual_list = args["list"] or args["oldlist"]		local has_manual_list = args["list"] or args[1] or args["oldlist"]
	local has_multiple_rows = args["result1"]		local has_multiple_rows = args["result1"]

Line 172:		Line 174:
	if (has_manual_list) then		if (has_manual_list) then
	text = mw.ustring.format("%s%s", text, manualList(args))		text = mw.ustring.format("%s%s", text, manualList(args))
			if (args[1]) then
			text = mw.ustring.format("%s%s", text, "[[Category:Talk pages listing old moves with the first unnamed parameter]]")
			end
	end		end
	if (not args["result"] and not has_multiple_rows and not has_manual_list and not has_log_args) then		if (not args["result"] and not has_multiple_rows and not has_manual_list and not has_log_args) then

Izno: use div instead of table

2025-02-03T02:38:13Z

use div instead of table

← Previous revision		Revision as of 02:38, 3 February 2025
Line 1:		Line 1:
	-- This module implements {{Old move}}.

	local p = {}		local p = {}

Line 17:		Line 15:

	-- Build the HTML for collapsing lists		-- Build the HTML for collapsing lists
	local function ~~makeTable~~(list, text)		local function makeCollapsible(list, text)
	local ~~collapseTable~~ = mw.html.create("~~table~~")		local box = mw.html.create("div")
			box
	collapseTable
		⚫	:addClass("mw-collapsible mw-collapsed")
	:attr({
		⚫	:css('padding', '0.5em 0 0')
⚫	~~class =~~ "mw-collapsible mw-collapsed",
			:tag('div'):css('font-weight', 'bold'):wikitext(text):done()
	cellspacing = 0,
		⚫	:tag("div"):addClass('mw-collapsible-content'):wikitext(mw.ustring.format("\n%s", list))
	cellpadding = 0
			return box
	})
	:css({
	width = "100%",
	background = "transparent",
	margin = 0,
⚫	padding ~~= "~~0.5em 0 0"
	})
	:tag("tr"):tag("th"):wikitext(text):cssText("text-align: left")
⚫	:tag("tr"):~~tag~~(~~"td"~~):wikitext(mw.ustring.format("\n%s", list))
	return collapseTable
	end		end

Line 128:		Line 117:
	manualListText = mw.ustring.format("%s\n'''Older discussions: '''\n%s", manualListText, tostring(args["oldlist"]))		manualListText = mw.ustring.format("%s\n'''Older discussions: '''\n%s", manualListText, tostring(args["oldlist"]))
	else		else
	manualListText = mw.ustring.format("%s\n%s", manualListText, tostring(~~makeTable~~(args["oldlist"], "Older discussions:")))		manualListText = mw.ustring.format("%s\n%s", manualListText, tostring(makeCollapsible(args["oldlist"], "Older discussions:")))
	end		end
	end		end
Line 198:		Line 187:
	if (yesno(args["collapse"]) or yesno(args["collapsed"])) then		if (yesno(args["collapse"]) or yesno(args["collapsed"])) then
	if (args["title"] or args["title1"]) then		if (args["title"] or args["title1"]) then
	list = mw.ustring.format("%s%s", blurb, tostring(~~makeTable~~(evaluate(args), "Logs and discussions:")))		list = mw.ustring.format("%s%s", blurb, tostring(makeCollapsible(evaluate(args), "Logs and discussions:")))
	else		else
	list = mw.ustring.format("%s%s", blurb, tostring(~~makeTable~~(evaluate(args), "Discussions:")))		list = mw.ustring.format("%s%s", blurb, tostring(makeCollapsible(evaluate(args), "Discussions:")))
	end		end
	else		else

Mz7: in the rare case where we don't have anything at all to list, then we can suppress the "Discussions:" text

2024-07-17T06:10:27Z

in the rare case where we don't have anything at all to list, then we can suppress the "Discussions:" text

← Previous revision		Revision as of 06:10, 17 July 2024
Line 183:		Line 183:
	if (has_manual_list) then		if (has_manual_list) then
	text = mw.ustring.format("%s%s", text, manualList(args))		text = mw.ustring.format("%s%s", text, manualList(args))
			end
			if (not args["result"] and not has_multiple_rows and not has_manual_list and not has_log_args) then
			return ""
	end		end

Mz7: rework to avoid duplication "Discussion:"

2024-07-17T05:58:20Z

rework to avoid duplication "Discussion:"

← Previous revision		Revision as of 05:58, 17 July 2024
Line 120:		Line 120:
	if (args["result"] or args["result1"]) then		if (args["result"] or args["result1"]) then
	manualListText = mw.ustring.format("%s\n'''Other discussions: '''\n%s", manualListText, args["list"])		manualListText = mw.ustring.format("%s\n'''Other discussions: '''\n%s", manualListText, args["list"])
⚫	~~elseif~~ (yesno(args["collapse"]) or yesno(args["collapsed"])~~) then~~
	manualListText = mw.ustring.format("%s\n%s", manualListText, args["list"])
	else		else
	manualListText = mw.ustring.format("%s~~\n'''Discussions: '''~~\n%s", manualListText, args["list"])		manualListText = mw.ustring.format("%s\n%s", manualListText, args["list"])
	end		end
	end		end
Line 156:		Line 154:
	local function evaluate(args)		local function evaluate(args)
	local text = ""		local text = ""
		⚫	local is_collapsed = yesno(args["collapse"]) or yesno(args["collapsed"])
			local has_log_args = args["title"] or args["title1"]
			local has_manual_list = args["list"] or args["oldlist"]
			local has_multiple_rows = args["result1"]

	if (~~args["title"] or args["title1"]~~) then		if (has_log_args) then
	text = mw.ustring.format("%s%s", text, showLogs(args))		text = mw.ustring.format("%s%s", text, showLogs(args))
	end		end

			if (not is_collapsed or has_log_args) then
			-- if the template is collapsed and there are no log arguments,
			-- then we already have "Discussions:" in bold text in bannerText(),
			-- so we don't need to repeat it here
			text = mw.ustring.format("%s\n'''Discussions: '''\n", text)
			end

	-- This makes sure we don't show the single version when there's a log or manual list		-- This makes sure we don't show the single version when there's a log or manual list
	if (args["result"]) then		if (args["result"]) then
	if (~~args["title"]~~ or ~~args["title1"] or args["list"] or args["oldlist"]~~ or args["result2"]) then		if (has_log_args or has_manual_list or args["result2"]) then
	text = mw.ustring.format("%s~~\n'''Discussions: '''\n~~%s", text, list(args))		text = mw.ustring.format("%s%s", text, list(args))
	else		else
	text = singleText(args)		text = singleText(args)
	end		end
	end		end
			if (has_multiple_rows) then
	if (args["result1"]) then -- Do we have multiple rows?
	text = mw.ustring.format("%s~~\n'''Discussions: '''\n~~%s", text, list(args))		text = mw.ustring.format("%s%s", text, list(args))
	end		end
	if (~~args["list"] or args["oldlist"]~~) then		if (has_manual_list) then
	text = mw.ustring.format("%s%s", text, manualList(args))		text = mw.ustring.format("%s%s", text, manualList(args))
	end		end

Mz7: adding blurb "Please review the prior discussions if you are considering re-nomination." to align with Template:Old XfD multi and also as good advice

2024-07-13T19:18:06Z

adding blurb "Please review the prior discussions if you are considering re-nomination." to align with Template:Old XfD multi and also as good advice

← Previous revision		Revision as of 19:18, 13 July 2024
Line 179:		Line 179:

	local function bannerText(args)		local function bannerText(args)
	local blurb = mw.ustring.format("This %s has previously been nominated to be moved.", ~~pageType)~~		local blurb = mw.ustring.format("This %s has previously been nominated to be moved." ..
			" Please review the prior discussions if you are considering re-nomination.", pageType)
	local list = ""		local list = ""

Mz7: Undid revision 1234321347 by Mz7 (talk): ugh, this is creating duplicates in the old implementation, e.g. Talk:George I of Great Britain

2024-07-13T19:11:02Z

Undid revision 1234321347 by Mz7 (talk): ugh, this is creating duplicates in the old implementation, e.g. Talk:George I of Great Britain

← Previous revision		Revision as of 19:11, 13 July 2024
Line 156:		Line 156:
	local function evaluate(args)		local function evaluate(args)
	local text = ""		local text = ""
	local is_collapsed = yesno(args["collapse"]) or yesno(args["collapsed"])
	local has_log_args = args["title"] or args["title1"]
	local has_manual_list = args["list"] or args["oldlist"]
	local has_multiple_rows = args["result1"]

	if (~~has_log_args~~) then		if (args["title"] or args["title1"]) then
	text = mw.ustring.format("%s%s", text, showLogs(args))		text = mw.ustring.format("%s%s", text, showLogs(args))
	end		end

	if (not is_collapsed or has_log_args) then
	-- if the template is collapsed and there are no log arguments,
	-- then we already have "Discussions:" in bold text in bannerText(),
	-- so we don't need to repeat it here
	text = mw.ustring.format("%s\n'''Discussions: '''\n", text)
	end

	-- This makes sure we don't show the single version when there's a log or manual list		-- This makes sure we don't show the single version when there's a log or manual list
	if (args["result"]) then		if (args["result"]) then
	if (~~has_log_args~~ or ~~has_manual_list~~ or args["result2"]) then		if (args["title"] or args["title1"] or args["list"] or args["oldlist"] or args["result2"]) then
	text = mw.ustring.format("%s%s", text, list(args))		text = mw.ustring.format("%s\n'''Discussions: '''\n%s", text, list(args))
	else		else
	text = singleText(args)		text = singleText(args)
	end		end
	end		end
			if (args["result1"]) then -- Do we have multiple rows?
	if (has_multiple_rows) then
	text = mw.ustring.format("%s%s", text, list(args))		text = mw.ustring.format("%s\n'''Discussions: '''\n%s", text, list(args))
	end		end
	if (~~has_manual_list~~) then		if (args["list"] or args["oldlist"]) then
	text = mw.ustring.format("%s%s", text, manualList(args))		text = mw.ustring.format("%s%s", text, manualList(args))
	end		end

Mz7: avoid duplicate "Discussion:" header in collapsed lists

2024-07-13T19:06:54Z

avoid duplicate "Discussion:" header in collapsed lists

← Previous revision		Revision as of 19:06, 13 July 2024
Line 156:		Line 156:
	local function evaluate(args)		local function evaluate(args)
	local text = ""		local text = ""
			local is_collapsed = yesno(args["collapse"]) or yesno(args["collapsed"])
			local has_log_args = args["title"] or args["title1"]
			local has_manual_list = args["list"] or args["oldlist"]
			local has_multiple_rows = args["result1"]

	if (~~args["title"] or args["title1"]~~) then		if (has_log_args) then
	text = mw.ustring.format("%s%s", text, showLogs(args))		text = mw.ustring.format("%s%s", text, showLogs(args))
	end		end

			if (not is_collapsed or has_log_args) then
			-- if the template is collapsed and there are no log arguments,
			-- then we already have "Discussions:" in bold text in bannerText(),
			-- so we don't need to repeat it here
			text = mw.ustring.format("%s\n'''Discussions: '''\n", text)
			end

	-- This makes sure we don't show the single version when there's a log or manual list		-- This makes sure we don't show the single version when there's a log or manual list
	if (args["result"]) then		if (args["result"]) then
	if (~~args["title"]~~ or ~~args["title1"] or args["list"] or args["oldlist"]~~ or args["result2"]) then		if (has_log_args or has_manual_list or args["result2"]) then
	text = mw.ustring.format("%s~~\n'''Discussions: '''\n~~%s", text, list(args))		text = mw.ustring.format("%s%s", text, list(args))
	else		else
	text = singleText(args)		text = singleText(args)
	end		end
	end		end
			if (has_multiple_rows) then
	if (args["result1"]) then -- Do we have multiple rows?
	text = mw.ustring.format("%s~~\n'''Discussions: '''\n~~%s", text, list(args))		text = mw.ustring.format("%s%s", text, list(args))
	end		end
	if (~~args["list"] or args["oldlist"]~~) then		if (has_manual_list) then
	text = mw.ustring.format("%s%s", text, manualList(args))		text = mw.ustring.format("%s%s", text, manualList(args))
	end		end

Hilst: MaterialWorks moved page Module:Old move to Module:Old moves without leaving a redirect: sync name with parent template

2023-08-16T14:43:26Z

MaterialWorks moved page Module:Old move to Module:Old moves without leaving a redirect: sync name with parent template

← Previous revision	Revision as of 14:43, 16 August 2023
(No difference)

Hilst: VERY dirty hotfix to make sure the multiple entry version shows up when there both non-numbered and numbered result params, will rework this later

2023-08-10T18:16:29Z

VERY dirty hotfix to make sure the multiple entry version shows up when there both non-numbered and numbered result params, will rework this later

← Previous revision		Revision as of 18:16, 10 August 2023
Line 162:		Line 162:
	-- This makes sure we don't show the single version when there's a log or manual list		-- This makes sure we don't show the single version when there's a log or manual list
	if (args["result"]) then		if (args["result"]) then
	if (args["title"] or args["title1"] or args["list"] or args["oldlist"]) then		if (args["title"] or args["title1"] or args["list"] or args["oldlist"] or args["result2"]) then
	text = mw.ustring.format("%s\n'''Discussions: '''\n%s", text, list(args))		text = mw.ustring.format("%s\n'''Discussions: '''\n%s", text, list(args))
	else		else
Line 188:		Line 188:
	list = mw.ustring.format("%s%s", blurb, tostring(makeTable(evaluate(args), "Discussions:")))		list = mw.ustring.format("%s%s", blurb, tostring(makeTable(evaluate(args), "Discussions:")))
	end		end
	elseif (args["result"] and not args["result1"] and not args["list"] and not args["oldlist"]) then
	list = singleText(args)
	else		else
	list = ~~mw.ustring.format("%s%s", blurb,~~ evaluate(args))		list = evaluate(args)

			if (mw.ustring.find(list, "proposed that this") == nil) then
			list = mw.ustring.format("%s%s", blurb, evaluate(args))
			end
	end		end

Hilst: simplify single entry evaluation, changed the argument used to check for multiple entries

2023-08-05T11:58:03Z

simplify single entry evaluation, changed the argument used to check for multiple entries

← Previous revision		Revision as of 11:58, 5 August 2023
Line 160:		Line 160:
	text = mw.ustring.format("%s%s", text, showLogs(args))		text = mw.ustring.format("%s%s", text, showLogs(args))
	end		end

	-- This makes sure we don't show the single version when there's a log or manual list		-- This makes sure we don't show the single version when there's a log or manual list
	if (args["result"]) then		if (args["result"]) then
	if (~~not~~ args["title"] ~~and not~~ args["title1"]) then		if (args["title"] or args["title1"] or args["list"] or args["oldlist"]) then
		⚫	text = mw.ustring.format("%s\n'''Discussions: '''\n%s", text, list(args))
	if (args["list"] or args["oldlist"]) then
⚫	text = mw.ustring.format("%s\n'''Discussions: '''\n%s", text, list(args))
	else
	text = singleText(args)
	end
	else		else
	text = ~~mw.ustring.format("%s\n'''Discussions: '''\n%s", text, list~~(args))		text = singleText(args)
	end		end
	end		end
	if (args["~~result2~~"]) then -- Do we have multiple rows?		if (args["result1"]) then -- Do we have multiple rows?
	text = mw.ustring.format("%s\n'''Discussions: '''\n%s", text, list(args))		text = mw.ustring.format("%s\n'''Discussions: '''\n%s", text, list(args))
	end		end
Line 193:		Line 188:
	list = mw.ustring.format("%s%s", blurb, tostring(makeTable(evaluate(args), "Discussions:")))		list = mw.ustring.format("%s%s", blurb, tostring(makeTable(evaluate(args), "Discussions:")))
	end		end
	elseif (args["result"] and not args["~~result2~~"] and not args["list"] and not args["oldlist"]) then		elseif (args["result"] and not args["result1"] and not args["list"] and not args["oldlist"]) then
	list = singleText(args)		list = singleText(args)
	else		else