Module:Split article - Revision history

MtPenguinMonster: Fix bug with text not being defined

2025-02-16T07:31:07Z

Fix bug with text not being defined

← Previous revision		Revision as of 07:31, 16 February 2025
Line 97:		Line 97:

	local function row(args, i)		local function row(args, i)
			local text = ""
	local to = delink(args["to" .. i] or (i == 1 and args["to"] or nil))		local to = delink(args["to" .. i] or (i == 1 and args["to"] or nil))
	local toTalk = mw.title.new(delink(to)).subjectPageTitle.prefixedText		local toTalk = mw.title.new(delink(to)).subjectPageTitle.prefixedText

Taavi: use codex design token for dark mode support

2024-10-05T06:28:16Z

use codex design token for dark mode support

← Previous revision		Revision as of 06:28, 5 October 2024
Line 173:		Line 173:
	target = mw.html.create("table")		target = mw.html.create("table")
	:attr("class", "mw-collapsible mw-collapsed")		:attr("class", "mw-collapsible mw-collapsed")
	:attr("style", "width:100%; background-color: #~~f8eaba~~;")		:attr("style", "width:100%; background-color: var( --background-color-warning-subtle, #fdf2d5 ); color: inherit;")
	:node(mw.html.create("tr")		:node(mw.html.create("tr")
	:node(mw.html.create("th")		:node(mw.html.create("th")
Line 186:		Line 186:
	target = mw.html.create("table")		target = mw.html.create("table")
	:attr("class", "mw-collapsible mw-collapsed")		:attr("class", "mw-collapsible mw-collapsed")
	:attr("style", "width:100%; background-color: #~~f8eaba~~;")		:attr("style", "width:100%; background-color: var( --background-color-warning-subtle, #fdf2d5 ); color: inherit;")
	:node(mw.html.create("tr")		:node(mw.html.create("tr")
	:node(mw.html.create("th")		:node(mw.html.create("th")

Chlod: fixing misnested tag (td without tr), rm logObject

2022-08-18T03:06:26Z

fixing misnested tag (td without tr), rm logObject

← Previous revision		Revision as of 03:06, 18 August 2022
Line 17:		Line 17:
	return nil		return nil
	end		end
	mw.logObject(require("Module:Delink")._delink{text})
	return string.gsub(string.gsub(require("Module:Delink")._delink{text}, "%[%[", ""), "%]%]", "")		return string.gsub(string.gsub(require("Module:Delink")._delink{text}, "%[%[", ""), "%]%]", "")
	end		end
Line 193:		Line 192:
	)		)
	)		)
	:node(mw.html.create("td"~~):node(ul)~~)		:node(mw.html.create("tr")
			:node(mw.html.create("td"):node(ul))
			)
	end		end

MusikBot II: Protected "Module:Split article": High-risk template or module: 3421 transclusions (more info) ([Edit=Require extended confirmed access] (indefinite) [Move=Require extended confirmed access] (indefinite))

2022-08-14T18:00:39Z

Protected "Module:Split article": High-risk template or module: 3421 transclusions (more info) ([Edit=Require extended confirmed access] (indefinite) [Move=Require extended confirmed access] (indefinite))

← Previous revision	Revision as of 18:00, 14 August 2022
(No difference)

Chlod: better delinking, fix flipped date/diff

2022-08-13T18:33:33Z

better delinking, fix flipped date/diff

← Previous revision		Revision as of 18:33, 13 August 2022
Line 14:		Line 14:

	local function delink(text)		local function delink(text)
			if text == nil then
			return nil
			end
			mw.logObject(require("Module:Delink")._delink{text})
	return string.gsub(string.gsub(require("Module:Delink")._delink{text}, "%[%[", ""), "%]%]", "")		return string.gsub(string.gsub(require("Module:Delink")._delink{text}, "%[%[", ""), "%]%]", "")
	end		end
Line 34:		Line 38:
	local text = ""		local text = ""
	-- from1 also included in case someone filed the template improperly		-- from1 also included in case someone filed the template improperly
	local from = args["from"] or args["from1"] or mw.title.getCurrentTitle().subjectPageTitle.prefixedText		local from = delink(args["from"] or args["from1"]) or mw.title.getCurrentTitle().subjectPageTitle.prefixedText
	local fromTalk = mw.title.new~~(delink~~(from)).talkPageTitle.prefixedText		local fromTalk = mw.title.new(from).talkPageTitle.prefixedText

	if args["from1"] then		if args["from1"] then
Line 45:		Line 49:

	-- Row-based parameters		-- Row-based parameters
	local to = args["to"] or args["to1"]		local to = delink(args["to"] or args["to1"])
	local diff = args["diff"] or args["diff1"]		local diff = args["diff"] or args["diff1"]
	local date = args["date"] or args["date1"]		local date = args["date"] or args["date1"]
Line 58:		Line 62:
	text = string.format("Material from [[%s]] was split to [[%s]]", from, to, date)		text = string.format("Material from [[%s]] was split to [[%s]]", from, to, date)
	if diff ~= nil and date ~= nil then		if diff ~= nil and date ~= nil then
	if ~~date~~:match("^[0123456789/]+$") then		if diff:match("^[0123456789/]+$") then
	text = string.format("%s on [[Special:Diff/%s\|%s]]", text, diff, date)		text = string.format("%s on [[Special:Diff/%s\|%s]]", text, diff, date)
	else		else
Line 94:		Line 98:

	local function row(args, i)		local function row(args, i)
	local to = args["to" .. i] or (i == 1 and args["to"] or nil)		local to = delink(args["to" .. i] or (i == 1 and args["to"] or nil))
	local toTalk = mw.title.new(delink(to)).subjectPageTitle.prefixedText		local toTalk = mw.title.new(delink(to)).subjectPageTitle.prefixedText
	local diff = args["diff" .. i] or (i == 1 and args["diff"] or nil)		local diff = args["diff" .. i] or (i == 1 and args["diff"] or nil)
Line 111:		Line 115:
	)		)
	if diff ~= nil and date ~= nil then		if diff ~= nil and date ~= nil then
	if ~~date~~:match("^[0123456789/]+$") then		if diff:match("^[0123456789/]+$") then
	rowText = string.format("%s on [[Special:Diff/%s\|%s]]", rowText, diff, date)		rowText = string.format("%s on [[Special:Diff/%s\|%s]]", rowText, diff, date)
	else		else
Line 133:		Line 137:
	local text = ""		local text = ""
	-- from1 also included in case someone filed the template improperly		-- from1 also included in case someone filed the template improperly
	local from = args["from"] or args["from1"] or mw.title.getCurrentTitle().subjectPageTitle.prefixedText		local from = delink(args["from"] or args["from1"]) or mw.title.getCurrentTitle().subjectPageTitle.prefixedText
	local fromTalk = mw.title.new(delink(from)).talkPageTitle.prefixedText		local fromTalk = mw.title.new(delink(from)).talkPageTitle.prefixedText
	local collapse = tonumber(args["collapse"]) or yesno(args["collapse"])		local collapse = tonumber(args["collapse"]) or yesno(args["collapse"])
Line 178:		Line 182:
	:node(mw.html.create("td"):node(ul))		:node(mw.html.create("td"):node(ul))
	elseif collapse == (i - 1) then		elseif collapse == (i - 1) then
	mw.logObject({1, i, ul})
	ul2 = ul		ul2 = ul
	ul = mw.html.create("ul")		ul = mw.html.create("ul")
	mw.logObject({2, i, ul, ul2})
	-- Only once threshold has been reached		-- Only once threshold has been reached
	target = mw.html.create("table")		target = mw.html.create("table")

Chlod: delink linked uses

2022-08-13T18:28:43Z

delink linked uses

← Previous revision		Revision as of 18:28, 13 August 2022
Line 11:		Line 11:
	mw.addWarning(text)		mw.addWarning(text)
	return nocat and "" or "[[Category:Pages with split article warnings]]"		return nocat and "" or "[[Category:Pages with split article warnings]]"
			end

			local function delink(text)
			return string.gsub(string.gsub(require("Module:Delink")._delink{text}, "%[%[", ""), "%]%]", "")
	end		end

Line 31:		Line 35:
	-- from1 also included in case someone filed the template improperly		-- from1 also included in case someone filed the template improperly
	local from = args["from"] or args["from1"] or mw.title.getCurrentTitle().subjectPageTitle.prefixedText		local from = args["from"] or args["from1"] or mw.title.getCurrentTitle().subjectPageTitle.prefixedText
	local fromTalk = mw.title.new(from).talkPageTitle.prefixedText		local fromTalk = mw.title.new(delink(from)).talkPageTitle.prefixedText

	if args["from1"] then		if args["from1"] then
Line 91:		Line 95:
	local function row(args, i)		local function row(args, i)
	local to = args["to" .. i] or (i == 1 and args["to"] or nil)		local to = args["to" .. i] or (i == 1 and args["to"] or nil)
	local toTalk = mw.title.new(to).subjectPageTitle.prefixedText		local toTalk = mw.title.new(delink(to)).subjectPageTitle.prefixedText
	local diff = args["diff" .. i] or (i == 1 and args["diff"] or nil)		local diff = args["diff" .. i] or (i == 1 and args["diff"] or nil)
	local date = args["date" .. i] or (i == 1 and args["date"] or nil)		local date = args["date" .. i] or (i == 1 and args["date"] or nil)
Line 130:		Line 134:
	-- from1 also included in case someone filed the template improperly		-- from1 also included in case someone filed the template improperly
	local from = args["from"] or args["from1"] or mw.title.getCurrentTitle().subjectPageTitle.prefixedText		local from = args["from"] or args["from1"] or mw.title.getCurrentTitle().subjectPageTitle.prefixedText
	local fromTalk = mw.title.new(from).talkPageTitle.prefixedText		local fromTalk = mw.title.new(delink(from)).talkPageTitle.prefixedText
	local collapse = tonumber(args["collapse"]) or yesno(args["collapse"])		local collapse = tonumber(args["collapse"]) or yesno(args["collapse"])
	if args["from1"] then		if args["from1"] then

Chlod: add conditions to support missing date

2022-08-13T17:57:39Z

add conditions to support missing date

← Previous revision		Revision as of 17:57, 13 August 2022
Line 53:		Line 53:

	text = string.format("Material from [[%s]] was split to [[%s]]", from, to, date)		text = string.format("Material from [[%s]] was split to [[%s]]", from, to, date)
	if diff ~= nil then		if diff ~= nil and date ~= nil then
	if date:match("^[0123456789/]+$") then		if date:match("^[0123456789/]+$") then
	text = string.format("%s on [[Special:Diff/%s\|%s]]", text, diff, date)		text = string.format("%s on [[Special:Diff/%s\|%s]]", text, diff, date)
Line 59:		Line 59:
	text = string.format("%s on [%s %s]", text, diff, date)		text = string.format("%s on [%s %s]", text, diff, date)
	end		end
			elseif date ~= nil then
	else
	text = string.format("%s on %s", text, date)		text = string.format("%s on %s", text, date)
	end		end
Line 106:		Line 106:
	to, toTalk, to, date, diff		to, toTalk, to, date, diff
	)		)
	if diff ~= nil then		if diff ~= nil and date ~= nil then
	if date:match("^[0123456789/]+$") then		if date:match("^[0123456789/]+$") then
	rowText = string.format("%s on [[Special:Diff/%s\|%s]]", rowText, diff, date)		rowText = string.format("%s on [[Special:Diff/%s\|%s]]", rowText, diff, date)
Line 112:		Line 112:
	rowText = string.format("%s on [%s %s]", rowText, diff, date)		rowText = string.format("%s on [%s %s]", rowText, diff, date)
	end		end
			elseif date ~= nil then
	else
	rowText = string.format("%s on %s", rowText, date)		rowText = string.format("%s on %s", rowText, date)
	end		end

Chlod: downgrade missing date to warning (to act more similar to plain wikitext implementation)

2022-08-13T17:56:28Z

downgrade missing date to warning (to act more similar to plain wikitext implementation)

← Previous revision		Revision as of 17:56, 13 August 2022
Line 49:		Line 49:
	end		end
	if date == nil then		if date == nil then
	~~return~~ ~~showError~~("Split date was not specified with <code>date</code>.", demoCheck(args))		text = text .. showWarning("Split date was not specified with <code>date</code>. This may cause an error in the future.", demoCheck(args))
	end		end

Line 96:		Line 96:
	local from_oldid = args["from_oldid" .. i] or (i == 1 and args["from_oldid"] or nil)		local from_oldid = args["from_oldid" .. i] or (i == 1 and args["from_oldid"] or nil)
	if date == nil then		if date == nil then
	~~return~~ ~~showError~~(string.format(		text = text .. showWarning(string.format(
	"Split date for \"%s\" (<code>to%s</code>) was not specified with <code>date</code>.",		"Split date for \"%s\" (<code>to%s</code>) was not specified with <code>date</code>. This may cause an error in the future.",
	to, i		to, i
	), demoCheck(args))		), demoCheck(args))

Chlod: fix misused parameter

2022-08-13T17:27:14Z

fix misused parameter

← Previous revision		Revision as of 17:27, 13 August 2022
Line 107:		Line 107:
	)		)
	if diff ~= nil then		if diff ~= nil then
	if ~~from_oldid~~:match("^[0123456789/]+$") then		if date:match("^[0123456789/]+$") then
	rowText = string.format("%s on [[Special:Diff/%s\|%s]]", rowText, diff, date)		rowText = string.format("%s on [[Special:Diff/%s\|%s]]", rowText, diff, date)
	else		else

Chlod: add support for collapse

2022-08-12T12:51:26Z

add support for collapse

@@ Line 1: / Line 1: @@
-local MessageBox = require('Module:Message box')
+local MessageBox = require("Module:Message box")
 local yesno = require("Module:Yesno")
@@ Line 131: / Line 131: @@
 	local from = args["from"] or args["from1"] or mw.title.getCurrentTitle().subjectPageTitle.prefixedText
 	local fromTalk = mw.title.new(from).talkPageTitle.prefixedText
 	if args["from1"] then
 		text = text .. showWarning(
@@ Line 153: / Line 154: @@
+	)
-	local li = mw.html.create("ul")
+	local ul = mw.html.create("ul")
 	-- Row-based parameters
 	local i = 1
 	while args["to" .. i] or (i == 1 and args["to"]) do
 		i = i + 1
 	end
@@ Line 169: / Line 201: @@
 	end
-	text = text .. " " .. tostring(li)
+	text = text .. " " .. tostring(ul2 or "") .. tostring(target)
 	return text