Merge "Various small cleanups to ResultWrapper"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Mon, 19 Sep 2016 16:18:50 +0000 (16:18 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Mon, 19 Sep 2016 16:18:50 +0000 (16:18 +0000)
1  2 
includes/libs/rdbms/database/DatabaseSqlite.php

@@@ -300,12 -300,12 +300,12 @@@ class DatabaseSqlite extends DatabaseBa
                $res = $this->mConn->query( $sql );
                if ( $res === false ) {
                        return false;
-               } else {
-                       $r = $res instanceof ResultWrapper ? $res->result : $res;
-                       $this->mAffectedRows = $r->rowCount();
-                       $res = new ResultWrapper( $this, $r->fetchAll() );
                }
  
+               $r = $res instanceof ResultWrapper ? $res->result : $res;
+               $this->mAffectedRows = $r->rowCount();
+               $res = new ResultWrapper( $this, $r->fetchAll() );
                return $res;
        }
  
                return 'CAST ( ' . $field . ' AS TEXT )';
        }
  
 -      /**
 -       * @return string
 -       */
 -      public function getSearchEngine() {
 -              return "SearchSqlite";
 -      }
 -
        /**
         * No-op version of deadlockLoop
         *
                return 'SQLite ' . (string)$this->mConn->getAttribute( PDO::ATTR_SERVER_VERSION );
        }
  
- } // end DatabaseSqlite class
+ }