jenkins-bot has submitted this change and it was merged.

Change subject: BagOStuff: Don't use $this in a closure
......................................................................


BagOStuff: Don't use $this in a closure

Change-Id: Ib9fb92a70adc2b4d75c30148e17dff6f7765afe1
---
M includes/libs/objectcache/BagOStuff.php
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Ori.livneh: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/libs/objectcache/BagOStuff.php 
b/includes/libs/objectcache/BagOStuff.php
index d72e1a5..2756143 100644
--- a/includes/libs/objectcache/BagOStuff.php
+++ b/includes/libs/objectcache/BagOStuff.php
@@ -294,7 +294,7 @@
                        $latency = .050; // latency skew (err towards keeping 
lock present)
                        $age = ( microtime( true ) - $lSince + $latency );
                        if ( ( $age + $latency ) >= $expiry ) {
-                               $this->logger->warning( "Lock for $key held too 
long ($age sec)." );
+                               $that->logger->warning( "Lock for $key held too 
long ($age sec)." );
                                return; // expired; it's not "safe" to delete 
the key
                        }
                        $that->unlock( $key );

-- 
To view, visit https://gerrit.wikimedia.org/r/233046
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib9fb92a70adc2b4d75c30148e17dff6f7765afe1
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Legoktm <[email protected]>
Gerrit-Reviewer: Ori.livneh <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to