Merge "rdbms: clarify IDatabase::setTransactionListener() comment"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Thu, 4 Oct 2018 02:46:19 +0000 (02:46 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Thu, 4 Oct 2018 02:46:19 +0000 (02:46 +0000)
1  2 
includes/libs/rdbms/database/IDatabase.php

@@@ -486,8 -486,8 +486,8 @@@ interface IDatabase 
         * Close the database connection
         *
         * This should only be called after any transactions have been resolved,
 -       * aside from read-only transactions (assuming no callbacks are registered).
 -       * If a transaction is still open anyway, it will be committed if possible.
 +       * aside from read-only automatic transactions (assuming no callbacks are registered).
 +       * If a transaction is still open anyway, it will be rolled back.
         *
         * @throws DBError
         * @return bool Operation success. true if already closed.
         * Escaping of untrusted input used in values of numeric keys should be done via
         * IDatabase::addQuotes()
         *
 +       * Use an empty array, string, or '*' to update all rows.
 +       *
         * @param string|array $options
         *
         * Optional: Array of query options. Boolean options are specified by
        public function onTransactionPreCommitOrIdle( callable $callback, $fname = __METHOD__ );
  
        /**
-        * Run a callback each time any transaction commits or rolls back
+        * Run a callback after each time any transaction commits or rolls back
         *
         * The callback takes two arguments:
         *   - IDatabase::TRIGGER_COMMIT or IDatabase::TRIGGER_ROLLBACK