Fix Status::getMessage accidentially returning string instead of Message
authorThiemo Mättig <thiemo.maettig@wikimedia.de>
Thu, 6 Mar 2014 20:16:27 +0000 (21:16 +0100)
committerThiemo Mättig <thiemo.maettig@wikimedia.de>
Mon, 17 Mar 2014 17:46:00 +0000 (18:46 +0100)
commit2dd32c82ffe5677dec64933c06e0ddbfa2d7c476
treefbede5d1e5505f08dc313c60910143c676e2bc68
parent705c4adb63fdb2a32b644a8b89596d9d8b779689
Fix Status::getMessage accidentially returning string instead of Message

This mistake was introduced in commit
92e284d3fa62f45e20fed34c4359c575481d583c and the reason for the two
disabled tests. I did not enabled the second test because of an
unrelated problem. The first enabled test already covers the fix.

The method should return Message objects only but did return a
string in that special case (multiple warnings set but no context
message key).

Unfortunatelly Status::getMessage does have many, many more
problems but I understand it's not a good idea to address them all
in a single confusing patch.

Change-Id: I0dc37e248f407019d5921aaaca3eabba338b0fd3
includes/Status.php
tests/phpunit/includes/StatusTest.php