Module:Infobox television - Revision history

Gonnym: new validations: external links as values; additional text in date fields

2025-03-03T13:51:05Z

new validations: external links as values; additional text in date fields

@@ Line 61: / Line 61: @@
 	if args.italic_title and args.italic_title ~= "no" then
 		return string.format(maintenance_categories.incorrectly_formatted, "italic_title")
 	end
 	return ""
@@ Line 112: / Line 140: @@
 	end
 	return ""
 end
@@ Line 169: / Line 227: @@
 		return maintenance_categories.dates_incorrectly_formatted
 	end
 	-- Check if start date is set to a future date.
 	if start_date ~= first_aired_future then
-		return is_start_date_in_the_future(start_date)
+		local category = is_start_date_in_the_future(start_date)
 	end
 	return ""
 end
@@ Line 927: / Line 1,004: @@
 	table.insert(categories, are_exclusive_parameter_sets_used(args))
 	local release_categories = is_release_information_formatted_correctly(args)
 	for _, v in ipairs(release_categories) do
         table.insert(categories, v)
 	end
 	table.insert(categories, is_italic_title_valid_value(args))

Gonnym: bugfix

2025-02-26T13:53:15Z

bugfix

← Previous revision		Revision as of 13:53, 26 February 2025
Line 108:		Line 108:

	-- The infobox does not allow for future dates.		-- The infobox does not allow for future dates.
	mw.log("current_timestamp: " .. current_timestamp)
	mw.log("start_date_timestamp: " .. extracted_date)
	if current_timestamp < start_date_timestamp then		if current_timestamp < start_date_timestamp then
	return maintenance_categories.dates_incorrectly_formatted		return maintenance_categories.dates_incorrectly_formatted
Line 296:		Line 294:
	local release_information = {"first_aired", "released", "last_aired", "network", "channel"}		local release_information = {"first_aired", "released", "last_aired", "network", "channel"}
	local categories = {}		local categories = {}

⚫
	for i = 1, number_of_network_sets do		for i = 1, number_of_network_sets do
	local num = ""		local num = ""
Line 307:		Line 305:
	numbered_args[v .. num] = args[v .. num]		numbered_args[v .. num] = args[v .. num]
	end		end
		⚫

	-- If current table is empty, break current cycle.		-- If current table is empty, break current cycle, but continue checking next tables instead of exiting completely.
			-- This is incorrect user usage, but we want to make sure that the data still passes validation, as it may be visible to readers.
	if next(numbered_args) then		if i == 1 or next(numbered_args) then
	table.insert(categories, does_release_information_have_extraneous_text(numbered_args))		table.insert(categories, does_release_information_have_extraneous_text(numbered_args))
	table.insert(categories, are_values_unlinked({network = args["network" .. num], channel = args["channel" .. num]}))		table.insert(categories, are_values_unlinked({network = args["network" .. num], channel = args["channel" .. num]}))

Gonnym: +exceptions

2025-01-29T11:49:02Z

+exceptions

← Previous revision		Revision as of 11:49, 29 January 2025
Line 465:		Line 465:
	"Booker",		"Booker",
	"Jack Trevor Story",		"Jack Trevor Story",
			"Tanveer Bookwala",
			"Sanford Bookstaver",
	}		}

Gonnym: commenting out language checking code

2024-08-19T06:21:08Z

commenting out language checking code

← Previous revision		Revision as of 06:21, 19 August 2024
Line 918:		Line 918:
	table.insert(categories, is_image_size_using_px(args.image_size))		table.insert(categories, is_image_size_using_px(args.image_size))
	--table.insert(categories, is_alt_name_in_italics(args.alt_name))		--table.insert(categories, is_alt_name_in_italics(args.alt_name))
	table.insert(categories, are_values_linked_or_formatted(args))		-- table.insert(categories, are_values_linked_or_formatted(args)) -- commenting out for now
	table.insert(categories, is_country_name_valid(args.country))		table.insert(categories, is_country_name_valid(args.country))
	table.insert(categories, has_flag_icon(args))		table.insert(categories, has_flag_icon(args))

Gonnym: fix

2024-08-18T13:05:50Z

fix

← Previous revision		Revision as of 13:05, 18 August 2024
Line 16:		Line 16:
	unnecessary_title_parameter = "[[Category:Pages using infobox television with unnecessary name parameter]]",		unnecessary_title_parameter = "[[Category:Pages using infobox television with unnecessary name parameter]]",
	unlinked_values = "[[Category:Pages using infobox television with unlinked values\|%s]]",		unlinked_values = "[[Category:Pages using infobox television with unlinked values\|%s]]",
	temp = "[[Category:Temp infobox television tracking category]]",		temp = "[[Category:Temp infobox television tracking category\|%s]]",
	}		}

Gonnym at 13:00, 18 August 2024

2024-08-18T13:00:45Z

← Previous revision		Revision as of 13:00, 18 August 2024
Line 537:		Line 537:
	if string.find(string.lower(credit), pattern) and not is_in_credit_exceptions(credit) then		if string.find(string.lower(credit), pattern) and not is_in_credit_exceptions(credit) then
	if invalid_credit == "lyric" or invalid_credit == "dialogue" then		if invalid_credit == "lyric" or invalid_credit == "dialogue" then
	return string.format(maintenance_categories.temp, ~~key~~)		return string.format(maintenance_categories.temp, invalid_credit)
	else		else
	return string.format(maintenance_categories.incorrectly_formatted, key)		return string.format(maintenance_categories.incorrectly_formatted, key)

Gonnym: fix

2024-08-18T11:45:29Z

fix

← Previous revision		Revision as of 11:45, 18 August 2024
Line 416:		Line 416:
	end		end

	--- Returns a maintenance category if \|based_on uses ~~one~~ of ~~the follows~~ instead		--- Returns a maintenance category if \|based_on uses a generic word instead
	--- of the title of the original work:		--- of the title of the original work:
⚫	--- "novel"
	--- "book"		--- "book"
		⚫	--- "novel"
	---		---
	--- Infobox parameters checked:		--- Infobox parameters checked:
Line 427:		Line 427:
	--- @return string		--- @return string
	local function is_based_on_used_correctly(based_on)		local function is_based_on_used_correctly(based_on)
			if not based_on then
⚫	local ~~incomplete~~ = {
	"~~novel~~",		return ""
			end

		⚫	local generic_titles = {
	"book",		"book",
			"novel",
	}		}

			-- Remove content enclosed by double single quotes (''text'')
	local delink = require("Module:Delink")._delink
			local stripped_text = based_on:gsub("''[^']*''", "")
⚫	for _, word in pairs(~~incomplete~~) do
			-- Remove content enclosed by double quotes ("text")
⚫	if string.find(~~string.lower(delink{based_on})~~, word) then
			stripped_text = based_on:gsub('"[^"]*"', "")
			-- Remove content enclosed by double square brackets ([[text]])
			stripped_text = based_on:gsub("%[%[.-%]%]", "")

			-- Convert to lowercase to ensure case-insensitive matching
			stripped_text = stripped_text:lower()

		⚫	for _, word in pairs(generic_titles) do
		⚫	if string.find(stripped_text, word) then
	return string.format(maintenance_categories.incorrectly_formatted, "based_on")		return string.format(maintenance_categories.incorrectly_formatted, "based_on")
	end		end

Gonnym: added |director= to the credits check; added a function that checks if based_on= uses a generic title instead of the actual title of the work

2024-08-18T09:37:47Z

added |director= to the credits check; added a function that checks if based_on= uses a generic title instead of the actual title of the work

@@ Line 413: / Line 413: @@
 	end
 	return ""
 end
@@ Line 443: / Line 469: @@
 ---
 --- Infobox parameters checked:
 --- - |editor=
 --- - |executive_producer=
@@ Line 455: / Line 482: @@
 local function is_credit_used_correctly(args)
 	local credits_list = {
 		"editor",
 		"executive_producer",
@@ Line 881: / Line 909: @@
 	table.insert(categories, has_flag_icon(args))
 	table.insert(categories, is_credit_used_correctly(args))
 	table.insert(categories, does_num_episodes_have_extraneous_text(args.num_episodes))
 	table.insert(categories, are_exclusive_parameter_sets_used(args))

Gonnym: updating image check to see if it uses a URL

2024-08-08T17:47:29Z

updating image check to see if it uses a URL

← Previous revision		Revision as of 17:47, 8 August 2024
Line 480:		Line 480:
	"on%-line",		"on%-line",
	"prod%.",		"prod%.",
⚫	"script",
	"supervising",		"supervising",
		⚫	"script",
	"screenplay",		"screenplay",
	"story",		"story",
Line 615:		Line 615:
	end		end

	--- Returns a maintenance category if the \|image= value includes the "File:" or "Image:" prefix.		--- Returns a maintenance category if the \|image= value includes the "File:"
			--- or "Image:" prefix, or if it is a URL.
	---		---
	--- Infobox parameters checked:		--- Infobox parameters checked:
Line 627:		Line 628:
	end		end

			local invalid_strings = {
	if string.find(image, "[Ff]ile:") or string.find(image, "[Ii]mage:") then
			"file:",
⚫	return string.format(maintenance_categories.incorrectly_formatted, "image")
			"image:",
			"http:",
			"https:",
			}

			image = string.lower(image)
			for _, invalid in ipairs(invalid_strings) do
			if string.find(image, invalid) then
		⚫	return string.format(maintenance_categories.incorrectly_formatted, "image")
			end
	end		end

Gonnym: fix

2024-08-07T12:57:15Z

fix

← Previous revision		Revision as of 12:57, 7 August 2024
Line 458:		Line 458:
	"executive_producer",		"executive_producer",
	"producer",		"producer",
⚫	"script",
	"screenplay",		"screenplay",
	"story",		"story",
Line 481:		Line 480:
	"on%-line",		"on%-line",
	"prod%.",		"prod%.",
		⚫	"script",
	"supervising",		"supervising",
	"screenplay",		"screenplay",