Module:Excerpt slideshow - Revision history

Aidan9382: ignore tags by default

2024-08-18T11:19:31Z

ignore <onlyinclude> tags by default

← Previous revision		Revision as of 11:19, 18 August 2024
Line 170:		Line 170:
	options.paraflags = excerptModule.numberFlags(args.paragraphs or "") -- parse paragraphs, e.g. "1,3-5" → {"1","3-5"}		options.paraflags = excerptModule.numberFlags(args.paragraphs or "") -- parse paragraphs, e.g. "1,3-5" → {"1","3-5"}
	options.fileflags = excerptModule.numberFlags(args.files or "") -- parse file numbers		options.fileflags = excerptModule.numberFlags(args.files or "") -- parse file numbers
			options.ignoreOnlyincludes = options.ignoreOnlyincludes == nil and true or options.ignoreOnlyincludes -- default to true
	if args.nostubs and isDeclined(args.nostubs) then		if args.nostubs and isDeclined(args.nostubs) then
	options.nostubs = false		options.nostubs = false

Od1n: fix for mutiple-word namespaces (see T369784); additionally, support underscores in parameter value (as they are equivalent to spaces in page names); after T369784 is resolved, the first line could be removed

2024-07-11T09:29:10Z

fix for mutiple-word namespaces (see T369784); additionally, support underscores in parameter value (as they are equivalent to spaces in page names); after T369784 is resolved, the first line could be removed

← Previous revision		Revision as of 09:29, 11 July 2024
Line 203:		Line 203:
	-- begin BHG addition for tracking source pages		-- begin BHG addition for tracking source pages
	local thisPage = mw.title.getCurrentTitle().nsText .. ":" .. mw.title.getCurrentTitle().text		local thisPage = mw.title.getCurrentTitle().nsText .. ":" .. mw.title.getCurrentTitle().text
			thisPage = thisPage:gsub('_', ' ') -- fix the nsText part, until [[phab:T369784]] is resolved
	local thisBareParam = mw.ustring.gsub(args[ii], "^([^#]+).*$", "%1", 1) -- strip any section anchor from the parameter's page name		local thisBareParam = mw.ustring.gsub(args[ii], "^([^#]+).*$", "%1", 1) -- strip any section anchor from the parameter's page name
			thisBareParam = thisBareParam:gsub('_', ' ') -- support underscores in parameter's page name
	if (thisPage == thisBareParam) then		if (thisPage == thisBareParam) then
	usesEmbeddedList = true;		usesEmbeddedList = true;

Aidan9382: update comments

2023-02-09T18:05:30Z

update comments

← Previous revision		Revision as of 18:05, 9 February 2023
Line 110:		Line 110:
	return frame:preprocess(t)		return frame:preprocess(t)
	end		end
	-- else ~~replace pipes and~~ remove internal linebreaks		-- else remove internal linebreaks
	return mw.ustring.gsub(t, '\n*', '')		return mw.ustring.gsub(t, '\n*', '')
	end		end
Line 117:		Line 117:
	-- strip galleries		-- strip galleries
	excerpt = mw.ustring.gsub(excerpt, "<%s[Gg]allery.->.-<%s/%s[Gg]allery%s>", "")		excerpt = mw.ustring.gsub(excerpt, "<%s[Gg]allery.->.-<%s/%s[Gg]allery%s>", "")
	-- strip tables and block templates; strip newlines ~~and replace pipes within inline templates~~		-- strip tables and block templates; strip newlines
	excerpt = mw.ustring.gsub(excerpt..'\n', '\n?%b{}\n?', processBraces)		excerpt = mw.ustring.gsub(excerpt..'\n', '\n?%b{}\n?', processBraces)
	-- replace wikitext bulleted lists with html bulleted lists		-- replace wikitext bulleted lists with html bulleted lists

Aidan9382: Avoid turning | into {{!}} due to fear of processing errors - Random slideshow now provides a much more logical response that doesn't need this

2022-11-04T12:36:26Z

Avoid turning | into {{!}} due to fear of processing errors - Random slideshow now provides a much more logical response that doesn't need this

← Previous revision		Revision as of 12:36, 4 November 2022
Line 88:		Line 88:
	end		end
	)		)
	end

	local replacePipesWithMagicword = function(t)
	return mw.ustring.gsub(t, '\|', '{{!}}')
	end		end

Line 115:		Line 111:
	end		end
	-- else replace pipes and remove internal linebreaks		-- else replace pipes and remove internal linebreaks
	return ~~replacePipesWithMagicword(~~mw.ustring.gsub(t, '\n*', ''))		return mw.ustring.gsub(t, '\n*', '')
	end		end

Line 123:		Line 119:
	-- strip tables and block templates; strip newlines and replace pipes within inline templates		-- strip tables and block templates; strip newlines and replace pipes within inline templates
	excerpt = mw.ustring.gsub(excerpt..'\n', '\n?%b{}\n?', processBraces)		excerpt = mw.ustring.gsub(excerpt..'\n', '\n?%b{}\n?', processBraces)
	-- replace pipes within links
	excerpt = mw.ustring.gsub(excerpt, '%b[]', replacePipesWithMagicword)
	-- replace other pipes with html entity
	excerpt = mw.ustring.gsub(excerpt, '\|', '\|')
	-- replace wikitext bulleted lists with html bulleted lists		-- replace wikitext bulleted lists with html bulleted lists
	excerpt = gsubWikitextLists(excerpt, '*', 'ul', 'li')		excerpt = gsubWikitextLists(excerpt, '*', 'ul', 'li')
Line 156:		Line 148:
	excerpt = excerpt .. " ('''[[" .. title .. "\|" .. options.more .. "]]''')"		excerpt = excerpt .. " ('''[[" .. title .. "\|" .. options.more .. "]]''')"
	end		end
	local text = '<div style{{=}}text-align:left;>' .. mw.ustring.gsub(excerpt, '%c', '<br>') .. '</div>'		local text = '<div style="text-align:left;">' .. mw.ustring.gsub(excerpt, '%c', '<br>') .. '</div>'
	table.insert(galleryArgs, 'File:Blank.png')		table.insert(galleryArgs, 'File:Blank.png')
	table.insert(galleryArgs, text)		table.insert(galleryArgs, text)

Evad37: Better mobile view handling per Module talk:Random slideshow#Mobile view improvement

2021-05-07T06:26:03Z

Better mobile view handling per Module talk:Random slideshow#Mobile view improvement

← Previous revision		Revision as of 06:26, 7 May 2021
Line 254:		Line 254:
	local parent = frame.getParent(frame)		local parent = frame.getParent(frame)
	local output = p._excerpt(parent.args, 'random')		local output = p._excerpt(parent.args, 'random')
			return frame:extensionTag{ name='templatestyles', args = { src='Module:Random slideshow/styles.css'} }
	~~return~~ frame:preprocess(output)		.. frame:preprocess(output)
	end		end

Line 261:		Line 262:
	local parent = frame.getParent(frame)		local parent = frame.getParent(frame)
	local output = p._excerpt(parent.args, 'linked')		local output = p._excerpt(parent.args, 'linked')
			return frame:extensionTag{ name='templatestyles', args = { src='Module:Random slideshow/styles.css'} }
	~~return~~ frame:preprocess(output)		.. frame:preprocess(output)
	end		end

Line 268:		Line 270:
	local parent = frame.getParent(frame)		local parent = frame.getParent(frame)
	local output = p._excerpt(parent.args, 'listitem')		local output = p._excerpt(parent.args, 'listitem')
			return frame:extensionTag{ name='templatestyles', args = { src='Module:Random slideshow/styles.css'} }
	~~return~~ frame:preprocess(output)		.. frame:preprocess(output)
	end		end

Andrybak: update outdated code comment

2020-11-24T10:07:42Z

update outdated code comment

← Previous revision		Revision as of 10:07, 24 November 2020
Line 151:		Line 151:
	excerpt = nil		excerpt = nil
	end		end
	if excerpt and excerpt ~= '' and #excerpt > 10 then -- check again in case we had a few characters plus ~~Read~~ ~~more~~...		if excerpt and excerpt ~= '' and #excerpt > 10 then -- check again in case we had a few characters plus (Full article...)
	excerpt = cleanUpExcerpt(excerpt)		excerpt = cleanUpExcerpt(excerpt)
	if options.more then		if options.more then

Andrybak: replace default text "Read more..." with "Full article..." as per discussion; use falsy values of parameter "more" for suppression of the link

2020-11-22T00:50:02Z

replace default text "Read more..." with "Full article..." as per discussion; use falsy values of parameter "more" for suppression of the link

← Previous revision		Revision as of 00:50, 22 November 2020
Line 276:		Line 276:
	p._excerpt = function(_args, method)		p._excerpt = function(_args, method)
	local args = cleanupArgs(_args)		local args = cleanupArgs(_args)
			args.more = excerptModule.getMoreLinkText(_args.more)
	-- check for blank value in more parameter
	if _args.more and not args.more then
	args.more = "Read more..." -- default text for blank more=
	end
	local options = makeOptions(args)		local options = makeOptions(args)
	local limit = args.limit and tonumber(args.limit) or DEFAULT_LIMIT		local limit = args.limit and tonumber(args.limit) or DEFAULT_LIMIT

Andrybak: fixup

2020-11-15T17:54:20Z

fixup

← Previous revision		Revision as of 17:54, 15 November 2020
Line 132:		Line 132:
	excerpt = gsubWikitextLists(excerpt, '#', 'ol', 'li')		excerpt = gsubWikitextLists(excerpt, '#', 'ol', 'li')
	excerpt = mw.text.trim(excerpt)		excerpt = mw.text.trim(excerpt)
			return excerpt
	end		end

Andrybak: extract function "cleanUpExcerpt" for reuse; export it for other modules as "p._cleanUpExcerpt"

2020-11-15T17:53:32Z

extract function "cleanUpExcerpt" for reuse; export it for other modules as "p._cleanUpExcerpt"

@@ Line 116: / Line 116: @@
 	-- else replace pipes and remove internal linebreaks
 	return replacePipesWithMagicword(mw.ustring.gsub(t, '\n*', ''))
 end
@@ Line 135: / Line 151: @@
 			end
 			if excerpt and excerpt ~= '' and #excerpt > 10 then -- check again in case we had a few characters plus Read more...
 				if options.more then
 					excerpt = excerpt .. " ('''[[" .. title .. "|" .. options.more .. "]]''')"
@@ Line 286: / Line 290: @@
 	return slideshowModule._main(galleryArgs, false, 'excerptSlideshow-container') .. checksourcepages()
 end
 -- begin BHG addition for tracking source pages

Andrybak: fix a bug: parameter "more" was ignored, because function "get" of Module:Excerpt/portals does not append the "Read more..." link. See tests in Template:Transclude linked excerpts as random slideshow/testcases#Parameter "more" tests

2020-11-06T11:35:46Z

fix a bug: parameter "more" was ignored, because function "get" of Module:Excerpt/portals does not append the "Read more..." link. See tests in Template:Transclude linked excerpts as random slideshow/testcases#Parameter "more" tests

← Previous revision		Revision as of 11:35, 6 November 2020
Line 1:		Line 1:
	local p = {}		local p = {}
	local excerptModule = require('Module:Excerpt/portals')		local excerptModule = require('Module:Excerpt/portals')
	local slideshowModule = require('Module:Random slideshow')		local slideshowModule = require('Module:Random slideshow')
	local randomModule = require('Module:Random')		local randomModule = require('Module:Random')
Line 133:		Line 133:
	mw.log("require('Module:Excerpt').get failed: " .. excerpt) -- probably got a redlink		mw.log("require('Module:Excerpt').get failed: " .. excerpt) -- probably got a redlink
	excerpt = nil		excerpt = nil
	end
	if excerpt and excerpt ~= '' and #excerpt > 10 then
	-- temporarily take off the '''[[Page title\|Read more...]]''' link if present
	readmore_start_index, readmore_end_index, readmore_text = mw.ustring.find(excerpt, "(%s*'''%b[]''')$", -350) --- Starting from end should improve efficiency. 350 characters allows for long page titles and/or a long custom label for the link
	if readmore_start_index then
	excerpt = mw.ustring.sub(excerpt, 1, readmore_start_index-1)
	end
	end		end
	if excerpt and excerpt ~= '' and #excerpt > 10 then -- check again in case we had a few characters plus Read more...		if excerpt and excerpt ~= '' and #excerpt > 10 then -- check again in case we had a few characters plus Read more...
Line 155:		Line 148:
	excerpt = gsubWikitextLists(excerpt, '#', 'ol', 'li')		excerpt = gsubWikitextLists(excerpt, '#', 'ol', 'li')
	excerpt = mw.text.trim(excerpt)		excerpt = mw.text.trim(excerpt)
		⚫	if options.more then
	-- add back the "Read more..." link if it was present
			excerpt = excerpt .. " ('''[[" .. title .. "\|" .. options.more .. "]]''')"
⚫	if ~~readmore_text~~ then
	excerpt = excerpt .. readmore_text
	end		end
	local text = '<div style{{=}}text-align:left;>' .. mw.ustring.gsub(excerpt, '%c', '<br>') .. '</div>'		local text = '<div style{{=}}text-align:left;>' .. mw.ustring.gsub(excerpt, '%c', '<br>') .. '</div>'