Module:Template parameter value/sandbox - Revision history

Ahecht: debug

2025-04-16T17:13:37Z

debug

← Previous revision		Revision as of 17:13, 16 April 2025
Line 111:		Line 111:

	local foundTemplates = 0		local foundTemplates = 0
			local foundTemplate
	for _,template in next,matchAllTemplates(content) do		for _,template in next,matchAllTemplates(content) do
	for _,wantedTemplate in pairs(templates) do		for _,wantedTemplate in pairs(templates) do
Line 122:		Line 123:
	end		end
	if string.match(template, "^{{%s"..wantedTemplate.."%s[\|}]") then		if string.match(template, "^{{%s"..wantedTemplate.."%s[\|}]") then
			foundTemplate = template
	foundTemplates = foundTemplates + 1		foundTemplates = foundTemplates + 1
	if foundTemplates == template_index then --Found our wanted template		if foundTemplates == template_index then --Found our wanted template
	return true, ~~template~~		return true, foundTemplate
	end		end
	end		end
Line 130:		Line 132:
	end		end
	if foundTemplates > 0 and template_index == -1 then		if foundTemplates > 0 and template_index == -1 then
	return true, ~~template~~		return true, foundTemplate
	end		end
	return false, "No valid template found"		return false, "No valid template found"

Ahecht: add -1 option for last occurrence of template

2025-04-16T17:08:56Z

add -1 option for last occurrence of template

← Previous revision		Revision as of 17:08, 16 April 2025
Line 128:		Line 128:
	end		end
	end		end
			end
			if foundTemplates > 0 and template_index == -1 then
			return true, template
	end		end
	return false, "No valid template found"		return false, "No valid template found"

Aidan9382: Undid revision 1211320205 by Aidan9382 (talk) actually, not sure this is actually a useful change. People can just .. the sides, and suddenly not having implicit full-match will be weird

2024-03-01T23:23:53Z

Undid revision 1211320205 by Aidan9382 (talk) actually, not sure this is actually a useful change. People can just .*.* the sides, and suddenly not having implicit full-match will be weird

← Previous revision		Revision as of 23:23, 1 March 2024
Line 121:		Line 121:
	wantedTemplate = "[" .. firstUpper .. firstLower .. "]" .. string.sub(wantedTemplate, 2)		wantedTemplate = "[" .. firstUpper .. firstLower .. "]" .. string.sub(wantedTemplate, 2)
	end		end
		⚫	if string.match(template, "^{{%s"..wantedTemplate.."%s[\|}]") then
	if not treat_as_regex then
	wantedTemplate = "^" .. wantedTemplate .. "$"
	end
	local templateName = string.match(template, "^{{%s(.-)%s[\|}]") or ""
⚫	if string.match(~~templateName~~, wantedTemplate) then
	foundTemplates = foundTemplates + 1		foundTemplates = foundTemplates + 1
	if foundTemplates == template_index then --Found our wanted template		if foundTemplates == template_index then --Found our wanted template

Aidan9382: Fix behaviour of ^ and $ if using treat_as_regex for matching template names (not releasing just yet, need to think if it would be smarter to just always force ^/$, since just treat_as_regex no longer being an implicit full-match might be weird)

2024-03-01T23:21:28Z

Fix behaviour of ^ and $ if using treat_as_regex for matching template names (not releasing just yet, need to think if it would be smarter to just always force ^/$, since just treat_as_regex no longer being an implicit full-match might be weird)

← Previous revision		Revision as of 23:21, 1 March 2024
Line 121:		Line 121:
	wantedTemplate = "[" .. firstUpper .. firstLower .. "]" .. string.sub(wantedTemplate, 2)		wantedTemplate = "[" .. firstUpper .. firstLower .. "]" .. string.sub(wantedTemplate, 2)
	end		end
			if not treat_as_regex then
⚫	if string.match(~~template~~, ~~"^{{%s"..~~wantedTemplate~~.."%s[\|}]"~~) then
			wantedTemplate = "^" .. wantedTemplate .. "$"
			end
			local templateName = string.match(template, "^{{%s(.-)%s[\|}]") or ""
		⚫	if string.match(templateName, wantedTemplate) then
	foundTemplates = foundTemplates + 1		foundTemplates = foundTemplates + 1
	if foundTemplates == template_index then --Found our wanted template		if foundTemplates == template_index then --Found our wanted template

Aidan9382: clarify

2024-02-23T23:26:59Z

clarify

← Previous revision		Revision as of 23:26, 23 February 2024
Line 164:		Line 164:

	--Template entry point. Returns either "yes" or nothing depending on if the wanted template is found		--Template entry point. Returns either "yes" or nothing depending on if the wanted template is found
			--Will return error text if no template is provided
	function p.hasTemplate(frame)		function p.hasTemplate(frame)
	local args = require('Module:Arguments').getArgs(frame)		local args = require('Module:Arguments').getArgs(frame)

Aidan9382: cleanup

2024-02-23T10:36:30Z

cleanup

← Previous revision		Revision as of 10:36, 23 February 2024
Line 101:		Line 101:
	-- (Modules can just provide a table so no concerns there)		-- (Modules can just provide a table so no concerns there)
	-- Comma splitting is a bad idea (lots of templates have a comma in their name)		-- Comma splitting is a bad idea (lots of templates have a comma in their name)
		⚫	templates = {templates}
	-- (Safe to remove for now since it's an unused feature afaict)
⚫	templates = {templates}~~--mw.text.split(templates, ", ?")~~
	end		end

Line 165:		Line 164:

	--Template entry point. Returns either "yes" or nothing depending on if the wanted template is found		--Template entry point. Returns either "yes" or nothing depending on if the wanted template is found
	--The return values can be changed via args 4 and 5 for yes and no respectively
	function p.hasTemplate(frame)		function p.hasTemplate(frame)
	local args = require('Module:Arguments').getArgs(frame)		local args = require('Module:Arguments').getArgs(frame)
Line 173:		Line 171:
	local template_index = tonumber(args[3] or args.N) or 1		local template_index = tonumber(args[3] or args.N) or 1
	if not template or template == "" then		if not template or template == "" then
	return '<span class="error">No template provided for hasTemplate</span>' ~~--is this a good idea?~~		return '<span class="error">No template provided for hasTemplate</span>'
	end		end
	local follow = yesno(args.follow) or false		local follow = yesno(args.follow) or false
Line 188:		Line 186:
	end		end

	--Template entry point. Returns an empty string upon failure		--Template entry point for getParameter. Returns an empty string upon failure
	function p.main(frame)		function p.main(frame)
	local args = require('Module:Arguments').getArgs(frame, {		local args = require('Module:Arguments').getArgs(frame, {

Aidan9382: implement follow at the template entry point

2024-02-17T09:54:33Z

implement follow at the template entry point

← Previous revision		Revision as of 09:54, 17 February 2024
Line 174:		Line 174:
	if not template or template == "" then		if not template or template == "" then
	return '<span class="error">No template provided for hasTemplate</span>' --is this a good idea?		return '<span class="error">No template provided for hasTemplate</span>' --is this a good idea?
			end
			local follow = yesno(args.follow) or false
			if follow then
			page = require("Module:Redirect").luaMain(page)
	end		end
	local options = {		local options = {

Aidan9382: this is excessive and probably not something we should allow to be done here (just {{#if:}} wrap it, makes it simpler)

2024-02-17T07:40:59Z

this is excessive and probably not something we should allow to be done here (just {{#if:}} wrap it, makes it simpler)

← Previous revision		Revision as of 07:40, 17 February 2024
Line 180:		Line 180:
	}		}

	local yes, no = "yes", ""
	if args.yes ~= nil then yes = args.yes end
	if args.no ~= nil then no = args.no end
	local success, _ = p.getTemplate(page, template, options)		local success, _ = p.getTemplate(page, template, options)
	return success and yes or no		return success and "yes" or ""
	end		end

Aidan9382: more parameter name stuff

2024-02-17T07:34:37Z

more parameter name stuff

← Previous revision		Revision as of 07:34, 17 February 2024
Line 171:		Line 171:
	local page = args[1] or args.page		local page = args[1] or args.page
	local template = args[2] or args.template		local template = args[2] or args.template
	local template_index = tonumber(args[3] or args.~~template_index~~) or 1		local template_index = tonumber(args[3] or args.N) or 1
	if not template or template == "" then		if not template or template == "" then
	return '<span class="error">No template provided for hasTemplate</span>' --is this a good idea?		return '<span class="error">No template provided for hasTemplate</span>' --is this a good idea?

Aidan9382: parameter name changes (avoid overusing positionals)

2024-02-16T21:45:20Z

parameter name changes (avoid overusing positionals)

← Previous revision		Revision as of 21:45, 16 February 2024
Line 181:		Line 181:

	local yes, no = "yes", ""		local yes, no = "yes", ""
	if args~~[4]~~ ~= nil then yes = args~~[4]~~ end		if args.yes ~= nil then yes = args.yes end
	if args~~[5]~~ ~= nil then no = args~~[5]~~ end		if args.no ~= nil then no = args.no end
	local success, _ = p.getTemplate(page, template, options)		local success, _ = p.getTemplate(page, template, options)
	return success and yes or no		return success and yes or no