Given that that transaction level counting is now implemented, there's no...

Given that that transaction level counting is now implemented, there's no reason to do it ourselves when committing artefacts.

htdocs/artefact/lib.php

@@ -364,9 +364,7 @@ abstract class ArtefactType {
             return;
         }
       
-        if (empty($GLOBALS['_TRANSACTION_STARTED'])) {
         db_begin();
-        }
 
         // Call delete() on children (if there are any)
         if ($children = $this->get_children_instances()) {
@@ -400,10 +398,8 @@ abstract class ArtefactType {
         $this->parentdirty = true;
         $this->deleted = true;
 
-        if (!empty($GLOBALS['_TRANSACTION_STARTED'])) {
         db_commit();
     }
-    }
 
     /**
     *