Module:GetShortDescription/sandbox - Revision history

Fred Gandt: detect redirects to an #id (section/anchor); suffering from fuzzy brain right now and have no idea what to do next

2023-02-22T01:42:09Z

detect redirects to an #id (section/anchor); suffering from fuzzy brain right now and have no idea what to do next

← Previous revision		Revision as of 01:42, 22 February 2023
Line 78:		Line 78:

	-- Better to lower/upper the case of the possibly very long page_content or this?		-- Better to lower/upper the case of the possibly very long page_content or this?
	local ~~redirect_page~~ = mw.ustring.match(page_content, '^%s#[Rr][Ee][Dd][Ii][Rr][Ee][Cc][Tt]%s%[%[')		local redirect_to_id = mw.ustring.match(page_content, '^%s#[Rr][Ee][Dd][Ii][Rr][Ee][Cc][Tt]%s%[%[.-#.-%]%]')
	if ~~redirect_page~~ then		if redirect_to_id then
	mw.log(~~'redirect page'~~)		mw.log(redirect_to_id)
	-- pass this info along		-- pass this info along
	end		end

Fred Gandt: test page content while we have it for redirect syntax; to finish later

2023-02-21T05:39:22Z

test page content while we have it for redirect syntax; to finish later

Show changes

Fred Gandt: testing redirected detection of explicits

2023-02-12T16:09:28Z

testing redirected detection of explicits

← Previous revision		Revision as of 16:09, 12 February 2023
Line 68:		Line 68:
	local function getShortDescription( args_name, args_name_title_table, title, title_table, fallback )		local function getShortDescription( args_name, args_name_title_table, title, title_table, fallback )
	local contents_of_all_short_description_templates = {}		local contents_of_all_short_description_templates = {}
			local redirected

	-- Check for short description templates on redirect pages.		-- Check for short description templates on redirect pages.
Line 73:		Line 74:
	contents_of_all_short_description_templates = getShortDescriptionTemplates( args_name_title_table )		contents_of_all_short_description_templates = getShortDescriptionTemplates( args_name_title_table )
	if contents_of_all_short_description_templates.redlink then return contents_of_all_short_description_templates end		if contents_of_all_short_description_templates.redlink then return contents_of_all_short_description_templates end
			redirected = false
	end		end

	if #contents_of_all_short_description_templates < 1 then		if #contents_of_all_short_description_templates < 1 then
	contents_of_all_short_description_templates = getShortDescriptionTemplates( title_table )		contents_of_all_short_description_templates = getShortDescriptionTemplates( title_table )
			if notEmpty( redirected ) then redirected = true end
	end		end

Line 140:		Line 143:
	-- Pop!		-- Pop!
	local short_description = short_descriptions[ #short_descriptions ].description		local short_description = short_descriptions[ #short_descriptions ].description
	if notEmpty( short_description ) then return { explicit = short_description, fellback = fallback } end		if notEmpty( short_description ) then return { explicit = short_description, fellback = fallback, redirected = redirected } end
	end		end

Fred Gandt: cleanup; still testing

2023-02-12T02:13:01Z

cleanup; still testing

← Previous revision		Revision as of 02:13, 12 February 2023
Line 72:		Line 72:
	if title ~= args_name then		if title ~= args_name then
	contents_of_all_short_description_templates = getShortDescriptionTemplates( args_name_title_table )		contents_of_all_short_description_templates = getShortDescriptionTemplates( args_name_title_table )
		⚫	if contents_of_all_short_description_templates.redlink then return contents_of_all_short_description_templates end
	end		end

⚫	if contents_of_all_short_description_templates.redlink then return contents_of_all_short_description_templates end
⚫
	-- If the provided name was not for a redirect page, or the redirect page had no short description templates:
	if #contents_of_all_short_description_templates < 1 then		if #contents_of_all_short_description_templates < 1 then
	contents_of_all_short_description_templates = getShortDescriptionTemplates( title_table )		contents_of_all_short_description_templates = getShortDescriptionTemplates( title_table )
Line 176:		Line 174:

	-- Literally testing if title_table.isRedirect can be expensive;		-- Literally testing if title_table.isRedirect can be expensive;
	-- processing this way ~~reolves~~ (multiple) redirects without the possibly expensive check.		-- processing this way resolves (multiple) redirects without the possibly expensive check.
	local function getTitleAndTable( orig_name )		local function getTitleAndTable( orig_name )
	local title_table = mw.title.new( orig_name )		local title_table = mw.title.new( orig_name )
Line 191:		Line 189:
	if isEmpty( args_name ) then return { alarm = 'requires a page name (including namespace)' } end		if isEmpty( args_name ) then return { alarm = 'requires a page name (including namespace)' } end

			-- Keep the orginal name, cleaned up, and its title_table for later.
	local args_name_title_table = mw.title.new( args_name )		local args_name_title_table = mw.title.new( args_name )
	args_name = args_name_title_table.prefixedText		args_name = args_name_title_table.prefixedText
		⚫
			if isSisterProjectLink( args_name ) then return nil end

	local title, title_table = getTitleAndTable( args_name )		local title, title_table = getTitleAndTable( args_name )

	if isSisterProjectLink( title ) then return nil end		if title ~= args_name then
			if isSisterProjectLink( title ) then return nil end
			end

	local only = args.only		local only = args.only
Line 244:		Line 248:
	local args = require( 'Module:Arguments' ).getArgs( frame )		local args = require( 'Module:Arguments' ).getArgs( frame )
	if isEmpty( args ) then return alarmingMessage( 'could not getArgs' ) end -- This really would be alarming.		if isEmpty( args ) then return alarmingMessage( 'could not getArgs' ) end -- This really would be alarming.
	~~local d =~~ main( args )		return main( args )
	mw.logObject( d )
	return d
	end		end

Fred Gandt: testing redirect changes

2023-02-12T01:44:41Z

testing redirect changes

← Previous revision		Revision as of 01:44, 12 February 2023
Line 45:		Line 45:
	end		end

	local function ~~getShortDescription~~( title_table~~, args_name, fallback~~ )		local function getShortDescriptionTemplates( title_table )
	local page_content = title_table:getContent()		local page_content = title_table:getContent()

Line 63:		Line 63:
	-- but on an appallingly formatted page, a short description template down by the categories would likely be missed.		-- but on an appallingly formatted page, a short description template down by the categories would likely be missed.
	end		end
			return contents_of_all_short_description_templates
			end

			local function getShortDescription( args_name, args_name_title_table, title, title_table, fallback )
			local contents_of_all_short_description_templates = {}
		⚫
			-- Check for short description templates on redirect pages.
			if title ~= args_name then
			contents_of_all_short_description_templates = getShortDescriptionTemplates( args_name_title_table )
			end

			if contents_of_all_short_description_templates.redlink then return contents_of_all_short_description_templates end

			-- If the provided name was not for a redirect page, or the redirect page had no short description templates:
			if #contents_of_all_short_description_templates < 1 then
			contents_of_all_short_description_templates = getShortDescriptionTemplates( title_table )
			end

			if contents_of_all_short_description_templates.redlink then return contents_of_all_short_description_templates end

	if #contents_of_all_short_description_templates < 1 then return nil end		if #contents_of_all_short_description_templates < 1 then return nil end
Line 156:		Line 175:
	end		end

			-- Literally testing if title_table.isRedirect can be expensive;
	-- Resolve redirects (including multiple) to their end target.
			-- processing this way reolves (multiple) redirects without the possibly expensive check.
	local function getTitleAndTable( orig_name )		local function getTitleAndTable( orig_name )
	local title_table = mw.title.new( orig_name )		local title_table = mw.title.new( orig_name )
Line 169:		Line 189:
	local function getDescription( args )		local function getDescription( args )
	local args_name = args.name		local args_name = args.name
⚫
	if isEmpty( args_name ) then return { alarm = 'requires a page name (including namespace)' } end		if isEmpty( args_name ) then return { alarm = 'requires a page name (including namespace)' } end

			local args_name_title_table = mw.title.new( args_name )
			args_name = args_name_title_table.prefixedText
	local title, title_table = getTitleAndTable( args_name )		local title, title_table = getTitleAndTable( args_name )

Line 182:		Line 203:

	if notEmpty( only ) then		if notEmpty( only ) then
	if only == 'explicit' then return getShortDescription( ~~title_table~~, ~~args_name~~ ) end		if only == 'explicit' then return getShortDescription( args_name, args_name_title_table, title, title_table ) end
	if only == 'wikidata' then return getWikidataDescription( title, args ) end		if only == 'wikidata' then return getWikidataDescription( title, args ) end
	return { alarm = 'accepts either "explicit" or "wikidata" as the value of \|only=' }		return { alarm = 'accepts either "explicit" or "wikidata" as the value of \|only=' }
Line 189:		Line 210:
	if notEmpty( prefer ) then		if notEmpty( prefer ) then
	if prefer == 'explicit' then		if prefer == 'explicit' then
	local short_description = getShortDescription( ~~title_table~~, ~~args_name~~ )		local short_description = getShortDescription( args_name, args_name_title_table, title, title_table )
	if notEmpty( short_description ) then		if notEmpty( short_description ) then
	-- Assume a Wikidata search would be a bad idea for an assumed nonexistent title.		-- Assume a Wikidata search would be a bad idea for an assumed nonexistent title.
Line 196:		Line 217:
	return getWikidataDescription( title, args, true )		return getWikidataDescription( title, args, true )
	end		end
	if prefer == 'wikidata' then return getWikidataDescription( title, args ) or getShortDescription( ~~title_table~~, ~~args_name~~, true ) end		if prefer == 'wikidata' then return getWikidataDescription( title, args ) or getShortDescription( args_name, args_name_title_table, title, title_table, true ) end
	return { alarm = 'accepts either "explicit" or "wikidata" as the value of \|prefer=' }		return { alarm = 'accepts either "explicit" or "wikidata" as the value of \|prefer=' }
	end		end
Line 223:		Line 244:
	local args = require( 'Module:Arguments' ).getArgs( frame )		local args = require( 'Module:Arguments' ).getArgs( frame )
	if isEmpty( args ) then return alarmingMessage( 'could not getArgs' ) end -- This really would be alarming.		if isEmpty( args ) then return alarmingMessage( 'could not getArgs' ) end -- This really would be alarming.
	~~return~~ main( args )		local d = main( args )
			mw.logObject( d )
			return d
	end		end

Fred Gandt: cleanup and notes

2023-02-11T16:38:27Z

cleanup and notes

Show changes

Fred Gandt: fix returning empty explicits + identify fellback

2023-02-11T01:16:03Z

fix returning empty explicits + identify fellback

← Previous revision		Revision as of 01:16, 11 February 2023
Line 23:		Line 23:
	end		end

	local function getWikidataDescription( title, args )		local function getWikidataDescription( title, args, fallback )
	local wikidata_id = mw.wikibase.getEntityIdForTitle( title )		local wikidata_id = mw.wikibase.getEntityIdForTitle( title )
	if isEmpty( wikidata_id ) then return nil end		if isEmpty( wikidata_id ) then return nil end
Line 41:		Line 41:
	}		}
	end		end
			result.fellback = fallback
	return result		return result
	end		end

	local function getShortDescription( title_table, args_name )		local function getShortDescription( title_table, args_name, fallback )
	local page_content = title_table:getContent()		local page_content = title_table:getContent()
	-- Assume no content is a nonexistent title because it's cheaper than testing if it exists.		-- Assume no content is a nonexistent title because it's cheaper than testing if it exists.
Line 106:		Line 107:
	-- Let previewWarning() work out if these numbers are bad.		-- Let previewWarning() work out if these numbers are bad.
	previewWarning( args_name, quantity_of_things )		previewWarning( args_name, quantity_of_things )
	if #short_descriptions >= 1 then ~~return { explicit~~ = short_descriptions[ #short_descriptions ].description ~~} end~~		if #short_descriptions >= 1 then
			local short_description = short_descriptions[ #short_descriptions ].description
			if notEmpty( short_description ) then return { explicit = short_description, fellback = fallback } end
			end
	return nil		return nil
	end		end
Line 166:		Line 170:
	if short_description.redlink or ( not isNone( short_description.explicit ) or args.none_is_valid ) then return short_description end		if short_description.redlink or ( not isNone( short_description.explicit ) or args.none_is_valid ) then return short_description end
	end		end
	return getWikidataDescription( title, args ) end		return getWikidataDescription( title, args, true ) end
	if prefer == 'wikidata' then return getWikidataDescription( title, args ) or getShortDescription( title_table, args_name ) end		if prefer == 'wikidata' then return getWikidataDescription( title, args ) or getShortDescription( title_table, args_name, true ) end
	return { alarm = 'accepts either "explicit" or "wikidata" as the value of \|prefer=' }		return { alarm = 'accepts either "explicit" or "wikidata" as the value of \|prefer=' }
	end		end
Line 196:		Line 200:

	--[[ DON'T FORGET THIS ]]		--[[ DON'T FORGET THIS ]]
	--local result = main( args )		local result = main( args )
	--mw.logObject( result )		--mw.logObject( result )
	return ~~main( args )~~		return result
	--[[ DON'T FORGET THIS ]]		--[[ DON'T FORGET THIS ]]

Fred Gandt: lua is -_-

2023-02-10T21:26:54Z

lua is -_-

← Previous revision		Revision as of 21:26, 10 February 2023
Line 163:		Line 163:
	if prefer == 'explicit' then		if prefer == 'explicit' then
	local short_description = getShortDescription( title_table, args_name )		local short_description = getShortDescription( title_table, args_name )
	if notEmpty( short_description ) ~~and~~ short_description.redlink or ( not isNone( short_description.explicit ) or args.none_is_valid ) then return short_description end		if notEmpty( short_description ) then
			if short_description.redlink or ( not isNone( short_description.explicit ) or args.none_is_valid ) then return short_description end
			end
	return getWikidataDescription( title, args ) end		return getWikidataDescription( title, args ) end
	if prefer == 'wikidata' then return getWikidataDescription( title, args ) or getShortDescription( title_table, args_name ) end		if prefer == 'wikidata' then return getWikidataDescription( title, args ) or getShortDescription( title_table, args_name ) end

Fred Gandt: + notEmpty

2023-02-10T21:19:22Z

+ notEmpty

← Previous revision		Revision as of 21:19, 10 February 2023
Line 163:		Line 163:
	if prefer == 'explicit' then		if prefer == 'explicit' then
	local short_description = getShortDescription( title_table, args_name )		local short_description = getShortDescription( title_table, args_name )
	if short_description and short_description.redlink or ( not isNone( short_description.explicit ) or args.none_is_valid ) then return short_description end		if notEmpty( short_description ) and short_description.redlink or ( not isNone( short_description.explicit ) or args.none_is_valid ) then return short_description end
	return getWikidataDescription( title, args ) end		return getWikidataDescription( title, args ) end
	if prefer == 'wikidata' then return getWikidataDescription( title, args ) or getShortDescription( title_table, args_name ) end		if prefer == 'wikidata' then return getWikidataDescription( title, args ) or getShortDescription( title_table, args_name ) end

Fred Gandt: alter alarming message and comment logging

2023-02-10T20:39:00Z

alter alarming message and comment logging

← Previous revision		Revision as of 20:39, 10 February 2023
Line 159:		Line 159:
	if only == 'explicit' then return getShortDescription( title_table, args_name ) end		if only == 'explicit' then return getShortDescription( title_table, args_name ) end
	if only == 'wikidata' then return getWikidataDescription( title, args ) end		if only == 'wikidata' then return getWikidataDescription( title, args ) end
	return { alarm = '~~only~~ accepts either "explicit" or "wikidata" as the value of \|only=' } end		return { alarm = 'accepts either "explicit" or "wikidata" as the value of \|only=' } end
	if notEmpty( prefer ) then		if notEmpty( prefer ) then
	if prefer == 'explicit' then		if prefer == 'explicit' then
Line 166:		Line 166:
	return getWikidataDescription( title, args ) end		return getWikidataDescription( title, args ) end
	if prefer == 'wikidata' then return getWikidataDescription( title, args ) or getShortDescription( title_table, args_name ) end		if prefer == 'wikidata' then return getWikidataDescription( title, args ) or getShortDescription( title_table, args_name ) end
	return { alarm = '~~only~~ accepts either "explicit" or "wikidata" as the value of \|prefer=' }		return { alarm = 'accepts either "explicit" or "wikidata" as the value of \|prefer=' }
	end		end
	end		end
Line 194:		Line 194:

	--[[ DON'T FORGET THIS ]]		--[[ DON'T FORGET THIS ]]
	local result = main( args )		--local result = main( args )
	mw.logObject( result )		--mw.logObject( result )
	return ~~result~~		return main( args )
	--[[ DON'T FORGET THIS ]]		--[[ DON'T FORGET THIS ]]