Module:Redirect: Difference between revisions

Module:Redirect (view source)

Revision as of 11:16, 10 September 2021

394 bytes removed , 10 September 2021

restore p.getTargetFromText which is used by Module:RfD which is causing "Lua error in Module:RfD at line 87: attempt to call upvalue 'getTargetFromText' (a nil value)"

Anonymous user

en>Johnuniq

@@ Line 17: / Line 17: @@
 -- redirect.
 function p.getTargetFromText(text)
-	return string.match(
+	local target = string.match(
 		text,
 		"^%s*#[Rr][Ee][Dd][Ii][Rr][Ee][Cc][Tt]%s*:?%s*%[%[([^%[%]|]-)%]%]"
@@ Line 24: / Line 24: @@
 		"^%s*#[Rr][Ee][Dd][Ii][Rr][Ee][Cc][Tt]%s*:?%s*%[%[([^%[%]|]-)|[^%[%]]-%]%]"
 	)
+	return target and mw.uri.decode(target, 'PATH')
 end
@@ Line 43: / Line 44: @@
 		), 2)
 	end
-	if not titleObj or not titleObj.isRedirect then
+	if not titleObj then
 		return nil
 	end
+	local targetTitle = titleObj.redirectTarget
-	-- Find the target by using string matching on the page content.
+	if targetTitle then
-	local target = p.getTargetFromText(titleObj:getContent() or "")
+		if fulltext then
-	if target then
+			return targetTitle.fullText
-		local targetTitle = getTitle(target)
-		if targetTitle then
-			if fulltext then
-				return targetTitle.fullText
-			else
-				return targetTitle.prefixedText
-			end
 		else
-			return nil
+			return targetTitle.prefixedText
 		end
 	else
-		-- The page is a redirect, but matching failed. This indicates a bug in
+		return nil
-		-- the redirect matching pattern, so throw an error.
-		error(string.format(
-			'could not parse redirect on page "%s"',
-			fulltext and titleObj.fullText or titleObj.prefixedText
-		))
 	end
 end
@@ Line 74: / Line 63: @@
 -- target page name, or the passed page name when not a redirect. The passed
 -- page name can be given as plain text or as a page link.
 --
 -- Returns page name as plain text, or when the bracket parameter is given, as a
 -- page link. Returns an error message when page does not exist or the redirect