Module:Portal - Revision history

Pppery: Reapply fixed version

2025-05-24T18:07:03Z

Reapply fixed version

← Previous revision		Revision as of 18:07, 24 May 2025
Line 205:		Line 205:
	end		end
	return root		return root
			end

			local function noviewer(portalImage)
			-- Function to add noviewer class to filespec for portalImage
			local portalImage, hasClass = mw.ustring.gsub(portalImage, "class%s*=[^%\|]+", "%0 noviewer")
			if hasClass == 0 then
			portalImage = portalImage.."\|class=noviewer"
			end
			return portalImage
	end		end

Line 252:		Line 261:
	if portalImage == defaultImage then		if portalImage == defaultImage then
	defaultUsed = portal		defaultUsed = portal
			else
			portalImage = noviewer(portalImage)
	end		end
	local image = string.format('[[File:%s\|32x28px~~\|class=noviewer~~]]',		local image = string.format('[[File:%s\|32x28px]]',
	portalImage)		portalImage)
	local link = string.format('[[Portal:%s\|%s%sportal]]',		local link = string.format('[[Portal:%s\|%s%sportal]]',
Line 278:		Line 289:
	-- Display the portals specified in the positional arguments.		-- Display the portals specified in the positional arguments.
	for _, fn in ipairs(imageList) do		for _, fn in ipairs(imageList) do
	local image = string.format('[[File:%s\|32x28px~~\|class=noviewer~~]]',fn)		local image = string.format('[[File:%s\|32x28px]]',noviewer(fn))
	local link = string.format('[[:File:%s\|%s]]',fn,fn)		local link = string.format('[[:File:%s\|%s]]',fn,fn)
	table.insert(contents,{image,link})		table.insert(contents,{image,link})
Line 294:		Line 305:
	local name = getImageName(portal)		local name = getImageName(portal)
	-- If keep is yes (or equivalent), then allow all metadata (like image borders) to be returned		-- If keep is yes (or equivalent), then allow all metadata (like image borders) to be returned
			-- also, add "noviewer" class to metadata
	local keepargs = yesno(keep)		local keepargs = yesno(keep)
			if keepargs then
			return noviewer(name)
			end
			-- otherwise, just keep filename, plus optional category
	local args = mw.text.split(name, "\|", true)		local args = mw.text.split(name, "\|", true)
	local result = {args[1]} -- the filename always comes first		local result = {args[1]} -- the filename always comes first
Line 301:		Line 317:
	for i = 2,#args do		for i = 2,#args do
	local m = mw.ustring.match(args[i], "^%scategory%s=")		local m = mw.ustring.match(args[i], "^%scategory%s=")
	if ~~keepargs or~~ m then		if m then
	table.insert(result, args[i])		table.insert(result, args[i])
	end		end

Giraffer: Undid revision 1292005673 by Pppery (talk) revert for now — this change is rendering errors in articlespace on multiple pages

2025-05-24T17:53:40Z

Undid revision 1292005673 by Pppery (talk) revert for now — this change is rendering errors in articlespace on multiple pages

← Previous revision		Revision as of 17:53, 24 May 2025
Line 205:		Line 205:
	end		end
	return root		return root
	end

	local function noviewer(portalImage)
	-- Function to add noviewer class to filespec for portalImage
	portalImage, hasClass = mw.ustring.gsub(portalImage, "class%s*=[^%\|]+", "%0 noviewer")
	if hasClass == 0 then
	portalImage = portalImage.."\|class=noviewer"
	end
	return portalImage
	end		end

Line 261:		Line 252:
	if portalImage == defaultImage then		if portalImage == defaultImage then
	defaultUsed = portal		defaultUsed = portal
	else
	portalImage = noviewer(portalImage)
	end		end
	local image = string.format('[[File:%s\|32x28px]]',		local image = string.format('[[File:%s\|32x28px\|class=noviewer]]',
	portalImage)		portalImage)
	local link = string.format('[[Portal:%s\|%s%sportal]]',		local link = string.format('[[Portal:%s\|%s%sportal]]',
Line 289:		Line 278:
	-- Display the portals specified in the positional arguments.		-- Display the portals specified in the positional arguments.
	for _, fn in ipairs(imageList) do		for _, fn in ipairs(imageList) do
	local image = string.format('[[File:%s\|32x28px]]',~~noviewer(~~fn))		local image = string.format('[[File:%s\|32x28px\|class=noviewer]]',fn)
	local link = string.format('[[:File:%s\|%s]]',fn,fn)		local link = string.format('[[:File:%s\|%s]]',fn,fn)
	table.insert(contents,{image,link})		table.insert(contents,{image,link})
Line 305:		Line 294:
	local name = getImageName(portal)		local name = getImageName(portal)
	-- If keep is yes (or equivalent), then allow all metadata (like image borders) to be returned		-- If keep is yes (or equivalent), then allow all metadata (like image borders) to be returned
	-- also, add "noviewer" class to metadata
	local keepargs = yesno(keep)		local keepargs = yesno(keep)
	if keepargs then
	return noviewer(name)
	end
	-- otherwise, just keep filename, plus optional category
	local args = mw.text.split(name, "\|", true)		local args = mw.text.split(name, "\|", true)
	local result = {args[1]} -- the filename always comes first		local result = {args[1]} -- the filename always comes first
Line 317:		Line 301:
	for i = 2,#args do		for i = 2,#args do
	local m = mw.ustring.match(args[i], "^%scategory%s=")		local m = mw.ustring.match(args[i], "^%scategory%s=")
	if m then		if keepargs or m then
	table.insert(result, args[i])		table.insert(result, args[i])
	end		end

Pppery: Sync from sandbox per request

2025-05-24T17:36:16Z

Sync from sandbox per request

← Previous revision		Revision as of 17:36, 24 May 2025
Line 205:		Line 205:
	end		end
	return root		return root
			end

			local function noviewer(portalImage)
			-- Function to add noviewer class to filespec for portalImage
			portalImage, hasClass = mw.ustring.gsub(portalImage, "class%s*=[^%\|]+", "%0 noviewer")
			if hasClass == 0 then
			portalImage = portalImage.."\|class=noviewer"
			end
			return portalImage
	end		end

Line 252:		Line 261:
	if portalImage == defaultImage then		if portalImage == defaultImage then
	defaultUsed = portal		defaultUsed = portal
			else
			portalImage = noviewer(portalImage)
	end		end
	local image = string.format('[[File:%s\|32x28px~~\|class=noviewer~~]]',		local image = string.format('[[File:%s\|32x28px]]',
	portalImage)		portalImage)
	local link = string.format('[[Portal:%s\|%s%sportal]]',		local link = string.format('[[Portal:%s\|%s%sportal]]',
Line 278:		Line 289:
	-- Display the portals specified in the positional arguments.		-- Display the portals specified in the positional arguments.
	for _, fn in ipairs(imageList) do		for _, fn in ipairs(imageList) do
	local image = string.format('[[File:%s\|32x28px~~\|class=noviewer~~]]',fn)		local image = string.format('[[File:%s\|32x28px]]',noviewer(fn))
	local link = string.format('[[:File:%s\|%s]]',fn,fn)		local link = string.format('[[:File:%s\|%s]]',fn,fn)
	table.insert(contents,{image,link})		table.insert(contents,{image,link})
Line 294:		Line 305:
	local name = getImageName(portal)		local name = getImageName(portal)
	-- If keep is yes (or equivalent), then allow all metadata (like image borders) to be returned		-- If keep is yes (or equivalent), then allow all metadata (like image borders) to be returned
			-- also, add "noviewer" class to metadata
	local keepargs = yesno(keep)		local keepargs = yesno(keep)
			if keepargs then
			return noviewer(name)
			end
			-- otherwise, just keep filename, plus optional category
	local args = mw.text.split(name, "\|", true)		local args = mw.text.split(name, "\|", true)
	local result = {args[1]} -- the filename always comes first		local result = {args[1]} -- the filename always comes first
Line 301:		Line 317:
	for i = 2,#args do		for i = 2,#args do
	local m = mw.ustring.match(args[i], "^%scategory%s=")		local m = mw.ustring.match(args[i], "^%scategory%s=")
	if ~~keepargs or~~ m then		if m then
	table.insert(result, args[i])		table.insert(result, args[i])
	end		end

Pppery: Changed protection settings for "Module:Portal": Up to full due to 9 million transclusions - the image modules are split by letter and can stay TPE ([Edit=Require administrator access] (indefinite) [Move=Require administrator access] (indefinite))

2024-11-25T21:42:52Z

Changed protection settings for "Module:Portal": Up to full due to 9 million transclusions - the image modules are split by letter and can stay TPE ([Edit=Require administrator access] (indefinite) [Move=Require administrator access] (indefinite))

← Previous revision	Revision as of 21:42, 25 November 2024
(No difference)

Hike395: strip File: and Image: in image data files

2024-10-01T12:33:57Z

strip File: and Image: in image data files

← Previous revision		Revision as of 12:33, 1 October 2024
Line 108:		Line 108:
	end		end
	s = mw.ustring.lower(s)		s = mw.ustring.lower(s)
	~~return~~ matchImagePage(s) or matchImagePage(getAlias(s)) or defaultImage		local image = matchImagePage(s) or matchImagePage(getAlias(s)) or defaultImage
			image = mw.ustring.gsub(image,'^File:','') --- strip mistaken leading File: or Image:
			image = mw.ustring.gsub(image,'^Image:','')
			return image
	end		end

	local function exists(title)		local function exists(title)
	local success, exists = pcall(function() return title.exists end)		local success, exists = pcall(function() return title.exists end)

Hike395: fix tracking logic

2024-05-23T01:44:10Z

fix tracking logic

← Previous revision		Revision as of 01:44, 23 May 2024
Line 255:		Line 255:
	table.insert(contents, {image, link})		table.insert(contents, {image, link})
	end		end
	if defaultUsed and checkTracking() then		if defaultUsed and args.tracking and checkTracking() then
	local cat = string.format('[[Category:Portal templates with default image\|%s]]',		local cat = string.format('[[Category:Portal templates with default image\|%s]]',
	defaultUsed)		defaultUsed)

Pppery: Avoid Lua error when out of expensive parser function calls

2023-10-04T22:49:12Z

Avoid Lua error when out of expensive parser function calls

← Previous revision		Revision as of 22:49, 4 October 2023
Line 109:		Line 109:
	s = mw.ustring.lower(s)		s = mw.ustring.lower(s)
	return matchImagePage(s) or matchImagePage(getAlias(s)) or defaultImage		return matchImagePage(s) or matchImagePage(getAlias(s)) or defaultImage
			end
			local function exists(title)
			local success, exists = pcall(function() return title.exists end)
			-- If success = false, then we're out of expensive parser function calls and can't check whether it exists
			-- in that case, don't throw a Lua error
			return not success or exists
	end		end

Line 154:		Line 160:
	local portalTitle = mw.title.new(portal,"Portal")		local portalTitle = mw.title.new(portal,"Portal")
	-- if portal exists, put it into list		-- if portal exists, put it into list
	if portalTitle and ~~portalTitle.~~exists then		if portalTitle and exists(portalTitle) then
	table.insert(existingPortals,portal)		table.insert(existingPortals,portal)
	-- otherwise set tracking cat		-- otherwise set tracking cat

Hike395: alphabetize "default image" tracking category

2023-04-17T15:55:14Z

alphabetize "default image" tracking category

← Previous revision		Revision as of 15:55, 17 April 2023
Line 237:		Line 237:
	local contents = {}		local contents = {}
	-- Display the portals specified in the positional arguments.		-- Display the portals specified in the positional arguments.
	local defaultUsed = ~~false~~		local defaultUsed = nil
	for _, portal in ipairs(portals) do		for _, portal in ipairs(portals) do
	local portalImage = getImageName(portal)		local portalImage = getImageName(portal)
	if portalImage == defaultImage then		if portalImage == defaultImage then
	defaultUsed = ~~true~~		defaultUsed = portal
	end		end
	local image = string.format('[[File:%s\|32x28px\|class=noviewer]]',		local image = string.format('[[File:%s\|32x28px\|class=noviewer]]',
Line 250:		Line 250:
	end		end
	if defaultUsed and checkTracking() then		if defaultUsed and checkTracking() then
	~~root:wikitext~~('[[Category:Portal templates with default image]]')		local cat = string.format('[[Category:Portal templates with default image\|%s]]',
			defaultUsed)
			root:wikitext(cat)
	end		end
	return tostring(fillBox(root, contents))		return tostring(fillBox(root, contents))

Hike395: cleanup/fix displayAll and imageDupes to remove stored image args

2023-03-10T10:20:59Z

cleanup/fix displayAll and imageDupes to remove stored image args

← Previous revision		Revision as of 10:20, 10 March 2023
Line 298:		Line 298:


	local function ~~getAllImageTables~~()		local function getAllImageTable()
	-- Returns an array containing all image subpages (minus aliases) as loaded by mw.loadData.		-- Returns an array containing all image subpages (minus aliases) as loaded by mw.loadData.
	local images = {}		local images = {}
	for i, subpage in ipairs{'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', 'other'} do		for i, subpage in ipairs{'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z', 'other'} do
	~~images[i]~~ = mw.loadData('Module:Portal/images/' .. subpage .. sandbox)		local imageTable = mw.loadData('Module:Portal/images/' .. subpage .. sandbox)
		⚫	for portal, image in pairs(imageTable) do
			local args = mw.text.split(image,"\|")
			images[portal] = args[1] -- just use image filename
		⚫	end
	end		end
	return images		return images
Line 312:		Line 316:
	-- names are capitalized, so the portal links may be broken.		-- names are capitalized, so the portal links may be broken.
	local lang = mw.language.getContentLanguage()		local lang = mw.language.getContentLanguage()
			portals = portals or {}
	local count = 1
	for ~~_, imageTable~~ in ~~ipairs~~(~~getAllImageTables~~()) do		for portal in pairs(getAllImageTable()) do
		⚫	table.insert(portals,lang:ucfirst(portal))
	for portal in pairs(imageTable) do
⚫	portals~~[count] =~~ lang:ucfirst(portal)
	count = count + 1
⚫	end
	end		end
			table.sort(portals)
			args.redlinks = args.redlinks or "yes"
	return p._portal(portals, args)		return p._portal(portals, args)
	end		end
Line 327:		Line 330:
	-- should be moved to a portal alias for ease of maintenance.		-- should be moved to a portal alias for ease of maintenance.
	local exists, dupes = {}, {}		local exists, dupes = {}, {}
	for _, ~~imageTable~~ in ~~ipairs~~(~~getAllImageTables~~()) do		for portal, image in pairs(getAllImageTable()) do
		⚫	if not exists[image] then
⚫	for portal, image in pairs(imageTable) do
	~~if not~~ exists[image] ~~then~~		exists[image] = portal
		⚫	else
⚫	exists[image] ~~= portal~~
		⚫	table.insert(dupes, string.format('The image "[[:File:%s\|%s]]" is used for both portals "%s" and "%s".', image, image, exists[image], portal))
⚫	else
⚫	table.insert(dupes, string.format('The image "[[:File:%s\|%s]]" is used for both portals "%s" and "%s".', image, image, exists[image], portal))
	end
	end		end
	end		end

Hike395: rm temporary tracking category --- it has done its job, and adds unwanted alt to images

2023-03-10T07:04:09Z

rm temporary tracking category --- it has done its job, and adds unwanted alt to images

← Previous revision		Revision as of 07:04, 10 March 2023
Line 288:		Line 288:
	-- parse name, looking for category arguments		-- parse name, looking for category arguments
	for i = 2,#args do		for i = 2,#args do
	local m = mw.ustring.match(args[i], "^%scategory%s=~~%s*(.+)~~")		local m = mw.ustring.match(args[i], "^%scategory%s=")
	if m then		if keepargs or m then
	category = category .. m
	elseif keepargs then
	table.insert(result, args[i])		table.insert(result, args[i])
	end		end
	end
	-- Temporary tracking category to clear out deleted portals
	local portalTitle = mw.title.new(portal,"Portal")
	if portalTitle and not portalTitle.exists then
	category = category .. "[[Category:Usage of portal image for non-existent portal]]"
	end		end
	-- reassemble arguments		-- reassemble arguments
	~~result =~~ table.concat(result,"\|")		return table.concat(result,"\|")
	-- reappend categories
	if #category > 0 then
	result = result .. "\|category=" .. category
	end
	return result
	end		end