Changes

Jump to navigation Jump to search
sync from sandbox;
Line 1: Line 1:  +
require ('Module:No globals');
   −
require('Module:No globals');
+
--[[--------------------------< F O R W A R D  D E C L A R A T I O N S >--------------------------------------
   −
--[[--------------------------< F O R W A R D  D E C L A R A T I O N S >--------------------------------------
   
each of these counts against the Lua upvalue limit
 
each of these counts against the Lua upvalue limit
 +
 
]]
 
]]
   Line 15: Line 16:  
local cfg = {}; -- table of configuration tables that are defined in Module:Citation/CS1/Configuration
 
local cfg = {}; -- table of configuration tables that are defined in Module:Citation/CS1/Configuration
 
local whitelist = {}; -- table of tables listing valid template parameter names; defined in Module:Citation/CS1/Whitelist
 
local whitelist = {}; -- table of tables listing valid template parameter names; defined in Module:Citation/CS1/Whitelist
 +
    
--[[------------------< P A G E  S C O P E  V A R I A B L E S >---------------
 
--[[------------------< P A G E  S C O P E  V A R I A B L E S >---------------
 +
 
declare variables here that have page-wide scope that are not brought in from
 
declare variables here that have page-wide scope that are not brought in from
 
other modules; that are created here and used here
 
other modules; that are created here and used here
 +
 
]]
 
]]
 +
 
local added_deprecated_cat; -- Boolean flag so that the category is added only once
 
local added_deprecated_cat; -- Boolean flag so that the category is added only once
local added_discouraged_cat; -- Boolean flag so that the category is added only once
   
local added_vanc_errs; -- Boolean flag so we only emit one Vancouver error / category
 
local added_vanc_errs; -- Boolean flag so we only emit one Vancouver error / category
 +
local added_generic_name_errs; -- Boolean flag so we only emit one generic name error / category and stop testing names once an error is encountered
 
local Frame; -- holds the module's frame table
 
local Frame; -- holds the module's frame table
 +
local is_preview_mode; -- true when article is in preview mode; false when using 'Preview page with this template' (previewing the module)
 +
local is_sandbox; -- true when using sandbox modules to render citation
 +
    
--[[--------------------------< F I R S T _ S E T >------------------------------------------------------------
 
--[[--------------------------< F I R S T _ S E T >------------------------------------------------------------
Line 60: Line 68:  
 
 
added_vanc_errs = true; -- note that we've added this category
 
added_vanc_errs = true; -- note that we've added this category
table.insert( z.message_tail, { utilities.set_message ( 'err_vancouver', {source, position}, true ) } );
+
utilities.set_message ('err_vancouver', {source, position});
 
end
 
end
   Line 274: Line 282:  
if utilities.is_set (orig) then
 
if utilities.is_set (orig) then
 
link = ''; -- unset
 
link = ''; -- unset
table.insert( z.message_tail, { utilities.set_message ( 'err_bad_paramlink', orig)}); -- URL or wikilink in |title= with |title-link=;
+
utilities.set_message ('err_bad_paramlink', orig); -- URL or wikilink in |title= with |title-link=;
 
end
 
end
 
 
Line 349: Line 357:  
]]
 
]]
   −
local function check_for_url (parameter_list)
+
local function check_for_url (parameter_list, error_list)
local error_message = '';
   
for k, v in pairs (parameter_list) do -- for each parameter in the list
 
for k, v in pairs (parameter_list) do -- for each parameter in the list
 
if is_parameter_ext_wikilink (v) then -- look at the value; if there is a URL add an error message
 
if is_parameter_ext_wikilink (v) then -- look at the value; if there is a URL add an error message
if utilities.is_set(error_message) then -- once we've added the first portion of the error message ...
+
table.insert (error_list, utilities.wrap_style ('parameter', k));
error_message = error_message .. ", "; -- ... add a comma space separator
  −
end
  −
error_message = error_message .. "&#124;" .. k .. "="; -- add the failed parameter
   
end
 
end
end
  −
if utilities.is_set (error_message) then -- done looping, if there is an error message, display it
  −
table.insert( z.message_tail, { utilities.set_message ( 'err_param_has_ext_link', {error_message}, true ) } );
   
end
 
end
 
end
 
end
Line 373: Line 374:  
local function safe_for_url( str )
 
local function safe_for_url( str )
 
if str:match( "%[%[.-%]%]" ) ~= nil then  
 
if str:match( "%[%[.-%]%]" ) ~= nil then  
table.insert( z.message_tail, { utilities.set_message ( 'err_wikilink_in_url', {}, true ) } );
+
utilities.set_message ('err_wikilink_in_url', {});
 
end
 
end
 
 
Line 389: Line 390:  
]]
 
]]
   −
local function external_link( URL, label, source, access)
+
local function external_link (URL, label, source, access)
local error_str = "";
+
local err_msg = '';
 
local domain;
 
local domain;
 
local path;
 
local path;
 
local base_url;
 
local base_url;
   −
if not utilities.is_set ( label ) then
+
if not utilities.is_set (label) then
 
label = URL;
 
label = URL;
if utilities.is_set ( source ) then
+
if utilities.is_set (source) then
error_str = utilities.set_message ( 'err_bare_url_missing_title', { utilities.wrap_style ('parameter', source) }, false, " " );
+
utilities.set_message ('err_bare_url_missing_title', {utilities.wrap_style ('parameter', source)});
 
else
 
else
error( cfg.messages["bare_url_no_origin"] );
+
error (cfg.messages["bare_url_no_origin"]);
 
end
 
end
 
end
 
end
if not check_url( URL ) then
+
if not check_url (URL) then
error_str = utilities.set_message ( 'err_bad_url', {utilities.wrap_style ('parameter', source)}, false, " " ) .. error_str;
+
utilities.set_message ('err_bad_url', {utilities.wrap_style ('parameter', source)});
 
end
 
end
 
 
Line 413: Line 414:  
end
 
end
   −
base_url = table.concat({ "[", URL, " ", safe_for_url (label), "]" }); -- assemble a wiki-markup URL
+
base_url = table.concat ({ "[", URL, " ", safe_for_url (label), "]" }); -- assemble a wiki-markup URL
    
if utilities.is_set (access) then -- access level (subscription, registration, limited)
 
if utilities.is_set (access) then -- access level (subscription, registration, limited)
 
base_url = utilities.substitute (cfg.presentation['ext-link-access-signal'], {cfg.presentation[access].class, cfg.presentation[access].title, base_url}); -- add the appropriate icon
 
base_url = utilities.substitute (cfg.presentation['ext-link-access-signal'], {cfg.presentation[access].class, cfg.presentation[access].title, base_url}); -- add the appropriate icon
 
end
 
end
+
 
return table.concat ({base_url, error_str});
+
return base_url;
 
end
 
end
   Line 436: Line 437:  
if not added_deprecated_cat then
 
if not added_deprecated_cat then
 
added_deprecated_cat = true; -- note that we've added this category
 
added_deprecated_cat = true; -- note that we've added this category
table.insert( z.message_tail, { utilities.set_message ( 'err_deprecated_params', {name}, true ) } ); -- add error message
+
utilities.set_message ('err_deprecated_params', {name}); -- add error message
end
  −
end
  −
 
  −
 
  −
--[[--------------------------< D I S C O U R A G E D _ P A R A M E T E R >------------------------------------
  −
 
  −
Categorize and emit an maintenance message when the citation contains one or more discouraged parameters.  Only
  −
one error message is emitted regardless of the number of discouraged parameters in the citation.
  −
 
  −
added_discouraged_cat is a Boolean declared in page scope variables above
  −
 
  −
]]
  −
 
  −
local function discouraged_parameter(name)
  −
if not added_discouraged_cat then
  −
added_discouraged_cat = true; -- note that we've added this category
  −
table.insert( z.message_tail, { utilities.set_message ( 'maint_discouraged', {name}, true ) } ); -- add maint message
   
end
 
end
 
end
 
end
Line 477: Line 461:  
local function kern_quotes (str)
 
local function kern_quotes (str)
 
local cap = '';
 
local cap = '';
local cap2 = '';
   
local wl_type, label, link;
 
local wl_type, label, link;
   Line 484: Line 467:  
if 1 == wl_type then -- [[D]] simple wikilink with or without quote marks
 
if 1 == wl_type then -- [[D]] simple wikilink with or without quote marks
 
if mw.ustring.match (str, '%[%[[\"“”\'‘’].+[\"“”\'‘’]%]%]') then -- leading and trailing quote marks
 
if mw.ustring.match (str, '%[%[[\"“”\'‘’].+[\"“”\'‘’]%]%]') then -- leading and trailing quote marks
str = utilities.substitute (cfg.presentation['kern-wl-both'], str);
+
str = utilities.substitute (cfg.presentation['kern-left'], str);
 +
str = utilities.substitute (cfg.presentation['kern-right'], str);
 
elseif mw.ustring.match (str, '%[%[[\"“”\'‘’].+%]%]') then -- leading quote marks
 
elseif mw.ustring.match (str, '%[%[[\"“”\'‘’].+%]%]') then -- leading quote marks
str = utilities.substitute (cfg.presentation['kern-wl-left'], str);
+
str = utilities.substitute (cfg.presentation['kern-left'], str);
 
elseif mw.ustring.match (str, '%[%[.+[\"“”\'‘’]%]%]') then -- trailing quote marks
 
elseif mw.ustring.match (str, '%[%[.+[\"“”\'‘’]%]%]') then -- trailing quote marks
str = utilities.substitute (cfg.presentation['kern-wl-right'], str);
+
str = utilities.substitute (cfg.presentation['kern-right'], str);
 
end
 
end
   Line 495: Line 479:  
label = mw.ustring.gsub (label, '[‘’]', '\''); -- replace ‘’ (U+2018 & U+2019) with ' (typewriter single quote mark)
 
label = mw.ustring.gsub (label, '[‘’]', '\''); -- replace ‘’ (U+2018 & U+2019) with ' (typewriter single quote mark)
   −
cap, cap2 = mw.ustring.match (label, "^([\"\'])([^\'].+)"); -- match leading double or single quote but not doubled single quotes (italic markup)
+
cap = mw.ustring.match (label, "^([\"\'][^\'].+)"); -- match leading double or single quote but not doubled single quotes (italic markup)
 
if utilities.is_set (cap) then
 
if utilities.is_set (cap) then
label = utilities.substitute (cfg.presentation['kern-left'], {cap, cap2});
+
label = utilities.substitute (cfg.presentation['kern-left'], cap);
 
end
 
end
 
 
cap, cap2 = mw.ustring.match (label, "^(.+[^\'])([\"\'])$") -- match trailing double or single quote but not doubled single quotes (italic markup)
+
cap = mw.ustring.match (label, "^(.+[^\'][\"\'])$") -- match trailing double or single quote but not doubled single quotes (italic markup)
 
if utilities.is_set (cap) then
 
if utilities.is_set (cap) then
label = utilities.substitute (cfg.presentation['kern-right'], {cap, cap2});
+
label = utilities.substitute (cfg.presentation['kern-right'], cap);
 
end
 
end
 
 
Line 545: Line 529:  
lang = script_value:match('^(%l%l%l?)%s*:%s*%S.*'); -- get the language prefix or nil if there is no script
 
lang = script_value:match('^(%l%l%l?)%s*:%s*%S.*'); -- get the language prefix or nil if there is no script
 
if not utilities.is_set (lang) then
 
if not utilities.is_set (lang) then
table.insert( z.message_tail, { utilities.set_message ( 'err_script_parameter', {script_param, 'missing title part'}, true ) } ); -- prefix without 'title'; add error message
+
utilities.set_message ('err_script_parameter', {script_param, cfg.err_msg_supl['missing title part']}); -- prefix without 'title'; add error message
 
return ''; -- script_value was just the prefix so return empty string
 
return ''; -- script_value was just the prefix so return empty string
 
end
 
end
Line 554: Line 538:  
-- is prefix one of these language codes?
 
-- is prefix one of these language codes?
 
if utilities.in_array (lang, cfg.script_lang_codes) then
 
if utilities.in_array (lang, cfg.script_lang_codes) then
utilities.add_prop_cat ('script_with_name', {name, lang})
+
utilities.add_prop_cat ('script', {name, lang})
 
else
 
else
table.insert( z.message_tail, { utilities.set_message ( 'err_script_parameter', {script_param, 'unknown language code'}, true ) } ); -- unknown script-language; add error message
+
utilities.set_message ('err_script_parameter', {script_param, cfg.err_msg_supl['unknown language code']}); -- unknown script-language; add error message
 
end
 
end
 
lang = ' lang="' .. lang .. '" '; -- convert prefix into a lang attribute
 
lang = ' lang="' .. lang .. '" '; -- convert prefix into a lang attribute
 
else
 
else
table.insert( z.message_tail, { utilities.set_message ( 'err_script_parameter', {script_param, 'invalid language code'}, true ) } ); -- invalid language code; add error message
+
utilities.set_message ('err_script_parameter', {script_param, cfg.err_msg_supl['invalid language code']}); -- invalid language code; add error message
 
lang = ''; -- invalid so set lang to empty string
 
lang = ''; -- invalid so set lang to empty string
 
end
 
end
 
else
 
else
table.insert( z.message_tail, { utilities.set_message ( 'err_script_parameter', {script_param, 'missing prefix'}, true ) } ); -- no language code prefix; add error message
+
utilities.set_message ('err_script_parameter', {script_param, cfg.err_msg_supl['missing prefix']}); -- no language code prefix; add error message
 
end
 
end
 
script_value = utilities.substitute (cfg.presentation['bdi'], {lang, script_value}); -- isolate in case script is RTL
 
script_value = utilities.substitute (cfg.presentation['bdi'], {lang, script_value}); -- isolate in case script is RTL
Line 574: Line 558:  
--[[--------------------------< S C R I P T _ C O N C A T E N A T E >------------------------------------------
 
--[[--------------------------< S C R I P T _ C O N C A T E N A T E >------------------------------------------
   −
Initially for |title= and |script-title=, this function concatenates those two parameter values after the script value has been  
+
Initially for |title= and |script-title=, this function concatenates those two parameter values after the script
wrapped in <bdi> tags.
+
value has been wrapped in <bdi> tags.
 +
 
 
]]
 
]]
   Line 673: Line 658:     
local function format_periodical (script_periodical, script_periodical_source, periodical, trans_periodical)
 
local function format_periodical (script_periodical, script_periodical_source, periodical, trans_periodical)
local periodical_error = '';
      
if not utilities.is_set (periodical) then
 
if not utilities.is_set (periodical) then
Line 689: Line 673:  
else -- here when trans-periodical without periodical or script-periodical
 
else -- here when trans-periodical without periodical or script-periodical
 
periodical = trans_periodical;
 
periodical = trans_periodical;
periodical_error = ' ' .. utilities.set_message ('err_trans_missing_title', {'periodical'});
+
utilities.set_message ('err_trans_missing_title', {'periodical'});
 
end
 
end
 
end
 
end
   −
return periodical .. periodical_error;
+
return periodical;
 
end
 
end
   Line 706: Line 690:     
local function format_chapter_title (script_chapter, script_chapter_source, chapter, chapter_source, trans_chapter, trans_chapter_source, chapter_url, chapter_url_source, no_quotes, access)
 
local function format_chapter_title (script_chapter, script_chapter_source, chapter, chapter_source, trans_chapter, trans_chapter_source, chapter_url, chapter_url_source, no_quotes, access)
local chapter_error = '';
  −
   
local ws_url, ws_label, L = wikisource_url_make (chapter); -- make a wikisource URL and label from a wikisource interwiki link
 
local ws_url, ws_label, L = wikisource_url_make (chapter); -- make a wikisource URL and label from a wikisource interwiki link
 
if ws_url then
 
if ws_url then
Line 739: Line 721:  
chapter = trans_chapter;
 
chapter = trans_chapter;
 
chapter_source = trans_chapter_source:match ('trans%-?(.+)'); -- when no chapter, get matching name from trans-<param>
 
chapter_source = trans_chapter_source:match ('trans%-?(.+)'); -- when no chapter, get matching name from trans-<param>
chapter_error = ' ' .. utilities.set_message ('err_trans_missing_title', {chapter_source});
+
utilities.set_message ('err_trans_missing_title', {chapter_source});
 
end
 
end
 
end
 
end
   −
return chapter .. chapter_error;
+
return chapter;
 
end
 
end
   Line 801: Line 783:  
end
 
end
   −
table.insert (z.message_tail, {utilities.set_message ('err_invisible_char', {err_msg, utilities.wrap_style ('parameter', param), position}, true)}); -- add error message
+
utilities.set_message ('err_invisible_char', {err_msg, utilities.wrap_style ('parameter', param), position}); -- add error message
 
return; -- and done with this parameter
 
return; -- and done with this parameter
 
end
 
end
Line 907: Line 889:       −
--[[--------------------------< H Y P H E N _ T O _ D A S H >--------------------------------------------------
+
--[[--------------------------< S A F E _ J O I N >-----------------------------
   −
Converts a hyphen to a dash under certain conditions.  The hyphen must separate
+
Joins a sequence of strings together while checking for duplicate separation characters.
like items; unlike items are returned unmodified.  These forms are modified:
  −
letter - letter (A - B)
  −
digit - digit (4-5)
  −
digit separator digit - digit separator digit (4.1-4.5 or 4-1-4-5)
  −
letterdigit - letterdigit (A1-A5) (an optional separator between letter and
  −
digit is supported – a.1-a.5 or a-1-a-5)
  −
digitletter - digitletter (5a - 5d) (an optional separator between letter and
  −
digit is supported – 5.a-5.d or 5-a-5-d)
  −
 
  −
any other forms are returned unmodified.
  −
 
  −
str may be a comma- or semicolon-separated list
      
]]
 
]]
   −
local function hyphen_to_dash( str )
+
local function safe_join( tbl, duplicate_char )
if not utilities.is_set (str) then
+
local f = {}; -- create a function table appropriate to type of 'duplicate character'
return str;
+
if 1 == #duplicate_char then -- for single byte ASCII characters use the string library functions
end
+
f.gsub = string.gsub
 
+
f.match = string.match
local accept; -- Boolean
+
f.sub = string.sub
 
+
else -- for multi-byte characters use the ustring library functions
str = str:gsub ('&[nm]dash;', {['&ndash;'] = '–', ['&mdash;'] = '—'}); -- replace &mdash; and &ndash; entities with their characters; semicolon mucks up the text.split
+
f.gsub = mw.ustring.gsub
str = str:gsub ('&#45;', '-'); -- replace HTML numeric entity with hyphen character
+
f.match = mw.ustring.match
 
+
f.sub = mw.ustring.sub
str = str:gsub ('&nbsp;', ' '); -- replace &nbsp; entity with generic keyboard space character
+
end
  −
local out = {};
  −
local list = mw.text.split (str, '%s*[,;]%s*'); -- split str at comma or semicolon separators if there are any
  −
 
  −
for _, item in ipairs (list) do -- for each item in the list
  −
item, accept = utilities.has_accept_as_written (item); -- remove accept-this-as-written markup when it wraps all of item
  −
if not accept and mw.ustring.match (item, '^%w*[%.%-]?%w+%s*[%-–—]%s*%w*[%.%-]?%w+$') then -- if a hyphenated range or has endash or emdash separators
  −
if item:match ('^%a+[%.%-]?%d+%s*%-%s*%a+[%.%-]?%d+$') or -- letterdigit hyphen letterdigit (optional separator between letter and digit)
  −
item:match ('^%d+[%.%-]?%a+%s*%-%s*%d+[%.%-]?%a+$') or -- digitletter hyphen digitletter (optional separator between digit and letter)
  −
item:match ('^%d+[%.%-]%d+%s*%-%s*%d+[%.%-]%d+$') or -- digit separator digit hyphen digit separator digit
  −
item:match ('^%d+%s*%-%s*%d+$') or -- digit hyphen digit
  −
item:match ('^%a+%s*%-%s*%a+$') then -- letter hyphen letter
  −
item = item:gsub ('(%w*[%.%-]?%w+)%s*%-%s*(%w*[%.%-]?%w+)', '%1–%2'); -- replace hyphen, remove extraneous space characters
  −
else
  −
item = mw.ustring.gsub (item, '%s*[–—]%s*', '–'); -- for endash or emdash separated ranges, replace em with en, remove extraneous whitespace
  −
end
  −
end
  −
table.insert (out, item); -- add the (possibly modified) item to the output table
  −
end
  −
 
  −
local temp_str = ''; -- concatenate the output table into a comma separated string
  −
temp_str, accept = utilities.has_accept_as_written (table.concat (out, ', ')); -- remove accept-this-as-written markup when it wraps all of concatenated out
  −
if accept then
  −
temp_str = utilities.has_accept_as_written (str); -- when global markup removed, return original str; do it this way to suppress boolean second return value
  −
return temp_str;
  −
else
  −
return temp_str; -- else, return assembled temp_str
  −
end
  −
end
  −
 
  −
 
  −
--[[--------------------------< S A F E _ J O I N >-----------------------------
  −
 
  −
Joins a sequence of strings together while checking for duplicate separation characters.
  −
 
  −
]]
  −
 
  −
local function safe_join( tbl, duplicate_char )
  −
local f = {}; -- create a function table appropriate to type of 'duplicate character'
  −
if 1 == #duplicate_char then -- for single byte ASCII characters use the string library functions
  −
f.gsub = string.gsub
  −
f.match = string.match
  −
f.sub = string.sub
  −
else -- for multi-byte characters use the ustring library functions
  −
f.gsub = mw.ustring.gsub
  −
f.match = mw.ustring.match
  −
f.sub = mw.ustring.sub
  −
end
      
local str = ''; -- the output string
 
local str = ''; -- the output string
Line 1,053: Line 975:  
--[[--------------------------< I S _ S U F F I X >-----------------------------
 
--[[--------------------------< I S _ S U F F I X >-----------------------------
   −
returns true is suffix is properly formed Jr, Sr, or ordinal in the range 1–9.
+
returns true if suffix is properly formed Jr, Sr, or ordinal in the range 1–9.
 
Puncutation not allowed.
 
Puncutation not allowed.
   Line 1,283: Line 1,205:  
return result, count; -- return name-list string and count of number of names (count used for editor names only)
 
return result, count; -- return name-list string and count of number of names (count used for editor names only)
 
end
 
end
 +
    
--[[--------------------< M A K E _ C I T E R E F _ I D >-----------------------
 
--[[--------------------< M A K E _ C I T E R E F _ I D >-----------------------
Line 1,310: Line 1,233:       −
--[[---------------------< N A M E _ H A S _ E T A L >--------------------------
+
--[[--------------------------< C I T E _ C L A S S _A T T R I B U T E _M A K E >------------------------------
   −
Evaluates the content of name parameters (author, editor, etc.) for variations on
+
construct <cite> tag class attribute for this citation.
the theme of et al.  If found, the et al. is removed, a flag is set to true and
  −
the function returns the modified name and the flag.
     −
This function never sets the flag to false but returns its previous state because
+
<cite_class> – config.CitationClass from calling template
it may have been set by previous passes through this function or by the associated
+
<mode> – value from |mode= parameter
|display-<names>=etal parameter
+
 
 +
]]
 +
 
 +
local function cite_class_attribute_make (cite_class, mode)
 +
local class_t = {};
 +
table.insert (class_t, 'citation'); -- required for blue highlight
 +
if 'citation' ~= cite_class then
 +
table.insert (class_t, cite_class); -- identify this template for user css
 +
table.insert (class_t, utilities.is_set (mode) and mode or 'cs1'); -- identify the citation style for user css or javascript
 +
else
 +
table.insert (class_t, utilities.is_set (mode) and mode or 'cs2'); -- identify the citation style for user css or javascript
 +
end
 +
for _, prop_key in ipairs (z.prop_keys_t) do
 +
table.insert (class_t, prop_key); -- identify various properties for user css or javascript
 +
end
 +
 
 +
return table.concat (class_t, ' '); -- make a big string and done
 +
end
 +
 
 +
 
 +
--[[---------------------< N A M E _ H A S _ E T A L >--------------------------
 +
 
 +
Evaluates the content of name parameters (author, editor, etc.) for variations on
 +
the theme of et al.  If found, the et al. is removed, a flag is set to true and
 +
the function returns the modified name and the flag.
 +
 
 +
This function never sets the flag to false but returns its previous state because
 +
it may have been set by previous passes through this function or by the associated
 +
|display-<names>=etal parameter
    
]]
 
]]
Line 1,332: Line 1,281:  
etal = true; -- set flag (may have been set previously here or by |display-<names>=etal)
 
etal = true; -- set flag (may have been set previously here or by |display-<names>=etal)
 
if not nocat then -- no categorization for |vauthors=
 
if not nocat then -- no categorization for |vauthors=
table.insert( z.message_tail, {utilities.set_message ('err_etal', {param})}); -- and set an error if not added
+
utilities.set_message ('err_etal', {param}); -- and set an error if not added
 
end
 
end
 
end
 
end
Line 1,356: Line 1,305:  
if mw.ustring.match (name, '^[%A]+$') then -- when name does not contain any letters
 
if mw.ustring.match (name, '^[%A]+$') then -- when name does not contain any letters
 
utilities.set_message ('maint_numeric_names', cfg.special_case_translation [list_name]); -- add a maint cat for this template
 
utilities.set_message ('maint_numeric_names', cfg.special_case_translation [list_name]); -- add a maint cat for this template
end
  −
end
  −
end
  −
  −
  −
--[[-------------------< N A M E _ H A S _ E D _ M A R K U P >------------------
  −
  −
Evaluates the content of author and editor parameters for extraneous editor annotations:
  −
ed, ed., eds, (Ed.), etc. These annotations do not belong in author parameters and
  −
are redundant in editor parameters.  If found, the function adds the editor markup
  −
maintenance category.
  −
  −
returns nothing
  −
  −
]]
  −
  −
local function name_has_ed_markup (name, list_name)
  −
local patterns = cfg.editor_markup_patterns; -- get patterns from configuration
  −
  −
if utilities.is_set (name) then
  −
for _, pattern in ipairs (patterns) do -- spin through patterns table and
  −
if name:match (pattern) then
  −
utilities.set_message ('maint_extra_text_names', cfg.special_case_translation [list_name]); -- add a maint cat for this template
  −
break;
  −
end
   
end
 
end
 
end
 
end
Line 1,419: Line 1,343:       −
--[[------------------------< N A M E _ C H E C K S >---------------------------
+
--[=[-------------------------< I S _ G E N E R I C >----------------------------------------------------------
 +
 
 +
Compares values assigned to various parameters according to the string provided as <item> in the function call.
 +
<item> can have on of two values:
 +
'generic_names' – for name-holding parameters: |last=, |first=, |editor-last=, etc
 +
'generic_titles' – for |title=
   −
This function calls various name checking functions used to validate the content
+
There are two types of generic tests.  The 'accept' tests look for a pattern that should not be rejected by the
of the various name-holding parameters.
+
'reject' test.  For example,
 +
|author=[[John Smith (author)|Smith, John]]
 +
would be rejected by the 'author' reject test.  But piped wikilinks with 'author' disambiguation should not be
 +
rejected so the 'accept' test prevents that from happening.  Accept tests are always performed before reject
 +
tests.
   −
]]
+
Each of the 'accept' and 'reject' sequence tables hold tables for en.wiki (['en']) and local.wiki (['local'])
 +
that each can hold a test sequence table  The sequence table holds, at index [1], a test pattern, and, at index
 +
[2], a boolean control value.  The control value tells string.find() or mw.ustring.find() to do plain-text search (true)
 +
or a pattern search (false).  The intent of all this complexity is to make these searches as fast as possible so
 +
that we don't run out of processing time on very large articles.
   −
local function name_checks (last, first, list_name)
+
Returns
local accept_name;
+
true when a reject test finds the pattern or string
 +
false when an accept test finds the pattern or string
 +
nil else
   −
if utilities.is_set (last) then
+
]=]
 +
 
 +
local function is_generic (item, value, wiki)
 +
local test_val;
 +
local str_lower = { -- use string.lower() for en.wiki (['en']) and use mw.ustring.lower() or local.wiki (['local'])
 +
['en'] = string.lower,
 +
['local'] = mw.ustring.lower,
 +
}
 +
local str_find = { -- use string.find() for en.wiki (['en']) and use mw.ustring.find() or local.wiki (['local'])
 +
['en'] = string.find,
 +
['local'] = mw.ustring.find,
 +
}
 +
 
 +
local function test (val, test_t, wiki) -- local function to do the testing; <wiki> selects lower() and find() functions
 +
val = test_t[2] and str_lower[wiki](value) or val; -- when <test_t[2]> set to 'true', plaintext search using lowercase value
 +
return str_find[wiki] (val, test_t[1], 1, test_t[2]); -- return nil when not found or matched
 +
end
 +
 +
local test_types_t = {'accept', 'reject'}; -- test accept patterns first, then reject patterns
 +
local wikis_t = {'en', 'local'}; -- do tests for each of these keys; en.wiki first, local.wiki second
 +
 
 +
for _, test_type in ipairs (test_types_t) do -- for each test type
 +
for _, generic_value in pairs (cfg.special_case_translation[item][test_type]) do -- spin through the list of generic value fragments to accept or reject
 +
for _, wiki in ipairs (wikis_t) do
 +
if generic_value[wiki] then
 +
if test (value, generic_value[wiki], wiki) then -- go do the test
 +
return ('reject' == test_type); -- param value rejected, return true; false else
 +
end
 +
end
 +
end
 +
end
 +
end
 +
end
 +
 
 +
 
 +
--[[--------------------------< N A M E _ I S _ G E N E R I C >------------------------------------------------
 +
 
 +
calls is_generic() to determine if <name> is a 'generic name' listed in cfg.generic_names; <name_alias> is the
 +
parameter name used in error messaging
 +
 
 +
]]
 +
 
 +
local function name_is_generic (name, name_alias)
 +
if not added_generic_name_errs  and is_generic ('generic_names', name) then
 +
utilities.set_message ('err_generic_name', name_alias); -- set an error message
 +
added_generic_name_errs = true;
 +
end
 +
end
 +
 
 +
 
 +
--[[--------------------------< N A M E _ C H E C K S >--------------------------------------------------------
 +
 
 +
This function calls various name checking functions used to validate the content of the various name-holding parameters.
 +
 
 +
]]
 +
 
 +
local function name_checks (last, first, list_name, last_alias, first_alias)
 +
local accept_name;
 +
 
 +
if utilities.is_set (last) then
 
last, accept_name = utilities.has_accept_as_written (last); -- remove accept-this-as-written markup when it wraps all of <last>
 
last, accept_name = utilities.has_accept_as_written (last); -- remove accept-this-as-written markup when it wraps all of <last>
+
 
 
if not accept_name then -- <last> not wrapped in accept-as-written markup
 
if not accept_name then -- <last> not wrapped in accept-as-written markup
 
name_has_mult_names (last, list_name); -- check for multiple names in the parameter (last only)
 
name_has_mult_names (last, list_name); -- check for multiple names in the parameter (last only)
name_has_ed_markup (last, list_name); -- check for extraneous 'editor' annotation
   
name_is_numeric (last, list_name); -- check for names that are composed of digits and punctuation
 
name_is_numeric (last, list_name); -- check for names that are composed of digits and punctuation
 +
name_is_generic (last, last_alias); -- check for names found in the generic names list
 
end
 
end
 
end
 
end
Line 1,443: Line 1,441:     
if not accept_name then -- <first> not wrapped in accept-as-written markup
 
if not accept_name then -- <first> not wrapped in accept-as-written markup
name_has_ed_markup (first, list_name); -- check for extraneous 'editor' annotation
   
name_is_numeric (first, list_name); -- check for names that are composed of digits and punctuation
 
name_is_numeric (first, list_name); -- check for names that are composed of digits and punctuation
 +
name_is_generic (first, first_alias); -- check for names found in the generic names list
 +
end
 +
local wl_type, D = utilities.is_wikilink (first);
 +
if 0 ~= wl_type then
 +
first = D;
 +
utilities.set_message ('err_bad_paramlink', first_alias);
 
end
 
end
 
end
 
end
Line 1,453: Line 1,456:     
--[[----------------------< E X T R A C T _ N A M E S >-------------------------
 
--[[----------------------< E X T R A C T _ N A M E S >-------------------------
 +
 
Gets name list from the input arguments
 
Gets name list from the input arguments
   Line 1,493: Line 1,497:  
last, etal = name_has_etal (last, etal, false, last_alias); -- find and remove variations on et al.
 
last, etal = name_has_etal (last, etal, false, last_alias); -- find and remove variations on et al.
 
first, etal = name_has_etal (first, etal, false, first_alias); -- find and remove variations on et al.
 
first, etal = name_has_etal (first, etal, false, first_alias); -- find and remove variations on et al.
last, first = name_checks (last, first, list_name); -- multiple names, extraneous annotation, etc. checks
+
last, first = name_checks (last, first, list_name, last_alias, first_alias); -- multiple names, extraneous annotation, etc. checks
+
 
 
if first and not last then -- if there is a firstn without a matching lastn
 
if first and not last then -- if there is a firstn without a matching lastn
 
local alias = first_alias:find ('given', 1, true) and 'given' or 'first'; -- get first or given form of the alias
 
local alias = first_alias:find ('given', 1, true) and 'given' or 'first'; -- get first or given form of the alias
table.insert (z.message_tail, { utilities.set_message ( 'err_first_missing_last', {
+
utilities.set_message ('err_first_missing_last', {
 
first_alias, -- param name of alias missing its mate
 
first_alias, -- param name of alias missing its mate
 
first_alias:gsub (alias, {['first'] = 'last', ['given'] = 'surname'}), -- make param name appropriate to the alias form
 
first_alias:gsub (alias, {['first'] = 'last', ['given'] = 'surname'}), -- make param name appropriate to the alias form
}, true ) } ); -- add this error message
+
}); -- add this error message
 
elseif not first and not last then -- if both firstn and lastn aren't found, are we done?
 
elseif not first and not last then -- if both firstn and lastn aren't found, are we done?
 
count = count + 1; -- number of times we haven't found last and first
 
count = count + 1; -- number of times we haven't found last and first
Line 1,509: Line 1,513:  
local result;
 
local result;
 
link = link_title_ok (link, link_alias, last, last_alias); -- check for improper wiki-markup
 
link = link_title_ok (link, link_alias, last, last_alias); -- check for improper wiki-markup
 +
 
if first then
 
if first then
 
link = link_title_ok (link, link_alias, first, first_alias); -- check for improper wiki-markup
 
link = link_title_ok (link, link_alias, first, first_alias); -- check for improper wiki-markup
Line 1,516: Line 1,521:  
n = n + 1; -- point to next location in the names table
 
n = n + 1; -- point to next location in the names table
 
if 1 == count then -- if the previous name was missing
 
if 1 == count then -- if the previous name was missing
table.insert( z.message_tail, { utilities.set_message ( 'err_missing_name', {list_name:match ("(%w+)List"):lower(), i - 1}, true ) } ); -- add this error message
+
utilities.set_message ('err_missing_name', {list_name:match ("(%w+)List"):lower(), i - 1}); -- add this error message
 
end
 
end
 
count = 0; -- reset the counter, we're looking for two consecutive missing names
 
count = 0; -- reset the counter, we're looking for two consecutive missing names
Line 1,527: Line 1,532:       −
--[[---------------------< G E T _ I S O 6 3 9 _ C O D E >----------------------
+
--[[--------------------------< N A M E _ T A G _ G E T >------------------------------------------------------
   −
Validates language names provided in |language= parameter if not an ISO639-1 or 639-2 code.
+
attempt to decode |language=<lang_param> and return language name and matching tag; nil else.
   −
Returns the language name and associated two- or three-character code. Because
+
This function looks for:
case of the source may be incorrect or different from the case that WikiMedia uses,
+
<lang_param> as a tag in cfg.lang_code_remap{}
the name comparisons are done in lower case and when a match is found, the Wikimedia
+
<lang_param> as a name in cfg.lang_name_remap{}
version (assumed to be correct) is returned along with the codeWhen there is no
+
match, we return the original language name string.
+
<lang_param> as a name in cfg.mw_languages_by_name_t
 +
<lang_param> as a tag in cfg.mw_languages_by_tag_t
 +
when those fail, presume that <lang_param> is an IETF-like tag that MediaWiki does not recognizeStrip all
 +
script, region, variant, whatever subtags from <lang_param> to leave just a two or three character language tag
 +
and look for the new <lang_param> in cfg.mw_languages_by_tag_t{}
   −
mw.language.fetchLanguageNames(<local wiki language>, 'all') returns a list of
+
on success, returns name (in properly capitalized form) and matching tag (in lowercase); on failure returns nil
languages that in some cases may include extensions. For example, code 'cbk-zam'
  −
and its associated name 'Chavacano de Zamboanga' (MediaWiki does not support
  −
code 'cbk' or name 'Chavacano'.  Most (all?) of these languages are not used a
  −
'language' codes per se, rather they are used as sub-domain names: cbk-zam.wikipedia.org.
  −
A list of language names and codes supported by fetchLanguageNames() can be found
  −
at Template:Citation Style documentation/language/doc
     −
Names that are included in the list will be found if that name is provided in the
+
]]
|language= parameter.  For example, if |language=Chavacano de Zamboanga, that name
  −
will be found with the associated code 'cbk-zam'.  When names are found and the
  −
associated code is not two or three characters, this function returns only the
  −
WikiMedia language name.
     −
Some language names have multiple entries under different codes:
+
local function name_tag_get (lang_param)
Aromanian has code rup and code roa-rup
+
local lang_param_lc = mw.ustring.lower (lang_param); -- use lowercase as an index into the various tables
When this occurs, this function returns the language name and the 2- or 3-character code
+
local name;
 +
local tag;
   −
Adapted from code taken from Module:Check ISO 639-1.
+
name = cfg.lang_code_remap[lang_param_lc]; -- assume <lang_param_lc> is a tag; attempt to get remapped language name
 +
if name then -- when <name>, <lang_param> is a tag for a remapped language name
 +
return name, lang_param_lc; -- so return <name> from remap and <lang_param_lc>
 +
end
   −
]]
+
tag = lang_param_lc:match ('^(%a%a%a?)%-.*'); -- still assuming that <lang_param_lc> is a tag; strip script, region, variant subtags
 +
name = cfg.lang_code_remap[tag]; -- attempt to get remapped language name with language subtag only
 +
if name then -- when <name>, <tag> is a tag for a remapped language name
 +
return name, tag; -- so return <name> from remap and <tag>
 +
end
 +
 
 +
if cfg.lang_name_remap[lang_param_lc] then -- not a tag, assume <lang_param_lc> is a name; attempt to get remapped language tag
 +
return cfg.lang_name_remap[lang_param_lc][1], cfg.lang_name_remap[lang_param_lc][2]; -- for this <lang_param_lc>, return a (possibly) new name and appropriate tag
 +
end
   −
local function get_iso639_code (lang, this_wiki_code)
+
tag = cfg.mw_languages_by_name_t[lang_param_lc]; -- assume that <lang_param_lc> is a language name; attempt to get its matching tag
if cfg.lang_name_remap[lang:lower()] then -- if there is a remapped name (because MediaWiki uses something that we don't think is correct)
+
return cfg.lang_name_remap[lang:lower()][1], cfg.lang_name_remap[lang:lower()][2]; -- for this language 'name', return a possibly new name and appropriate code
+
if tag then
 +
return cfg.mw_languages_by_tag_t[tag], tag; -- <lang_param_lc> is a name so return the name from the table and <tag>
 
end
 
end
   −
local ietf_code; -- because some languages have both IETF-like codes and ISO 639-like codes
+
name = cfg.mw_languages_by_tag_t[lang_param_lc]; -- assume that <lang_param_lc> is a tag; attempt to get its matching language name
local ietf_name;
   
 
local langlc = mw.ustring.lower (lang); -- lower-case version for comparisons
+
if name then
 +
return name, lang_param_lc; -- <lang_param_lc> is a tag so return it and <name>
 +
end
 +
 +
tag = lang_param_lc:match ('^(%a%a%a?)%-.*'); -- is <lang_param_lc> an IETF-like tag that MediaWiki doesn't recognize? <tag> gets the language subtag; nil else
   −
for code, name in pairs (cfg.languages) do -- scan the list to see if we can find our language
+
if tag then
if langlc == mw.ustring.lower (name) then
+
name = cfg.mw_languages_by_tag_t[tag]; -- attempt to get a language name using the shortened <tag>
if 2 == #code or 3 == #code then -- two- or three-character codes only; IETF extensions not supported
+
if name then
return name, code; -- so return the name and the code
+
return name, tag; -- <lang_param_lc> is an unrecognized IETF-like tag so return <name> and language subtag
end
  −
ietf_code = code; -- remember that we found an IETF-like code and save its name
  −
ietf_name = name; -- but keep looking for a 2- or 3-char code
   
end
 
end
 
end
 
end
-- didn't find name with 2- or 3-char code; if IETF-like code found return
  −
return ietf_code and ietf_name or lang; -- associated name; return original language text else
   
end
 
end
   Line 1,605: Line 1,615:     
local function language_parameter (lang)
 
local function language_parameter (lang)
local code; -- the two- or three-character language code
+
local tag; -- some form of IETF-like language tag; language subtag with optional region, sript, vatiant, etc subtags
 +
local lang_subtag; -- ve populates |language= with mostly unecessary region subtags the MediaWiki does not recognize; this is the base language subtag
 
local name; -- the language name
 
local name; -- the language name
 
local language_list = {}; -- table of language names to be rendered
 
local language_list = {}; -- table of language names to be rendered
local names_table = {}; -- table made from the value assigned to |language=
+
local names_t = {}; -- table made from the value assigned to |language=
    
local this_wiki_name = mw.language.fetchLanguageName (cfg.this_wiki_code, cfg.this_wiki_code); -- get this wiki's language name
 
local this_wiki_name = mw.language.fetchLanguageName (cfg.this_wiki_code, cfg.this_wiki_code); -- get this wiki's language name
   −
names_table = mw.text.split (lang, '%s*,%s*'); -- names should be a comma separated list
+
names_t = mw.text.split (lang, '%s*,%s*'); -- names should be a comma separated list
 
  −
for _, lang in ipairs (names_table) do -- reuse lang
  −
name = cfg.lang_code_remap[lang:lower()]; -- first see if this is a code that is not supported by MediaWiki but is in remap
     −
if name then -- there was a remapped code so
+
for _, lang in ipairs (names_t) do -- reuse lang here because we don't yet know if lang is a language name or a language tag
if not lang:match ('^%a%a%a?%-x%-%a+$') then -- if not a private IETF tag
+
name, tag = name_tag_get (lang); -- attempt to get name/tag pair for <lang>; <name> has proper capitalization; <tag> is lowercase
lang = lang:gsub ('^(%a%a%a?)%-.*', '%1'); -- strip IETF tags from code
  −
end
  −
else
  −
lang = lang:gsub ('^(%a%a%a?)%-.*', '%1'); -- strip any IETF-like tags from code
  −
if 2 == lang:len() or 3 == lang:len() then -- if two-or three-character code
  −
name = mw.language.fetchLanguageName (lang:lower(), cfg.this_wiki_code); -- get language name if |language= is a proper code
  −
end
  −
end
     −
if utilities.is_set (name) then -- if |language= specified a valid code
+
if utilities.is_set (tag) then
code = lang:lower(); -- save it
+
lang_subtag = tag:gsub ('^(%a%a%a?)%-.*', '%1'); -- for categorization, strip any IETF-like tags from language tag
else
  −
name, code = get_iso639_code (lang, cfg.this_wiki_code); -- attempt to get code from name (assign name here so that we are sure of proper capitalization)
  −
end
  −
  −
if utilities.is_set (code) then -- only 2- or 3-character codes
  −
name = cfg.lang_code_remap[code] or name; -- override wikimedia when they misuse language codes/names
     −
if cfg.this_wiki_code ~= code then -- when the language is not the same as this wiki's language
+
if cfg.this_wiki_code ~= lang_subtag then -- when the language is not the same as this wiki's language
if 2 == code:len() then -- and is a two-character code
+
if 2 == lang_subtag:len() then -- and is a two-character tag
utilities.add_prop_cat ('foreign_lang_source' .. code, {name, code}); -- categorize it; code appended to allow for multiple language categorization
+
-- utilities.add_prop_cat ('foreign-lang-source', {name, lang_subtag}, lang_subtag); -- categorize it; tag appended to allow for multiple language categorization
else -- or is a recognized language (but has a three-character code)
+
utilities.add_prop_cat ('foreign-lang-source', {name, tag}, lang_subtag); -- categorize it; tag appended to allow for multiple language categorization
utilities.add_prop_cat ('foreign_lang_source_2' .. code, {code}); -- categorize it differently TODO: support multiple three-character code categories per cs1|2 template
+
else -- or is a recognized language (but has a three-character tag)
 +
utilities.add_prop_cat ('foreign-lang-source-2', {lang_subtag}, lang_subtag); -- categorize it differently TODO: support multiple three-character tag categories per cs1|2 template?
 
end
 
end
 
elseif cfg.local_lang_cat_enable then -- when the language and this wiki's language are the same and categorization is enabled
 
elseif cfg.local_lang_cat_enable then -- when the language and this wiki's language are the same and categorization is enabled
utilities.add_prop_cat ('local_lang_source', {name, code}); -- categorize it
+
utilities.add_prop_cat ('local-lang-source', {name, lang_subtag}); -- categorize it
 
end
 
end
 
else
 
else
 +
name = lang; -- return whatever <lang> has so that we show something
 
utilities.set_message ('maint_unknown_lang'); -- add maint category if not already added
 
utilities.set_message ('maint_unknown_lang'); -- add maint category if not already added
 
end
 
end
Line 1,655: Line 1,651:  
   
 
   
 
name = utilities.make_sep_list (#language_list, language_list);
 
name = utilities.make_sep_list (#language_list, language_list);
 
+
if (1 == #language_list) and (lang_subtag == cfg.this_wiki_code) then -- when only one language, find lang name in this wiki lang name; for |language=en-us, 'English' in 'American English'
if this_wiki_name == name then
   
return ''; -- if one language and that language is this wiki's return an empty string (no annotation)
 
return ''; -- if one language and that language is this wiki's return an empty string (no annotation)
 
end
 
end
Line 1,664: Line 1,659:  
]]
 
]]
 
end
 
end
 +
    
--[[-----------------------< S E T _ C S _ S T Y L E >--------------------------
 
--[[-----------------------< S E T _ C S _ S T Y L E >--------------------------
 +
 
Gets the default CS style configuration for the given mode.
 
Gets the default CS style configuration for the given mode.
 
Returns default separator and either postscript as passed in or the default.
 
Returns default separator and either postscript as passed in or the default.
 
In CS1, the default postscript and separator are '.'.
 
In CS1, the default postscript and separator are '.'.
 
In CS2, the default postscript is the empty string and the default separator is ','.
 
In CS2, the default postscript is the empty string and the default separator is ','.
 +
 
]]
 
]]
 +
 
local function set_cs_style (postscript, mode)
 
local function set_cs_style (postscript, mode)
 
if utilities.is_set(postscript) then
 
if utilities.is_set(postscript) then
Line 1,685: Line 1,684:     
--[[--------------------------< S E T _ S T Y L E >-----------------------------
 
--[[--------------------------< S E T _ S T Y L E >-----------------------------
 +
 
Sets the separator and postscript styles. Checks the |mode= first and the
 
Sets the separator and postscript styles. Checks the |mode= first and the
 
#invoke CitationClass second. Removes the postscript if postscript == none.
 
#invoke CitationClass second. Removes the postscript if postscript == none.
 +
 
]]
 
]]
 +
 
local function set_style (mode, postscript, cite_class)
 
local function set_style (mode, postscript, cite_class)
 
local sep;
 
local sep;
Line 1,711: Line 1,713:  
return sep, postscript
 
return sep, postscript
 
end
 
end
 +
    
--[=[-------------------------< I S _ P D F >-----------------------------------
 
--[=[-------------------------< I S _ P D F >-----------------------------------
Line 1,742: Line 1,745:  
format = utilities.wrap_style ('format', format); -- add leading space, parentheses, resize
 
format = utilities.wrap_style ('format', format); -- add leading space, parentheses, resize
 
if not utilities.is_set (url) then
 
if not utilities.is_set (url) then
format = format .. ' ' .. utilities.set_message ( 'err_format_missing_url', {fmt_param, url_param} ); -- add an error message
+
utilities.set_message ('err_format_missing_url', {fmt_param, url_param}); -- add an error message
 
end
 
end
 
elseif is_pdf (url) then -- format is not set so if URL is a PDF file then
 
elseif is_pdf (url) then -- format is not set so if URL is a PDF file then
Line 1,784: Line 1,787:  
max = tonumber (max); -- make it a number
 
max = tonumber (max); -- make it a number
 
if max >= count then -- if |display-xxxxors= value greater than or equal to number of authors/editors
 
if max >= count then -- if |display-xxxxors= value greater than or equal to number of authors/editors
table.insert( z.message_tail, {utilities.set_message ('err_disp_name', {param, max}, true)}); -- add error message
+
utilities.set_message ('err_disp_name', {param, max}); -- add error message
 
max = nil;
 
max = nil;
 
end
 
end
 
else -- not a valid keyword or number
 
else -- not a valid keyword or number
table.insert( z.message_tail, {utilities.set_message ('err_disp_name', {param, max}, true)}); -- add error message
+
utilities.set_message ('err_disp_name', {param, max}); -- add error message
 
max = nil; -- unset; as if |display-xxxxors= had not been set
 
max = nil; -- unset; as if |display-xxxxors= had not been set
 
end
 
end
Line 1,813: Line 1,816:  
for _, pattern in ipairs (cfg.vol_iss_pg_patterns.bad_ppatterns) do -- spin through the selected sequence table of patterns
 
for _, pattern in ipairs (cfg.vol_iss_pg_patterns.bad_ppatterns) do -- spin through the selected sequence table of patterns
 
if val:match (pattern) then -- when a match, error so
 
if val:match (pattern) then -- when a match, error so
table.insert (z.message_tail, {utilities.set_message ('err_extra_text_pages', {name}, true)}); -- add error message
+
utilities.set_message ('err_extra_text_pages', name); -- add error message
 
return; -- and done
 
return; -- and done
 
end
 
end
Line 1,856: Line 1,859:  
for _, pattern in ipairs (patterns) do -- spin through the selected sequence table of patterns
 
for _, pattern in ipairs (patterns) do -- spin through the selected sequence table of patterns
 
if val:match (pattern) then -- when a match, error so
 
if val:match (pattern) then -- when a match, error so
table.insert (z.message_tail, {utilities.set_message (handler, {name}, true)}); -- add error message
+
utilities.set_message (handler, name); -- add error message
 
return; -- and done
 
return; -- and done
 
end
 
end
Line 2,028: Line 2,031:  
err_name = 'editor';
 
err_name = 'editor';
 
end
 
end
table.insert( z.message_tail, { utilities.set_message ( 'err_redundant_parameters',
+
utilities.set_message ('err_redundant_parameters', err_name .. '-name-list parameters'); -- add error message
{err_name .. '-name-list parameters'}, true ) } ); -- add error message
   
end
 
end
   Line 2,046: Line 2,048:  
of the list of allowed values returns the translated value; else, emits an error message and returns the value
 
of the list of allowed values returns the translated value; else, emits an error message and returns the value
 
specified by ret_val.
 
specified by ret_val.
 +
 +
TODO: explain <invert>
    
]]
 
]]
   −
local function is_valid_parameter_value (value, name, possible, ret_val)
+
local function is_valid_parameter_value (value, name, possible, ret_val, invert)
 
if not utilities.is_set (value) then
 
if not utilities.is_set (value) then
 
return ret_val; -- an empty parameter is ok
 
return ret_val; -- an empty parameter is ok
elseif utilities.in_array (value, possible) then
+
end
 +
 
 +
if (not invert and utilities.in_array (value, possible)) then -- normal; <value> is in <possible> table
 
return cfg.keywords_xlate[value]; -- return translation of parameter keyword
 
return cfg.keywords_xlate[value]; -- return translation of parameter keyword
 +
elseif invert and not utilities.in_array (value, possible) then -- invert; <value> is not in <possible> table
 +
return value; -- return <value> as it is
 
else
 
else
table.insert( z.message_tail, { utilities.set_message ( 'err_invalid_param_val', {name, value}, true ) } ); -- not an allowed value so add error message
+
utilities.set_message ('err_invalid_param_val', {name, value}); -- not an allowed value so add error message
 
return ret_val;
 
return ret_val;
 
end
 
end
Line 2,092: Line 2,100:  
return '';
 
return '';
 
end
 
end
 +
 +
-- same condition as in format_pages_sheets()
 +
local is_journal = 'journal' == cite_class or (utilities.in_array (cite_class, {'citation', 'map', 'interview'}) and 'journal' == origin);
 +
 +
local is_numeric_vol = volume and (volume:match ('^[MDCLXVI]+$') or volume:match ('^%d+$')); -- is only uppercase roman numerals or only digits?
 +
local is_long_vol = volume and (4 < mw.ustring.len(volume)); -- is |volume= value longer than 4 characters?
 
 
if 'magazine' == cite_class or (utilities.in_array (cite_class, {'citation', 'map'}) and 'magazine' == origin) then
+
if volume and (not is_numeric_vol and is_long_vol) then -- when not all digits or Roman numerals, is |volume= longer than 4 characters?
if utilities.is_set (volume) and utilities.is_set (issue) then
+
utilities.add_prop_cat ('long-vol'); -- yes, add properties cat
return wrap_msg ('vol-no', {sepc, hyphen_to_dash (volume), issue}, lower);
  −
elseif utilities.is_set (volume) then
  −
return wrap_msg ('vol', {sepc, hyphen_to_dash (volume)}, lower);
  −
else
  −
return wrap_msg ('issue', {sepc, issue}, lower);
  −
end
   
end
 
end
   −
if 'podcast' == cite_class and utilities.is_set (issue) then
+
if is_journal then -- journal-style formatting
return wrap_msg ('issue', {sepc, issue}, lower);
+
local vol = '';
end
  −
 
  −
local vol = ''; -- here for all cites except magazine
   
 
if utilities.is_set (volume) then
+
if utilities.is_set (volume) then
if volume:match ('^[MDCLXVI]+$') or volume:match ('^%d+$') then -- volume value is all digits or all uppercase Roman numerals
+
if is_numeric_vol then -- |volume= value all digits or all uppercase Roman numerals?
vol = utilities.substitute (cfg.presentation['vol-bold'], {sepc, volume}); -- render in bold face
+
vol = utilities.substitute (cfg.presentation['vol-bold'], {sepc, volume}); -- render in bold face
elseif (4 < mw.ustring.len(volume)) then -- not all digits or Roman numerals and longer than 4 characters
+
elseif is_long_vol then -- not all digits or Roman numerals; longer than 4 characters?
vol = utilities.substitute (cfg.messages['j-vol'], {sepc, hyphen_to_dash (volume)}); -- not bold
+
vol = utilities.substitute (cfg.messages['j-vol'], {sepc, utilities.hyphen_to_dash (volume)}); -- not bold
utilities.add_prop_cat ('long_vol');
+
else -- four or fewer characters
else -- four or less characters
+
vol = utilities.substitute (cfg.presentation['vol-bold'], {sepc, utilities.hyphen_to_dash (volume)}); -- bold
vol = utilities.substitute (cfg.presentation['vol-bold'], {sepc, hyphen_to_dash (volume)}); -- bold
+
end
 
end
 
end
 +
if utilities.is_set (issue) then
 +
return vol .. utilities.substitute (cfg.messages['j-issue'], issue);
 +
end
 +
return vol;
 +
end
 +
 +
if 'podcast' == cite_class and utilities.is_set (issue) then
 +
return wrap_msg ('issue', {sepc, issue}, lower);
 
end
 
end
if utilities.is_set (issue) then
+
 
return vol .. utilities.substitute (cfg.messages['j-issue'], issue);
+
-- all other types of citation
 +
if utilities.is_set (volume) and utilities.is_set (issue) then
 +
return wrap_msg ('vol-no', {sepc, utilities.hyphen_to_dash (volume), issue}, lower);
 +
elseif utilities.is_set (volume) then
 +
return wrap_msg ('vol', {sepc, utilities.hyphen_to_dash (volume)}, lower);
 +
else
 +
return wrap_msg ('issue', {sepc, issue}, lower);
 
end
 
end
return vol;
   
end
 
end
   Line 2,243: Line 2,261:  
if utilities.is_set (archive) then
 
if utilities.is_set (archive) then
 
if archive == url or archive == c_url then
 
if archive == url or archive == c_url then
table.insert (z.message_tail, {utilities.set_message ('err_bad_url', {utilities.wrap_style ('parameter', source)}, true)}); -- add error message
+
utilities.set_message ('err_bad_url', {utilities.wrap_style ('parameter', source)}); -- add error message
 
return '', ''; -- unset |archive-url= and |archive-date= because same as |url= or |chapter-url=
 
return '', ''; -- unset |archive-url= and |archive-date= because same as |url= or |chapter-url=
 
end
 
end
Line 2,300: Line 2,318:  
else
 
else
 
path, timestamp, flag = url:match('//web%.archive%.org/([^%d]*)(%d+)([^/]*)/'); -- split out some of the URL parts for evaluation
 
path, timestamp, flag = url:match('//web%.archive%.org/([^%d]*)(%d+)([^/]*)/'); -- split out some of the URL parts for evaluation
+
if not path then -- malformed in some way; pattern did not match
if not utilities.is_set (timestamp) or 14 ~= timestamp:len() then -- path and flag optional, must have 14-digit timestamp here
+
err_msg = cfg.err_msg_supl.timestamp;
 +
elseif 14 ~= timestamp:len() then -- path and flag optional, must have 14-digit timestamp here
 
err_msg = cfg.err_msg_supl.timestamp;
 
err_msg = cfg.err_msg_supl.timestamp;
 
if '*' ~= flag then
 
if '*' ~= flag then
url=url:gsub ('(//web%.archive%.org/[^%d]*%d?%d?%d?%d?%d?%d?)[^/]*', '%1*', 1) -- for preview, modify ts to be yearmo* max (0-6 digits plus splat)
+
local replacement = timestamp:match ('^%d%d%d%d%d%d') or timestamp:match ('^%d%d%d%d'); -- get the first 6 (YYYYMM) or first 4 digits (YYYY)
 +
if replacement then -- nil if there aren't at least 4 digits (year)
 +
replacement = replacement .. string.rep ('0', 14 - replacement:len()); -- year or yearmo (4 or 6 digits) zero-fill to make 14-digit timestamp
 +
url=url:gsub ('(//web%.archive%.org/[^%d]*)%d[^/]*', '%1' .. replacement .. '*', 1) -- for preview, modify ts to 14 digits plus splat for calendar display
 +
end
 
end
 
end
 
elseif utilities.is_set (path) and 'web/' ~= path then -- older archive URLs do not have the extra 'web/' path element
 
elseif utilities.is_set (path) and 'web/' ~= path then -- older archive URLs do not have the extra 'web/' path element
Line 2,317: Line 2,340:  
end
 
end
 
-- if here, something not right so
 
-- if here, something not right so
table.insert( z.message_tail, { utilities.set_message ( 'err_archive_url', {err_msg}, true ) } ); -- add error message and
+
utilities.set_message ('err_archive_url', {err_msg}); -- add error message and
if utilities.is_set (Frame:preprocess('{{REVISIONID}}')) then
+
 
 +
if is_preview_mode then
 +
return url, date; -- preview mode so return ArchiveURL and ArchiveDate
 +
else
 
return '', ''; -- return empty strings for ArchiveURL and ArchiveDate
 
return '', ''; -- return empty strings for ArchiveURL and ArchiveDate
else
  −
return url, date; -- preview mode so return ArchiveURL and ArchiveDate
   
end
 
end
 
end
 
end
Line 2,345: Line 2,369:  
 
 
return param_val; -- and done
 
return param_val; -- and done
end
  −
  −
  −
--[[--------------------------< I S _ G E N E R I C _ T I T L E >----------------------------------------------
  −
  −
compares |title= value against list of known generic title patterns.  Returns true when pattern matches; nil else
  −
  −
the k/v pairs in 'generic_titles' each contain two tables, one for English and one for another 'local' language
  −
Each of those tables contain another table that holds the string or pattern (whole title or title fragment) in
  −
index [1].  index [2] is a Boolean that tells string.find() or mw.ustring.find() to do plain-text search (true)
  −
or a pattern search (false).  The intent of all this complexity is to make these searches as fast as possible so
  −
that we don't run out of processing time on very large articles.
  −
  −
]]
  −
  −
local function is_generic_title (title)
  −
title = mw.ustring.lower(title); -- switch title to lower case
  −
for _, generic_title in ipairs (cfg.special_case_translation['generic_titles']) do -- spin through the list of known generic title fragments
  −
if title:find (generic_title['en'][1], 1, generic_title['en'][2]) then
  −
return true; -- found English generic title so done
  −
elseif generic_title['local'] then -- to keep work load down, generic_title['local'] should be nil except when there is a local version of the generic title
  −
if mw.ustring.find (title, generic_title['local'][1], 1, generic_title['local'][2]) then -- mw.ustring() because might not be Latin script
  −
return true; -- found local generic title so done
  −
end
  −
end
  −
end
   
end
 
end
   Line 2,459: Line 2,457:  
if 0 < #c then
 
if 0 < #c then
 
if not utilities.is_set (Contribution) then -- |contributor= requires |contribution=
 
if not utilities.is_set (Contribution) then -- |contributor= requires |contribution=
table.insert( z.message_tail, { utilities.set_message ( 'err_contributor_missing_required_param', 'contribution')}); -- add missing contribution error message
+
utilities.set_message ('err_contributor_missing_required_param', 'contribution'); -- add missing contribution error message
 
c = {}; -- blank the contributors' table; it is used as a flag later
 
c = {}; -- blank the contributors' table; it is used as a flag later
 
end
 
end
 
if 0 == #a then -- |contributor= requires |author=
 
if 0 == #a then -- |contributor= requires |author=
table.insert( z.message_tail, { utilities.set_message ( 'err_contributor_missing_required_param', 'author')}); -- add missing author error message
+
utilities.set_message ('err_contributor_missing_required_param', 'author'); -- add missing author error message
 
c = {}; -- blank the contributors' table; it is used as a flag later
 
c = {}; -- blank the contributors' table; it is used as a flag later
 
end
 
end
Line 2,469: Line 2,467:  
else -- if not a book cite
 
else -- if not a book cite
 
if utilities.select_one (args, cfg.aliases['ContributorList-Last'], 'err_redundant_parameters', 1 ) then -- are there contributor name list parameters?
 
if utilities.select_one (args, cfg.aliases['ContributorList-Last'], 'err_redundant_parameters', 1 ) then -- are there contributor name list parameters?
table.insert( z.message_tail, { utilities.set_message ( 'err_contributor_ignored')}); -- add contributor ignored error message
+
utilities.set_message ('err_contributor_ignored'); -- add contributor ignored error message
 
end
 
end
 
Contribution = nil; -- unset
 
Contribution = nil; -- unset
Line 2,479: Line 2,477:  
local auto_select = ''; -- default is auto
 
local auto_select = ''; -- default is auto
 
local accept_link;
 
local accept_link;
TitleLink, accept_link = utilities.has_accept_as_written(TitleLink, true); -- test for accept-this-as-written markup
+
TitleLink, accept_link = utilities.has_accept_as_written (TitleLink, true); -- test for accept-this-as-written markup
 
if (not accept_link) and utilities.in_array (TitleLink, {'none', 'pmc', 'doi'}) then -- check for special keywords
 
if (not accept_link) and utilities.in_array (TitleLink, {'none', 'pmc', 'doi'}) then -- check for special keywords
 
auto_select = TitleLink; -- remember selection for later
 
auto_select = TitleLink; -- remember selection for later
Line 2,500: Line 2,498:  
Periodical, i = utilities.strip_apostrophe_markup (Periodical); -- strip apostrophe markup so that metadata isn't contaminated  
 
Periodical, i = utilities.strip_apostrophe_markup (Periodical); -- strip apostrophe markup so that metadata isn't contaminated  
 
if i then -- non-zero when markup was stripped so emit an error message
 
if i then -- non-zero when markup was stripped so emit an error message
table.insert( z.message_tail, {utilities.set_message ('err_apostrophe_markup', {Periodical_origin}, true)});
+
utilities.set_message ('err_apostrophe_markup', {Periodical_origin});
 
end
 
end
 
end
 
end
    
if 'mailinglist' == config.CitationClass then -- special case for {{cite mailing list}}
 
if 'mailinglist' == config.CitationClass then -- special case for {{cite mailing list}}
if utilities.is_set (Periodical) and utilities.is_set (A ['MailingList']) then -- both set emit an error
+
if utilities.is_set (Periodical) and utilities.is_set (A ['MailingList']) then -- both set emit an error TODO: make a function for this and similar?
table.insert( z.message_tail, {utilities.set_message ('err_redundant_parameters', {utilities.wrap_style ('parameter', Periodical_origin) .. ' and ' .. utilities.wrap_style ('parameter', 'mailinglist')}, true )});
+
utilities.set_message ('err_redundant_parameters', {utilities.wrap_style ('parameter', Periodical_origin) .. ' and ' .. utilities.wrap_style ('parameter', 'mailinglist')});
 
end
 
end
   Line 2,521: Line 2,519:  
local p = {['journal'] = 'journal', ['magazine'] = 'magazine'}; -- for error message
 
local p = {['journal'] = 'journal', ['magazine'] = 'magazine'}; -- for error message
 
if p[config.CitationClass]  then
 
if p[config.CitationClass]  then
table.insert( z.message_tail, {utilities.set_message ('err_missing_periodical', {config.CitationClass, p[config.CitationClass]}, true)});
+
utilities.set_message ('err_missing_periodical', {config.CitationClass, p[config.CitationClass]});
 
end
 
end
 
end
 
end
Line 2,529: Line 2,527:  
if 'citation' == config.CitationClass then
 
if 'citation' == config.CitationClass then
 
if utilities.is_set (Periodical) then
 
if utilities.is_set (Periodical) then
if not utilities.in_array (Periodical_origin, {'website', 'mailinglist'}) then -- {{citation}} does not render volume for these 'periodicals'
+
if not utilities.in_array (Periodical_origin, cfg.citation_no_volume_t) then -- {{citation}} does not render |volume= when these parameters are used
 
Volume = A['Volume']; -- but does for all other 'periodicals'
 
Volume = A['Volume']; -- but does for all other 'periodicals'
 
end
 
end
Line 2,546: Line 2,544:  
local Issue;
 
local Issue;
 
if 'citation' == config.CitationClass then
 
if 'citation' == config.CitationClass then
if utilities.is_set (Periodical) and utilities.in_array (Periodical_origin, {'journal', 'magazine', 'newspaper', 'periodical', 'work'}) or -- {{citation}} renders issue for these 'periodicals'
+
if utilities.is_set (Periodical) and utilities.in_array (Periodical_origin, cfg.citation_issue_t) then -- {{citation}} may render |issue= when these parameters are used
utilities.is_set (ScriptPeriodical) and utilities.in_array (ScriptPeriodical_origin, {'script-journal', 'script-magazine', 'script-newspaper', 'script-periodical', 'script-work'}) then -- and these 'script-periodicals'
+
Issue = utilities.hyphen_to_dash (A['Issue']);
Issue = hyphen_to_dash (A['Issue']);
   
end
 
end
 
elseif utilities.in_array (config.CitationClass, cfg.templates_using_issue) then -- conference & map books do not support issue; {{citation}} listed here because included in settings table
 
elseif utilities.in_array (config.CitationClass, cfg.templates_using_issue) then -- conference & map books do not support issue; {{citation}} listed here because included in settings table
 
if not (utilities.in_array (config.CitationClass, {'conference', 'map', 'citation'}) and not (utilities.is_set (Periodical) or utilities.is_set (ScriptPeriodical))) then
 
if not (utilities.in_array (config.CitationClass, {'conference', 'map', 'citation'}) and not (utilities.is_set (Periodical) or utilities.is_set (ScriptPeriodical))) then
Issue = hyphen_to_dash (A['Issue']);
+
Issue = utilities.hyphen_to_dash (A['Issue']);
 
end
 
end
 
end
 
end
Line 2,562: Line 2,559:  
if not utilities.in_array (config.CitationClass, cfg.templates_not_using_page) then
 
if not utilities.in_array (config.CitationClass, cfg.templates_not_using_page) then
 
Page = A['Page'];
 
Page = A['Page'];
Pages = hyphen_to_dash (A['Pages']);
+
Pages = utilities.hyphen_to_dash (A['Pages']);
 
At = A['At'];
 
At = A['At'];
 
end
 
end
Line 2,576: Line 2,573:  
PublisherName, i = utilities.strip_apostrophe_markup (PublisherName); -- strip apostrophe markup so that metadata isn't contaminated; publisher is never italicized
 
PublisherName, i = utilities.strip_apostrophe_markup (PublisherName); -- strip apostrophe markup so that metadata isn't contaminated; publisher is never italicized
 
if i then -- non-zero when markup was stripped so emit an error message
 
if i then -- non-zero when markup was stripped so emit an error message
table.insert( z.message_tail, {utilities.set_message ('err_apostrophe_markup', {PublisherName_origin}, true)});
+
utilities.set_message ('err_apostrophe_markup', {PublisherName_origin});
 
end
 
end
 
end
 
end
Line 2,585: Line 2,582:  
if 'newsgroup' == config.CitationClass then
 
if 'newsgroup' == config.CitationClass then
 
if utilities.is_set (PublisherName) then -- general use parameter |publisher= not allowed in cite newsgroup
 
if utilities.is_set (PublisherName) then -- general use parameter |publisher= not allowed in cite newsgroup
local error_text, error_state = utilities.set_message ('err_parameter_ignored', {PublisherName_origin}, true);
+
utilities.set_message ('err_parameter_ignored', {PublisherName_origin});
if utilities.is_set (error_text) then
  −
table.insert( z.message_tail, {error_text, error_state} );
  −
end
   
end
 
end
   Line 2,594: Line 2,588:  
end
 
end
   −
local URL = A['URL']
+
local URL = A['URL']; -- TODO: better way to do this for URL, ChapterURL, and MapURL?
 
local UrlAccess = is_valid_parameter_value (A['UrlAccess'], A:ORIGIN('UrlAccess'), cfg.keywords_lists['url-access'], nil);
 
local UrlAccess = is_valid_parameter_value (A['UrlAccess'], A:ORIGIN('UrlAccess'), cfg.keywords_lists['url-access'], nil);
 
 
 
if not utilities.is_set (URL) and utilities.is_set (UrlAccess) then
 
if not utilities.is_set (URL) and utilities.is_set (UrlAccess) then
 
UrlAccess = nil;
 
UrlAccess = nil;
table.insert( z.message_tail, { utilities.set_message ( 'err_param_access_requires_param', {'url'}, true ) } );
+
utilities.set_message ('err_param_access_requires_param', 'url');
 
end
 
end
 
 
Line 2,606: Line 2,600:  
if not utilities.is_set (ChapterURL) and utilities.is_set (ChapterUrlAccess) then
 
if not utilities.is_set (ChapterURL) and utilities.is_set (ChapterUrlAccess) then
 
ChapterUrlAccess = nil;
 
ChapterUrlAccess = nil;
table.insert( z.message_tail, { utilities.set_message ( 'err_param_access_requires_param', {A:ORIGIN('ChapterUrlAccess'):gsub ('%-access', '')}, true ) } );
+
utilities.set_message ('err_param_access_requires_param', {A:ORIGIN('ChapterUrlAccess'):gsub ('%-access', '')});
 
end
 
end
   Line 2,612: Line 2,606:  
if not utilities.is_set (A['MapURL']) and utilities.is_set (MapUrlAccess) then
 
if not utilities.is_set (A['MapURL']) and utilities.is_set (MapUrlAccess) then
 
MapUrlAccess = nil;
 
MapUrlAccess = nil;
table.insert( z.message_tail, { utilities.set_message ( 'err_param_access_requires_param', {'map-url'}, true ) } );
+
utilities.set_message ('err_param_access_requires_param', {'map-url'});
 
end
 
end
   Line 2,640: Line 2,634:     
if utilities.is_set (PublicationPlace) and utilities.is_set (Place) then -- both |publication-place= and |place= (|location=) allowed if different
 
if utilities.is_set (PublicationPlace) and utilities.is_set (Place) then -- both |publication-place= and |place= (|location=) allowed if different
utilities.add_prop_cat ('location test'); -- add property cat to evaluate how often PublicationPlace and Place are used together
+
utilities.add_prop_cat ('location-test'); -- add property cat to evaluate how often PublicationPlace and Place are used together
 
if PublicationPlace == Place then
 
if PublicationPlace == Place then
 
Place = ''; -- unset; don't need both if they are the same
 
Place = ''; -- unset; don't need both if they are the same
Line 2,678: Line 2,672:  
if utilities.is_set (Encyclopedia) then -- emit error message when Encyclopedia set but template is other than {{cite encyclopedia}} or {{citation}}
 
if utilities.is_set (Encyclopedia) then -- emit error message when Encyclopedia set but template is other than {{cite encyclopedia}} or {{citation}}
 
if 'encyclopaedia' ~= config.CitationClass and 'citation' ~= config.CitationClass then
 
if 'encyclopaedia' ~= config.CitationClass and 'citation' ~= config.CitationClass then
table.insert (z.message_tail, {utilities.set_message ('err_parameter_ignored', {A:ORIGIN ('Encyclopedia')}, true)});
+
utilities.set_message ('err_parameter_ignored', {A:ORIGIN ('Encyclopedia')});
 
Encyclopedia = nil; -- unset because not supported by this template
 
Encyclopedia = nil; -- unset because not supported by this template
 
end
 
end
Line 2,684: Line 2,678:     
if ('encyclopaedia' == config.CitationClass) or ('citation' == config.CitationClass and utilities.is_set (Encyclopedia)) then
 
if ('encyclopaedia' == config.CitationClass) or ('citation' == config.CitationClass and utilities.is_set (Encyclopedia)) then
if utilities.is_set (Periodical) and utilities.is_set (Encyclopedia) then -- when both set emit an error
+
if utilities.is_set (Periodical) and utilities.is_set (Encyclopedia) then -- when both set emit an error TODO: make a function for this and similar?
table.insert (z.message_tail, {utilities.set_message ('err_redundant_parameters', {utilities.wrap_style ('parameter', A:ORIGIN ('Encyclopedia')) .. ' and ' .. utilities.wrap_style ('parameter', Periodical_origin)}, true )});
+
utilities.set_message ('err_redundant_parameters', {utilities.wrap_style ('parameter', A:ORIGIN ('Encyclopedia')) .. ' and ' .. utilities.wrap_style ('parameter', Periodical_origin)});
 
end
 
end
   Line 2,717: Line 2,711:  
ScriptTitle = '';
 
ScriptTitle = '';
 
end
 
end
elseif utilities.is_set (Chapter) then -- |title= not set
+
elseif utilities.is_set (Chapter) or utilities.is_set (ScriptChapter) then -- |title= not set
 
Title = Periodical; -- |encyclopedia= set and |article= set so map |encyclopedia= to |title=
 
Title = Periodical; -- |encyclopedia= set and |article= set so map |encyclopedia= to |title=
 
Periodical = ''; -- redundant so unset
 
Periodical = ''; -- redundant so unset
Line 2,731: Line 2,725:  
ID = A['Number']; -- yes, use it
 
ID = A['Number']; -- yes, use it
 
else -- ID has a value so emit error message
 
else -- ID has a value so emit error message
table.insert( z.message_tail, { utilities.set_message ('err_redundant_parameters', {utilities.wrap_style ('parameter', 'id') .. ' and ' .. utilities.wrap_style ('parameter', 'number')}, true )});
+
utilities.set_message ('err_redundant_parameters', {utilities.wrap_style ('parameter', 'id') .. ' and ' .. utilities.wrap_style ('parameter', 'number')});
 
end
 
end
 
end
 
end
Line 2,776: Line 2,770:  
local Sheets = A['Sheets'] or '';
 
local Sheets = A['Sheets'] or '';
 
if config.CitationClass == "map" then
 
if config.CitationClass == "map" then
if utilities.is_set (Chapter) then
+
if utilities.is_set (Chapter) then --TODO: make a function for this and similar?
table.insert( z.message_tail, { utilities.set_message ( 'err_redundant_parameters', {utilities.wrap_style ('parameter', 'map') .. ' and ' .. utilities.wrap_style ('parameter', Chapter_origin)}, true ) } ); -- add error message
+
utilities.set_message ('err_redundant_parameters', {utilities.wrap_style ('parameter', 'map') .. ' and ' .. utilities.wrap_style ('parameter', Chapter_origin)}); -- add error message
 
end
 
end
 
Chapter = A['Map'];
 
Chapter = A['Map'];
Line 2,819: Line 2,813:  
local SeriesNumber = A['SeriesNumber'];
 
local SeriesNumber = A['SeriesNumber'];
   −
if utilities.is_set (Season) and utilities.is_set (SeriesNumber) then -- these are mutually exclusive so if both are set
+
if utilities.is_set (Season) and utilities.is_set (SeriesNumber) then -- these are mutually exclusive so if both are set TODO: make a function for this and similar?
table.insert( z.message_tail, { utilities.set_message ( 'err_redundant_parameters', {utilities.wrap_style ('parameter', 'season') .. ' and ' .. utilities.wrap_style ('parameter', 'seriesno')}, true ) } ); -- add error message
+
utilities.set_message ('err_redundant_parameters', {utilities.wrap_style ('parameter', 'season') .. ' and ' .. utilities.wrap_style ('parameter', 'seriesno')}); -- add error message
 
SeriesNumber = ''; -- unset; prefer |season= over |seriesno=
 
SeriesNumber = ''; -- unset; prefer |season= over |seriesno=
 
end
 
end
Line 2,837: Line 2,831:  
ChapterUrlAccess = UrlAccess;
 
ChapterUrlAccess = UrlAccess;
 
ChapterURL_origin = URL_origin;
 
ChapterURL_origin = URL_origin;
+
ChapterFormat = Format;
 +
 
 
Title = Series; -- promote series to title
 
Title = Series; -- promote series to title
 
TitleLink = SeriesLink;
 
TitleLink = SeriesLink;
Line 2,850: Line 2,845:  
TransTitle = '';
 
TransTitle = '';
 
ScriptTitle = '';
 
ScriptTitle = '';
 +
Format = '';
 
 
 
else -- now oddities that are cite serial
 
else -- now oddities that are cite serial
Line 2,865: Line 2,861:  
local TitleType = A['TitleType'];
 
local TitleType = A['TitleType'];
 
local Degree = A['Degree'];
 
local Degree = A['Degree'];
if utilities.in_array (config.CitationClass, {"AV-media-notes", "interview", "mailinglist", "map", "podcast", "pressrelease", "report", "techreport", "thesis"}) then
+
if utilities.in_array (config.CitationClass, {'AV-media-notes', 'interview', 'mailinglist', 'map', 'podcast', 'pressrelease', 'report', 'speech', 'techreport', 'thesis'}) then
 
TitleType = set_titletype (config.CitationClass, TitleType);
 
TitleType = set_titletype (config.CitationClass, TitleType);
 
if utilities.is_set (Degree) and "Thesis" == TitleType then -- special case for cite thesis
 
if utilities.is_set (Degree) and "Thesis" == TitleType then -- special case for cite thesis
Line 2,946: Line 2,942:  
-- start temporary Julian / Gregorian calendar uncertainty categorization
 
-- start temporary Julian / Gregorian calendar uncertainty categorization
 
if COinS_date.inter_cal_cat then
 
if COinS_date.inter_cal_cat then
utilities.add_prop_cat ('jul_greg_uncertainty');
+
utilities.add_prop_cat ('jul-greg-uncertainty');
 
end
 
end
 
-- end temporary Julian / Gregorian calendar uncertainty categorization
 
-- end temporary Julian / Gregorian calendar uncertainty categorization
Line 2,957: Line 2,953:  
local modified = false; -- flag
 
local modified = false; -- flag
 
 
if validation.edtf_transform (date_parameters_list) then -- edtf dates to MOS compliant format
  −
modified = true;
  −
end
  −
   
if utilities.is_set (DF) then -- if we need to reformat dates
 
if utilities.is_set (DF) then -- if we need to reformat dates
 
modified = validation.reformat_dates (date_parameters_list, DF); -- reformat to DF format, use long month names if appropriate
 
modified = validation.reformat_dates (date_parameters_list, DF); -- reformat to DF format, use long month names if appropriate
Line 2,970: Line 2,962:  
end
 
end
 
 
-- for those wikis that can and want to have English date names translated to the local language,
+
-- for those wikis that can and want to have English date names translated to the local language; not supported at en.wiki
-- uncomment the next three lines.  Not supported by en.wiki (for obvious reasons)
+
if cfg.date_name_auto_xlate_enable and validation.date_name_xlate (date_parameters_list, cfg.date_digit_auto_xlate_enable ) then
-- set validation.date_name_xlate() second argument to true to translate English digits to local digits (will translate ymd dates)
+
utilities.set_message ('maint_date_auto_xlated'); -- add maint cat
-- if validation.date_name_xlate (date_parameters_list, false) then
+
modified = true;
-- modified = true;
+
end
-- end
      
if modified then -- if the date_parameters_list values were modified
 
if modified then -- if the date_parameters_list values were modified
Line 2,986: Line 2,977:  
end
 
end
 
else
 
else
table.insert (z.message_tail, {utilities.set_message ('err_bad_date', {utilities.make_sep_list (#error_list, error_list)}, true)}); -- add this error message
+
utilities.set_message ('err_bad_date', {utilities.make_sep_list (#error_list, error_list)}); -- add this error message
 
end
 
end
 
end -- end of do
 
end -- end of do
Line 3,005: Line 2,996:  
if utilities.in_array (config.CitationClass, whitelist.preprint_template_list) then
 
if utilities.in_array (config.CitationClass, whitelist.preprint_template_list) then
 
if not utilities.is_set (ID_list_coins[config.CitationClass:upper()]) then -- |arxiv= or |eprint= required for cite arxiv; |biorxiv= & |citeseerx= required for their templates
 
if not utilities.is_set (ID_list_coins[config.CitationClass:upper()]) then -- |arxiv= or |eprint= required for cite arxiv; |biorxiv= & |citeseerx= required for their templates
table.insert (z.message_tail, {utilities.set_message ('err_' .. config.CitationClass .. '_missing', {}, true)}); -- add error message
+
utilities.set_message ('err_' .. config.CitationClass .. '_missing'); -- add error message
 
end
 
end
   Line 3,027: Line 3,018:  
  end
 
  end
   −
if utilities.is_set (URL) and utilities.is_set (AccessDate) then -- access date requires |url=; identifier-created URL is not |url=
+
if utilities.is_set (URL) then -- set when using an identifier-created URL
table.insert( z.message_tail, { utilities.set_message ( 'err_accessdate_missing_url', {}, true ) } ); -- add an error message
+
if utilities.is_set (AccessDate) then -- |access-date= requires |url=; identifier-created URL is not |url=
AccessDate = ''; -- unset
+
utilities.set_message ('err_accessdate_missing_url'); -- add an error message
 +
AccessDate = ''; -- unset
 +
end
 +
 
 +
if utilities.is_set (ArchiveURL) then -- |archive-url= requires |url=; identifier-created URL is not |url=
 +
utilities.set_message ('err_archive_missing_url'); -- add an error message
 +
ArchiveURL = ''; -- unset
 +
end
 
end
 
end
 
end
 
end
Line 3,036: Line 3,034:  
-- Test if citation has no title
 
-- Test if citation has no title
 
if not utilities.is_set (Title) and not utilities.is_set (TransTitle) and not utilities.is_set (ScriptTitle) then -- has special case for cite episode
 
if not utilities.is_set (Title) and not utilities.is_set (TransTitle) and not utilities.is_set (ScriptTitle) then -- has special case for cite episode
table.insert( z.message_tail, { utilities.set_message ( 'err_citation_missing_title', {'episode' == config.CitationClass and 'series' or 'title'}, true ) } );
+
utilities.set_message ('err_citation_missing_title', {'episode' == config.CitationClass and 'series' or 'title'});
 
end
 
end
   Line 3,046: Line 3,044:  
utilities.set_message ('maint_untitled'); -- add maint cat
 
utilities.set_message ('maint_untitled'); -- add maint cat
 
end
 
end
  −
check_for_url ({ -- add error message when any of these parameters hold a URL
  −
['title'] = Title,
  −
[A:ORIGIN('Chapter')] = Chapter,
  −
[Periodical_origin] = Periodical,
  −
[PublisherName_origin] = PublisherName
  −
});
      
-- COinS metadata (see <http://ocoins.info/>) for automated parsing of citation information.
 
-- COinS metadata (see <http://ocoins.info/>) for automated parsing of citation information.
Line 3,073: Line 3,064:  
 
 
local QuotePage = A['QuotePage'];
 
local QuotePage = A['QuotePage'];
local QuotePages = hyphen_to_dash (A['QuotePages']);
+
local QuotePages = utilities.hyphen_to_dash (A['QuotePages']);
    
-- this is the function call to COinS()
 
-- this is the function call to COinS()
Line 3,079: Line 3,070:  
['Periodical'] = utilities.strip_apostrophe_markup (Periodical), -- no markup in the metadata
 
['Periodical'] = utilities.strip_apostrophe_markup (Periodical), -- no markup in the metadata
 
['Encyclopedia'] = Encyclopedia, -- just a flag; content ignored by ~/COinS
 
['Encyclopedia'] = Encyclopedia, -- just a flag; content ignored by ~/COinS
['Chapter'] = metadata.make_coins_title (coins_chapter, ScriptChapter), -- Chapter and ScriptChapter stripped of bold / italic wiki-markup
+
['Chapter'] = metadata.make_coins_title (coins_chapter, ScriptChapter), -- Chapter and ScriptChapter stripped of bold / italic / accept-as-written markup
 
['Degree'] = Degree; -- cite thesis only
 
['Degree'] = Degree; -- cite thesis only
['Title'] = metadata.make_coins_title (coins_title, ScriptTitle), -- Title and ScriptTitle stripped of bold / italic wiki-markup
+
['Title'] = metadata.make_coins_title (coins_title, ScriptTitle), -- Title and ScriptTitle stripped of bold / italic / accept-as-written markup
 
['PublicationPlace'] = PublicationPlace,
 
['PublicationPlace'] = PublicationPlace,
 
['Date'] = COinS_date.rftdate, -- COinS_date has correctly formatted date if Date is valid;
 
['Date'] = COinS_date.rftdate, -- COinS_date has correctly formatted date if Date is valid;
Line 3,188: Line 3,179:  
if utilities.in_array (config.CitationClass, {"web", "podcast", "mailinglist"}) or -- |url= required for cite web, cite podcast, and cite mailinglist
 
if utilities.in_array (config.CitationClass, {"web", "podcast", "mailinglist"}) or -- |url= required for cite web, cite podcast, and cite mailinglist
 
('citation' == config.CitationClass and ('website' == Periodical_origin or 'script-website' == ScriptPeriodical_origin)) then -- and required for {{citation}} with |website= or |script-website=
 
('citation' == config.CitationClass and ('website' == Periodical_origin or 'script-website' == ScriptPeriodical_origin)) then -- and required for {{citation}} with |website= or |script-website=
table.insert( z.message_tail, { utilities.set_message ( 'err_cite_web_url', {}, true ) } );
+
utilities.set_message ('err_cite_web_url');
 
end
 
end
 
 
 
-- do we have |accessdate= without either |url= or |chapter-url=?
 
-- do we have |accessdate= without either |url= or |chapter-url=?
 
if utilities.is_set (AccessDate) and not utilities.is_set (ChapterURL) then -- ChapterURL may be set when URL is not set;
 
if utilities.is_set (AccessDate) and not utilities.is_set (ChapterURL) then -- ChapterURL may be set when URL is not set;
table.insert( z.message_tail, { utilities.set_message ( 'err_accessdate_missing_url', {}, true ) } );
+
utilities.set_message ('err_accessdate_missing_url');
 
AccessDate = '';
 
AccessDate = '';
 
end
 
end
Line 3,228: Line 3,219:  
UrlAccess = nil; -- restricted access levels do not make sense for archived URLs
 
UrlAccess = nil; -- restricted access levels do not make sense for archived URLs
 
end
 
end
  end
+
end
 +
elseif utilities.is_set (UrlStatus) then -- if |url-status= is set when |archive-url= is not set
 +
  utilities.set_message ('maint_url_status'); -- add maint cat
 
end
 
end
   Line 3,247: Line 3,240:     
if utilities.is_set (chap_param) then -- if we found one
 
if utilities.is_set (chap_param) then -- if we found one
table.insert( z.message_tail, { utilities.set_message ( 'err_chapter_ignored', {chap_param}, true ) } ); -- add error message
+
utilities.set_message ('err_chapter_ignored', {chap_param}); -- add error message
 
Chapter = ''; -- and set them to empty string to be safe with concatenation
 
Chapter = ''; -- and set them to empty string to be safe with concatenation
 
TransChapter = '';
 
TransChapter = '';
Line 3,287: Line 3,280:  
end
 
end
   −
if not accept_title then -- <Title> not wrapped in accept-as-written markup
+
if not accept_title then -- <Title> not wrapped in accept-as-written markup
 
if '...' == Title:sub (-3) then -- if ellipsis is the last three characters of |title=
 
if '...' == Title:sub (-3) then -- if ellipsis is the last three characters of |title=
 
Title = Title:gsub ('(%.%.%.)%.+$', '%1'); -- limit the number of dots to three
 
Title = Title:gsub ('(%.%.%.)%.+$', '%1'); -- limit the number of dots to three
Line 3,299: Line 3,292:  
end
 
end
   −
if is_generic_title (Title) then
+
if is_generic ('generic_titles', Title) then
table.insert (z.message_tail, {utilities.set_message ( 'err_generic_title', {}, true ) } ); -- set an error message
+
utilities.set_message ('err_generic_title'); -- set an error message
 
end
 
end
 
end
 
end
Line 3,311: Line 3,304:  
Title = script_concatenate (Title, ScriptTitle, 'script-title'); -- <bdi> tags, lang attribute, categorization, etc.; must be done after title is wrapped
 
Title = script_concatenate (Title, ScriptTitle, 'script-title'); -- <bdi> tags, lang attribute, categorization, etc.; must be done after title is wrapped
 
TransTitle = utilities.wrap_style ('trans-quoted-title', TransTitle );
 
TransTitle = utilities.wrap_style ('trans-quoted-title', TransTitle );
elseif plain_title or ('report' == config.CitationClass) then -- no styling for cite report and descriptive titles (otherwise same as above)
+
elseif plain_title or ('report' == config.CitationClass) then -- no styling for cite report and descriptive titles (otherwise same as above)
 
Title = script_concatenate (Title, ScriptTitle, 'script-title'); -- <bdi> tags, lang attribute, categorization, etc.; must be done after title is wrapped
 
Title = script_concatenate (Title, ScriptTitle, 'script-title'); -- <bdi> tags, lang attribute, categorization, etc.; must be done after title is wrapped
TransTitle = utilities.wrap_style ('trans-quoted-title', TransTitle ); -- for cite report, use this form for trans-title
+
TransTitle = utilities.wrap_style ('trans-quoted-title', TransTitle ); -- for cite report, use this form for trans-title
 
else
 
else
 
Title = utilities.wrap_style ('italic-title', Title);
 
Title = utilities.wrap_style ('italic-title', Title);
Line 3,320: Line 3,313:  
end
 
end
   −
local TransError = "";
   
if utilities.is_set (TransTitle) then
 
if utilities.is_set (TransTitle) then
 
if utilities.is_set (Title) then
 
if utilities.is_set (Title) then
 
TransTitle = " " .. TransTitle;
 
TransTitle = " " .. TransTitle;
 
else
 
else
TransError = " " .. utilities.set_message ( 'err_trans_missing_title', {'title'} );
+
utilities.set_message ('err_trans_missing_title', {'title'});
 
end
 
end
 
end
 
end
Line 3,331: Line 3,323:  
if utilities.is_set (Title) then -- TODO: is this the right place to be making Wikisource URLs?
 
if utilities.is_set (Title) then -- TODO: is this the right place to be making Wikisource URLs?
 
if utilities.is_set (TitleLink) and utilities.is_set (URL) then
 
if utilities.is_set (TitleLink) and utilities.is_set (URL) then
table.insert( z.message_tail, { utilities.set_message ( 'err_wikilink_in_url', {}, true ) } ); -- set an error message because we can't have both
+
utilities.set_message ('err_wikilink_in_url'); -- set an error message because we can't have both
 
TitleLink = ''; -- unset
 
TitleLink = ''; -- unset
 
end
 
end
 
 
 
if not utilities.is_set (TitleLink) and utilities.is_set (URL) then
 
if not utilities.is_set (TitleLink) and utilities.is_set (URL) then
Title = external_link (URL, Title, URL_origin, UrlAccess) .. TransTitle .. TransError .. Format;
+
Title = external_link (URL, Title, URL_origin, UrlAccess) .. TransTitle .. Format;
 
URL = ''; -- unset these because no longer needed
 
URL = ''; -- unset these because no longer needed
 
Format = "";
 
Format = "";
Line 3,345: Line 3,337:  
Title = external_link (ws_url, Title .. '&nbsp;', 'ws link in title-link'); -- space char after Title to move icon away from italic text; TODO: a better way to do this?
 
Title = external_link (ws_url, Title .. '&nbsp;', 'ws link in title-link'); -- space char after Title to move icon away from italic text; TODO: a better way to do this?
 
Title = utilities.substitute (cfg.presentation['interwiki-icon'], {cfg.presentation['class-wikisource'], TitleLink, Title});
 
Title = utilities.substitute (cfg.presentation['interwiki-icon'], {cfg.presentation['class-wikisource'], TitleLink, Title});
Title = Title .. TransTitle .. TransError;
+
Title = Title .. TransTitle;
 
else
 
else
Title = utilities.make_wikilink (TitleLink, Title) .. TransTitle .. TransError;
+
Title = utilities.make_wikilink (TitleLink, Title) .. TransTitle;
 
end
 
end
 
else
 
else
Line 3,356: Line 3,348:  
Title = external_link (ws_url, Title .. '&nbsp;', 'ws link in title'); -- space char after Title to move icon away from italic text; TODO: a better way to do this?
 
Title = external_link (ws_url, Title .. '&nbsp;', 'ws link in title'); -- space char after Title to move icon away from italic text; TODO: a better way to do this?
 
Title = utilities.substitute (cfg.presentation['interwiki-icon'], {cfg.presentation['class-wikisource'], L, Title});
 
Title = utilities.substitute (cfg.presentation['interwiki-icon'], {cfg.presentation['class-wikisource'], L, Title});
Title = Title .. TransTitle .. TransError;
+
Title = Title .. TransTitle;
 
else
 
else
Title = Title .. TransTitle .. TransError;
+
Title = Title .. TransTitle;
 
end
 
end
 
end
 
end
 
else
 
else
Title = TransTitle .. TransError;
+
Title = TransTitle;
 
end
 
end
   Line 3,385: Line 3,377:     
if utilities.is_set (Minutes) then
 
if utilities.is_set (Minutes) then
if utilities.is_set (Time) then
+
if utilities.is_set (Time) then --TODO: make a function for this and similar?
table.insert( z.message_tail, { utilities.set_message ( 'err_redundant_parameters', {utilities.wrap_style ('parameter', 'minutes') .. ' and ' .. utilities.wrap_style ('parameter', 'time')}, true ) } );
+
utilities.set_message ('err_redundant_parameters', {utilities.wrap_style ('parameter', 'minutes') .. ' and ' .. utilities.wrap_style ('parameter', 'time')});
 
end
 
end
 
Position = " " .. Minutes .. " " .. cfg.messages['minutes'];
 
Position = " " .. Minutes .. " " .. cfg.messages['minutes'];
Line 3,448: Line 3,440:  
if utilities.is_set (Edition) then
 
if utilities.is_set (Edition) then
 
if Edition:match ('%f[%a][Ee]d%n?%.?$') or Edition:match ('%f[%a][Ee]dition$') then -- Ed, ed, Ed., ed., Edn, edn, Edn., edn.
 
if Edition:match ('%f[%a][Ee]d%n?%.?$') or Edition:match ('%f[%a][Ee]dition$') then -- Ed, ed, Ed., ed., Edn, edn, Edn., edn.
table.insert( z.message_tail, { utilities.set_message ( 'err_extra_text_edition')}); -- add error
+
utilities.set_message ('err_extra_text_edition'); -- add error message
 
end
 
end
 
Edition = " " .. wrap_msg ('edition', Edition);
 
Edition = " " .. wrap_msg ('edition', Edition);
Line 3,542: Line 3,534:  
-- TODO: Should we check a specific pattern?
 
-- TODO: Should we check a specific pattern?
 
if utilities.is_set(PostScript) and mw.ustring.len(PostScript) > 1 then
 
if utilities.is_set(PostScript) and mw.ustring.len(PostScript) > 1 then
utilities.set_message('maint_postscript')
+
utilities.set_message ('maint_postscript')
 
end
 
end
 
 
local Archived
+
local Archived;
 
if utilities.is_set (ArchiveURL) then
 
if utilities.is_set (ArchiveURL) then
 
local arch_text;
 
local arch_text;
 
if not utilities.is_set (ArchiveDate) then
 
if not utilities.is_set (ArchiveDate) then
ArchiveDate = utilities.set_message ('err_archive_missing_date');
+
utilities.set_message ('err_archive_missing_date');
 +
ArchiveDate = ''; -- empty string for concatenation
 
end
 
end
 
if "live" == UrlStatus then
 
if "live" == UrlStatus then
 
arch_text = cfg.messages['archived'];
 
arch_text = cfg.messages['archived'];
 
if sepc ~= "." then arch_text = arch_text:lower() end
 
if sepc ~= "." then arch_text = arch_text:lower() end
Archived = sepc .. " " .. utilities.substitute ( cfg.messages['archived-live'],
+
if utilities.is_set (ArchiveDate) then
{ external_link( ArchiveURL, arch_text, A:ORIGIN('ArchiveURL'), nil ) .. ArchiveFormat, ArchiveDate } );
+
Archived = sepc .. ' ' .. utilities.substitute ( cfg.messages['archived-live'],
 +
{external_link( ArchiveURL, arch_text, A:ORIGIN('ArchiveURL'), nil) .. ArchiveFormat, ArchiveDate } );
 +
else
 +
Archived = '';
 +
end
 
if not utilities.is_set (OriginalURL) then
 
if not utilities.is_set (OriginalURL) then
Archived = Archived .. " " .. utilities.set_message ('err_archive_missing_url');  
+
utilities.set_message ('err_archive_missing_url');
 +
Archived = ''; -- empty string for concatenation
 
end
 
end
 
elseif utilities.is_set (OriginalURL) then -- UrlStatus is empty, 'dead', 'unfit', 'usurped', 'bot: unknown'
 
elseif utilities.is_set (OriginalURL) then -- UrlStatus is empty, 'dead', 'unfit', 'usurped', 'bot: unknown'
Line 3,563: Line 3,561:  
arch_text = cfg.messages['archived-unfit'];
 
arch_text = cfg.messages['archived-unfit'];
 
if sepc ~= "." then arch_text = arch_text:lower() end
 
if sepc ~= "." then arch_text = arch_text:lower() end
Archived = sepc .. " " .. arch_text .. ArchiveDate; -- format already styled
+
Archived = sepc .. ' ' .. arch_text .. ArchiveDate; -- format already styled
 
if 'bot: unknown' == UrlStatus then
 
if 'bot: unknown' == UrlStatus then
 
utilities.set_message ('maint_bot_unknown'); -- and add a category if not already added
 
utilities.set_message ('maint_bot_unknown'); -- and add a category if not already added
Line 3,572: Line 3,570:  
arch_text = cfg.messages['archived-dead'];
 
arch_text = cfg.messages['archived-dead'];
 
if sepc ~= "." then arch_text = arch_text:lower() end
 
if sepc ~= "." then arch_text = arch_text:lower() end
Archived = sepc .. " " .. utilities.substitute ( arch_text,
+
if utilities.is_set (ArchiveDate) then
{ external_link( OriginalURL, cfg.messages['original'], OriginalURL_origin, OriginalAccess ) .. OriginalFormat, ArchiveDate } ); -- format already styled
+
Archived = sepc .. " " .. utilities.substitute ( arch_text,
 +
{ external_link( OriginalURL, cfg.messages['original'], OriginalURL_origin, OriginalAccess ) .. OriginalFormat, ArchiveDate } ); -- format already styled
 +
else
 +
Archived = ''; -- unset for concatenation
 +
end
 
end
 
end
 
else -- OriginalUrl not set
 
else -- OriginalUrl not set
 
arch_text = cfg.messages['archived-missing'];
 
arch_text = cfg.messages['archived-missing'];
 
if sepc ~= "." then arch_text = arch_text:lower() end
 
if sepc ~= "." then arch_text = arch_text:lower() end
Archived = sepc .. " " .. utilities.substitute ( arch_text,
+
utilities.set_message ('err_archive_missing_url');
{ utilities.set_message ('err_archive_missing_url'), ArchiveDate } );
+
Archived = ''; -- empty string for concatenation
 
end
 
end
 
elseif utilities.is_set (ArchiveFormat) then
 
elseif utilities.is_set (ArchiveFormat) then
 
Archived = ArchiveFormat; -- if set and ArchiveURL not set ArchiveFormat has error message
 
Archived = ArchiveFormat; -- if set and ArchiveURL not set ArchiveFormat has error message
 
else
 
else
Archived = ""
+
Archived = '';
 
end
 
end
 
 
Line 3,664: Line 3,666:  
]]
 
]]
 
if "speech" == config.CitationClass then -- cite speech only
 
if "speech" == config.CitationClass then -- cite speech only
TitleNote = " (Speech)"; -- annotate the citation
+
TitleNote = TitleType; -- move TitleType to TitleNote so that it renders ahead of |event=
 +
TitleType = ''; -- and unset
 +
 
 
if utilities.is_set (Periodical) then -- if Periodical, perhaps because of an included |website= or |journal= parameter  
 
if utilities.is_set (Periodical) then -- if Periodical, perhaps because of an included |website= or |journal= parameter  
 
if utilities.is_set (Conference) then -- and if |event= is set
 
if utilities.is_set (Conference) then -- and if |event= is set
Line 3,803: Line 3,807:     
-- Now enclose the whole thing in a <cite> element
 
-- Now enclose the whole thing in a <cite> element
local options = {};
+
local options_t = {};
+
options_t.class = cite_class_attribute_make (config.CitationClass, Mode);
if utilities.is_set (config.CitationClass) and config.CitationClass ~= "citation" then
+
 
options.class = string.format ('%s %s %s', 'citation', config.CitationClass, utilities.is_set (Mode) and Mode or 'cs1'); -- class=citation required for blue highlight when used with |ref=
+
local Ref = is_valid_parameter_value (A['Ref'], A:ORIGIN('Ref'), cfg.keywords_lists['ref'], nil, true); -- nil when |ref=harv; A['Ref'] else
else
  −
options.class = string.format ('%s %s', 'citation', utilities.is_set (Mode) and Mode or 'cs2');
  −
end
     −
local Ref = A['Ref'];
+
if 'none' ~= cfg.keywords_xlate[(Ref and Ref:lower()) or ''] then
if 'harv' == Ref then -- need to check this before setting to default
+
local namelist_t = {}; -- holds selected contributor, author, editor name list
utilities.set_message ('maint_ref_harv'); -- add maint cat to identify templates that have this now-extraneous param value
  −
elseif not utilities.is_set (Ref) then
  −
Ref = 'harv'; -- set as default when not set externally
  −
end
  −
if 'none' ~= cfg.keywords_xlate[Ref:lower()] then
  −
local id = Ref
  −
local namelist = {}; -- holds selected contributor, author, editor name list
   
local year = first_set ({Year, anchor_year}, 2); -- Year first for legacy citations and for YMD dates that require disambiguation
 
local year = first_set ({Year, anchor_year}, 2); -- Year first for legacy citations and for YMD dates that require disambiguation
    
if #c > 0 then -- if there is a contributor list
 
if #c > 0 then -- if there is a contributor list
namelist = c; -- select it
+
namelist_t = c; -- select it
 
elseif #a > 0 then -- or an author list
 
elseif #a > 0 then -- or an author list
namelist = a;
+
namelist_t = a;
 
elseif #e > 0 then -- or an editor list
 
elseif #e > 0 then -- or an editor list
namelist = e;
+
namelist_t = e;
 
end
 
end
local citeref_id
+
local citeref_id;
if #namelist > 0 then -- if there are names in namelist
+
if #namelist_t > 0 then -- if there are names in namelist_t
citeref_id = make_citeref_id (namelist, year); -- go make the CITEREF anchor
+
citeref_id = make_citeref_id (namelist_t, year); -- go make the CITEREF anchor
 +
if mw.uri.anchorEncode (citeref_id) == ((Ref and mw.uri.anchorEncode (Ref)) or '') then -- Ref may already be encoded (by {{sfnref}}) so citeref_id must be encoded before comparison
 +
utilities.set_message ('maint_ref_duplicates_default');
 +
end
 
else
 
else
 
citeref_id = ''; -- unset
 
citeref_id = ''; -- unset
 
end
 
end
if citeref_id == Ref then
+
options_t.id = Ref or citeref_id;
utilities.set_message ('maint_ref_duplicates_default');
  −
end
  −
if 'harv' == Ref then
  −
id = citeref_id
  −
end
  −
options.id = id;
   
end
 
end
+
 
if string.len(text:gsub("<span[^>/]*>(.-)</span>", "%1"):gsub("%b<>", "")) <= 2 then -- remove <span> tags and other HTML-like markup; then get length of what remains
+
if string.len (text:gsub('%b<>', '')) <= 2 then -- remove html and html-like tags; then get length of what remains;
z.error_categories = {};
+
z.error_cats_t = {}; -- blank the categories list
text = utilities.set_message ('err_empty_citation');
+
z.error_msgs_t = {}; -- blank the error messages list
z.message_tail = {};
+
OCinSoutput = nil; -- blank the metadata string
 +
text = ''; -- blank the the citation
 +
utilities.set_message ('err_empty_citation'); -- set empty citation message and category
 
end
 
end
 
 
local render = {}; -- here we collect the final bits for concatenation into the rendered citation
+
local render_t = {}; -- here we collect the final bits for concatenation into the rendered citation
   −
if utilities.is_set (options.id) then -- here we wrap the rendered citation in <cite ...>...</cite> tags
+
if utilities.is_set (options_t.id) then -- here we wrap the rendered citation in <cite ...>...</cite> tags
table.insert (render, utilities.substitute (cfg.presentation['cite-id'], {mw.uri.anchorEncode(options.id), mw.text.nowiki(options.class), text})); -- when |ref= is set
+
table.insert (render_t, utilities.substitute (cfg.presentation['cite-id'], {mw.uri.anchorEncode(options_t.id), mw.text.nowiki(options_t.class), text})); -- when |ref= is set or when there is a namelist
 
else
 
else
table.insert (render, utilities.substitute (cfg.presentation['cite'], {mw.text.nowiki(options.class), text})); -- all other cases
+
table.insert (render_t, utilities.substitute (cfg.presentation['cite'], {mw.text.nowiki(options_t.class), text})); -- when |ref=none or when namelist_t empty and |ref= is missing or is empty
 
end
 
end
   −
table.insert (render, utilities.substitute (cfg.presentation['ocins'], {OCinSoutput})); -- append metadata to the citation
+
if OCinSoutput then -- blanked when citation is 'empty' so don't bother to add boilerplate metadata span
 +
table.insert (render_t, utilities.substitute (cfg.presentation['ocins'], OCinSoutput)); -- format and append metadata to the citation
 +
end
 +
 
 +
local template_name = ('citation' == config.CitationClass) and 'citation' or 'cite ' .. (cfg.citation_class_map_t[config.CitationClass] or config.CitationClass);
 +
local template_link = '[[Template:' .. template_name .. '|' .. template_name .. ']]';
 +
local msg_prefix = '<code class="cs1-code">{{' .. template_link .. '}}</code>: ';
 +
 
 +
if 0 ~= #z.error_msgs_t then
 +
mw.addWarning (utilities.substitute (cfg.messages.warning_msg_e, template_link));
   −
if 0 ~= #z.message_tail then
+
table.insert (render_t, ' '); -- insert a space between citation and its error messages
table.insert (render, ' ');
+
table.sort (z.error_msgs_t); -- sort the error messages list; sorting includes wrapping <span> and <code> tags; hidden-error sorts ahead of visible-error
for i,v in ipairs( z.message_tail ) do
+
 
if utilities.is_set (v[1]) then
+
local hidden = true; -- presume that the only error messages emited by this template are hidden
if i == #z.message_tail then
+
for _, v in ipairs (z.error_msgs_t) do -- spin through the list of error messages
table.insert (render, utilities.error_comment ( v[1], v[2] ));
+
if v:find ('cs1-visible-error', 1, true) then -- look for the visible error class name
else
+
hidden = false; -- found one; so don't hide the error message prefix
table.insert (render, utilities.error_comment ( v[1] .. "; ", v[2] ));
+
break; -- and done because no need to look further
end
   
end
 
end
 
end
 
end
 +
 +
z.error_msgs_t[1] = table.concat ({utilities.error_comment (msg_prefix, hidden), z.error_msgs_t[1]}); -- add error message prefix to first error message to prevent extraneous punctuation
 +
table.insert (render_t, table.concat (z.error_msgs_t, '; ')); -- make a big string of error messages and add it to the rendering
 
end
 
end
   −
if 0 ~= #z.maintenance_cats then
+
if 0 ~= #z.maint_cats_t then
local maint_msgs = {}; -- here we collect all of the maint messages
+
mw.addWarning (utilities.substitute (cfg.messages.warning_msg_m, template_link));
for _, v in ipairs( z.maintenance_cats ) do -- append maintenance categories
+
 
local maint = {}; -- here we assemble a maintenence message
+
table.sort (z.maint_cats_t); -- sort the maintenance messages list
table.insert (maint, v); -- maint msg is the category name
+
 
table.insert (maint, ' ('); -- open the link text
+
local maint_msgs_t = {}; -- here we collect all of the maint messages
table.insert (maint, utilities.substitute (cfg.messages[':cat wikilink'], {v})); -- add the link
+
 
table.insert (maint, ')'); -- and close it
+
if 0 == #z.error_msgs_t then -- if no error messages
table.insert (maint_msgs, table.concat (maint)); -- assemble new maint message and add it to the maint_msgs table
+
table.insert (maint_msgs_t, msg_prefix); -- insert message prefix in maint message livery
 
end
 
end
table.insert (render, utilities.substitute (cfg.presentation['hidden-maint'], table.concat (maint_msgs, ' '))); -- wrap the group of maint message with proper presentation and save
+
 +
for _, v in ipairs( z.maint_cats_t ) do -- append maintenance categories
 +
table.insert (maint_msgs_t, -- assemble new maint message and add it to the maint_msgs_t table
 +
table.concat ({v, ' (', utilities.substitute (cfg.messages[':cat wikilink'], v), ')'})
 +
);
 +
end
 +
table.insert (render_t, utilities.substitute (cfg.presentation['hidden-maint'], table.concat (maint_msgs_t, ' '))); -- wrap the group of maint messages with proper presentation and save
 
end
 
end
+
 
 
if not no_tracking_cats then
 
if not no_tracking_cats then
for _, v in ipairs( z.error_categories ) do -- append error categories
+
for _, v in ipairs (z.error_cats_t) do -- append error categories
table.insert (render, utilities.substitute (cfg.messages['cat wikilink'], {v}));
+
table.insert (render_t, utilities.substitute (cfg.messages['cat wikilink'], v));
 
end
 
end
for _, v in ipairs( z.maintenance_cats ) do -- append maintenance categories
+
for _, v in ipairs (z.maint_cats_t) do -- append maintenance categories
table.insert (render, utilities.substitute (cfg.messages['cat wikilink'], {v}));
+
table.insert (render_t, utilities.substitute (cfg.messages['cat wikilink'], v));
 
end
 
end
for _, v in ipairs( z.properties_cats ) do -- append properties categories
+
for _, v in ipairs (z.prop_cats_t) do -- append properties categories
table.insert (render, utilities.substitute (cfg.messages['cat wikilink'], {v}));
+
table.insert (render_t, utilities.substitute (cfg.messages['cat wikilink'], v));
 
end
 
end
 
end
 
end
   −
return table.concat (render);
+
return table.concat (render_t); -- make a big string and done
 
end
 
end
   Line 3,924: Line 3,934:  
return true;
 
return true;
 
end
 
end
if 'discouraged' == state then
+
if 'tracked' == state then
discouraged_parameter (name); -- parameter is discouraged but still supported
+
local base_name = name:gsub ('%d', ''); -- strip enumerators from parameter names that have them to get the base name
 +
utilities.add_prop_cat ('tracked-param', {base_name}, base_name); -- add a properties category; <base_name> modifies <key>
 
return true;
 
return true;
 
end
 
end
Line 3,983: Line 3,994:  
 
 
if prefix and cfg.inter_wiki_map[prefix:lower()] then -- if prefix is in the map, needs preceding colon so
 
if prefix and cfg.inter_wiki_map[prefix:lower()] then -- if prefix is in the map, needs preceding colon so
table.insert( z.message_tail, {utilities.set_message ('err_bad_paramlink', parameter)}); -- emit an error message
+
utilities.set_message ('err_bad_paramlink', parameter); -- emit an error message
 
_, value, _ = utilities.is_wikilink (value); -- extract label portion from wikilink
 
_, value, _ = utilities.is_wikilink (value); -- extract label portion from wikilink
 
end
 
end
Line 4,010: Line 4,021:  
capture = value:match ('%s+(%a[%w%-]+)%s*=') or value:match ('^(%a[%w%-]+)%s*='); -- find and categorize parameters with possible missing pipes
 
capture = value:match ('%s+(%a[%w%-]+)%s*=') or value:match ('^(%a[%w%-]+)%s*='); -- find and categorize parameters with possible missing pipes
 
if capture and validate (capture) then -- if the capture is a valid parameter name
 
if capture and validate (capture) then -- if the capture is a valid parameter name
table.insert( z.message_tail, {utilities.set_message ('err_missing_pipe', parameter)});
+
utilities.set_message ('err_missing_pipe', parameter);
 
end
 
end
 
end
 
end
Line 4,032: Line 4,043:  
 
 
if value:match ('[,;:]$') then
 
if value:match ('[,;:]$') then
 +
utilities.set_message ('maint_extra_punct'); -- has extraneous punctuation; add maint cat
 +
end
 +
if value:match ('^=') then -- sometimes an extraneous '=' character appears ...
 
utilities.set_message ('maint_extra_punct'); -- has extraneous punctuation; add maint cat
 
utilities.set_message ('maint_extra_punct'); -- has extraneous punctuation; add maint cat
 +
end
 +
end
 +
 +
 +
--[[--------------------------< H A S _ E X T R A N E O U S _ U R L >------------------------------------------
 +
 +
look for extraneous url parameter values; parameters listed in skip table are not checked
 +
 +
]]
 +
 +
local function has_extraneous_url (url_param_t)
 +
local url_error_t = {};
 +
 +
check_for_url (url_param_t, url_error_t); -- extraneous url check
 +
if 0 ~= #url_error_t then -- non-zero when there are errors
 +
table.sort (url_error_t);
 +
utilities.set_message ('err_param_has_ext_link', {utilities.make_sep_list (#url_error_t, url_error_t)}); -- add this error message
 
end
 
end
 
end
 
end
Line 4,045: Line 4,076:  
local function citation(frame)
 
local function citation(frame)
 
Frame = frame; -- save a copy in case we need to display an error message in preview mode
 
Frame = frame; -- save a copy in case we need to display an error message in preview mode
 +
local sandbox = '/sandbox' -- i18n: replace this rvalue with the name that your wiki uses to identify sandbox subpages
 +
is_sandbox = nil ~= string.find (frame:getTitle(), sandbox, 1, true); -- is this invoke the sandbox module?
 +
sandbox = is_sandbox and sandbox or ''; -- use i18n sandbox to load sandbox modules when this module is the sandox; live modules else
 +
 
local pframe = frame:getParent()
 
local pframe = frame:getParent()
 
local styles;
 
local styles;
 
 
if nil ~= string.find (frame:getTitle(), 'sandbox', 1, true) then -- did the {{#invoke:}} use sandbox version?
+
cfg = mw.loadData ('Module:Citation/CS1/Configuration' .. sandbox); -- load sandbox versions of support modules when {{#invoke:Citation/CS1/sandbox|...}}; live modules else
cfg = mw.loadData ('Module:Citation/CS1/Configuration/sandbox'); -- load sandbox versions of support modules
+
whitelist = mw.loadData ('Module:Citation/CS1/Whitelist' .. sandbox);
whitelist = mw.loadData ('Module:Citation/CS1/Whitelist/sandbox');
+
utilities = require ('Module:Citation/CS1/Utilities' .. sandbox);
utilities = require ('Module:Citation/CS1/Utilities/sandbox');
+
validation = require ('Module:Citation/CS1/Date_validation' .. sandbox);
validation = require ('Module:Citation/CS1/Date_validation/sandbox');
+
identifiers = require ('Module:Citation/CS1/Identifiers' .. sandbox);
identifiers = require ('Module:Citation/CS1/Identifiers/sandbox');
+
metadata = require ('Module:Citation/CS1/COinS' .. sandbox);
metadata = require ('Module:Citation/CS1/COinS/sandbox');
+
styles = 'Module:Citation/CS1' .. sandbox .. '/styles.css';
styles = 'Module:Citation/CS1/sandbox/styles.css';
  −
  −
else -- otherwise
  −
cfg = mw.loadData ('Module:Citation/CS1/Configuration'); -- load live versions of support modules
  −
whitelist = mw.loadData ('Module:Citation/CS1/Whitelist');
  −
utilities = require ('Module:Citation/CS1/Utilities');
  −
validation = require ('Module:Citation/CS1/Date_validation');
  −
identifiers = require ('Module:Citation/CS1/Identifiers');
  −
metadata = require ('Module:Citation/CS1/COinS');
  −
styles = 'Module:Citation/CS1/styles.css';
  −
end
      
utilities.set_selected_modules (cfg); -- so that functions in Utilities can see the selected cfg tables
 
utilities.set_selected_modules (cfg); -- so that functions in Utilities can see the selected cfg tables
Line 4,073: Line 4,097:     
z = utilities.z; -- table of error and category tables in Module:Citation/CS1/Utilities
 
z = utilities.z; -- table of error and category tables in Module:Citation/CS1/Utilities
 +
 +
is_preview_mode = not utilities.is_set (frame:preprocess ('{{REVISIONID}}'));
    
local args = {}; -- table where we store all of the template's arguments
 
local args = {}; -- table where we store all of the template's arguments
 
local suggestions = {}; -- table where we store suggestions if we need to loadData them
 
local suggestions = {}; -- table where we store suggestions if we need to loadData them
local error_text, error_state;
+
local error_text; -- used as a flag
    
local config = {}; -- table to store parameters from the module {{#invoke:}}
 
local config = {}; -- table to store parameters from the module {{#invoke:}}
Line 4,093: Line 4,119:  
end
 
end
 
if not validate( k, config.CitationClass ) then
 
if not validate( k, config.CitationClass ) then
error_text = "";
+
if type (k) ~= 'string' then -- exclude empty numbered parameters
if type( k ) ~= 'string' then
  −
-- exclude empty numbered parameters
   
if v:match("%S+") ~= nil then
 
if v:match("%S+") ~= nil then
error_text, error_state = utilities.set_message ( 'err_text_ignored', {v}, true );
+
error_text = utilities.set_message ('err_text_ignored', {v});
 
end
 
end
elseif validate( k:lower(), config.CitationClass ) then  
+
elseif validate (k:lower(), config.CitationClass) then  
error_text, error_state = utilities.set_message ( 'err_parameter_ignored_suggest', {k, k:lower()}, true ); -- suggest the lowercase version of the parameter
+
error_text = utilities.set_message ('err_parameter_ignored_suggest', {k, k:lower()}); -- suggest the lowercase version of the parameter
 
else
 
else
 
if nil == suggestions.suggestions then -- if this table is nil then we need to load it
 
if nil == suggestions.suggestions then -- if this table is nil then we need to load it
if nil ~= string.find (frame:getTitle(), 'sandbox', 1, true) then -- did the {{#invoke:}} use sandbox version?
+
if is_sandbox then -- did the {{#invoke:}} use sandbox version?
 
suggestions = mw.loadData( 'Module:Citation/CS1/Suggestions/sandbox' ); -- use the sandbox version
 
suggestions = mw.loadData( 'Module:Citation/CS1/Suggestions/sandbox' ); -- use the sandbox version
 
else
 
else
Line 4,114: Line 4,138:  
param = utilities.substitute (param, capture); -- add the capture to the suggested parameter (typically the enumerator)
 
param = utilities.substitute (param, capture); -- add the capture to the suggested parameter (typically the enumerator)
 
if validate (param, config.CitationClass) then -- validate the suggestion to make sure that the suggestion is supported by this template (necessary for limited parameter lists)
 
if validate (param, config.CitationClass) then -- validate the suggestion to make sure that the suggestion is supported by this template (necessary for limited parameter lists)
error_text, error_state = utilities.set_message ('err_parameter_ignored_suggest', {k, param}, true); -- set the suggestion error message
+
error_text = utilities.set_message ('err_parameter_ignored_suggest', {k, param}); -- set the suggestion error message
 
else
 
else
error_text, error_state = utilities.set_message ( 'err_parameter_ignored', {k}, true ); -- suggested param not supported by this template
+
error_text = utilities.set_message ('err_parameter_ignored', {k}); -- suggested param not supported by this template
 
v = ''; -- unset
 
v = ''; -- unset
 
end
 
end
Line 4,123: Line 4,147:  
if not utilities.is_set (error_text) then -- couldn't match with a pattern, is there an explicit suggestion?
 
if not utilities.is_set (error_text) then -- couldn't match with a pattern, is there an explicit suggestion?
 
if (suggestions.suggestions[ k:lower() ] ~= nil) and validate (suggestions.suggestions[ k:lower() ], config.CitationClass) then
 
if (suggestions.suggestions[ k:lower() ] ~= nil) and validate (suggestions.suggestions[ k:lower() ], config.CitationClass) then
error_text, error_state = utilities.set_message ( 'err_parameter_ignored_suggest', {k, suggestions.suggestions[ k:lower() ]}, true );
+
utilities.set_message ('err_parameter_ignored_suggest', {k, suggestions.suggestions[ k:lower() ]});
 
else
 
else
error_text, error_state = utilities.set_message ( 'err_parameter_ignored', {k}, true );
+
utilities.set_message ('err_parameter_ignored', {k});
 
v = ''; -- unset value assigned to unrecognized parameters (this for the limited parameter lists)
 
v = ''; -- unset value assigned to unrecognized parameters (this for the limited parameter lists)
 
end
 
end
 
end
 
end
 
end    
 
end    
if error_text ~= '' then
  −
table.insert( z.message_tail, {error_text, error_state} );
  −
end
   
end
 
end
   Line 4,149: Line 4,170:     
if 0 ~= #empty_unknowns then -- create empty unknown error message
 
if 0 ~= #empty_unknowns then -- create empty unknown error message
table.insert (z.message_tail, {utilities.set_message ('err_param_unknown_empty', {
+
utilities.set_message ('err_param_unknown_empty', {
 
1 == #empty_unknowns and '' or 's',
 
1 == #empty_unknowns and '' or 's',
 
utilities.make_sep_list (#empty_unknowns, empty_unknowns)
 
utilities.make_sep_list (#empty_unknowns, empty_unknowns)
}, true )});
+
});
 
end
 
end
 +
 +
local url_param_t = {};
    
for k, v in pairs( args ) do
 
for k, v in pairs( args ) do
Line 4,162: Line 4,185:  
missing_pipe_check (k, v); -- do we think that there is a parameter that is missing a pipe?
 
missing_pipe_check (k, v); -- do we think that there is a parameter that is missing a pipe?
 
args[k] = inter_wiki_check (k, v); -- when language interwiki-linked parameter missing leading colon replace with wiki-link label
 
args[k] = inter_wiki_check (k, v); -- when language interwiki-linked parameter missing leading colon replace with wiki-link label
 +
 +
if 'string' == type (k) and not cfg.url_skip[k] then -- when parameter k is not positional and not in url skip table
 +
url_param_t[k] = v; -- make a parameter/value list for extraneous url check
 +
end
 
end
 
end
 +
 +
has_extraneous_url (url_param_t); -- look for url in parameter values where a url does not belong
    
return table.concat ({
 
return table.concat ({
Line 4,169: Line 4,198:  
});
 
});
 
end
 
end
 +
    
--[[--------------------------< E X P O R T E D  F U N C T I O N S >------------------------------------------
 
--[[--------------------------< E X P O R T E D  F U N C T I O N S >------------------------------------------
Anonymous user

Navigation menu