Module:If empty - Revision history

ZooWiki: 1 revision imported

2023-11-30T01:01:42Z

1 revision imported

← Older revision	Revision as of 01:01, 30 November 2023
(No difference)

Template>MSGJ: lastk is not needed

2023-01-26T18:33:28Z

lastk is not needed

← Older revision		Revision as of 18:33, 26 January 2023
Line 3:		Line 3:
	function p.main(frame)		function p.main(frame)
	local args = require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})		local args = require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})

	~~-- For backwards compatibility reasons, the first 8 parameters can be unset instead of being blank,~~
	~~-- even though there's really no legitimate use case for this. At some point, this will be removed.~~
	~~local lowestNil = math.huge~~
	~~for i = 8,1,-1 do~~
	~~if args[i] == nil then~~
	~~args[i] = ''~~
	~~lowestNil = i~~
	~~end~~
	~~end~~

	for k,v in ipairs(args) do		for k,v in ipairs(args) do
	if v ~= '' then		if v ~= '' then
	~~if lowestNil < k then~~
	~~-- If any uses of this template depend on the behavior above, add them to a tracking category.~~
	~~-- This is a rather fragile, convoluted, hacky way to do it, but it ensures that this module's output won't be modified~~
	~~-- by it.~~
	~~frame:extensionTag('ref', '[[Category:Instances of Template:If_empty missing arguments]]', {group = 'TrackingCategory'})~~
	~~frame:extensionTag('references', '', {group = 'TrackingCategory'})~~
	~~end~~
	return v		return v
	end		end
	end		end

	end		end

	return p		return p

Ookami: 24 revisions imported

2020-09-03T04:38:23Z

24 revisions imported

← Older revision	Revision as of 04:38, 3 September 2020
(No difference)

meta>Jackmcbarn: actually, only the first 8 need this check

2014-12-23T21:56:09Z

actually, only the first 8 need this check

← Older revision		Revision as of 21:56, 23 December 2014
Line 4:		Line 4:
	local args = require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})		local args = require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})

	-- For backwards compatibility reasons, the first 9 parameters can be unset instead of being blank,		-- For backwards compatibility reasons, the first 8 parameters can be unset instead of being blank,
	-- even though there's really no legitimate use case for this. At some point, this will be removed.		-- even though there's really no legitimate use case for this. At some point, this will be removed.
	local lowestNil = math.huge		local lowestNil = math.huge
	for i = 9,1,-1 do		for i = 8,1,-1 do
	if args[i] == nil then		if args[i] == nil then
	args[i] = ''		args[i] = ''

meta>Jackmcbarn: Protected Module:If empty: High-risk Lua module ([Edit=Allow only template editors and admins] (indefinite) [Move=Allow only template editors and admins] (indefinite))

2014-12-23T21:55:09Z

Protected Module:If empty: High-risk Lua module ([Edit=Allow only template editors and admins] (indefinite) [Move=Allow only template editors and admins] (indefinite))

← Older revision	Revision as of 21:55, 23 December 2014
(No difference)

meta>Jackmcbarn: add a tracking category to any pages that use this behavior

2014-12-23T21:37:01Z

add a tracking category to any pages that use this behavior

← Older revision		Revision as of 21:37, 23 December 2014
Line 6:		Line 6:
	-- For backwards compatibility reasons, the first 9 parameters can be unset instead of being blank,		-- For backwards compatibility reasons, the first 9 parameters can be unset instead of being blank,
	-- even though there's really no legitimate use case for this. At some point, this will be removed.		-- even though there's really no legitimate use case for this. At some point, this will be removed.
	for i = 1,9 do		local lowestNil = math.huge
			for i = 9,1,-1 do
	if args[i] == nil then		if args[i] == nil then
	args[i] = ''		args[i] = ''
			lowestNil = i
	end		end
	end		end

	for _,v in ipairs(args) do		for k,v in ipairs(args) do
	if v ~= '' then		if v ~= '' then
			if lowestNil < k then
			-- If any uses of this template depend on the behavior above, add them to a tracking category.
			-- This is a rather fragile, convoluted, hacky way to do it, but it ensures that this module's output won't be modified
			-- by it.
			frame:extensionTag('ref', '[[Category:Instances of Template:If_empty missing arguments]]', {group = 'TrackingCategory'})
			frame:extensionTag('references', '', {group = 'TrackingCategory'})
			end
	return v		return v
	end		end

meta>Jackmcbarn: add comment

2014-12-23T21:24:50Z

add comment

← Older revision		Revision as of 21:24, 23 December 2014
Line 3:		Line 3:
	function p.main(frame)		function p.main(frame)
	local args = require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})		local args = require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})

			-- For backwards compatibility reasons, the first 9 parameters can be unset instead of being blank,
			-- even though there's really no legitimate use case for this. At some point, this will be removed.
	for i = 1,9 do		for i = 1,9 do
	if args[i] == nil then		if args[i] == nil then
Line 8:		Line 11:
	end		end
	end		end

	for _,v in ipairs(args) do		for _,v in ipairs(args) do
	if v ~= '' then		if v ~= '' then

meta>Jackmcbarn: properly handle blanks acting like empties (I'm not sure it's a good idea to do this at all, but if it is, this is how you should do it)

2014-12-23T21:23:22Z

properly handle blanks acting like empties (I'm not sure it's a good idea to do this at all, but if it is, this is how you should do it)

← Older revision		Revision as of 21:23, 23 December 2014
Line 2:		Line 2:

	function p.main(frame)		function p.main(frame)
	~~for _,v in ipairs(~~require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})) do		local args = require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})
			for i = 1,9 do
			if args[i] == nil then
			args[i] = ''
			end
			end
			for _,v in ipairs(args) do
	if v ~= '' then		if v ~= '' then
	return v		return v

meta>Jackmcbarn: when frame args are relevant, they'll get picked up by Module:Arguments, and when they're not, that's just a waste of CPU cycles

2014-12-23T21:21:17Z

when frame args are relevant, they'll get picked up by Module:Arguments, and when they're not, that's just a waste of CPU cycles

← Older revision		Revision as of 21:21, 23 December 2014
Line 2:		Line 2:

	function p.main(frame)		function p.main(frame)
	~~for _,v in ipairs(frame.args) do~~
	~~if v ~= '' then~~
	~~return v~~
	~~end~~
	~~end~~
	for _,v in ipairs(require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})) do		for _,v in ipairs(require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})) do
	if v ~= '' then		if v ~= '' then

meta>Codehydro: simpliy; just noticed the "preview page with this module" box... will no longer be doing weird tests XD

2014-12-23T20:58:12Z

simpliy; just noticed the "preview page with this module" box... will no longer be doing weird tests XD

← Older revision		Revision as of 20:58, 23 December 2014
Line 2:		Line 2:

	function p.main(frame)		function p.main(frame)
	for ~~i=1~~,~~9,1~~ do		for _,v in ipairs(frame.args) do
	if ~~frame.args[i] ~= nil and frame.args[i]~~ ~= '' then		if v ~= '' then
	return ~~frame.args[i]~~		return v
	end		end
	end		end