Module:Pagetype/sandbox - Revision history

Aidan9382: Make the error clearer when an impossible title gets passed

2024-11-12T19:30:54Z

Make the error clearer when an impossible title gets passed

← Previous revision		Revision as of 19:30, 12 November 2024
Line 243:		Line 243:
	if args.page then		if args.page then
	title = mw.title.new(args.page)		title = mw.title.new(args.page)
			if not title then
			error(string.format("Invalid page title '%s'", args.page))
			end
	else		else
	title = mw.title.getCurrentTitle()		title = mw.title.getCurrentTitle()

2024-10-10T22:35:45Z

Removed protection from "Module:Pagetype/sandbox": not needed

← Previous revision	Revision as of 22:35, 10 October 2024
(No difference)

2024-10-09T18:00:29Z

Protected "Module:Pagetype/sandbox": High-risk template or module: 36033 transclusions (more info) ([Edit=Require template editor access] (indefinite) [Move=Require template editor access] (indefinite))

← Previous revision	Revision as of 18:00, 9 October 2024
(No difference)

2024-05-21T11:57:04Z

check files exist (including commons)

← Previous revision		Revision as of 11:57, 21 May 2024
Line 161:		Line 161:
	exists = true		exists = true
	elseif title.namespace==8 and mw.message.new(title.text):exists() then		elseif title.namespace==8 and mw.message.new(title.text):exists() then
			exists = true
			elseif title.namespace==6 and title.fileExists then
	exists = true		exists = true
	end		end

2024-05-20T09:33:40Z

rmv namespace

← Previous revision		Revision as of 09:33, 20 May 2024
Line 160:		Line 160:
	if title.exists then -- not an article if it does not exist		if title.exists then -- not an article if it does not exist
	exists = true		exists = true
	elseif title.namespace==8 and mw.message.new(title.~~prefixedText~~):exists() then		elseif title.namespace==8 and mw.message.new(title.text):exists() then
	exists = true		exists = true
	end		end

2024-05-20T09:30:05Z

clearer code

← Previous revision		Revision as of 09:30, 20 May 2024
Line 157:		Line 157:
	return nil		return nil
	end		end
			local exists = false
⚫	~~if not (title.exists or~~ title.namespace==8 and mw.message.new(title.prefixedText):exists()) then ~~-- not an article if it does not exist~~
			if title.exists then -- not an article if it does not exist
			exists = true
		⚫	elseif title.namespace==8 and mw.message.new(title.prefixedText):exists() then
			exists = true
			end
			if not exists then
	if type(arg) == 'string' then		if type(arg) == 'string' then
	return arg		return arg

2024-05-20T09:26:41Z

treat valid message as though it exists

← Previous revision		Revision as of 09:26, 20 May 2024
Line 157:		Line 157:
	return nil		return nil
	end		end
	if not title.exists then -- not an article if it does not exist		if not (title.exists or title.namespace==8 and mw.message.new(title.prefixedText):exists()) then -- not an article if it does not exist
	if type(arg) == 'string' then		if type(arg) == 'string' then
	return arg		return arg

2024-05-15T09:50:20Z

simp

← Previous revision		Revision as of 09:50, 15 May 2024
Line 10:		Line 10:

	-- Load required modules.		-- Load required modules.
	local getArgs = require('Module:Arguments').getArgs
	local yesno = require('Module:Yesno')		local yesno = require('Module:Yesno')
	local p = {}		local p = {}
Line 263:		Line 262:

	function p.main(frame)		function p.main(frame)
	local args = getArgs(frame)		local args = require('Module:Arguments').getArgs(frame)
	return p._main(args)		return p._main(args)
	end		end

2024-05-14T13:43:09Z

simp

@@ Line 6: / Line 6: @@
 --                                                                            --
 --------------------------------------------------------------------------------
 -- Load config.
 local cfg = mw.loadData('Module:Pagetype/config/sandbox')
@@ Line 233: / Line 232: @@
 end
-local function getPageType(title, args)
+function p._main(args)
 		or nonExistent(title, args)
 		or parseContent(title, args, {
@@ Line 246: / Line 253: @@
 		or getNamespaceArgPageType(title, args)
 		or getOtherPageType(title, args)
 	if yesno(args.plural, false) then
 		pageType = pluralize(pageType)

2024-05-14T13:40:34Z

simp

← Previous revision		Revision as of 13:40, 14 May 2024
Line 249:		Line 249:
	end		end

		⚫	function p._main(args)
	-- Get the Scribunto title object to fetch the page type of
	local function getTitle(args)
	local title		local title
	if args.page then		if args.page then
	title = mw.title.new(args.page)		title = mw.title.new(args.page)
	if not title then
	return nil
	end
	else		else
	title = mw.title.getCurrentTitle()		title = mw.title.getCurrentTitle()
	end		end
	if not yesno(args.talk, true) and args[cfg.defaultns] ~= cfg.defaultnsAll then		if title and not yesno(args.talk, true) and args[cfg.defaultns] ~= cfg.defaultnsAll then
	~~return~~ title.subjectPageTitle		title = title.subjectPageTitle
	else
	return title
	end		end
	end

⚫	function p._main(args)
	local title = getTitle(args)
	local pageType = getPageType(title, args)		local pageType = getPageType(title, args)
	if yesno(args.plural, false) then		if yesno(args.plural, false) then