Module:WDL - Revision history

Paine Ellsworth: per edit request on talk page - fix totals background color

2025-03-30T22:08:39Z

per edit request on talk page - fix totals background color

← Previous revision		Revision as of 22:08, 30 March 2025
Line 50:		Line 50:
	function p.main(frame, otherargs)		function p.main(frame, otherargs)
	local args = otherargs or getArgs(frame)		local args = otherargs or getArgs(frame)
	local tableprefix = string.format('\| style="%stext-align:%s" \|', args.total and 'font-weight:bold;background:#~~efefef~~;' or '', args.align or 'center')		local tableprefix = string.format('\| style="%stext-align:%s" \|', args.total and 'font-weight:bold;background:#eaecf0;' or '', args.align or 'center')
	local retval = tableprefix .. total(frame, args[1], args[2], args[3], args[4], args.demospace and '' or '[[Category:WDL error]]') .. '\n'		local retval = tableprefix .. total(frame, args[1], args[2], args[3], args[4], args.demospace and '' or '[[Category:WDL error]]') .. '\n'
	retval = retval .. tableprefix .. frame:expandTemplate{title = 'Number table sorting', args = { args[2] }} .. '\n'		retval = retval .. tableprefix .. frame:expandTemplate{title = 'Number table sorting', args = { args[2] }} .. '\n'

Legoktm: Replace Module:No globals with require( "strict" )

2022-10-21T18:45:35Z

Replace Module:No globals with require( "strict" )

← Previous revision		Revision as of 18:45, 21 October 2022
Line 1:		Line 1:
	require('~~Module:No globals~~')		require('strict')

	local getArgs = require('Module:Arguments').getArgs		local getArgs = require('Module:Arguments').getArgs

Pppery: Remove use of Module:Decimals per TfD of Template:Decimals

2019-03-18T00:22:40Z

Remove use of Module:Decimals per TfD of Template:Decimals

← Previous revision		Revision as of 00:22, 18 March 2019
Line 2:		Line 2:

	local getArgs = require('Module:Arguments').getArgs		local getArgs = require('Module:Arguments').getArgs
	local roundAndPad = require('Module:~~Decimals~~').~~_main~~		local roundAndPad = require('Module:Math')._precision_format

	local p = {}		local p = {}

JJMC89: Changed protection level for "Module:WDL": Highly visible module: match template ([Edit=Require template editor access] (indefinite) [Move=Require template editor access] (indefinite))

2019-02-25T04:02:06Z

Changed protection level for "Module:WDL": Highly visible module: match template ([Edit=Require template editor access] (indefinite) [Move=Require template editor access] (indefinite))

← Previous revision	Revision as of 04:02, 25 February 2019
(No difference)

Plastikspork: Make it so we can call this from another module

2018-12-30T16:03:17Z

Make it so we can call this from another module

← Previous revision		Revision as of 16:03, 30 December 2018
Line 48:		Line 48:
	end		end

	function p.main(frame)		function p.main(frame, otherargs)
	local args = getArgs(frame)		local args = otherargs or getArgs(frame)
	local tableprefix = string.format('\| style="%stext-align:%s" \|', args.total and 'font-weight:bold;background:#efefef;' or '', args.align or 'center')		local tableprefix = string.format('\| style="%stext-align:%s" \|', args.total and 'font-weight:bold;background:#efefef;' or '', args.align or 'center')
	local retval = tableprefix .. total(frame, args[1], args[2], args[3], args[4], args.demospace and '' or '[[Category:WDL error]]') .. '\n'		local retval = tableprefix .. total(frame, args[1], args[2], args[3], args[4], args.demospace and '' or '[[Category:WDL error]]') .. '\n'

MusikAnimal: Protected "Module:WDL": High-risk Lua module; 1,000+ transclusions ([Edit=Require autoconfirmed or confirmed access] (indefinite))

2017-10-10T17:47:29Z

Protected "Module:WDL": High-risk Lua module; 1,000+ transclusions ([Edit=Require autoconfirmed or confirmed access] (indefinite))

← Previous revision	Revision as of 17:47, 10 October 2017
(No difference)

Frietjes: adjust decimal alignment and some logic for percent (see List of Real Madrid C.F. managers for an example), will also add more examples to the doc

2017-07-03T15:26:36Z

adjust decimal alignment and some logic for percent (see List of Real Madrid C.F. managers for an example), will also add more examples to the doc

← Previous revision		Revision as of 15:26, 3 July 2017
Line 24:		Line 24:
	local retval = ''		local retval = ''
	if winPercent < 10 then		if winPercent < 10 then
		⚫	retval = '<span style="visibility:hidden;color:transparent;">00</span>'
			elseif winPercent < 100 then
	retval = '<span style="visibility:hidden;color:transparent;">0</span>'		retval = '<span style="visibility:hidden;color:transparent;">0</span>'
	end		end
	~~retval =~~ retval .. frame:expandTemplate{title = 'Number table sorting', args = { roundAndPad(winPercent, decimals or 2) }}		return retval .. frame:expandTemplate{title = 'Number table sorting', args = { roundAndPad(winPercent, decimals or 2) }}
	if winPercent >= 100 then -- XXX: Should >= be ==? This should never be over 100, but that's what the original template had
⚫	~~return~~ retval .. '<span style="visibility:hidden;color:transparent;">0</span>'
	else
	return retval
	end
	end		end

Line 43:		Line 40:
	return '<span style="display:none">!</span>—'		return '<span style="display:none">!</span>—'
	end		end
	return displayWinPercent(frame, 100 * (won or 1) / ((won or 1) + (drawn or 0) + (lost or 0)), decimals) ~~-- XXX: Why is 1 used in the numerator instead of 0 if won isn't set? It's not like that in the case where played is set. It makes sense in the denominator to avoid a divide by zero~~		return displayWinPercent(frame, 100 * (won or 0) / (((won or 0) + (drawn or 0) + (lost or 0)) or 1), decimals)
	elseif tonumber(played) <= 0 then		elseif tonumber(played) <= 0 then
	return '<span style="display:none">!</span>—'		return '<span style="display:none">!</span>—'

Rolapib: Undid revision 784165209 by Rolapib (talk)

2017-06-06T20:54:56Z

Undid revision 784165209 by Rolapib (talk)

← Previous revision		Revision as of 20:54, 6 June 2017
Line 27:		Line 27:
	end		end
	retval = retval .. frame:expandTemplate{title = 'Number table sorting', args = { roundAndPad(winPercent, decimals or 2) }}		retval = retval .. frame:expandTemplate{title = 'Number table sorting', args = { roundAndPad(winPercent, decimals or 2) }}
	if winPercent > 100 then --~~preventing~~ ~~incorrect~~ ~~alignment~~ ~~when~~ = ~~100%~~ ~~(as~~ ~~per~~ ~~List~~ of ~~Real~~ ~~Madrid~~ ~~C.F.~~ ~~managers)~~		if winPercent >= 100 then -- XXX: Should >= be ==? This should never be over 100, but that's what the original template had
	return retval .. '<span style="visibility:hidden;color:transparent;">0</span>'		return retval .. '<span style="visibility:hidden;color:transparent;">0</span>'
	else		else

Rolapib at 20:53, 6 June 2017

2017-06-06T20:53:11Z

← Previous revision		Revision as of 20:53, 6 June 2017
Line 27:		Line 27:
	end		end
	retval = retval .. frame:expandTemplate{title = 'Number table sorting', args = { roundAndPad(winPercent, decimals or 2) }}		retval = retval .. frame:expandTemplate{title = 'Number table sorting', args = { roundAndPad(winPercent, decimals or 2) }}
	if winPercent >= 100 then -- ~~XXX:~~ ~~Should~~ ~~>= be~~ ==? ~~This~~ ~~should never be over 100,~~ ~~but~~ ~~that's~~ ~~what~~ ~~the~~ ~~original~~ ~~template~~ ~~had~~		if winPercent > 100 then --preventing incorrect alignment when = 100% (as per List of Real Madrid C.F. managers)
	return retval .. '<span style="visibility:hidden;color:transparent;">0</span>'		return retval .. '<span style="visibility:hidden;color:transparent;">0</span>'
	else		else

Jackmcbarn: compare with tonumber

2015-08-08T00:25:47Z

compare with tonumber

← Previous revision		Revision as of 00:25, 8 August 2015
Line 66:		Line 66:
	if args.diff == 'yes' then		if args.diff == 'yes' then
	if tonumber(args['for']) and tonumber(args.against) then		if tonumber(args['for']) and tonumber(args.against) then
	retval = retval .. tableprefix .. string.format('%s%d\n', args['for'] < args.against and '−' or '+', math.abs(args['for'] - args.against))		retval = retval .. tableprefix .. string.format('%s%d\n', tonumber(args['for']) < tonumber(args.against) and '−' or '+', math.abs(args['for'] - args.against))
	else		else
	retval = retval .. tableprefix .. '<span style="display:none">!</span>—\n'		retval = retval .. tableprefix .. '<span style="display:none">!</span>—\n'