Module:External links - Revision history

S.A. Julio: Reduce number of links shown by default. Per WP:LINKFARM: Wikipedia is not a repository of links... excessive lists can dwarf articles and detract from the purpose of Wikipedia; Per WP:ELPOINTS: Links in the "External links" section should be kept to a minimum

2024-05-28T06:09:32Z

Reduce number of links shown by default. Per WP:LINKFARM: Wikipedia is not a repository of links... excessive lists can dwarf articles and detract from the purpose of Wikipedia; Per WP:ELPOINTS: Links in the "External links" section should be kept to a minimum

← Previous revision		Revision as of 06:09, 28 May 2024
Line 925:		Line 925:
	end		end
	if not links_shown or links_shown == '' then		if not links_shown or links_shown == '' then
	links_shown = limits['links-shown'] and limits['links-shown'] or 10		links_shown = limits['links-shown'] and limits['links-shown'] or 5
	else		else
	links_shown = tonumber(links_shown)		links_shown = tonumber(links_shown)

Lemondoge: Finally acquiring the bravery to fix these bugs (bad parentheses, ternary expression short-circuit). This change shouldn't actually affect any articles at all, if I'm not mistaken; the "not (local_value or local_value == '') instances are in functions that aren't called by any existing confs, and short_links is never set to true by any templates yet (which ends up giving the right result for somedataonwikidata)

2023-05-09T18:43:10Z

Finally acquiring the bravery to fix these bugs (bad parentheses, ternary expression short-circuit). This change shouldn't actually affect any articles at all, if I'm not mistaken; the "not (local_value or local_value == '') instances are in functions that aren't called by any existing confs, and short_links is never set to true by any templates yet (which ends up giving the right result for somedataonwikidata)

← Previous revision		Revision as of 18:43, 9 May 2023
Line 560:		Line 560:
	return nil		return nil
	end		end
	if not (local_value or local_value == '') then		if not local_value or local_value == '' then
	-- bail out if no value is present		-- bail out if no value is present
	return output		return output
Line 664:		Line 664:
	return nil		return nil
	end		end
	if not (local_value or local_value == '') then		if not local_value or local_value == '' then
	-- bail out if no value is present		-- bail out if no value is present
	return output		return output
Line 929:		Line 929:
	links_shown = tonumber(links_shown)		links_shown = tonumber(links_shown)
	end		end
	local somedataonwikidata = (short_links ~~and false or true)~~		local somedataonwikidata = not short_links
	--mw.log("conf_claims="..dump(conf_claims))		--mw.log("conf_claims="..dump(conf_claims))
	--mw.log("req_prop="..dump(req_prop))		--mw.log("req_prop="..dump(req_prop))

WOSlinker: use require('strict') instead of require('Module:No globals')

2022-10-21T21:56:23Z

use require('strict') instead of require('Module:No globals')

← Previous revision		Revision as of 21:56, 21 October 2022
Line 1:		Line 1:
	require('~~Module:No globals~~')		require('strict')
	-- local genitive = require('Module:Genitive')._genitive		-- local genitive = require('Module:Genitive')._genitive
	local contLang = mw.language.getContentLanguage()		local contLang = mw.language.getContentLanguage()

MusikBot II: Changed protection settings for "Module:External links": High-risk template or module 16175 transclusions (more info) ([Edit=Require template editor access] (indefinite) [Move=Require template editor access] (indefinite))

2021-08-17T01:23:59Z

Changed protection settings for "Module:External links": High-risk template or module 16175 transclusions (more info) ([Edit=Require template editor access] (indefinite) [Move=Require template editor access] (indefinite))

← Previous revision	Revision as of 01:23, 17 August 2021
(No difference)

MusikBot II: Protected "Module:External links": High-risk template or module (more info) ([Edit=Require autoconfirmed or confirmed access] (indefinite) [Move=Require autoconfirmed or confirmed access] (indefinite))

2019-02-18T23:58:01Z

Protected "Module:External links": High-risk template or module (more info) ([Edit=Require autoconfirmed or confirmed access] (indefinite) [Move=Require autoconfirmed or confirmed access] (indefinite))

← Previous revision	Revision as of 23:58, 18 February 2019
(No difference)

Stigmj: removed property 2439 according to https://www.wikidata.org/w/index.php?title=Wikidata:Properties_for_deletion&oldid=679782022

2018-05-15T06:25:31Z

removed property 2439 according to https://www.wikidata.org/w/index.php?title=Wikidata:Properties_for_deletion&oldid=679782022

← Previous revision		Revision as of 06:25, 15 May 2018
Line 44:		Line 44:

	pval.P407 = { -- language of work or name		pval.P407 = { -- language of work or name
	types = {
	snaktype = 'value',
	datatype = 'wikibase-item',
	datavalue = {
	type = 'wikibase-entityid',
	}
	},
	}

	pval.P2439 = { -- language
	types = {		types = {
	snaktype = 'value',		snaktype = 'value',
Line 256:		Line 246:
	end		end

	local langqvalorder = {'P407~~','P2439~~','P364'}		local langqvalorder = {'P407','P364'}
	local otherqvalorder = {'P582'}		local otherqvalorder = {'P582'}

Zyxw: Don't display msg-langcode if language associated with link is same as current wiki's language - e.g. don't add "(en)" or "(English)" on enwiki.

2017-05-10T14:05:53Z

Don't display msg-langcode if language associated with link is same as current wiki's language - e.g. don't add "(en)" or "(English)" on enwiki.

Zyxw: allow msg-langcode to use $2 for language name

2017-05-10T13:36:40Z

allow msg-langcode to use $2 for language name

← Previous revision		Revision as of 13:36, 10 May 2017
Line 430:		Line 430:
	end		end
	if url ~= '' then		if url ~= '' then
			local this_wiki = mw.getContentLanguage()
⚫	local langlink = (value.langcode and value.langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), value.langcode) or ""
			local this_wiki_code = this_wiki:getCode()
		⚫	local langlink = (value.langcode and value.langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), value.langcode, mw.language.fetchLanguageName(value.langcode, this_wiki_code)) or ""
	if short_links and props.short then		if short_links and props.short then
	output[#output].text =		output[#output].text =
Line 532:		Line 534:
	end		end
	if url ~= '' then		if url ~= '' then
			local this_wiki = mw.getContentLanguage()
⚫	local langlink = (value.langcode and value.langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), value.langcode) or ""
			local this_wiki_code = this_wiki:getCode()
		⚫	local langlink = (value.langcode and value.langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), value.langcode, mw.language.fetchLanguageName(value.langcode, this_wiki_code)) or ""
	if short_links and props.short then		if short_links and props.short then
	output[#output].text =		output[#output].text =
Line 637:		Line 641:
	return {}		return {}
	end		end
			local this_wiki = mw.getContentLanguage()
⚫	local langlink = (output[#output].langcode and output[#output].langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), props.langcode) or ""
			local this_wiki_code = this_wiki:getCode()
		⚫	local langlink = (output[#output].langcode and output[#output].langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), props.langcode, mw.language.fetchLanguageName(props.langcode, this_wiki_code)) or ""
	if short_links and props.short then		if short_links and props.short then
	output[#output].text =		output[#output].text =
Line 735:		Line 741:
	return {}		return {}
	end		end
			local this_wiki = mw.getContentLanguage()
⚫	local langlink = (output[#output].langcode and output[#output].langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), props.langcode) or ""
			local this_wiki_code = this_wiki:getCode()
		⚫	local langlink = (output[#output].langcode and output[#output].langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), props.langcode, mw.language.fetchLanguageName(props.langcode, this_wiki_code)) or ""
	if short_links and props.short then		if short_links and props.short then
	output[#output].text =		output[#output].text =

Zyxw: update addLinkback, less padding before link icon and use text-top instead of baseline for alignment, similar to Module:EditAtWikidata

2017-05-10T12:25:22Z

update addLinkback, less padding before link icon and use text-top instead of baseline for alignment, similar to Module:EditAtWikidata

← Previous revision		Revision as of 12:25, 10 May 2017
Line 781:		Line 781:

	local title = conf:g('wikidata-linkback-edit')		local title = conf:g('wikidata-linkback-edit')
	local icon = '[%s [[File:Blue pencil.svg\|%s\|10px\|~~baseline~~\|link=]] ]'		local icon = '[%s [[File:Blue pencil.svg\|%s\|10px\|text-top\|link=]] ]'
	url = tostring(url)		url = tostring(url)
	local v = mw.html.create('span')		local v = mw.html.create('span')
Line 788:		Line 788:
	:tag('span')		:tag('span')
	:addClass('noprint plainlinks wikidata-linkback')		:addClass('noprint plainlinks wikidata-linkback')
	:css('padding-left', '.~~5em~~')		:css('padding-left', '.3em')
	:wikitext(icon:format(url, title))		:wikitext(icon:format(url, title))
	:allDone()		:allDone()

Zyxw: Undid revision 779585169 by Zyxw (talk)

2017-05-10T10:36:32Z

Undid revision 779585169 by Zyxw (talk)

← Previous revision		Revision as of 10:36, 10 May 2017
Line 1:		Line 1:
	require('Module:No globals')		require('Module:No globals')
	-- local ~~genitiv~~ = require('~~Modul~~:~~Genitiv~~').~~_genitiv~~		-- local genitive = require('Module:Genitive')._genitive
	local contLang = mw.language.getContentLanguage()		local contLang = mw.language.getContentLanguage()

Line 11:		Line 11:
	local p = {}		local p = {}

	local function getLabel(entity, ~~use_genitiv~~, pagetitle)		local function getLabel(entity, use_genitive, pagetitle)
	local label = (pagetitle and pagetitle ~= '') and pagetitle or nil		local label = (pagetitle and pagetitle ~= '') and pagetitle or nil
	if not label and not entity then		if not label and not entity then
Line 18:		Line 18:
	label = mw.wikibase.label(entity.id) or mw.title.getCurrentTitle().text		label = mw.wikibase.label(entity.id) or mw.title.getCurrentTitle().text
	end		end
	-- return ~~use_genitiv~~ and ~~genitiv~~(label, 'sitt') or label		-- return use_genitive and genitive(label, 'sitt') or label
	return ~~use_genitiv~~ and label .. "'s" or label		return use_genitive and label .. "'s" or label
	end		end

Line 434:		Line 434:
	output[#output].text =		output[#output].text =
	mw.message.newRawMessage(props.short,		mw.message.newRawMessage(props.short,
	getLabel(entity, props.~~genitiv~~, pagetitle),		getLabel(entity, props.genitive, pagetitle),
	url,		url,
	langlink,		langlink,
Line 443:		Line 443:
	output[#output].text =		output[#output].text =
	mw.message.newRawMessage(props.message,		mw.message.newRawMessage(props.message,
	getLabel(entity, props.~~genitiv~~, pagetitle),		getLabel(entity, props.genitive, pagetitle),
	url,		url,
	langlink,		langlink,
Line 536:		Line 536:
	output[#output].text =		output[#output].text =
	mw.message.newRawMessage(props.short,		mw.message.newRawMessage(props.short,
	getLabel(entity, props.~~genitiv~~, pagetitle),		getLabel(entity, props.genitive, pagetitle),
	url,		url,
	langlink,		langlink,
Line 545:		Line 545:
	output[#output].text =		output[#output].text =
	mw.message.newRawMessage(props.message,		mw.message.newRawMessage(props.message,
	getLabel(entity, props.~~genitiv~~, pagetitle),		getLabel(entity, props.genitive, pagetitle),
	url,		url,
	langlink,		langlink,
Line 641:		Line 641:
	output[#output].text =		output[#output].text =
	mw.message.newRawMessage(props.short,		mw.message.newRawMessage(props.short,
	getLabel(nil, props.~~genitiv~~, pagetitle),		getLabel(nil, props.genitive, pagetitle),
	url,		url,
	langlink,		langlink,
Line 650:		Line 650:
	output[#output].text =		output[#output].text =
	mw.message.newRawMessage(props.message,		mw.message.newRawMessage(props.message,
	getLabel(nil, props.~~genitiv~~, pagetitle),		getLabel(nil, props.genitive, pagetitle),
	url,		url,
	langlink,		langlink,
Line 739:		Line 739:
	output[#output].text =		output[#output].text =
	mw.message.newRawMessage(props.short,		mw.message.newRawMessage(props.short,
	getLabel(nil, props.~~genitiv~~, pagetitle),		getLabel(nil, props.genitive, pagetitle),
	url,		url,
	langlink,		langlink,
Line 748:		Line 748:
	output[#output].text =		output[#output].text =
	mw.message.newRawMessage(props.message,		mw.message.newRawMessage(props.message,
	getLabel(nil, props.~~genitiv~~, pagetitle),		getLabel(nil, props.genitive, pagetitle),
	url,		url,
	langlink,		langlink,

← Previous revision		Revision as of 14:05, 10 May 2017
Line 432:		Line 432:
	local this_wiki = mw.getContentLanguage()		local this_wiki = mw.getContentLanguage()
	local this_wiki_code = this_wiki:getCode()		local this_wiki_code = this_wiki:getCode()
	local langlink = (value.langcode and value.langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), value.langcode, mw.language.fetchLanguageName(value.langcode, this_wiki_code)) or ""		local langlink = (value.langcode and value.langcode ~= '' and value.langcode ~= this_wiki_code) and mw.message.newRawMessage(conf:g('msg-langcode'), value.langcode, mw.language.fetchLanguageName(value.langcode, this_wiki_code)) or ""
	if short_links and props.short then		if short_links and props.short then
	output[#output].text =		output[#output].text =
Line 536:		Line 536:
	local this_wiki = mw.getContentLanguage()		local this_wiki = mw.getContentLanguage()
	local this_wiki_code = this_wiki:getCode()		local this_wiki_code = this_wiki:getCode()
	local langlink = (value.langcode and value.langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), value.langcode, mw.language.fetchLanguageName(value.langcode, this_wiki_code)) or ""		local langlink = (value.langcode and value.langcode ~= '' and value.langcode ~= this_wiki_code) and mw.message.newRawMessage(conf:g('msg-langcode'), value.langcode, mw.language.fetchLanguageName(value.langcode, this_wiki_code)) or ""
	if short_links and props.short then		if short_links and props.short then
	output[#output].text =		output[#output].text =
Line 643:		Line 643:
	local this_wiki = mw.getContentLanguage()		local this_wiki = mw.getContentLanguage()
	local this_wiki_code = this_wiki:getCode()		local this_wiki_code = this_wiki:getCode()
	local langlink = (output[#output].langcode and output[#output].langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), props.langcode, mw.language.fetchLanguageName(props.langcode, this_wiki_code)) or ""		local langlink = (output[#output].langcode and output[#output].langcode ~= '' and output[#output].langcode ~= this_wiki_code) and mw.message.newRawMessage(conf:g('msg-langcode'), props.langcode, mw.language.fetchLanguageName(props.langcode, this_wiki_code)) or ""
	if short_links and props.short then		if short_links and props.short then
	output[#output].text =		output[#output].text =
Line 743:		Line 743:
	local this_wiki = mw.getContentLanguage()		local this_wiki = mw.getContentLanguage()
	local this_wiki_code = this_wiki:getCode()		local this_wiki_code = this_wiki:getCode()
	local langlink = (output[#output].langcode and output[#output].langcode ~= '') and mw.message.newRawMessage(conf:g('msg-langcode'), props.langcode, mw.language.fetchLanguageName(props.langcode, this_wiki_code)) or ""		local langlink = (output[#output].langcode and output[#output].langcode ~= '' and output[#output].langcode ~= this_wiki_code) and mw.message.newRawMessage(conf:g('msg-langcode'), props.langcode, mw.language.fetchLanguageName(props.langcode, this_wiki_code)) or ""
	if short_links and props.short then		if short_links and props.short then
	output[#output].text =		output[#output].text =