Fix RecentChange::parseParams @return docs
authoraddshore <addshorewiki@gmail.com>
Mon, 18 Apr 2016 10:06:09 +0000 (11:06 +0100)
committerAddshore <addshorewiki@gmail.com>
Mon, 18 Apr 2016 10:13:10 +0000 (10:13 +0000)
Per the docs of unserialize:
In case the passed string is not unserializeable, false is returned.

Also as we can not guarentee an array was serialized in
rc_params we cant say it will always be an array...
Quite often this is just an empty string which will
result in: "s:0:"";" (not an array)....

Change-Id: Ib22f9f7468a23ce3d4f145d47f8514d08be6c957

includes/changes/RecentChange.php

index 2508304..3fe5f2f 100644 (file)
@@ -973,7 +973,7 @@ class RecentChange {
         *
         * @since 1.26
         *
-        * @return array|null
+        * @return mixed|bool false on failed unserialization
         */
        public function parseParams() {
                $rcParams = $this->getAttribute( 'rc_params' );