Page history

Psmithca

1 revision imported

w>MusikAnimal

1 revision imported

w>MusikAnimal

Protected "Module:Arguments": High-risk template ([Edit=Allow only administrators] (indefinite) [Move=Allow only administrators] (indefinite))

>Xaosflux

Changed protection level of Module:Arguments: used in the mediawiki interface ex: MediaWiki:Watchlist-details ([Edit=Allow only administrators] (indefinite) [Move=Allow only administrators] (indefinite))

>Jackmcbarn

add argument translation from sandbox - all tests pass

>Jackmcbarn

apply changes from sandbox - all tests pass

>Mr. Stradivarius

fix bug where explicitly deleted args were still appearing when iterated over with pairs or ipairs - code courtesy of User:Jackmcbarn

>Mr. Stradivarius

allow using both the frame and parent frame arguments with the wrappers option if specifically requested, per protected edit request by User:Jackmcbarn

>Mr. Stradivarius

add a "wrappers" option and fix bug when frame:getParent() returns nil, per protected edit request by User:Jackmcbarn

>Mr. Stradivarius

memoize nils using a separate nilArgs table; this fixes a bug where nil values were being iterated over with pairs()

>Mr. Stradivarius

fix undefined next() behaviour bug by checking for metatable.donePairs in the __index metamethod; also, format the module so it fits into 80 characters

>Mr. Stradivarius

tweak one of the comments and make some beautification fixes, now that this is in the job queue

>Anomie

Fix __pairs and __ipairs functions to handle the nil marker

>Mr. Stradivarius

Changed protection level of Module:Arguments: High-risk Lua module ([Edit=Allow only template editors and admins] (indefinite) [Move=Allow only template editors and admins] (indefinite))

>Mr. Stradivarius

tweak comment

>Mr. Stradivarius

memoize nils, add type checks for frame and options, use string library functions instead of mw.text.trim and mw.ustring.find, define four different tidyVal functions to avoid checking options every time

>Mr. Stradivarius

don't use "if frame == mw.getCurrentFrame()"

>Mr. Stradivarius

Protected Module:Arguments: High-risk Lua module ([Edit=Allow only autoconfirmed users] (indefinite) [Move=Allow only autoconfirmed users] (indefinite))

>Mr. Stradivarius

oops, forgot the first parameter to tidyVal

>Mr. Stradivarius

add options.frameOnly and options.parentOnly

>Mr. Stradivarius

use an array to hold argument tables rather than firstArgs and secondArgs variables

>Mr. Stradivarius

check fargs and pargs before writing new values to args if options.noOverwrite is set

>Mr. Stradivarius

add a more detailed comment about metatable behaviour

>Mr. Stradivarius

return a table rather than a function, improve the comments

>Mr. Stradivarius

create argument-processing module