mw.Map: Check presence of value argument in set()
authorTimo Tijhof <krinklemail@gmail.com>
Sun, 11 Jan 2015 14:03:34 +0000 (06:03 -0800)
committerTimo Tijhof <krinklemail@gmail.com>
Tue, 13 Jan 2015 07:13:04 +0000 (23:13 -0800)
commitd04b7618b566b02cea5e6544c1b4169104da5374
tree7e5f977e22fe4f5004332d1d73e46779206ebaba
parent00f6574a1a6bb3c0891f336702ee5539d7bb057a
mw.Map: Check presence of value argument in set()

Add back the `arguments.length > 1` check (accidentally removed
in 24f84b0). Otherwise it inadvertently uses the `value`
parameter, causing it to set undefined as the value.

There was already a test ensuring undefined can be set as value,
but a test to ensure it doesn't default to undefined was missing.

Change-Id: I4c69f0c11f165640a9b387a72c77c48eb6aa9e72
resources/src/mediawiki/mediawiki.js
tests/qunit/suites/resources/mediawiki/mediawiki.test.js