Difference between revisions of "Module:Effective protection level"

Module:Effective protection level (view source)

Revision as of 15:05, 1 July 2014

12 bytes removed , 15:05, 1 July 2014

cleaner indentation and better title check

Anonymous user

>Jackmcbarn

@@ Line 5: / Line 5: @@
 function p._main(action, pagename)
 	local title
-	if type(pagename) == 'table' then
+	if type(pagename) == 'table' and pagename.prefixedText then
 		title = pagename
+	elseif pagename then
+		title = mw.title.new(pagename)
 	else
-		title = pagename and mw.title.new(pagename) or mw.title.getCurrentTitle()
+		title = mw.title.getCurrentTitle()
 	end
 	pagename = title.prefixedText
@@ Line 27: / Line 29: @@
 	elseif title.namespace == 2 and ( mw.ustring.find( pagename, '/.*%.js$') or mw.ustring.find( pagename, '/.*%.css$') ) then -- user .js or .css page
 		return 'sysop'
-	else
+	end
-		local level = title.protectionLevels[action] and title.protectionLevels[action][1]
+	local level = title.protectionLevels[action] and title.protectionLevels[action][1]
-		if level == 'sysop' then
+	if level == 'sysop' then
-			return 'sysop'
+		return 'sysop'
-		elseif mw.getCurrentFrame():callParserFunction('CASCADINGSOURCES', pagename) ~= '' then -- used by a cascading-protected page
+	elseif mw.getCurrentFrame():callParserFunction('CASCADINGSOURCES', pagename) ~= '' then -- used by a cascading-protected page
-			return 'sysop'
+		return 'sysop'
-		elseif level == 'templateeditor' then
+	elseif level == 'templateeditor' then
-			return 'templateeditor'
+		return 'templateeditor'
-		elseif action == 'move' then
+	elseif action == 'move' then
-			local blacklistentry = mw.ext.TitleBlacklist.test('edit', pagename) -- Testing action edit is correct, since this is for the source page. The target page name gets tested with action move.
+		local blacklistentry = mw.ext.TitleBlacklist.test('edit', pagename) -- Testing action edit is correct, since this is for the source page. The target page name gets tested with action move.
-			if blacklistentry and not blacklistentry.params.autoconfirmed then
+		if blacklistentry and not blacklistentry.params.autoconfirmed then
-				return 'accountcreator'
+			return 'accountcreator'
-			elseif title.namespace == 6 then
+		elseif title.namespace == 6 then
-				return 'filemover'
+			return 'filemover'
-			else
-				return 'autoconfirmed'
-			end
 		else
-			local blacklistentry = mw.ext.TitleBlacklist.test(action, pagename)
+			return 'autoconfirmed'
-			if blacklistentry then
-				return blacklistentry.params.autoconfirmed and 'autoconfirmed' or 'accountcreator'
-			elseif level then
-				return level
-			elseif action == 'upload' then
-				return 'autoconfirmed'
-			elseif action == 'create' and title.namespace % 2 == 0 and title.namespace ~= 118 then -- You need to be registered, but not autoconfirmed, to create non-talk pages other than drafts
-				return 'user'
-			else
-				return '*'
-			end
 		end
+	end
+	local blacklistentry = mw.ext.TitleBlacklist.test(action, pagename)
+	if blacklistentry then
+		return blacklistentry.params.autoconfirmed and 'autoconfirmed' or 'accountcreator'
+	elseif level then
+		return level
+	elseif action == 'upload' then
+		return 'autoconfirmed'
+	elseif action == 'create' and title.namespace % 2 == 0 and title.namespace ~= 118 then -- You need to be registered, but not autoconfirmed, to create non-talk pages other than drafts
+		return 'user'
+	else
+		return '*'
 	end
 end