Module:URL/sandbox - Revision history

Paine Ellsworth: sync to live module

2022-09-11T18:57:54Z

sync to live module

← Previous revision		Revision as of 18:57, 11 September 2022
Line 54:		Line 54:
	-- If it's an HTTP[S] URL without the double slash, fix it.		-- If it's an HTTP[S] URL without the double slash, fix it.
	url = mw.ustring.gsub(url, '^[Hh][Tt][Tt][Pp]([Ss]?):(/?)([^/])', 'http%1://%3')		url = mw.ustring.gsub(url, '^[Hh][Tt][Tt][Pp]([Ss]?):(/?)([^/])', 'http%1://%3')

	-- Handle URLs from Wikidata of the format http://
	url = mw.ustring.gsub(url, '^[Hh][Tt][Tt][Pp]([Ss]?)://', 'http%1://')

	local uri = safeUri(url)		local uri = safeUri(url)

RexxS: operator precedence

2021-01-03T13:35:03Z

operator precedence

← Previous revision		Revision as of 13:35, 3 January 2021
Line 124:		Line 124:
	-- truncate anything after a space		-- truncate anything after a space
	url = url:gsub("%%20", " "):gsub(" .*", "")		url = url:gsub("%%20", " "):gsub(" .*", "")
	return p._url(url, text, msg) or "" .. penicon or ""		return (p._url(url, text, msg) or "") .. (penicon or "")
	end		end

RexxS: _url may return nil

2021-01-03T13:34:10Z

_url may return nil

← Previous revision		Revision as of 13:34, 3 January 2021
Line 124:		Line 124:
	-- truncate anything after a space		-- truncate anything after a space
	url = url:gsub("%%20", " "):gsub(" .*", "")		url = url:gsub("%%20", " "):gsub(" .*", "")
	return p._url(url, text, msg) .. (penicon or "")		return p._url(url, text, msg) or "" .. penicon or ""
	end		end

RexxS: try to capture pen icon and append it later

2021-01-03T13:30:40Z

try to capture pen icon and append it later

← Previous revision		Revision as of 13:30, 3 January 2021
Line 116:		Line 116:
	local msg = templateArgs.msg or parentArgs.msg or 'no'		local msg = templateArgs.msg or parentArgs.msg or 'no'
	url = url or extractUrl(templateArgs) or extractUrl(parentArgs) or ''		url = url or extractUrl(templateArgs) or extractUrl(parentArgs) or ''
			-- if the url came from a Wikidata call, it might have a pen icon appended
			-- we want to keep that and add it back at the end.
			local u1, penicon = mw.ustring.match( url, "(.)( <span class='penicon.)" )
			if penicon then url = u1 end
	-- strip out html tags and [ ] from url		-- strip out html tags and [ ] from url
	url = (url or ''):gsub("<[^>]*>", ""):gsub("[%[%]]", "")		url = (url or ''):gsub("<[^>]*>", ""):gsub("[%[%]]", "")
	-- truncate anything after a space		-- truncate anything after a space
	url = url:gsub("%%20", " "):gsub(" .*", "")		url = url:gsub("%%20", " "):gsub(" .*", "")
	return p._url(url, text, msg)		return p._url(url, text, msg) .. (penicon or "")
	end		end

Pppery: Undid revision 907987740 by Pppery (talk)

2019-07-26T16:43:14Z

Undid revision 907987740 by Pppery (talk)

← Previous revision		Revision as of 16:43, 26 July 2019
Line 17:		Line 17:
	local function extractUrl(args)		local function extractUrl(args)
	for name, val in pairs(args) do		for name, val in pairs(args) do
	if name ~= "msg" then		if name ~= 2 and name ~= "msg" then
	local url = name .. "=" .. val;		local url = name .. "=" .. val;
	url = mw.ustring.gsub(url, '^[Hh][Tt][Tt][Pp]([Ss]?):(/?)([^/])', 'http%1://%3')		url = mw.ustring.gsub(url, '^[Hh][Tt][Tt][Pp]([Ss]?):(/?)([^/])', 'http%1://%3')

Pppery: Partial revert; this condition can only be true in contrived cases like {{URL|https://example.com?key=value|2=text}}, where the template need not produce a sensible result

2019-07-26T16:40:23Z

Partial revert; this condition can only be true in contrived cases like {{URL|https://example.com?key=value|2=text}}, where the template need not produce a sensible result

← Previous revision		Revision as of 16:40, 26 July 2019
Line 17:		Line 17:
	local function extractUrl(args)		local function extractUrl(args)
	for name, val in pairs(args) do		for name, val in pairs(args) do
	if ~~name ~= 2 and~~ name ~= "msg" then		if name ~= "msg" then
	local url = name .. "=" .. val;		local url = name .. "=" .. val;
	url = mw.ustring.gsub(url, '^[Hh][Tt][Tt][Pp]([Ss]?):(/?)([^/])', 'http%1://%3')		url = mw.ustring.gsub(url, '^[Hh][Tt][Tt][Pp]([Ss]?):(/?)([^/])', 'http%1://%3')

Pppery: Fix link to testcases

2019-07-26T16:28:38Z

Fix link to testcases

← Previous revision		Revision as of 16:28, 26 July 2019
Line 2:		Line 2:
	-- This module implements {{URL}}		-- This module implements {{URL}}
	--		--
	-- See unit tests at [[Module:URL/~~tests~~]]		-- See unit tests at [[Module:URL/testcases]]

	local p = {}		local p = {}

Danski454: made sure expected parameters are not extracted as URLs

2019-07-24T23:29:20Z

made sure expected parameters are not extracted as URLs

← Previous revision		Revision as of 23:29, 24 July 2019
Line 17:		Line 17:
	local function extractUrl(args)		local function extractUrl(args)
	for name, val in pairs(args) do		for name, val in pairs(args) do
	~~local~~ ~~url~~ = name .. "=" ~~.. val;~~		if name ~= 2 and name ~= "msg" then
	url = mw.ustring.gsub(url, '^[Hh][Tt][Tt][Pp]([Ss]?):(/?)([^/])', 'http%1://%3')		local url = name .. "=" .. val;
			url = mw.ustring.gsub(url, '^[Hh][Tt][Tt][Pp]([Ss]?):(/?)([^/])', 'http%1://%3')
	local uri = safeUri(url);		local uri = safeUri(url);
	if uri and uri.host then		if uri and uri.host then
	return url		return url
			end
	end		end
	end		end

Danski454: Undid revision 907739008 by Danski454 (talk)

2019-07-24T23:20:22Z

Undid revision 907739008 by Danski454 (talk)

← Previous revision		Revision as of 23:20, 24 July 2019
Line 94:		Line 94:
	local templateArgs = frame.args		local templateArgs = frame.args
	local parentArgs = frame:getParent().args		local parentArgs = frame:getParent().args
	local url = templateArgs[1] or parentArgs[1] ~~or ''~~		local url = templateArgs[1] or parentArgs[1]
	local text = templateArgs[2] or parentArgs[2]		local text = templateArgs[2] or parentArgs[2] or ''
	local msg = templateArgs.msg or parentArgs.msg or ''		local msg = templateArgs.msg or parentArgs.msg or ''
		⚫	url = url or extractUrl(templateArgs) or extractUrl(parentArgs) or ''
	if not text then
⚫	url = url or extractUrl(templateArgs) or extractUrl(parentArgs)
	end
	text = text or ''
	return p._url(url, text, msg)		return p._url(url, text, msg)
	end		end
Line 112:		Line 109:
	local templateArgs = frame.args		local templateArgs = frame.args
	local parentArgs = frame:getParent().args		local parentArgs = frame:getParent().args
	local url = templateArgs[1] or parentArgs[1] ~~or ''~~		local url = templateArgs[1] or parentArgs[1]
	local text = templateArgs[2] or parentArgs[2] or ''		local text = templateArgs[2] or parentArgs[2] or ''
	-- default to no message		-- default to no message
	local msg = templateArgs.msg or parentArgs.msg or 'no'		local msg = templateArgs.msg or parentArgs.msg or 'no'
		⚫	url = url or extractUrl(templateArgs) or extractUrl(parentArgs) or ''
	if text == '' then
⚫	url = url or extractUrl(templateArgs) or extractUrl(parentArgs)
	end
	-- strip out html tags and [ ] from url		-- strip out html tags and [ ] from url
	url = (url or ''):gsub("<[^>]*>", ""):gsub("[%[%]]", "")		url = (url or ''):gsub("<[^>]*>", ""):gsub("[%[%]]", "")

Danski454: Undid revision 907738735 by Danski454 (talk)

2019-07-24T23:17:57Z

Undid revision 907738735 by Danski454 (talk)

← Previous revision		Revision as of 23:17, 24 July 2019
Line 94:		Line 94:
	local templateArgs = frame.args		local templateArgs = frame.args
	local parentArgs = frame:getParent().args		local parentArgs = frame:getParent().args
	local url = templateArgs[1] or parentArgs[1]		local url = templateArgs[1] or parentArgs[1] or ''
	local text = templateArgs[2] or parentArgs[2] ~~or ''~~		local text = templateArgs[2] or parentArgs[2]
	local msg = templateArgs.msg or parentArgs.msg or ''		local msg = templateArgs.msg or parentArgs.msg or ''
			if not text then
	url = url or extractUrl(templateArgs) or extractUrl(parentArgs) ~~or ''~~		url = url or extractUrl(templateArgs) or extractUrl(parentArgs)
			end
			text = text or ''
	return p._url(url, text, msg)		return p._url(url, text, msg)
	end		end
Line 109:		Line 112:
	local templateArgs = frame.args		local templateArgs = frame.args
	local parentArgs = frame:getParent().args		local parentArgs = frame:getParent().args
	local url = templateArgs[1] or parentArgs[1]		local url = templateArgs[1] or parentArgs[1] or ''
	local text = templateArgs[2] or parentArgs[2] or ''		local text = templateArgs[2] or parentArgs[2] or ''
	-- default to no message		-- default to no message
	local msg = templateArgs.msg or parentArgs.msg or 'no'		local msg = templateArgs.msg or parentArgs.msg or 'no'
			if text == '' then
	url = url or extractUrl(templateArgs) or extractUrl(parentArgs) ~~or ''~~		url = url or extractUrl(templateArgs) or extractUrl(parentArgs)
			end
	-- strip out html tags and [ ] from url		-- strip out html tags and [ ] from url
	url = (url or ''):gsub("<[^>]*>", ""):gsub("[%[%]]", "")		url = (url or ''):gsub("<[^>]*>", ""):gsub("[%[%]]", "")