Module:Namespace detect - Revision history

Mr. Stradivarius: use demopage instead of page as the main "page" parameter

2014-04-05T17:01:23Z

use demopage instead of page as the main "page" parameter

← Previous revision		Revision as of 17:01, 5 April 2014
Line 72:		Line 72:
	local function getNamespace(args)		local function getNamespace(args)
	-- This function gets the namespace name from the page object.		-- This function gets the namespace name from the page object.
	local page = fetchValue(args, argKeys.~~page~~)		local page = fetchValue(args, argKeys.demopage)
	if page == '' then		if page == '' then
	page = nil		page = nil

Mr. Stradivarius: cache mappings using mw.loadData to increase performance, enable multiple parameter config values, switch p.main to Module:Arguments so arguments are only processed when necessary, streamline p._main code, and other minor fixes

2014-03-24T11:54:00Z

cache mappings using mw.loadData to increase performance, enable multiple parameter config values, switch p.main to Module:Arguments so arguments are only processed when necessary, streamline p._main code, and other minor fixes

Show changes

Mr. Stradivarius: fix the education program namespace bug by replacing underscores with spaces in the getNamespace function

2013-10-28T07:47:48Z

fix the education program namespace bug by replacing underscores with spaces in the getNamespace function

← Previous revision		Revision as of 07:47, 28 October 2013
Line 126:		Line 126:
	end		end
	end		end
			ret = mw.ustring.gsub(ret, '_', ' ')
	return mw.ustring.lower(ret)		return mw.ustring.lower(ret)
	end		end

Mr. Stradivarius: allow other modules to return false if it is specified, and fix an error in the comments for the namespace mappings

2013-10-26T08:53:38Z

allow other modules to return false if it is specified, and fix an error in the comments for the namespace mappings

← Previous revision		Revision as of 08:53, 26 October 2013
Line 71:		Line 71:
	names, in lower case, and the values are the possible parameter names for that namespace, also in		names, in lower case, and the values are the possible parameter names for that namespace, also in
	lower case. The table entries are structured like this:		lower case. The table entries are structured like this:
	~~[''] =~~ {		{
	{'main'},		[''] = {'main'},
		⚫	['wikipedia'] = {'wikipedia', 'project', 'wp'},
	},
			...
	['wikipedia'] = {
⚫	{'wikipedia', 'project', 'wp'}
	}		}
	]]		]]
Line 141:		Line 140:
	-- {{namespace detect}} checked these before alias names.		-- {{namespace detect}} checked these before alias names.
	for _, param in ipairs(params) do		for _, param in ipairs(params) do
	if args[param] then		if args[param] ~= nil then
	return args[param]		return args[param]
	end		end
Line 151:		Line 150:
	-- is not a valid namespace. Note that the parameter for the detected namespace must be		-- is not a valid namespace. Note that the parameter for the detected namespace must be
	-- completely absent for this to happen, not merely blank.		-- completely absent for this to happen, not merely blank.
	if args[cfg.other] then		if args[cfg.other] ~= nil then
	return args[cfg.other]		return args[cfg.other]
	end		end

Mr. Stradivarius: add the ability to use the subject namespace if on a talk page, switch indentation to tabs, and clean up some of the comments

2013-10-24T06:00:56Z

add the ability to use the subject namespace if on a talk page, switch indentation to tabs, and clean up some of the comments

Show changes

Mr. Stradivarius: allow access to the p._main() function from other modules, add an option to use the cfg.talk parameter in p.table(), and whitespace tweaks

2013-10-20T10:34:43Z

allow access to the p._main() function from other modules, add an option to use the cfg.talk parameter in p.table(), and whitespace tweaks

Show changes

Mr. Stradivarius: make some of the functions public for use in Module:Category handler, redo compare() code, use local variables instead of globals where possible

2013-07-01T11:01:43Z

make some of the functions public for use in Module:Category handler, redo compare() code, use local variables instead of globals where possible

Show changes

Mr. Stradivarius: Protected Module:Namespace detect: High-risk Lua module ([Edit=Block all non-admin users] (indefinite) [Move=Block all non-admin users] (indefinite))

2013-06-24T14:20:43Z

Protected Module:Namespace detect: High-risk Lua module ([Edit=Block all non-admin users] (indefinite) [Move=Block all non-admin users] (indefinite))

← Previous revision	Revision as of 14:20, 24 June 2013
(No difference)

Mr. Stradivarius: undo - on second thoughts, these failure modes are pretty arbitrary (bad titles actually default to "talk"), so changing them to both default to "other" is probably for the best

2013-06-24T14:10:40Z

undo - on second thoughts, these failure modes are pretty arbitrary (bad titles actually default to "talk"), so changing them to both default to "other" is probably for the best

← Previous revision		Revision as of 14:10, 24 June 2013
Line 98:		Line 98:
	ret = detectNamespace( getPageObject() )		ret = detectNamespace( getPageObject() )
	else		else
	-- ~~If pageObject is~~ nil, we ~~are~~ ~~either~~ ~~over~~ ~~the~~ ~~expensive~~		return nil -- return nil if the page object doesn't exist.
	-- function limit, or the page title is invalid. In both of
	-- these cases {{NAMESPACE}} defaults to the blank string, so
	-- we should do so too.
	ret = ''
	end		end
	end		end

Mr. Stradivarius: properly handle bad titles and pages exceeding the expensive function limit

2013-06-24T13:56:32Z

properly handle bad titles and pages exceeding the expensive function limit

← Previous revision		Revision as of 13:56, 24 June 2013
Line 98:		Line 98:
	ret = detectNamespace( getPageObject() )		ret = detectNamespace( getPageObject() )
	else		else
	~~return nil~~ -- ~~return~~ nil if ~~the~~ ~~page~~ ~~object~~ ~~doesn't~~ ~~exist.~~		-- If pageObject is nil, we are either over the expensive
			-- function limit, or the page title is invalid. In both of
			-- these cases {{NAMESPACE}} defaults to the blank string, so
			-- we should do so too.
			ret = ''
	end		end
	end		end