Page history

m>Mr. Stradivarius

use hyphens for message names rather than camelCase, in line with MediaWiki

m>Mr. Stradivarius

make 'string' the default message type in the message function to avoid having 'string' all over the place

m>Mr. Stradivarius

remove the err function, as it's not being used any more

m>Mr. Stradivarius

allow documentation passed with the |content= parameter to show even if we get errors from the env table

m>Mr. Stradivarius

simplify the start box code and attempt to get some more output if the env table contains errors

m>Mr. Stradivarius

use "ugsub" rather than "gsub" as the abbreviation for mw.ustring.gsub, to make it clearer that it isn't the regular string.gsub

m>Mr. Stradivarius

oops, looks like I issued a command that put everything in lower case somewhere along the line

m>Mr. Stradivarius

only generate the URL for the compare link in the env table - the rest is now done elsewhere so that the display can be different in different places

m>Mr. Stradivarius

fix subjectSpace var in makeSubpagesBlurb

m>Mr. Stradivarius

clean up the env table functions and the comments

m>Mr. Stradivarius

fix revision id code

m>Mr. Stradivarius

sandbox notice - separate test cases blurb with a line break rather than a space

m>Mr. Stradivarius

convert the sandbox notice code to Lua

m>Mr. Stradivarius

move diff link to the env table

m>Mr. Stradivarius

p.makeStartBoxLinksData - fix title objects that weren't being passed through

m>Mr. Stradivarius

get rid of the env:grab function

m>Mr. Stradivarius

use the err function for all the errors

m>Mr. Stradivarius

finish moving end box code into sub-functions

m>Mr. Stradivarius

simplify the content function

m>Mr. Stradivarius

need to check type as well

m>Mr. Stradivarius

control for blank values of linksData

m>Mr. Stradivarius

replace another argument name with a hardcoded value

m>Mr. Stradivarius

hardcode "off" in "link box=off" as well, for the benefit of other modules/templates

m>Mr. Stradivarius

don't use messages for args either - would be a pain for other modules that have to use this module

m>Mr. Stradivarius

don't use messages for fmbox parameter names

m>Mr. Stradivarius

use messages for the err function instead of hard-coded values

m>Mr. Stradivarius

add a categories blurb function

m>Mr. Stradivarius

add a templateTitle object to the env functions, and reformat the experimentBlurb function to make it fit on the page

m>Mr. Stradivarius

allow for nil values of makeDocPageBlurb

m>Mr. Stradivarius

split out docpage blurb code

m>Mr. Stradivarius

fix the docpage thing again

m>Mr. Stradivarius

split out experiment blurb code

m>Mr. Stradivarius

use docTitle.prefixedText instead of docpage

m>Mr. Stradivarius

finish splitting the _startBox function into smaller functions

m>Mr. Stradivarius

add functions for rendering start box links and for making the start box link data

m>Mr. Stradivarius

create new function to separate html rendering in the start box

m>Mr. Stradivarius

fix typo

m>Mr. Stradivarius

add local subjectSpace variable to _startBox

m>Mr. Stradivarius

make all the helper functions available in the p table

m>Mr. Stradivarius

make a couple of the helper functions available for testing

m>Mr. Stradivarius

convert env table to use a metatable so we only process things when we need to; add a grab method and an err function for dealing with errors (main functions need to be converted to use these)

m>Mr. Stradivarius

finish converting existing values to use env table

m>Mr. Stradivarius

start conversion to a p.getEnv function rather than using globals; I've only got as far as the start box, so the module currently won't run

m>Mr. Stradivarius

rename formatMessage --> message, as it is no longer just doing formatting; tweak the function's comment as well

m>Mr. Stradivarius

found it - fix misspelled message name

m>Mr. Stradivarius

test otherNamespaceHeading

m>Mr. Stradivarius

add some newlines to make the html slightly more readable

m>Mr. Stradivarius

check for type of cfg messages rather than just for nil

m>Mr. Stradivarius

check existence of all cfg messages - this should help avoid hard-to-track-down bugs and silent failures if translators make a coding mistake

m>Mr. Stradivarius

set the proper error level for formatMessage, although it looks like it doesn't work because of the upvalues