HOR-4658

workflow/engine/classes/Applications.php

@@ -1187,6 +1187,9 @@ class Applications
                 case 'EXTERNAL':
                     $stepTitle = 'unknown ' . $caseStep->getStepUidObj();
                     $oPluginRegistry = PluginRegistry::loadSingleton();
+                    if (empty($externalSteps[$caseStep->getStepUidObj()])) {
+                        throw new Exception(G::LoadTranslation('ID_EXTERNAL_STEP_MISSING', SYS_LANG, ['plugin' => $stepTitle]));
+                    }
                     $externalStep = $externalSteps[$caseStep->getStepUidObj()];
                     $stepItem['id'] = $externalStep->getStepId();
                     $stepItem['title'] = $externalStep->getStepTitle();

workflow/engine/methods/services/ActionsByEmail.php

@@ -64,6 +64,7 @@ if (isset($_GET['BROWSER_TIME_ZONE_OFFSET'])) {
                         }
 
                         ChangeLog::getChangeLog()
+                                ->getUsrIdByUsrUid($caseFieldsABE['CURRENT_USER_UID'], true)
                                 ->setSourceId(ChangeLog::FromABE);
 
                         $caseFieldsABE['CURRENT_DYNAFORM'] = '';

workflow/engine/methods/services/ActionsByEmailDataFormPost.php

@@ -56,6 +56,7 @@ if (PMLicensedFeatures::getSingleton()
         $casesFields['USER_UID'] = $casesFields['CURRENT_USER_UID'];
 
         ChangeLog::getChangeLog()
+                ->getUsrIdByUsrUid($casesFields['USER_UID'], true)
                 ->setSourceId(ChangeLog::FromABE);
 
         //Update case info

workflow/engine/src/ProcessMaker/BusinessModel/Cases/ChangeLog.php

@@ -262,7 +262,7 @@ class ChangeLog
             return false;
         }
         foreach ($this->permissions as $type => $ids) {
-            if (array_search($uid, $ids) !== false) {
+            if (is_array($ids) && array_search($uid, $ids) !== false) {
                 return true;
             }
         }