IDatabase::commit: Docs re 'flush' exception on explicit transaction
authorMatthew Flaschen <mflaschen@wikimedia.org>
Wed, 24 Feb 2016 23:22:30 +0000 (18:22 -0500)
committerMatthew Flaschen <mflaschen@wikimedia.org>
Wed, 24 Feb 2016 23:22:30 +0000 (18:22 -0500)
Change-Id: I44087e92d2186f4e931b3f0766caba1f90f4ebc4

includes/db/IDatabase.php

index e1d1173..7855861 100644 (file)
@@ -1334,9 +1334,13 @@ interface IDatabase {
         * @param string $fname
         * @param string $flush Flush flag, set to 'flush' to disable warnings about
         *   explicitly committing implicit transactions, or calling commit when no
-        *   transaction is in progress. This will silently break any ongoing
-        *   explicit transaction. Only set the flush flag if you are sure that it
-        *   is safe to ignore these warnings in your context.
+        *   transaction is in progress.
+        *
+        *   This will trigger an exception if there is an ongoing explicit transaction.
+        *
+        *   Only set the flush flag if you are sure that these warnings are not applicable,
+        *   and no explicit transactions are open.
+        *
         * @throws DBUnexpectedError
         */
        public function commit( $fname = __METHOD__, $flush = '' );