Merge pull request #93 from spryker/bugfix/supesc-946-delete-triggered-on-0-events

SUPESC-946 EventBehaviorFacade::triggerRuntimeEvents triggers delete when there are 0 event triggered

Author: spryker-release-bot

.github/workflows/ci.yml

@@ -62,7 +62,7 @@ jobs:
         run: |
           echo "::set-output name=dir::$(composer config cache-files-dir)"
 
-      - uses: actions/cache@v2
+      - uses: actions/cache@v3
         with:
           path: ${{ steps.composer-cache.outputs.dir }}
           key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
@@ -120,7 +120,7 @@ jobs:
             run: |
                 echo "::set-output name=dir::$(composer config cache-files-dir)"
 
-          - uses: actions/cache@v2
+          - uses: actions/cache@v3
             with:
                 path: ${{ steps.composer-cache.outputs.dir }}
                 key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}

src/Spryker/Zed/EventBehavior/Business/Model/TriggerManager.php

@@ -127,7 +127,7 @@ public function triggerRuntimeEvents()
             $offset += $limit;
         } while ($countEvents === $limit);
 
-        if ($countEvents === $triggeredEvents) {
+        if ($countEvents === $triggeredEvents && $triggeredEvents > 0) {
             $this->eventBehaviorEntityManager->deleteEventBehaviorEntityByProcessId($processId);
 
             return;

src/Spryker/Zed/EventBehavior/Persistence/Propel/Behavior/EventBehavior.php

@@ -105,33 +105,27 @@ public function addParameter(array $parameter): void
     {
         $parameter = array_change_key_case($parameter, CASE_LOWER);
 
-        // @phpstan-ignore-next-line
         $this->parameters[$parameter['name']] = [];
 
         if (!isset($parameter['column'])) {
             throw new PropelException(sprintf('"column" attribute for %s event behavior is missing', $parameter['name']));
         }
 
-        // @phpstan-ignore-next-line
         $this->parameters[$parameter['name']]['column'] = $parameter['column'];
 
         if (isset($parameter['value'])) {
-            // @phpstan-ignore-next-line
             $this->parameters[$parameter['name']]['value'] = $parameter['value'];
         }
 
         if (isset($parameter['operator'])) {
-            // @phpstan-ignore-next-line
             $this->parameters[$parameter['name']]['operator'] = $parameter['operator'];
         }
 
         if (isset($parameter['keep-original'])) {
-            // @phpstan-ignore-next-line
             $this->parameters[$parameter['name']]['keep-original'] = $parameter['keep-original'];
         }
 
         if (isset($parameter['keep-additional'])) {
-            // @phpstan-ignore-next-line
             $this->parameters[$parameter['name']]['keep-additional'] = $parameter['keep-additional'];
         }
     }