Module:Check for deprecated parameters/sandbox - Revision history

Lemondoge: Simplifications

2023-04-08T20:39:00Z

Simplifications

← Previous revision		Revision as of 20:39, 8 April 2023
Line 17:		Line 17:
	local dep_values = {}		local dep_values = {}
	for k, v in pairs(args) do		for k, v in pairs(args) do
	if k == 'ignoreblank' or k == 'preview' or k == 'deprecated' then ~~else~~		if not (k == 'ignoreblank' or k == 'preview' or k == 'deprecated') then
	dep_values[k] = v		dep_values[k] = v
	end		end
Line 27:		Line 27:
	local pargs = frame:getParent().args		local pargs = frame:getParent().args
	for k, v in pairs(pargs) do		for k, v in pairs(pargs) do
	if ignoreblank then		if dep_values[k] and (ignoreblank or v ~= '') then
		⚫	table.insert(values, k)
	if dep_values[k] and v~='' then
⚫	table.insert(values, k)
	end
	else
	if dep_values[k] then
	table.insert(values, k)
	end
	end		end
	end		end
Line 40:		Line 34:
	-- add results to the output tables		-- add results to the output tables
	local res = {}		local res = {}
	if #values > 0 then		if #values ~= 0 then
	local preview = frame.args['preview'] or		local preview = frame.args['preview'] or
	'Page using [['..frame:getParent():getTitle()..']] with deprecated parameter _VALUE_'		'Page using [['..frame:getParent():getTitle()..']] with deprecated parameter _VALUE_'

Izno: make those inserts make sense again

2021-05-09T23:49:41Z

make those inserts make sense again

← Previous revision		Revision as of 23:49, 9 May 2021
Line 54:		Line 54:
	local category = frame.args['category']		local category = frame.args['category']
	for k, v in pairs(values) do		for k, v in pairs(values) do
	~~table.insert(res,~~ category:gsub('_VALUE_', v))		local c = category:gsub('_VALUE_', v)
		⚫	table.insert(res, c)
	if v == '' then		if v == '' then
	-- Fix odd bug for \| = which gets stripped to the empty string		-- Fix odd bug for \| = which gets stripped to the empty string
Line 60:		Line 61:
	v = ' '		v = ' '
	end		end
		⚫	local p = preview:gsub(
⚫	table.insert(
	~~res~~,		'_VALUE_',
		⚫	'"'..v..'". Replace with "'..dep_values[v]..'".'
⚫	preview:gsub(
	'_VALUE_',
⚫	'"'..v..'". Replace with "'..dep_values[v]..'".'
	)
	)		)
			table.insert(res, p )
	end		end
	end		end

Izno: ugh my brainnnnnn

2021-05-09T23:23:49Z

ugh my brainnnnnn

@@ Line 41: / Line 41: @@
 	local res = {}
 	if #values > 0 then
-		local preview = frame.args['preview'] or 'Page using [['..frame:getParent():getTitle()..']] with deprecated parameter _VALUE_'
+		local preview = frame.args['preview'] or
-		if isnotempty(preview) then
+			preview = require('Module:If preview')._warning({
 		end
-			end
+				v = ' '
 			end
 		end
 	end

Izno: typo

2021-05-07T16:47:50Z

typo

← Previous revision		Revision as of 16:47, 7 May 2021
Line 37:		Line 37:
	end		end
	end		end

	-- add ~~resuls~~ to the output tables		-- add results to the output tables
	local res = {}		local res = {}
	if #values > 0 then		if #values > 0 then

Izno: in fact, we can move that in here

2021-05-07T16:46:11Z

in fact, we can move that in here

← Previous revision		Revision as of 16:46, 7 May 2021
Line 36:		Line 36:
	end		end
	end		end
⚫	end

⚫	local preview = frame.args['preview'] or 'Page using [['..frame:getParent():getTitle()..']] with deprecated parameter _VALUE_'
⚫	local deprecated = frame.args['category']
⚫	if isnotempty(preview) then
⚫	preview = '<div class="hatnote" style="color:red"><strong>Warning:</strong> ' .. preview .. ' (this message is shown only in preview).</div>'
	end		end
	-- add resuls to the output tables		-- add resuls to the output tables
	local res = {}		local res = {}
	if #values > 0 then		if #values > 0 then
		⚫	local preview = frame.args['preview'] or 'Page using [['..frame:getParent():getTitle()..']] with deprecated parameter _VALUE_'
		⚫	local deprecated = frame.args['category']
		⚫	if isnotempty(preview) then
		⚫	preview = '<div class="hatnote" style="color:red"><strong>Warning:</strong> ' .. preview .. ' (this message is shown only in preview).</div>'
		⚫	end
	if frame:preprocess( "{{REVISIONID}}" ) == "" then		if frame:preprocess( "{{REVISIONID}}" ) == "" then
	deprecated = preview		deprecated = preview

Izno: preview is never nil here

2021-05-07T16:42:47Z

preview is never nil here

← Previous revision		Revision as of 16:42, 7 May 2021
Line 42:		Line 42:
	if isnotempty(preview) then		if isnotempty(preview) then
	preview = '<div class="hatnote" style="color:red"><strong>Warning:</strong> ' .. preview .. ' (this message is shown only in preview).</div>'		preview = '<div class="hatnote" style="color:red"><strong>Warning:</strong> ' .. preview .. ' (this message is shown only in preview).</div>'
	elseif preview == nil then
	preview = deprecated
	end		end
	-- add resuls to the output tables		-- add resuls to the output tables

Izno: init these much closer to use

2021-05-07T16:41:34Z

init these much closer to use

@@ Line 20: / Line 20: @@
 			dep_values[k] = v
 		end
 	end
@@ Line 45: / Line 37: @@
 		end
 	end
 	-- add resuls to the output tables
 	local res = {}

Izno: move stuff around to where it makes sense

2021-05-07T01:15:26Z

move stuff around to where it makes sense

← Previous revision		Revision as of 01:15, 7 May 2021
Line 13:		Line 13:

	function p.check (frame)		function p.check (frame)
		⚫	-- create the table of deprecated values and their matching new value
	local args = frame.args		local args = frame.args
⚫	local pargs = frame:getParent().args
⚫	local ignoreblank = isnotempty(frame.args['ignoreblank'])
⚫	local deprecated = frame.args['category']
⚫	local preview = frame.args['preview'] or 'Page using [['..frame:getParent():getTitle()..']] with deprecated parameter _VALUE_'

	local dep_values = {}		local dep_values = {}
⚫	local values = {}
⚫	local res = {}

⚫	-- create the table of deprecated values and their matching new value
	for k, v in pairs(args) do		for k, v in pairs(args) do
	if k == 'ignoreblank' or k == 'preview' or k == 'deprecated' then else		if k == 'ignoreblank' or k == 'preview' or k == 'deprecated' then else
Line 30:		Line 22:
	end		end

		⚫	local preview = frame.args['preview'] or 'Page using [['..frame:getParent():getTitle()..']] with deprecated parameter _VALUE_'
		⚫	local deprecated = frame.args['category']
	if isnotempty(preview) then		if isnotempty(preview) then
	preview = '<div class="hatnote" style="color:red"><strong>Warning:</strong> ' .. preview .. ' (this message is shown only in preview).</div>'		preview = '<div class="hatnote" style="color:red"><strong>Warning:</strong> ' .. preview .. ' (this message is shown only in preview).</div>'
Line 37:		Line 31:

	-- loop over the parent args and see if any are deprecated		-- loop over the parent args and see if any are deprecated
		⚫	local values = {}
		⚫	local ignoreblank = isnotempty(frame.args['ignoreblank'])
		⚫	local pargs = frame:getParent().args
	for k, v in pairs(pargs) do		for k, v in pairs(pargs) do
	if ignoreblank then		if ignoreblank then
Line 50:		Line 47:

	-- add resuls to the output tables		-- add resuls to the output tables
		⚫	local res = {}
	if #values > 0 then		if #values > 0 then
	if frame:preprocess( "{{REVISIONID}}" ) == "" then		if frame:preprocess( "{{REVISIONID}}" ) == "" then

Izno: make a sandy box

2021-05-07T01:11:15Z

make a sandy box

New page

-- This module may be used to compare the arguments passed to the parent
-- with a list of arguments, returning a specified result if an argument is
-- on the list
local p = {}

local function trim(s)
return s:match('^%s*(.-)%s*$')
end

local function isnotempty(s)
return s and trim(s) ~= ''
end

function p.check (frame)
local args = frame.args
local pargs = frame:getParent().args
local ignoreblank = isnotempty(frame.args['ignoreblank'])
local deprecated = frame.args['category']
local preview = frame.args['preview'] or 'Page using [['..frame:getParent():getTitle()..']] with deprecated parameter _VALUE_'

local dep_values = {}
local values = {}
local res = {}

-- create the table of deprecated values and their matching new value
for k, v in pairs(args) do
if k == 'ignoreblank' or k == 'preview' or k == 'deprecated' then else
dep_values[k] = v
end
end

if isnotempty(preview) then
preview = '<div class="hatnote" style="color:red"><strong>Warning:</strong> ' .. preview .. ' (this message is shown only in preview).</div>'
elseif preview == nil then
preview = deprecated
end

-- loop over the parent args and see if any are deprecated
for k, v in pairs(pargs) do
if ignoreblank then
if dep_values[k] and v~='' then
table.insert(values, k)
end
else
if dep_values[k] then
table.insert(values, k)
end
end
end

-- add resuls to the output tables
if #values > 0 then
if frame:preprocess( "{{REVISIONID}}" ) == "" then
deprecated = preview
for k, v in pairs(values) do
if v == '' then
-- Fix odd bug for | = which gets stripped to the empty string and
-- breaks category links
v = ' '
end
local r = deprecated:gsub('_VALUE_', ('"'..v..'". replace with "'..dep_values[v]..'"'))
table.insert(res, r)
end
else
for k, v in pairs(values) do
local r = deprecated:gsub('_VALUE_', v)
table.insert(res, r)
end
end
end

return table.concat(res)
end

return p