Fixes #903: Balanced reverting user context

modules/com_vtiger_workflow/tasks/VTCreateEntityTask.inc

@@ -32,6 +32,7 @@ class VTCreateEntityTask extends VTTask {
 
 		$entityType = $this->entity_type;		
 		if(!vtlib_isModuleActive($entityType)) {
+			$util->revertUser();
 			return;
 		}
 
@@ -198,6 +199,7 @@ class VTCreateEntityTask extends VTTask {
 			} catch (Exception $e) {
 			}
 			$util->revertUser();
+			return;
 		}
 
 		$util->revertUser();
@@ -225,4 +227,4 @@ class VTCreateEntityTask extends VTTask {
 	}
 
 }
-?>
\ No newline at end of file
+?>

modules/com_vtiger_workflow/tasks/VTUpdateFieldsTask.inc

@@ -345,7 +345,6 @@ class VTUpdateFieldsTask extends VTTask {
 
 			// Reverting back the action name as there can be some dependencies on this.
 			$_REQUEST['action'] = $actionName;
-			$util->revertUser();
 		}
 		$util->revertUser();
 		$_FILES = $tmpFiles;
@@ -419,4 +418,4 @@ class VTUpdateFieldsTask extends VTTask {
 
 }
 
-?>
\ No newline at end of file
+?>