Module:Sports table/argcheck/sandbox - Revision history

Lemondoge: simplified regex

2023-04-19T15:01:43Z

simplified regex

← Previous revision		Revision as of 15:01, 19 April 2023
Line 287:		Line 287:
	local unknown = {}		local unknown = {}

	local mstyle = mw.ustring.gsub(targs['style'] or '', '[%s_][%s_]*', ' ')		local mstyle = mw.ustring.gsub(targs['style'] or '', '[%s_]+', ' ')
	mstyle = tracked_styles[mstyle]		mstyle = tracked_styles[mstyle]

Line 355:		Line 355:
	for k,v in pairs(targs) do		for k,v in pairs(targs) do
	if type(k) == 'string' then		if type(k) == 'string' then
	if k:find('^team%d%d*$') then		if k:find('^team%d+$') then
	teams[v] = 1		teams[v] = 1
	targs[k] = ''		targs[k] = ''
Line 386:		Line 386:
	targs['hth_' .. v] = ''		targs['hth_' .. v] = ''
	end		end
	elseif k:find('^result%d%d$') or k:find('^._result%d%d*$') then		elseif k:find('^result%d+$') or k:find('^.*_result%d+$') then
	local pre = mw.ustring.gsub(k, '^(.)result%d%d$', '%1')		local pre = mw.ustring.gsub(k, '^(.*)result%d+$', '%1')
	results[v] = 1		results[v] = 1
	targs[k] = ''		targs[k] = ''
Line 453:		Line 453:
	end		end
	end		end
	if (good_args[k] or k:find('^split[%d][%d]*$')) then		if (good_args[k] or k:find('^split[%d]+$')) then
	-- Great!		-- Great!
	elseif k == 'showteam' then		elseif k == 'showteam' then

Lemondoge: Massively simplified logic

2023-04-07T19:51:25Z

Massively simplified logic

Show changes

Frietjes at 16:41, 13 March 2023

2023-03-13T16:41:46Z

← Previous revision		Revision as of 16:41, 13 March 2023
Line 15:		Line 15:

	local good_args = {		local good_args = {
	['article_section'] = 1,
	['away_goals_header'] = 1,		['away_goals_header'] = 1,
	['bonus_header'] = 1,		['bonus_header'] = 1,

Frietjes at 00:12, 7 December 2022

2022-12-07T00:12:08Z

← Previous revision		Revision as of 00:12, 7 December 2022
Line 11:		Line 11:
	possible_incorrect_winpoints = 'Category:Pages using sports table with possible incorrect winpoints',		possible_incorrect_winpoints = 'Category:Pages using sports table with possible incorrect winpoints',
	possibly_ignored_parameters = 'Category:Pages using sports table with possibly ignored parameters',		possibly_ignored_parameters = 'Category:Pages using sports table with possibly ignored parameters',
	unknown_parameters = 'Category:Pages using sports table with unknown parameters',		unknown_parameters = 'Category:Pages using sports table with unknown parameters'
	deprecated_parameters = 'Category:Pages using sports table with deprecated parameters'
	}		}

Line 81:		Line 80:
	['team_order_start'] = 1,		['team_order_start'] = 1,
	['teamwidth'] = 1,		['teamwidth'] = 1,
	['template_name'] = 1,
	['title'] = 1,		['title'] = 1,
	['title_source'] = 1,		['title_source'] = 1,
Line 283:		Line 281:
	local possibly_ignored = {}		local possibly_ignored = {}
	local unknown = {}		local unknown = {}
	local deprecated = {}

	local mstyle = mw.ustring.gsub(targs['style'] or '', '[%s_][%s_]*', ' ')		local mstyle = mw.ustring.gsub(targs['style'] or '', '[%s_][%s_]*', ' ')
Line 459:		Line 456:
	end		end
	end		end
	end
	if k == 'template_name' then
	table.insert(deprecated, k)
	end		end
	if found_arg == 0 and (good_args[k] or k:find('^split[%d][%d]*$')) then		if found_arg == 0 and (good_args[k] or k:find('^split[%d][%d]*$')) then
Line 566:		Line 560:
	validate_table_values(ignored, categories.ignored_parameters, bad_char, false)		validate_table_values(ignored, categories.ignored_parameters, bad_char, false)
	validate_table_values(possibly_ignored, categories.possibly_ignored_parameters, bad_char, true)		validate_table_values(possibly_ignored, categories.possibly_ignored_parameters, bad_char, true)
	validate_table_values(deprecated, categories.deprecated_parameters, bad_char, true)

	if possible_missing_winpoints > 1 then		if possible_missing_winpoints > 1 then

Frietjes at 17:16, 6 December 2022

2022-12-06T17:16:54Z

@@ Line 126: / Line 126: @@
 	['OTloss_after_loss'] = 1,
 	['tiebr_header'] = 1,
+}
@@ Line 184: / Line 203: @@
 	['OTwin'] = 1,
 	['tiebr'] = 1
+}
@@ Line 195: / Line 245: @@
 	['WL OT'] = 'PKOT',
 	['WL PK'] = 'PKOT',
-	['WL OTL tiebreak'] = 'PKOT'
+	['WL OTL tiebreak'] = 'PKOT',
+}
@@ Line 260: / Line 312: @@
 		end
 		for k, v in pairs(col_names_PKOT) do
 			col_names[k] = v ~= 0 and v or nil
 		end
@@ Line 391: / Line 461: @@
 				end
 				if k == 'template_name' then
-					if mw.ustring.match(v, '%#') then
+					table.insert(deprecated, k)
 				end
 				if found_arg == 0 and (good_args[k] or k:find('^split[%d][%d]*$')) then
@@ Line 412: / Line 478: @@
 					table.insert(ignored, k)
 				end
-				if found_arg == 0 and k:find('^[A-Za-z]*_.*$') then
+				if found_arg == 0 and k:find('^[A-Za-z3-5]*_.*$') then
-					local p = mw.ustring.gsub(k, '^([A-Za-z]*)_.*$', '%1')
+					local p = mw.ustring.gsub(k, '^([A-Za-z3-5]*)_.*$', '%1')
 					if col_names[p] then
 						-- Great!

Frietjes at 19:05, 5 December 2022

2022-12-05T19:05:30Z

Show changes

Frietjes at 14:58, 18 May 2021

2021-05-18T14:58:16Z

← Previous revision		Revision as of 14:58, 18 May 2021
Line 7:		Line 7:
	['draw_header'] = 1,		['draw_header'] = 1,
	['drawpoints'] = 1,		['drawpoints'] = 1,
			['float'] = 1,
	['for_against_style'] = 1,		['for_against_style'] = 1,
	['GB_team'] = 1,		['GB_team'] = 1,
Line 13:		Line 14:
	['hide_for_against_columns'] = 1,		['hide_for_against_columns'] = 1,
	['hide_played'] = 1,		['hide_played'] = 1,
			['hide_results'] = 1,
	['highest_pos'] = 1,		['highest_pos'] = 1,
	['legs'] = 1,		['legs'] = 1,
Line 26:		Line 28:
	['only_totals'] = 1,		['only_totals'] = 1,
	['only_fa_totals'] = 1,		['only_fa_totals'] = 1,
			['pct_style'] = 1,
	['pld_header'] = 1,		['pld_header'] = 1,
	['postitle'] = 1,		['postitle'] = 1,
Line 48:		Line 51:
	['show_rw'] = 1,		['show_rw'] = 1,
	['show_status'] = 1,		['show_status'] = 1,
			['show_totals'] = 1,
	['solid_cell'] = 1,		['solid_cell'] = 1,
	['sortable_table'] = 1,		['sortable_table'] = 1,
Line 57:		Line 61:
	['style'] = 1,		['style'] = 1,
	['table_header'] = 1,		['table_header'] = 1,
			['table_header_source'] = 1,
	['team_header'] = 1,		['team_header'] = 1,
	['team_order'] = 1,		['team_order'] = 1,
Line 63:		Line 68:
	['template_name'] = 1,		['template_name'] = 1,
	['title'] = 1,		['title'] = 1,
			['title_source'] = 1,
	['transcludesection'] = 1,		['transcludesection'] = 1,
	['update'] = 1,		['update'] = 1,
Line 71:		Line 77:
	['use_goals_scored'] = 1,		['use_goals_scored'] = 1,
	['use_point_percentage'] = 1,		['use_point_percentage'] = 1,
			['use_goals_per_match'] = 1,
			['use_points_per_match'] = 1,
	['use_tie'] = 1,		['use_tie'] = 1,
	['winpoints'] = 1,		['winpoints'] = 1,

Frietjes at 17:23, 9 February 2021

2021-02-09T17:23:43Z

← Previous revision		Revision as of 17:23, 9 February 2021
Line 59:		Line 59:
	['team_header'] = 1,		['team_header'] = 1,
	['team_order'] = 1,		['team_order'] = 1,
			['team_order_start'] = 1,
	['teamwidth'] = 1,		['teamwidth'] = 1,
	['template_name'] = 1,		['template_name'] = 1,

NHammen: sync

2020-04-06T22:49:23Z

sync

Show changes

Frietjes at 14:04, 26 November 2019

2019-11-26T14:04:01Z

@@ Line 205: / Line 205: @@
 	end
 	-- Step 1: Generate a team and result list
 	for k,v in pairs(targs) do