Module:DYK queue formatting check - Revision history

VectorWorld: detect https://en.wikipedia.org/w/index.php?title=Template:Did_you_know/Preparation_area_4&diff=prev&oldid=1283687597

2025-04-06T02:47:27Z

detect https://en.wikipedia.org/w/index.php?title=Template:Did_you_know/Preparation_area_4&diff=prev&oldid=1283687597

← Previous revision		Revision as of 02:47, 6 April 2025
Line 43:		Line 43:

	-- Per-hook checks that may have occasional exceptions		-- Per-hook checks that may have occasional exceptions
	local function check_hook(wikitextLine, expandedLine, hookNum)		local function check_hook(wikitextLine, expandedLine, decodedLine, hookNum)
			if ((expandedLine:find("{{") and not wikitextLine:find("{{%("))
			or (expandedLine:find("}}") and not wikitextLine:find("{{%)"))
			or (expandedLine:find("%[%[") and not wikitextLine:find("{{!"))
			or (expandedLine:find("%]%]") and not wikitextLine:find("!}}"))) then
			-- "code" is not ignored because some hooks use the word "codename"
			-- Use templates or HTML alternatives from
			-- [[Template:Escape template list]] if there's actually code in a hook.
			return "Potentially broken wikitext"
			end
	local byteCount = -1 -- Counting ends at the question mark		local byteCount = -1 -- Counting ends at the question mark
	if wikitextLine:find("''%([^)]+%)''") then		if wikitextLine:find("''%([^)]+%)''") then
Line 58:		Line 67:
	boldLinkFound = true		boldLinkFound = true
	end		end
	byteCount = byteCount + #~~expandedLine~~		byteCount = byteCount + #decodedLine
	if byteCount > 200 then		if byteCount > 200 then
	return "[[WP:DYK200]]: The hook cannot exceed 200 prose characters. " ..		return "[[WP:DYK200]]: The hook cannot exceed 200 prose characters. " ..
Line 78:		Line 87:
	-- First hook might have been skipped by opt-out		-- First hook might have been skipped by opt-out
	else		else
	if not ~~expandedLine~~:find("%([^)]+%)") then		if not decodedLine:find("%([^)]+%)") then
	return "[[WP:DYKIMG]]/[[WP:DYKMOS]]: Missing ''(pictured)'' or " ..		return "[[WP:DYKIMG]]/[[WP:DYKMOS]]: Missing ''(pictured)'' or " ..
	"alternative"		"alternative"
Line 89:		Line 98:
	or string.find(wikitextLine, "%]%] '''+ '''+ *%[%[")) then		or string.find(wikitextLine, "%]%] '''+ '''+ *%[%[")) then
	return "[[WP:DYKMOS]]/[[MOS:SEAOFBLUE]]: Two non-boldlinks or " ..		return "[[WP:DYKMOS]]/[[MOS:SEAOFBLUE]]: Two non-boldlinks or " ..
	"two boldlinks must be kept separate: " ~~.. expandedLine~~		"two boldlinks must be kept separate"
	end		end
	end		end
	if ~~expandedLine~~:byte(-1) ~= 46 then -- Not an empty hook in prep area		if decodedLine:byte(-1) ~= 46 then -- Not an empty hook in prep area
	if not boldLinkFound then		if not boldLinkFound then
	return "[[WP:DYKMOS]]: Every eligible article in the hook should be " ..		return "[[WP:DYKMOS]]: Every eligible article in the hook should be " ..
	"linked and wrapped in bold markup"		"linked and wrapped in bold markup"
	end		end
	if ~~expandedLine~~:byte(-1) ~= 63 then		if decodedLine:byte(-1) ~= 63 then
	return "[[WP:DYKMOS]]: A hook should end in a question mark"		return "[[WP:DYKMOS]]: A hook should end in a question mark"
	end		end
	end		end
	if mw.ustring.find(~~expandedLine~~, ("[“”]")) then		if mw.ustring.find(decodedLine, ("[“”]")) then
	return '[[MOS:CURLY]]: Use "straight" quotation marks, not “curly” ones'		return '[[MOS:CURLY]]: Use "straight" quotation marks, not “curly” ones'
	end		end
	if mw.ustring.find(~~expandedLine~~, "[‘’]") then		if mw.ustring.find(decodedLine, "[‘’]") then
	return "[[MOS:APOSTROPHE]]: Use straight apostrophes ('), " ..		return "[[MOS:APOSTROPHE]]: Use straight apostrophes ('), " ..
	"not curly apostrophes (‘ or ’)"		"not curly apostrophes (‘ or ’)"
Line 199:		Line 208:
	local expandedLine = frame:preprocess(wikitextLine)		local expandedLine = frame:preprocess(wikitextLine)
	expandedLine = plainText(expandedLine, false)		expandedLine = plainText(expandedLine, false)
	~~expandedLine~~ = decode(expandedLine)		local decodedLine = decode(expandedLine)
	local hookProblem = check_hook(wikitextLine, expandedLine, ~~hookNum)~~		local hookProblem = check_hook(wikitextLine, expandedLine,
			decodedLine, hookNum)
	if hookProblem then		if hookProblem then
	return hookProblem .. ": " .. ~~expandedLine~~		return hookProblem .. ": " .. decodedLine
	end		end

VectorWorld: detect https://en.wikipedia.org/w/index.php?title=Template:Did_you_know/Queue/2&diff=prev&oldid=1280004514

2025-04-06T02:22:46Z

detect https://en.wikipedia.org/w/index.php?title=Template:Did_you_know/Queue/2&diff=prev&oldid=1280004514

← Previous revision		Revision as of 02:22, 6 April 2025
Line 150:		Line 150:
	return "[[WP:DYKMOS]]: {{`}} (or {{`s}}) is for adjacent bold " ..		return "[[WP:DYKMOS]]: {{`}} (or {{`s}}) is for adjacent bold " ..
	"markup; {{'}} (or {{'s}}) is for adjacent italic (or bold italic)"		"markup; {{'}} (or {{'s}}) is for adjacent italic (or bold italic)"
			end
			if (delinkedContent:find("[^'] *{{'s}}")) then
			return "[[WP:DYKMOS]]: {{'s}} is only for italics"
	end		end
	if content:find("\n%*%.%.%.") then		if content:find("\n%*%.%.%.") then

VectorWorld: silence on https://en.wikipedia.org/w/index.php?title=Template:Did_you_know/Preparation_area_5&oldid=1283491162

2025-04-01T21:35:12Z

silence on https://en.wikipedia.org/w/index.php?title=Template:Did_you_know/Preparation_area_5&oldid=1283491162

← Previous revision		Revision as of 21:35, 1 April 2025
Line 216:		Line 216:

	-- No empty hooks ("... that ...") = queue or completed prep area		-- No empty hooks ("... that ...") = queue or completed prep area
	if not content:find("\n%* %.%.%. that %.%.%.\n") then		if not content:find("\n%* %.%.%. that[^\n]*%.%.%.\n") then
	local queueProblem = check_queue(content)		local queueProblem = check_queue(content)
	if queueProblem then		if queueProblem then

173.206.124.146: fix detection of italics/bold inside piped links, document non-detection of contractions and missing "that"

2025-03-31T00:49:30Z

fix detection of italics/bold inside piped links, document non-detection of contractions and missing "that"

← Previous revision		Revision as of 00:49, 31 March 2025
Line 110:		Line 110:
	-- Only humans can check if {{lang}} and {{transl}} are needed i.e. if		-- Only humans can check if {{lang}} and {{transl}} are needed i.e. if
	-- the "non-English and transliterated text" is in "common English usage"		-- the "non-English and transliterated text" is in "common English usage"
			-- [[WP:DYKMOS]] says that only ''most'' hooks begin with "that"
			-- [[MOS:CONTRACTION]] doesn't apply to quotes
	end		end

Line 130:		Line 132:
	"disambiguation pages"		"disambiguation pages"
	end		end
	if (content:~~find~~("~~[^']''~~%?")		local delinkedContent = content:gsub("%]", "")
	or ~~content~~:find("''''%?")		if (delinkedContent:find("[^']''%?")
	or ~~content~~:find("~~[ ;?]~~%?")~~) then~~		or delinkedContent:find("''''%?")
			or delinkedContent:find("[ ;?]%?")) then
	return "[[WP:DYKMOS]]: There should not be a space before the " ..		return "[[WP:DYKMOS]]: There should not be a space before the " ..
	"question mark, but if the text directly preceding it is " ..		"question mark, but if the text directly preceding it is " ..
	"italicized, the {{-?}} tag can offset it."		"italicized, the {{-?}} tag can offset it."
	end		end
	if ~~content~~:find("''s ") then		if delinkedContent:find("''s ") then
	return "[[WP:DYKMOS]]: Keep the bold / italic (or bold italic) text and " ..		return "[[WP:DYKMOS]]: Keep the bold / italic (or bold italic) text and " ..
	"the apostrophe distinct using {{`s}}/{{'s}} respectively"		"the apostrophe distinct using {{`s}}/{{'s}} respectively"
	end		end
	if (~~content~~:find("''''{{`") -- Use {{'}} with bold italic		if (delinkedContent:find("''''{{`") -- Use {{'}} with bold italic
	or ~~content~~:find("[^']''{{`") -- Use {{'}} with italic		or delinkedContent:find("[^']''{{`") -- Use {{'}} with italic
	or ~~content~~:find("[^']'''{{'")) then -- Use {{`}} with bold		or delinkedContent:find("[^']'''{{'")) then -- Use {{`}} with bold
	-- Adapted from [[Template:Quotation mark templates]]		-- Adapted from [[Template:Quotation mark templates]]
	return "[[WP:DYKMOS]]: {{`}} (or {{`s}}) is for adjacent bold " ..		return "[[WP:DYKMOS]]: {{`}} (or {{`s}}) is for adjacent bold " ..

173.206.124.146: detect "ArticleName" per hidden comment

2025-03-29T07:21:34Z

detect "ArticleName" per hidden comment

← Previous revision		Revision as of 07:21, 29 March 2025
Line 165:		Line 165:
	end		end
	for article in content:gmatch("\n%* *%{%{DYK[^\n\|]+\|([^\n\|]+)\|") do		for article in content:gmatch("\n%* *%{%{DYK[^\n\|]+\|([^\n\|]+)\|") do
	if article == ~~"Example" then~~		article = article:lower()
			if article == "example" or article == "articlename" then
	return "[[WP:DYKPBI]]: DYKmake or DYKnom article is still " ..		return "[[WP:DYKPBI]]: DYKmake or DYKnom article is still " ..
	'"Example"'		'"Example"/"ArticleName"'
	end		end
	end		end

173.206.124.146: improve message for https://en.wikipedia.org/w/index.php?title=Template:Did_you_know/Preparation_area_1&diff=prev&oldid=1282875365

2025-03-29T07:02:35Z

improve message for https://en.wikipedia.org/w/index.php?title=Template:Did_you_know/Preparation_area_1&diff=prev&oldid=1282875365

← Previous revision		Revision as of 07:02, 29 March 2025
Line 62:		Line 62:
	return "[[WP:DYK200]]: The hook cannot exceed 200 prose characters. " ..		return "[[WP:DYK200]]: The hook cannot exceed 200 prose characters. " ..
	"It is currently " .. byteCount .. " characters"		"It is currently " .. byteCount .. " characters"
			elseif byteCount < 1 then
			if hookNum == 1 then
			return 'Empty first hooks are usually written as "... that ... ' ..
			"''(pictured)'' ...\""
			else
			return 'Empty hooks are usually written as "... that ..."'
			end
	end		end
	if hookNum > 1 then		if hookNum > 1 then

216.58.24.27: rm nowiki opt-out feature, as requested by https://en.wikipedia.org/wiki/Wikipedia_talk:Did_you_know#c-RoySmith-20250328120200-RoySmith-20250328113900

2025-03-28T12:27:16Z

rm nowiki opt-out feature, as requested by https://en.wikipedia.org/wiki/Wikipedia_talk:Did_you_know#c-RoySmith-20250328120200-RoySmith-20250328113900

← Previous revision		Revision as of 12:27, 28 March 2025
Line 3:		Line 3:
	-- [[WP:DYKMOS]] and [[MOS:CQ]] to keep formatting errors from [[WP:MP/E]]		-- [[WP:DYKMOS]] and [[MOS:CQ]] to keep formatting errors from [[WP:MP/E]]

			-- If there's a false positive with a hook and the warning is too annoying,
	-- To disable most of this module for YOUR hook, add "<nowiki/>"
			-- the current method is to just disableAll.

	-- This module doesn't show up on the main page or prevent promotion,		-- This module doesn't show up on the main page or prevent promotion,
Line 177:		Line 178:
	-- Counting starts from after the space following the three dots		-- Counting starts from after the space following the three dots
	for wikitextLine in content:gmatch("\n%* %.%.%. ([^\n]*)") do		for wikitextLine in content:gmatch("\n%* %.%.%. ([^\n]*)") do
			if false then -- "Skip one hook" feature was removed as under discussion
	-- "nowiki" anywhere = skip this hook
	if wikitextLine:find("nowiki") then
	-- Disable missing ''(pictured)'' check		-- Disable missing ''(pictured)'' check
	-- as first hook might have been skipped		-- as first hook might have been skipped

216.58.24.27: general_checks are still run with nowiki

2025-03-28T07:54:28Z

general_checks are still run with nowiki

← Previous revision		Revision as of 07:54, 28 March 2025
Line 3:		Line 3:
	-- [[WP:DYKMOS]] and [[MOS:CQ]] to keep formatting errors from [[WP:MP/E]]		-- [[WP:DYKMOS]] and [[MOS:CQ]] to keep formatting errors from [[WP:MP/E]]

	-- To disable this module for YOUR hook, add "<nowiki/>"		-- To disable most of this module for YOUR hook, add "<nowiki/>"

	-- This module doesn't show up on the main page or prevent promotion,		-- This module doesn't show up on the main page or prevent promotion,

216.58.24.27: fix example-serious detection

2025-03-28T07:41:27Z

fix example-serious detection

← Previous revision		Revision as of 07:41, 28 March 2025
Line 150:		Line 150:
	-- Checks to use for queues and complete prep areas		-- Checks to use for queues and complete prep areas
	local function check_queue(content)		local function check_queue(content)
	if content:find("example-serious%.jpg") then		if content:find("example%-serious%.jpg") then
	return "Image is still \"example-serious.jpg\""		return "Image is still \"example-serious.jpg\""
	end		end

216.58.24.27: replace missing space opt-out as requested by https://en.wikipedia.org/wiki/Wikipedia_talk:Did_you_know#c-RoySmith-20250328021900-Ganesha811-20250328015400

2025-03-28T07:35:26Z

replace missing space opt-out as requested by https://en.wikipedia.org/wiki/Wikipedia_talk:Did_you_know#c-RoySmith-20250328021900-Ganesha811-20250328015400

← Previous revision		Revision as of 07:35, 28 March 2025
Line 3:		Line 3:
	-- [[WP:DYKMOS]] and [[MOS:CQ]] to keep formatting errors from [[WP:MP/E]]		-- [[WP:DYKMOS]] and [[MOS:CQ]] to keep formatting errors from [[WP:MP/E]]

	-- To disable this module for YOUR hook, ~~edit it and replace~~ "* ..." ~~with "*..."~~		-- To disable this module for YOUR hook, add "<nowiki/>"

	-- This module doesn't show up on the main page or prevent promotion,		-- This module doesn't show up on the main page or prevent promotion,
Line 42:		Line 42:

	-- Per-hook checks that may have occasional exceptions		-- Per-hook checks that may have occasional exceptions
	local function check_hook(wikitextLine, expandedLine, ~~hookNumber~~)		local function check_hook(wikitextLine, expandedLine, hookNum)
	local byteCount = -1 -- Counting ends at the question mark		local byteCount = -1 -- Counting ends at the question mark
	if wikitextLine:find("''%([^)]+%)''") then		if wikitextLine:find("''%([^)]+%)''") then
Line 62:		Line 62:
	"It is currently " .. byteCount .. " characters"		"It is currently " .. byteCount .. " characters"
	end		end
	if ~~hookNumber~~ > 1 then		if hookNum > 1 then
	if wikitextLine:find("''%([^)]+%)''") then		if wikitextLine:find("''%([^)]+%)''") then
	return "[[WP:DYKIMG]]/[[WP:DYKMOS]]: Extra ''(pictured)'' or " ..		return "[[WP:DYKIMG]]/[[WP:DYKMOS]]: Extra ''(pictured)'' or " ..
	"alternative"		"alternative"
	end		end
	elseif ~~hookNumber~~ < 0 then		elseif hookNum < 0 then
	-- First hook might have been skipped by opt-out		-- First hook might have been skipped by opt-out
	else		else
Line 139:		Line 139:
	return "[[WP:DYKMOS]]: {{`}} (or {{`s}}) is for adjacent bold " ..		return "[[WP:DYKMOS]]: {{`}} (or {{`s}}) is for adjacent bold " ..
	"markup; {{'}} (or {{'s}}) is for adjacent italic (or bold italic)"		"markup; {{'}} (or {{'s}}) is for adjacent italic (or bold italic)"
		⚫	end
		⚫	if content:find("\n%*%.%.%.") then
		⚫	return "[[WP:DYKMOS]]: The three dots should be preceded by a space"
		⚫	end
		⚫	if content:find("\n%* %.%.%.that") then
			return "[[WP:DYKMOS]]: The three dots should be followed by a space"
	end		end
	end		end
Line 167:		Line 173:
	end		end

	local ~~hookCount~~ = 1		local hookNum = 1
⚫	if content:find("\n%*%.%.%.") then
⚫	-- Disable missing ''(pictured)'' check ~~as first hook might be skipped~~
⚫	~~hookCount~~ = -1
⚫	end
⚫	if content:find("\n%* %.%.%.that") then
⚫	return "[[WP:DYKMOS]]: The three dots should be ~~followed~~ by a space"
⚫	end

	local frame = mw.getCurrentFrame()		local frame = mw.getCurrentFrame()
	local foundEmptyHook = false
	-- Counting starts from after the space following the three dots		-- Counting starts from after the space following the three dots
		⚫	for wikitextLine in content:gmatch("\n%* %.%.%. ([^\n]*)") do
	-- A missing space in "* ..." = disable this module for that line
			-- "nowiki" anywhere = skip this hook
⚫	for wikitextLine in content:gmatch("\n%* %.%.%. ([^\n]*)") do
			if wikitextLine:find("nowiki") then
⚫	local expandedLine = ~~plainText(~~frame:preprocess(wikitextLine~~), false~~)
		⚫	-- Disable missing ''(pictured)'' check
⚫	expandedLine = decode(expandedLine)
			-- as first hook might have been skipped
		⚫	hookNum = -1
			else
		⚫	local expandedLine = frame:preprocess(wikitextLine)
			expandedLine = plainText(expandedLine, false)
		⚫	expandedLine = decode(expandedLine)
			local hookProblem = check_hook(wikitextLine, expandedLine, hookNum)
		⚫	if hookProblem then
		⚫	return hookProblem .. ": " .. expandedLine
		⚫	end

	~~local~~ ~~hookProblem~~ = ~~check_hook(~~wikitextLine~~, expandedLine, hookCount~~)		for page in wikitextLine:gmatch("%[%[([^]\|]+)\|?[^]]*%]%]") do
		⚫	local linkProblem = check_link(page)
⚫	if hookProblem then
		⚫	if linkProblem then
⚫	return hookProblem .. ": " .. expandedLine
		⚫	return linkProblem
⚫	end
		⚫	end
	if expandedLine:byte(-1) == 46 then -- Empty hook in prep area
		⚫	end
	foundEmptyHook = true
⚫	end

		⚫	if hookNum > 0 then
	for page in wikitextLine:gmatch("%[%[([^]\|]+)\|?[^]]*%]%]") do
			hookNum = hookNum + 1
⚫	local linkProblem = check_link(page)
⚫	if linkProblem then
⚫	return linkProblem
	end		end
⚫	end

⚫	if ~~hookCount~~ > 0 then
	hookCount = hookCount + 1
	end		end
	end		end

			-- No empty hooks ("... that ...") = queue or completed prep area
	if not foundEmptyHook then
			if not content:find("\n%* %.%.%. that %.%.%.\n") then
	local queueProblem = check_queue(content)		local queueProblem = check_queue(content)
	if queueProblem then		if queueProblem then