Module:Television critical response - Revision history

Gonnym: when a multiseries, don't add to category if one of the series has only one row, that is a valid use here

2024-11-27T12:06:11Z

when a multiseries, don't add to category if one of the series has only one row, that is a valid use here

← Previous revision		Revision as of 12:06, 27 November 2024
Line 234:		Line 234:
	else		else
	-- One row entries, only categorized in the mainspace		-- One row entries, only categorized in the mainspace
	if SeasonCount == 1 then		if not args.series and SeasonCount == 1 then
	categories = categories .. '[[Category:Articles using Template:Television critical response with only one row]]'		categories = categories .. '[[Category:Articles using Template:Television critical response with only one row]]'
	end		end

WOSlinker: use require('strict') instead of require('Module:No globals')

2022-10-21T21:21:15Z

use require('strict') instead of require('Module:No globals')

← Previous revision		Revision as of 21:21, 21 October 2022
Line 1:		Line 1:
	-- This module implements {{Television critical response}}.		-- This module implements {{Television critical response}}.

	require('~~Module:No globals~~')		require('strict')
	local yesno = require('Module:Yesno')		local yesno = require('Module:Yesno')

Tholme: clear:right when float right set

2022-07-07T08:29:55Z

clear:right when float right set

← Previous revision		Revision as of 08:29, 7 July 2022
Line 106:		Line 106:
	if not args.multiseries then		if not args.multiseries then
	root:css('float', 'right')		root:css('float', 'right')
			root:css('clear', 'right')
	root:css('margin', '10px')		root:css('margin', '10px')
	end		end

Certes: Promote sandbox per WP:VPT discussion. # doesn't count string keys.

2022-05-09T21:01:35Z

Promote sandbox per WP:VPT discussion. # doesn't count string keys.

← Previous revision		Revision as of 21:01, 9 May 2022
Line 144:		Line 144:
	-- Extract seasons info and place into a 3D array		-- Extract seasons info and place into a 3D array
	local SeasonEntries = {}		local SeasonEntries = {}
			local SeasonCount = 0
	for k,v in pairs(args) do		for k,v in pairs(args) do
	local str, num, str2 = string.match(k, '([^%d])(%d)(%a*)')		local str, num, str2 = string.match(k, '([^%d])(%d)(%a*)')
Line 157:		Line 158:
	if not SeasonEntries[num] then		if not SeasonEntries[num] then
	SeasonEntries[num] = {}		SeasonEntries[num] = {}
			SeasonCount = SeasonCount + 1
	end		end
	SeasonEntries[num][str .. str2] = v		SeasonEntries[num][str .. str2] = v
Line 231:		Line 233:
	else		else
	-- One row entries, only categorized in the mainspace		-- One row entries, only categorized in the mainspace
	if ~~#SeasonEntries~~ == 1 then		if SeasonCount == 1 then
	categories = categories .. '[[Category:Articles using Template:Television critical response with only one row]]'		categories = categories .. '[[Category:Articles using Template:Television critical response with only one row]]'
	end		end

Alex 21: Apply changes per talk page

2021-07-24T04:22:18Z

Apply changes per talk page

← Previous revision		Revision as of 04:22, 24 July 2021
Line 121:		Line 121:
	-- Caption		-- Caption
	if not args.series then		if not args.series then
			local formatted_caption
⚫	if args.title and args.multiseries then

⚫	root:tag('caption'):wikitext(frame:expandTemplate{title='Screen reader-only',args={~~'Critical response of ' .. args.title~~}})
			if args.caption then
			formatted_caption = args.caption
		⚫	elseif args.title and args.multiseries then
			formatted_caption = 'Critical response of ' .. args.title
	elseif args.title then		elseif args.title then
	~~root:tag('caption'):wikitext(frame:expandTemplate{title~~=~~'Screen~~ ~~reader-only',args={~~'Critical response of <i>' .. args.title .. '</i>'~~}})~~		formatted_caption = 'Critical response of <i>' .. args.title .. '</i>'
	else		else
	~~root:tag('caption'):wikitext(frame:expandTemplate{title~~=~~'Screen~~ ~~reader-only',args={~~'Critical response of <i>' .. series_name .. '</i>'~~}})~~		formatted_caption = 'Critical response of <i>' .. series_name .. '</i>'
			end

			if args.show_caption then
			-- Visible caption option
			root:tag('caption'):wikitext(formatted_caption)
			elseif formatted_caption then
			-- If a visible caption isn't defined, then default to the screenreader-only caption
		⚫	root:tag('caption'):wikitext(frame:expandTemplate{title='Screen reader-only',args={formatted_caption}})
	end		end
	end		end

Alex 21 at 05:34, 16 January 2021

2021-01-16T05:34:54Z

← Previous revision		Revision as of 05:34, 16 January 2021
Line 304:		Line 304:
	end -- End 'do' season rows		end -- End 'do' season rows

	return tostring(root) .. categories		return (args.dontclose and mw.ustring.gsub(tostring(root), "</table>", "") or tostring(root)) .. categories
	end		end

Alex 21 at 05:31, 16 January 2021

2021-01-16T05:31:20Z

← Previous revision		Revision as of 05:31, 16 January 2021
Line 90:		Line 90:
	local hide_metacritic = yesno(args.hide_metacritic)		local hide_metacritic = yesno(args.hide_metacritic)
	local show_cinemascore = yesno(args.show_cinemascore)		local show_cinemascore = yesno(args.show_cinemascore)
	local ~~filmonly~~ = yesno(args.~~filmonly~~)		local hide_season = yesno(args.hide_season)
	local series_name = tostring(title):gsub("%s%((.-)%)","")		local series_name = tostring(title):gsub("%s%((.-)%)","")

Line 174:		Line 174:
	:attr('scope', 'col')		:attr('scope', 'col')
	:css('padding', cellPadding)		:css('padding', cellPadding)
	:wikitext(~~args.seriesT or~~ '~~Series~~')		:wikitext('Title')
	end		end

	-- Season header		-- Season header
	if not ~~filmonly~~ then		if not hide_season then
	headerRow:tag('th')		headerRow:tag('th')
	:attr('scope', 'col')		:attr('scope', 'col')
Line 184:		Line 184:
	:css('padding', cellPadding)		:css('padding', cellPadding)
	:addClass('unsortable')		:addClass('unsortable')
	:wikitext(args.seasonT or 'Season')		:wikitext(args.seriesT or args.seasonT or 'Season')
	end		end

Line 250:		Line 250:

	-- Season number link, included only in the first row		-- Season number link, included only in the first row
	if not ~~filmonly~~ then		if not hide_season then
	if k == '' or k == 'A' then		if k == '' or k == 'A' then
	seasonRow:tag(args.series and 'td' or 'th')		seasonRow:tag(args.series and 'td' or 'th')

Alex 21 at 05:24, 16 January 2021

2021-01-16T05:24:00Z

← Previous revision		Revision as of 05:24, 16 January 2021
Line 89:		Line 89:
	local hide_rotten_tomatoes = yesno(args.hide_rotten_tomatoes)		local hide_rotten_tomatoes = yesno(args.hide_rotten_tomatoes)
	local hide_metacritic = yesno(args.hide_metacritic)		local hide_metacritic = yesno(args.hide_metacritic)
	local ~~hide_cinemascore~~ = ~~(args.cinemascore and~~ yesno(args.~~cinemascore) or true~~)		local show_cinemascore = yesno(args.show_cinemascore)
	local filmonly = yesno(args.filmonly)		local filmonly = yesno(args.filmonly)
	local series_name = tostring(title):gsub("%s%((.-)%)","")		local series_name = tostring(title):gsub("%s%((.-)%)","")
Line 202:		Line 202:

	-- CinemaScore header		-- CinemaScore header
	if ~~not hide_cinemascore~~ then		if show_cinemascore then
	headerRow:tag('th')		headerRow:tag('th')
	:attr('scope', 'col')		:attr('scope', 'col')
Line 287:		Line 287:

	-- CinemaScore		-- CinemaScore
	if ~~not hide_cinemascore~~ and entry['cinemascore' .. k] ~= 'rotten_tomatoes' then		if show_cinemascore and entry['cinemascore' .. k] ~= 'rotten_tomatoes' then
	if entry['cinemascore' .. k] then		if entry['cinemascore' .. k] then
	local thisCell = CriticalResponse.season_cell(entry['cinemascore' .. k], frame)		local thisCell = CriticalResponse.season_cell(entry['cinemascore' .. k], frame)

Alex 21 at 00:36, 15 January 2021

2021-01-15T00:36:43Z

← Previous revision		Revision as of 00:36, 15 January 2021
Line 202:		Line 202:

	-- CinemaScore header		-- CinemaScore header
	if not ~~cinemascore~~ then		if not hide_cinemascore then
	headerRow:tag('th')		headerRow:tag('th')
	:attr('scope', 'col')		:attr('scope', 'col')

Alex 21: Adding new options/formatting

2021-01-15T00:36:12Z

Adding new options/formatting

← Previous revision		Revision as of 00:36, 15 January 2021
Line 89:		Line 89:
	local hide_rotten_tomatoes = yesno(args.hide_rotten_tomatoes)		local hide_rotten_tomatoes = yesno(args.hide_rotten_tomatoes)
	local hide_metacritic = yesno(args.hide_metacritic)		local hide_metacritic = yesno(args.hide_metacritic)
			local hide_cinemascore = (args.cinemascore and yesno(args.cinemascore) or true)
			local filmonly = yesno(args.filmonly)
	local series_name = tostring(title):gsub("%s%((.-)%)","")		local series_name = tostring(title):gsub("%s%((.-)%)","")

Line 172:		Line 174:
	:attr('scope', 'col')		:attr('scope', 'col')
	:css('padding', cellPadding)		:css('padding', cellPadding)
	:wikitext('Series')		:wikitext(args.seriesT or 'Series')
	end		end

	-- Season header		-- Season header
			if not filmonly then
⚫	headerRow:tag('th')
	:~~attr~~('~~scope', 'col~~')		headerRow:tag('th')
	:~~css~~('~~min-width~~', '~~50px~~')		:attr('scope', 'col')
	:css('~~padding~~', ~~cellPadding~~)		:css('min-width', '50px')
	:~~addClass~~('~~unsortable~~')		:css('padding', cellPadding)
			:addClass('unsortable')
	:wikitext(~~args.seriesT or~~ args.seasonT or 'Season')		:wikitext(args.seasonT or 'Season')
			end

	-- Rotten Tomatoes header		-- Rotten Tomatoes header
Line 195:		Line 199:
	:attr('scope', 'col')		:attr('scope', 'col')
	:wikitext('[[Metacritic]]')		:wikitext('[[Metacritic]]')
			end

			-- CinemaScore header
			if not cinemascore then
		⚫	headerRow:tag('th')
			:attr('scope', 'col')
			:wikitext('[[CinemaScore]]')
	end		end
	end		end
Line 228:		Line 239:

	-- New season row		-- New season row
	local seasonRow = (entry['rotten_tomatoes' .. k] or entry['metacritic' .. k]) and root:tag('tr') or mw.html.create('tr')		local seasonRow = (entry['rotten_tomatoes' .. k] or entry['metacritic' .. k] or entry['cinemascore' .. k]) and root:tag('tr') or mw.html.create('tr')

	-- Series name for group overviews		-- Series name for group overviews
Line 239:		Line 250:

	-- Season number link, included only in the first row		-- Season number link, included only in the first row
	if k ~~== '' or k == 'A'~~ then		if not filmonly then
	~~seasonRow:tag(args.series~~ ~~and~~ 'td' or 'th')		if k == '' or k == 'A' then
	:~~attr~~('~~scope~~', '~~row~~')		seasonRow:tag(args.series and 'td' or 'th')
	:attr('~~colspan~~', ~~entry.special or 1~~)		:attr('scope', 'row')
	:~~css~~('~~text-align~~', ~~'center'~~)		:attr('colspan', entry.special or 1)
			:css('text-align', 'center')
	:wikitext((entry.link and '[[' .. entry.link .. '\|' .. (entry.linkT or season) .. ']]' or (entry.linkT or season)) .. (entry.linkR or ''))		:wikitext((entry.link and '[[' .. entry.link .. '\|' .. (entry.linkT or season) .. ']]' or (entry.linkT or season)) .. (entry.linkR or ''))
			end
	end		end

Line 264:		Line 277:
	if entry['metacritic' .. k] then		if entry['metacritic' .. k] then
	local thisCell = CriticalResponse.season_cell(entry['metacritic' .. k], frame)		local thisCell = CriticalResponse.season_cell(entry['metacritic' .. k], frame)
			:css('padding',cellPadding)
			seasonRow:node(thisCell)
			else
			local infoCell = CriticalResponse.series_attributes( frame:expandTemplate{title='N/A'} )
			infoCell:css('padding',cellPadding)
			seasonRow:node(infoCell)
			end
			end

			-- CinemaScore
			if not hide_cinemascore and entry['cinemascore' .. k] ~= 'rotten_tomatoes' then
			if entry['cinemascore' .. k] then
			local thisCell = CriticalResponse.season_cell(entry['cinemascore' .. k], frame)
	:css('padding',cellPadding)		:css('padding',cellPadding)
	seasonRow:node(thisCell)		seasonRow:node(thisCell)