MDL-20699 more abstraction in default exception handler

nilo72 · Nov 1, 2009 · a56c457 · a56c457
1 parent 695c5ec
commit a56c457
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 7 deletions.
diff --git a/lib/setuplib.php b/lib/setuplib.php
@@ -168,11 +168,6 @@ function __construct($hint, $debuginfo=null) {
 function default_exception_handler($ex) {
     global $DB, $OUTPUT;
 
-    if ($DB) {
-        // If you enable db debugging and exception is thrown, the print footer prints a lot of rubbish
-        $DB->set_debug(0);
-    }
-
     // detect active db transactions, rollback and log as error
     abort_all_db_transactions();
 
@@ -187,6 +182,10 @@ function default_exception_handler($ex) {
         echo bootstrap_renderer::early_error($info->message, $info->moreinfourl, $info->link, $info->backtrace, $info->debuginfo);
     } else {
         try {
+            if ($DB) {
+                // If you enable db debugging and exception is thrown, the print footer prints a lot of rubbish
+                $DB->set_debug(0);
+            }
             echo $OUTPUT->fatal_error($info->message, $info->moreinfourl, $info->link, $info->backtrace, $info->debuginfo);
         } catch (Exception $out_ex) {
             // default exception handler MUST not throw any exceptions!!

diff --git a/lib/upgradelib.php b/lib/upgradelib.php
@@ -840,6 +840,9 @@ function external_delete_descriptions($component) {
  * upgrade logging functions
  */
 function upgrade_handle_exception($ex, $plugin = null) {
+    // rollback everything, we need to log all upgrade problems
+    abort_all_db_transactions();
+
     $info = get_exception_info($ex);
 
     // First log upgrade error
@@ -1174,7 +1177,6 @@ function install_core($version, $verbose) {
 
         print_upgrade_part_end(null, true, $verbose);
     } catch (exception $ex) {
-        //TODO: unconditionally rollback DB transaction if active
         upgrade_handle_exception($ex);
     }
 }
@@ -1225,7 +1227,6 @@ function upgrade_core($version, $verbose) {
 
         print_upgrade_part_end('moodle', false, $verbose);
     } catch (Exception $ex) {
-        //TODO: unconditionally rollback DB transaction if active
         upgrade_handle_exception($ex);
     }
 }