changetags: $param type should be string and not array
authorDerick Alangi <alangiderick@gmail.com>
Mon, 15 Apr 2019 20:33:18 +0000 (21:33 +0100)
committerUmherirrender <umherirrender_de.wp@web.de>
Tue, 16 Apr 2019 20:02:38 +0000 (20:02 +0000)
Per the call to updateTagsWithChecks(), $params should be a string
and not an array as soft type hinted. The previous calls updateTags()
which uses $param as a string or null.

Change-Id: I87fa39682d91b7cb3b0844d0aabbfaea0cb6f206

includes/changetags/ChangeTags.php
includes/changetags/ChangeTagsLogList.php

index 3a93e57..2169a4d 100644 (file)
@@ -604,7 +604,7 @@ class ChangeTags {
         * @param int|null $rc_id The rc_id of the change to add the tags to
         * @param int|null $rev_id The rev_id of the change to add the tags to
         * @param int|null $log_id The log_id of the change to add the tags to
-        * @param string $params Params to put in the ct_params field of table
+        * @param string|null $params Params to put in the ct_params field of table
         * 'change_tag' when adding tags
         * @param string $reason Comment for the log
         * @param User $user Who to give credit for the action
index 7e1f1d1..bf455af 100644 (file)
@@ -66,7 +66,7 @@ class ChangeTagsLogList extends ChangeTagsList {
         *
         * @param array $tagsToAdd
         * @param array $tagsToRemove
-        * @param array $params
+        * @param string|null $params
         * @param string $reason
         * @param User $user
         * @return Status