Make RecentChange::addTags() accept a string
authorKunal Mehta <legoktm@member.fsf.org>
Tue, 27 Sep 2016 21:50:13 +0000 (14:50 -0700)
committerKunal Mehta <legoktm@member.fsf.org>
Tue, 27 Sep 2016 21:50:13 +0000 (14:50 -0700)
Most planned callers of RecentChange::addTags() only want to add a
single tag, so forcing them to create an array seems unfriendly. Plus
this lets us avoid a array_merge call.

Change-Id: If001ba9ff01a33be158e7edd5c5e4de9825fa180

includes/changes/RecentChange.php

index 7ae1f29..584c813 100644 (file)
@@ -1037,9 +1037,13 @@ class RecentChange {
         *
         * @since 1.28
         *
-        * @param array $tags
+        * @param string|array $tags
         */
        public function addTags( $tags ) {
-               $this->tags = array_merge( $tags, $this->tags );
+               if ( is_string( $tags ) ) {
+                       $this->tags[] = $tags;
+               } else {
+                       $this->tags = array_merge( $tags, $this->tags );
+               }
        }
 }