31 October 2022
1 April 2020
13 November 2017
11 November 2014
5 April 2014
m>Mr. Stradivarius
bug fix - use the demospace parameter as both key and value in the argKeys table
+43
m>Mr. Stradivarius
use "demopage" instead of "page" as the default page parameter - needed to fix bug on en.wikisource where "Page" is a namespace name
+4
24 March 2014
m>Mr. Stradivarius
Protected Module:Namespace detect/data: High-risk Lua module ([Edit=Allow only administrators] (indefinite) [Move=Allow only administrators] (indefinite))
mm>Mr. Stradivarius
found it - mappings[nsname] was getting an upper-case key
−42
m>Mr. Stradivarius
Undid revision 601018314 by Mr. Stradivarius (talk) looks like that broke something
+42
m>Mr. Stradivarius
avoid using local variables to save table lookups per Jackmcbarn's suggestion, and because this will be cached with mw.loadData so performance of this function is not such a worry
−42
22 March 2014
m>Mr. Stradivarius
give the argKeys table global scope so that getParamMappings can access it without having to process the cfg table again
−54
m>Mr. Stradivarius
add a getArgKeys function, so that we can have multiple keys for each argument (helpful for localisation) and so that we only have to create that table once per page
+1,003
21 March 2014
m>Jackmcbarn
Undid revision 600634263 by Jackmcbarn (talk): Oh, this seems to have been for performance. Still not sure if it's a good idea, but will leave for now
+95
m>Jackmcbarn
IMO, aliasing variables like that makes readability worse
−95
m>Mr. Stradivarius
use a dedicated config page - Module:Namespace detect/config - for configuration data, and try and reduce unnecessary table lookups in the getParamMappings function
−1,166