Module:Check for unknown parameters/sandbox - Revision history

Gonnym at 12:52, 16 January 2024

2024-01-16T12:52:41Z

← Previous revision		Revision as of 12:52, 16 January 2024
Line 109:		Line 109:
	end		end

	return table.concat(previews) .. table.concat(categories)		return table.concat(previews) .. "\n" .. table.concat(categories)
	end		end

Gonnym at 12:43, 16 January 2024

2024-01-16T12:43:01Z

← Previous revision		Revision as of 12:43, 16 January 2024
Line 89:		Line 89:

	-- add results to the output tables		-- add results to the output tables
	local ~~res~~ = {}		local previews = {}
			local categories = {}
	local unknown_category = args['unknown']		local unknown_category = args['unknown']
	local preview_text = args['preview'] or 'Found _VALUE_, '		local preview_text = args['preview'] or 'Found _VALUE_, '
Line 101:		Line 102:

	-- avoid error with v = 'example%2' ("invalid capture index")		-- avoid error with v = 'example%2' ("invalid capture index")
	local r = unknown_category:gsub('_VALUE_', {_VALUE_ = v})		local category = unknown_category:gsub('_VALUE_', {_VALUE_ = v})
	local r2 = preview_text:gsub('_VALUE_', {_VALUE_ = v})		local preview = preview_text:gsub('_VALUE_', {_VALUE_ = v})
	table.insert(~~res~~, r)		table.insert(categories, category)
	table.insert(~~res~~, r2)		table.insert(previews, preview)
	end		end
	end		end

	return table.concat(~~res~~)		return table.concat(previews) .. table.concat(categories)
	end		end

Gonnym: test

2024-01-16T12:39:38Z

test

← Previous revision		Revision as of 12:39, 16 January 2024
Line 15:		Line 15:
	-- Return text cleaned for display and truncated if too long.		-- Return text cleaned for display and truncated if too long.
	-- Strip markers are replaced with dummy text representing the original wikitext.		-- Strip markers are replaced with dummy text representing the original wikitext.
	local ~~pos,~~ truncated		local truncated
	local function truncate(text)		local function truncate(text)
	if truncated then		if truncated then
Line 27:		Line 27:
	end		end
	local parts = {}		local parts = {}
			local pos
	for before, tag, remainder in text:gmatch('([^\127])\127[^\127]%-(%l+)%-[^\127]*\127()') do		for before, tag, remainder in text:gmatch('([^\127])\127[^\127]%-(%l+)%-[^\127]*\127()') do
	pos = remainder		pos = remainder
Line 35:		Line 36:
	end		end

	function ~~p._check~~(args, pargs)		local function get_values(args, pargs)
⚫	if type(args) ~= "table" or type(pargs) ~= "table" then
⚫	-- TODO: error handling
⚫	return
⚫	end

	-- create the list of known args, regular expressions, and the return string		-- create the list of known args, regular expressions, and the return string
	local knownargs = {}		local knownargs = {}
Line 52:		Line 48:
	end		end
	end		end

	-- loop over the parent args, and make sure they are on the list		-- loop over the parent args, and make sure they are on the list
	local ignoreblank = isnotempty(args['ignoreblank'])		local ignoreblank = isnotempty(args['ignoreblank'])
Line 82:		Line 77:
	end		end
	end		end
			return values
			end

			function p._check(args, pargs)
		⚫	if type(args) ~= "table" or type(pargs) ~= "table" then
		⚫	-- TODO: error handling
		⚫	return
		⚫	end

			local values = get_values(args, pargs)

	-- add results to the output tables		-- add results to the output tables
	local res = {}		local res = {}
			local unknown_category = args['unknown']
		⚫	local preview_text = args['preview'] or 'Found _VALUE_, '
	if #values > 0 then		if #values > 0 then
		⚫	preview_text = require('Module:If preview')._warning({preview_text})
⚫	local ~~unknown_text~~ = args['~~unknown~~'] or 'Found _VALUE_, '

	if mw.getCurrentFrame():preprocess( "{{REVISIONID}}" ) == "" then
	local preview_text = args['preview']
	if isnotempty(preview_text) then
⚫	preview_text = require('Module:If preview')._warning({preview_text})
	elseif preview == nil then
	preview_text = unknown_text
⚫	end
	unknown_text = preview_text
	end
	for _, v in pairs(values) do		for _, v in pairs(values) do
	-- Fix odd bug for \| = which gets stripped to the empty string and		-- Fix odd bug for \| = which gets stripped to the empty string and
Line 103:		Line 101:

	-- avoid error with v = 'example%2' ("invalid capture index")		-- avoid error with v = 'example%2' ("invalid capture index")
	local r = ~~unknown_text~~:gsub('_VALUE_', {_VALUE_ = v})		local r = unknown_category:gsub('_VALUE_', {_VALUE_ = v})
			local r2 = preview_text:gsub('_VALUE_', {_VALUE_ = v})
	table.insert(res, r)		table.insert(res, r)
			table.insert(res, r2)
	end		end
	end		end

Gonnym: sync with live

2024-01-16T12:00:02Z

sync with live

← Previous revision		Revision as of 12:00, 16 January 2024
Line 2:		Line 2:
	-- with a list of arguments, returning a specified result if an argument is		-- with a list of arguments, returning a specified result if an argument is
	-- not on the list		-- not on the list
	require('strict');

	local p = {}		local p = {}

Line 10:		Line 8:
	end		end

	local function ~~hasvalue~~(s)		local function isnotempty(s)
	return s and s:match('%S')		return s and s:match('%S')
	end		end
Line 37:		Line 35:
	end		end

	function p._check(args, pargs~~, frame~~)		function p._check(args, pargs)
	if type(args) ~= "table" or type(pargs) ~= "table" then		if type(args) ~= "table" or type(pargs) ~= "table" then
	-- TODO: error handling		-- TODO: error handling
Line 46:		Line 44:
	local knownargs = {}		local knownargs = {}
	local regexps = {}		local regexps = {}
	local template_name = hasvalue (args.autofetch) and frame:getParent():getTitle() or nil;
	if template_name then -- if a template name is supplied
	local template_content = mw.title.new (template_name):getContent() or nil;
	if template_content then -- if we could get unparsed template content
	template_content = template_content:gsub ('<!%-%-.-%-%->', ''); -- strip html comments; any unique parameters in comments are not supported
	for param in template_content:gmatch ('{{{([^\|}]+)') do -- get the next parameter name from template
	param = mw.text.trim (param); -- trim wihtespace if any
	knownargs[param] = 1; -- save it
	end
	end
	-- mw.logObject (knownargs)
	end
	for k, v in pairs(args) do		for k, v in pairs(args) do
	if type(k) == 'number' then		if type(k) == 'number' then
Line 68:		Line 54:

	-- loop over the parent args, and make sure they are on the list		-- loop over the parent args, and make sure they are on the list
	local ignoreblank = ~~hasvalue~~(args['ignoreblank'])		local ignoreblank = isnotempty(args['ignoreblank'])
	local showblankpos = ~~hasvalue~~(args['showblankpositional'])		local showblankpos = isnotempty(args['showblankpositional'])
	local values = {}		local values = {}
	for k, v in pairs(pargs) do		for k, v in pairs(pargs) do
Line 80:		Line 66:
	end		end
	end		end
	if not knownflag and ( not ignoreblank or ~~hasvalue~~(v) ) then		if not knownflag and ( not ignoreblank or isnotempty(v) ) then
	table.insert(values, clean(k))		table.insert(values, clean(k))
	end		end
Line 91:		Line 77:
	end		end
	end		end
	if not knownflag and ( showblankpos or ~~hasvalue~~(v) ) then		if not knownflag and ( showblankpos or isnotempty(v) ) then
	table.insert(values, k .. ' = ' .. clean(v))		table.insert(values, k .. ' = ' .. clean(v))
	end		end
Line 104:		Line 90:
	if mw.getCurrentFrame():preprocess( "{{REVISIONID}}" ) == "" then		if mw.getCurrentFrame():preprocess( "{{REVISIONID}}" ) == "" then
	local preview_text = args['preview']		local preview_text = args['preview']
	if ~~hasvalue~~(preview_text) then		if isnotempty(preview_text) then
	preview_text = require('Module:If preview')._warning({preview_text})		preview_text = require('Module:If preview')._warning({preview_text})
	elseif ~~preview_text~~ == nil then		elseif preview == nil then
	preview_text = unknown_text		preview_text = unknown_text
	end		end
Line 128:		Line 114:
	local args = frame.args		local args = frame.args
	local pargs = frame:getParent().args		local pargs = frame:getParent().args
	return p._check(args, pargs~~, frame~~)		return p._check(args, pargs)
	end		end

Dinoguy1000: it's generally poor practice to name stuff with a negative, so change `isnotempty()` to `hasvalue()`

2023-10-09T22:05:47Z

it's generally poor practice to name stuff with a negative, so change `isnotempty()` to `hasvalue()`

← Previous revision		Revision as of 22:05, 9 October 2023
Line 10:		Line 10:
	end		end

	local function ~~isnotempty~~(s)		local function hasvalue(s)
	return s and s:match('%S')		return s and s:match('%S')
	end		end
Line 46:		Line 46:
	local knownargs = {}		local knownargs = {}
	local regexps = {}		local regexps = {}
	local template_name = ~~isnotempty~~ (args.autofetch) and frame:getParent():getTitle() or nil;		local template_name = hasvalue (args.autofetch) and frame:getParent():getTitle() or nil;
	if template_name then -- if a template name is supplied		if template_name then -- if a template name is supplied
	local template_content = mw.title.new (template_name):getContent() or nil;		local template_content = mw.title.new (template_name):getContent() or nil;
Line 68:		Line 68:

	-- loop over the parent args, and make sure they are on the list		-- loop over the parent args, and make sure they are on the list
	local ignoreblank = ~~isnotempty~~(args['ignoreblank'])		local ignoreblank = hasvalue(args['ignoreblank'])
	local showblankpos = ~~isnotempty~~(args['showblankpositional'])		local showblankpos = hasvalue(args['showblankpositional'])
	local values = {}		local values = {}
	for k, v in pairs(pargs) do		for k, v in pairs(pargs) do
Line 80:		Line 80:
	end		end
	end		end
	if not knownflag and ( not ignoreblank or ~~isnotempty~~(v) ) then		if not knownflag and ( not ignoreblank or hasvalue(v) ) then
	table.insert(values, clean(k))		table.insert(values, clean(k))
	end		end
Line 91:		Line 91:
	end		end
	end		end
	if not knownflag and ( showblankpos or ~~isnotempty~~(v) ) then		if not knownflag and ( showblankpos or hasvalue(v) ) then
	table.insert(values, k .. ' = ' .. clean(v))		table.insert(values, k .. ' = ' .. clean(v))
	end		end
Line 104:		Line 104:
	if mw.getCurrentFrame():preprocess( "{{REVISIONID}}" ) == "" then		if mw.getCurrentFrame():preprocess( "{{REVISIONID}}" ) == "" then
	local preview_text = args['preview']		local preview_text = args['preview']
	if ~~isnotempty~~(preview_text) then		if hasvalue(preview_text) then
	preview_text = require('Module:If preview')._warning({preview_text})		preview_text = require('Module:If preview')._warning({preview_text})
	elseif preview_text == nil then		elseif preview_text == nil then

حبيشان: fix:undeclared variable

2023-04-15T10:12:18Z

fix:undeclared variable

← Previous revision		Revision as of 10:12, 15 April 2023
Line 106:		Line 106:
	if isnotempty(preview_text) then		if isnotempty(preview_text) then
	preview_text = require('Module:If preview')._warning({preview_text})		preview_text = require('Module:If preview')._warning({preview_text})
	elseif ~~preview~~ == nil then		elseif preview_text == nil then
	preview_text = unknown_text		preview_text = unknown_text
	end		end

WOSlinker: use require('strict') instead of require('Module:No globals')

2022-10-21T09:54:56Z

use require('strict') instead of require('Module:No globals')

← Previous revision		Revision as of 09:54, 21 October 2022
Line 2:		Line 2:
	-- with a list of arguments, returning a specified result if an argument is		-- with a list of arguments, returning a specified result if an argument is
	-- not on the list		-- not on the list
	require('~~Module:No globals~~');		require('strict');

	local p = {}		local p = {}

Trappist the monk at 14:56, 5 September 2022

2022-09-05T14:56:59Z

← Previous revision		Revision as of 14:56, 5 September 2022
Line 50:		Line 50:
	local template_content = mw.title.new (template_name):getContent() or nil;		local template_content = mw.title.new (template_name):getContent() or nil;
	if template_content then -- if we could get unparsed template content		if template_content then -- if we could get unparsed template content
			template_content = template_content:gsub ('<!%-%-.-%-%->', ''); -- strip html comments; any unique parameters in comments are not supported
	for param in template_content:gmatch ('{{{([^\|}]+)') do -- get the next parameter name from template		for param in template_content:gmatch ('{{{([^\|}]+)') do -- get the next parameter name from template
			param = mw.text.trim (param); -- trim wihtespace if any
	knownargs[param] = 1; -- save it		knownargs[param] = 1; -- save it
	end		end

Trappist the monk at 13:07, 5 September 2022

2022-09-05T13:07:25Z

← Previous revision		Revision as of 13:07, 5 September 2022
Line 13:		Line 13:
	return s and s:match('%S')		return s and s:match('%S')
	end		end

	local function is_empty (s)
	return not (isnotempty(s))
	end


	local function clean(text)		local function clean(text)
Line 51:		Line 46:
	local knownargs = {}		local knownargs = {}
	local regexps = {}		local regexps = {}
	-- local template_name = isnotempty (args.template_name) and args.template_name or nil;
	local template_name = isnotempty (args.autofetch) and frame:getParent():getTitle() or nil;		local template_name = isnotempty (args.autofetch) and frame:getParent():getTitle() or nil;
	if template_name then -- if a template name is supplied		if template_name then -- if a template name is supplied
	-- template_name = template_name:find ('^[Tt]emplate:') and template_name or ('Template:' .. template_name) -- this test not necessary if we get template name from parent frame
	--TODO: assume that this module is only used in templates so no need to check template_name for Template: namespace prefix?
	--TODO: instead of reading the template wikitext, read the template's TemplateData and mw.text.jsonDecode()?
	local template_content = mw.title.new (template_name):getContent() or nil;		local template_content = mw.title.new (template_name):getContent() or nil;
	-- local template_content = mw.title.new ('Template:Infobox dog breed/sandbox'):getContent() or nil;
	if template_content then -- if we could get unparsed template content		if template_content then -- if we could get unparsed template content
	for param in template_content:gmatch ('{{{([^\|}]+)') do -- get the next parameter name from template		for param in template_content:gmatch ('{{{([^\|}]+)') do -- get the next parameter name from template
Line 64:		Line 54:
	end		end
	end		end
	mw.logObject (knownargs)		-- mw.logObject (knownargs)
	end		end
	for k, v in pairs(args) do		for k, v in pairs(args) do

Trappist the monk at 13:05, 5 September 2022

2022-09-05T13:05:45Z

← Previous revision		Revision as of 13:05, 5 September 2022
Line 42:		Line 42:
	end		end

	function p._check(args, pargs)		function p._check(args, pargs, frame)
	if type(args) ~= "table" or type(pargs) ~= "table" then		if type(args) ~= "table" or type(pargs) ~= "table" then
	-- TODO: error handling		-- TODO: error handling
Line 51:		Line 51:
	local knownargs = {}		local knownargs = {}
	local regexps = {}		local regexps = {}
	local template_name = isnotempty (args.template_name) and args.template_name or nil;		-- local template_name = isnotempty (args.template_name) and args.template_name or nil;
			local template_name = isnotempty (args.autofetch) and frame:getParent():getTitle() or nil;
	if template_name then -- if a template name is supplied		if template_name then -- if a template name is supplied
	template_name = template_name:find ('^[Tt]emplate:') and template_name or ('Template:' .. template_name)		-- template_name = template_name:find ('^[Tt]emplate:') and template_name or ('Template:' .. template_name) -- this test not necessary if we get template name from parent frame
			--TODO: assume that this module is only used in templates so no need to check template_name for Template: namespace prefix?

			--TODO: instead of reading the template wikitext, read the template's TemplateData and mw.text.jsonDecode()?
	local template_content = mw.title.new (template_name):getContent() or nil;		local template_content = mw.title.new (template_name):getContent() or nil;
	-- local template_content = mw.title.new ('Template:Infobox dog breed/sandbox'):getContent() or nil;		-- local template_content = mw.title.new ('Template:Infobox dog breed/sandbox'):getContent() or nil;
Line 62:		Line 64:
	end		end
	end		end
	-- mw.logObject (knownargs)		mw.logObject (knownargs)
	end		end
	for k, v in pairs(args) do		for k, v in pairs(args) do
Line 134:		Line 136:
	local args = frame.args		local args = frame.args
	local pargs = frame:getParent().args		local pargs = frame:getParent().args
	return p._check(args, pargs)		return p._check(args, pargs, frame)
	end		end