Fix IDEA Ultimate warnings in ChangeTags
authorAaron Schulz <aschulz@wikimedia.org>
Tue, 20 Sep 2016 18:12:26 +0000 (11:12 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Tue, 20 Sep 2016 18:12:26 +0000 (11:12 -0700)
Change-Id: Ie0778e5e32be81bda246d231526630db700b8538

includes/changetags/ChangeTags.php

index 3ef9641..6455a3a 100644 (file)
@@ -571,7 +571,7 @@ class ChangeTags {
                        // This function is from revision deletion logic and has nothing to do with
                        // change tags, but it appears to be the only other place in core where we
                        // perform logged actions on log items.
-                       $logEntry->setTarget( RevDelLogList::suggestTarget( 0, [ $log_id ] ) );
+                       $logEntry->setTarget( RevDelLogList::suggestTarget( null, [ $log_id ] ) );
                }
 
                if ( !$logEntry->getTarget() ) {
@@ -1038,7 +1038,7 @@ class ChangeTags {
                // let's not allow error results, as the actual tag deletion succeeded
                if ( !$status->isOK() ) {
                        wfDebug( 'ChangeTagAfterDelete error condition downgraded to warning' );
-                       $status->ok = true;
+                       $status->setOK( true );
                }
 
                // clear the memcache of defined tags