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

Change subject: Fix notices on undefined $fileHandle and $res
......................................................................


Fix notices on undefined $fileHandle and $res

Change-Id: If9f37549af85df0a418dbd12a7793e5a45868f61
---
M maintenance/cdb.php
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Aaron Schulz: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/maintenance/cdb.php b/maintenance/cdb.php
index bda64f3..1772b5f 100644
--- a/maintenance/cdb.php
+++ b/maintenance/cdb.php
@@ -52,7 +52,7 @@
        $bad = false;
        $showhelp = false;
        $quit = false;
-       static $fileHandle;
+       static $fileHandle = false;
 
        $line = Maintenance::readconsole();
        if ( $line === false ) {
@@ -98,6 +98,7 @@
                                $res = $fileHandle->get( $args[0] );
                        } catch ( CdbException $e ) {
                                print "Unable to read key from file\n";
+                               break;
                        }
                        if ( $res === false ) {
                                print "No such key/value pair\n";

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

Gerrit-MessageType: merged
Gerrit-Change-Id: If9f37549af85df0a418dbd12a7793e5a45868f61
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Chad <ch...@wikimedia.org>
Gerrit-Reviewer: Aaron Schulz <asch...@wikimedia.org>
Gerrit-Reviewer: Parent5446 <tylerro...@gmail.com>
Gerrit-Reviewer: jenkins-bot

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to