Module:If empty - Revision history

MSGJ: lastk is not needed

2023-01-26T18:33:28Z

lastk is not needed

← Previous revision		Revision as of 18:33, 26 January 2023
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})

	local lastk = 0
	for k,v in ipairs(args) do		for k,v in ipairs(args) do
	if v ~= '' then		if v ~= '' then
	return v		return v
	end		end
	lastk = k
	end		end

Dinoguy1000: update from sandbox, per Template talk:If empty#Backcompat

2022-05-06T04:38:40Z

update from sandbox, per Template talk:If empty#Backcompat

@@ Line 4: / Line 4: @@
 	local args = require('Module:Arguments').getArgs(frame, {wrappers = 'Template:If empty', removeBlanks = false})
 	for k,v in ipairs(args) do
 		if v ~= '' then
 			return v
 		end
 	end
 end

Primefac: Changed protection settings for "Module:If empty": High-risk Lua module ([Edit=Require administrator access] (indefinite) [Move=Require administrator access] (indefinite))

2022-04-03T09:08:23Z

Changed protection settings for "Module:If empty": High-risk Lua module ([Edit=Require administrator access] (indefinite) [Move=Require administrator access] (indefinite))

← Previous revision	Revision as of 09:08, 3 April 2022
(No difference)

Jackmcbarn: actually, only the first 8 need this check

2014-12-23T21:56:09Z

actually, only the first 8 need this check

← Previous 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] = ''

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))

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

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

← Previous 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.
			local lowestNil = math.huge
	for i = 1,9 do		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

Jackmcbarn: add comment

2014-12-23T21:24:50Z

add comment

← Previous 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

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)

← Previous 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

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

← Previous 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

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

← Previous 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