feature/FOUR-8121: IT6 QA observations

ProcessMaker/Events/AuthClientCreated.php

@@ -11,42 +11,46 @@ class AuthClientCreated implements SecurityLogEventInterface
 
     private array $data;
 
-    private array $changes;
-
     /**
      * Create a new event instance.
      *
+     * @param array $data
+     *
      * @return void
      */
-    public function __construct(array $created_values)
+    public function __construct(array $data)
     {
-        $this->data = [
-            'auth_client_id' => $created_values['id'],
-            'name' => $created_values['name'],
-            'user_id' => $created_values['user_id'],
-            'revoked' => $created_values['revoked'],
-            'provider' => $created_values['provider'],
-            'redirect' => $created_values['redirect'],
-            'password_client' => $created_values['password_client'],
-            'personal_access_client' => $created_values['personal_access_client'],
-        ];
-        $this->changes = $created_values;
+        $this->data = $data;
     }
 
     /**
      * Return event data
      */
     public function getData(): array
     {
-        return $this->data;
+        return [
+            'name' => [
+                'label' => $this->data['name'],
+                'link' => route('auth-clients.index'),
+            ],
+            'auth_client_id' => $this->data['id'] ?? '',
+            'user_id' => $this->data['user_id'] ?? '',
+            'revoked' => $this->data['revoked'] ?? '',
+            'provider' => $this->data['provider'] ?? '',
+            'redirect' => $this->data['redirect'] ?? '',
+            'personal_access_client' => $this->data['personal_access_client'] ?? '',
+            'created_at' => $this->data['created_at'] ?? '',
+        ];
     }
 
     /**
      * Return event changes
      */
     public function getChanges(): array
     {
-        return $this->changes;
+        return [
+            'id' => $this->data['id']
+        ];
     }
 
     /**

ProcessMaker/Events/AuthClientUpdated.php

@@ -13,45 +13,44 @@ class AuthClientUpdated implements SecurityLogEventInterface
     use FormatSecurityLogChanges;
 
     private array $original;
-
-    private array $data;
-
     private array $changes;
-
     private string $clientId;
+    private string $clientName;
 
     /**
      * Create a new event instance.
      *
      * @return void
      */
-    public function __construct(string $clientId, array $originalValues, array $changedValues)
+    public function __construct(string $clientId, array $originalValues, array $changedValues, $name = '')
     {
         $this->original = $originalValues;
         $this->changes = $changedValues;
         $this->clientId = $clientId;
+        $this->clientName = $name;
     }
 
     /**
-     * Return event data
      * Return event data
      */
     public function getData(): array
     {
         return array_merge([
-            'auth_client_id' => $this->clientId,
+            'name' => [
+                'label' => $this->changes['name'] ?? $this->clientName,
+                'link' => route('auth-clients.index'),
+            ],
             'last_modified' => $this->changes['updated_at'] ?? Carbon::now(),
         ], $this->formatChanges($this->changes, $this->original));
     }
 
     /**
-     * Return event changes
      * Return event changes
      */
     public function getChanges(): array
     {
         return [
-            'auth_client_id' => $this->clientId,
+            'id' => $this->clientId,
         ];
     }
 

ProcessMaker/Events/CustomizeUiUpdated.php

@@ -39,7 +39,6 @@ public function __construct(array $original, array $changes)
      * Building the data
      */
     public function buildData()
-    public function buildData()
     {
         if (isset($this->changes['variables'])) {
             $varChanges = [];
@@ -56,7 +55,13 @@ public function buildData()
             $this->original['variables'] = $varOriginal;
         }
         $this->data = array_merge(
-            ['last_modified' => Carbon::now()],
+            [
+                'name' => [
+                    'label' => 'Customize Ui',
+                    'link' => route('customize-ui.edit'),
+                ],
+                'last_modified' => Carbon::now()
+            ],
             $this->formatChanges($this->changes, $this->original)
         );
     }

ProcessMaker/Events/PermissionUpdated.php

@@ -9,7 +9,8 @@
 
 class PermissionUpdated implements SecurityLogEventInterface
 {
-    use Dispatchable, FormatSecurityLogChanges;
+    use Dispatchable;
+    use FormatSecurityLogChanges;
 
     private array $changedPermissions;
 
@@ -24,8 +25,12 @@ class PermissionUpdated implements SecurityLogEventInterface
      *
      * @return void
      */
-    public function __construct(array $changedPermissions, array $originalPermissions, bool $permissionType, string $userId)
-    {
+    public function __construct(
+        array $changedPermissions,
+        array $originalPermissions,
+        bool $permissionType,
+        string $userId
+    ) {
         $this->changedPermissions = $changedPermissions;
         $this->originalPermissions = $originalPermissions;
         $this->permissionType = $permissionType;

ProcessMaker/Events/ProcessCreated.php

@@ -18,6 +18,8 @@ class ProcessCreated implements SecurityLogEventInterface
 
     public const TEMPLATE_CREATION = 'TEMPLATE';
 
+    public const BPMN_CREATION = 'BPMN';
+
     /**
      * Create a new event instance.
      *

ProcessMaker/Events/ProcessPublished.php

@@ -42,9 +42,11 @@ public function __construct(Process $data, array $changes, array $original)
         $this->original = array_diff_key($original, array_flip($this::REMOVE_KEYS));
 
         // Get category name
-        $this->original['process_category'] = isset($original['process_category_id']) ? ProcessCategory::getNamesByIds($this->original['process_category_id']) : '';
+        $this->original['process_category'] = isset($original['process_category_id'])
+        ? ProcessCategory::getNamesByIds($this->original['process_category_id']) : '';
         unset($this->original['process_category_id']);
-        $this->changes['process_category'] = isset($changes['process_category_id']) ? ProcessCategory::getNamesByIds($this->changes['process_category_id']) : '';
+        $this->changes['process_category'] = isset($changes['process_category_id'])
+        ? ProcessCategory::getNamesByIds($this->changes['process_category_id']) : '';
         unset($this->changes['process_category_id']);
     }
 

ProcessMaker/Events/QueueManagementAccessed.php

@@ -0,0 +1,54 @@
+<?php
+
+namespace ProcessMaker\Events;
+
+use Carbon\Carbon;
+use Illuminate\Foundation\Events\Dispatchable;
+use ProcessMaker\Contracts\SecurityLogEventInterface;
+
+class QueueManagementAccessed implements SecurityLogEventInterface
+{
+    use Dispatchable;
+
+    /**
+     * Create a new event instance.
+     *
+     * @return void
+     */
+    public function __construct()
+    {
+    }
+
+    /**
+     * Get specific data related to the event
+     *
+     * @return array
+     */
+    public function getData(): array
+    {
+        return [
+            'name' => 'Laravel Horizon',
+            'accessed_at' => Carbon::now(),
+        ];
+    }
+
+    /**
+     * Get specific changes without format related to the event
+     *
+     * @return array
+     */
+    public function getChanges(): array
+    {
+        return [];
+    }
+
+    /**
+     * Get the Event name with the syntax ‘[Past-test Action] [Object]’
+     *
+     * @return string
+     */
+    public function getEventName(): string
+    {
+        return 'QueueManagementAccessed';
+    }
+}

ProcessMaker/Events/ScreenCreated.php

@@ -2,6 +2,7 @@
 
 namespace ProcessMaker\Events;
 
+use Carbon\Carbon;
 use Illuminate\Foundation\Events\Dispatchable;
 use ProcessMaker\Contracts\SecurityLogEventInterface;
 use ProcessMaker\Models\ScreenCategory;
@@ -21,9 +22,9 @@ public function __construct(array $newScreen)
     {
         $this->newScreen = $newScreen;
 
-        if (isset($newScreen['screen_category_id'])) {
-            $this->newScreen['screen_category'] = ScreenCategory::getNamesByIds($newScreen['screen_category_id']);
-            unset($this->newScreen['screen_category_id']);
+        if (isset($newScreen['tmp_screen_category_id'])) {
+            $this->newScreen['screen_category'] = ScreenCategory::getNamesByIds($newScreen['tmp_screen_category_id']);
+            unset($newScreen['tmp_screen_category_id']);
         }
     }
 
@@ -35,10 +36,14 @@ public function __construct(array $newScreen)
     public function getData(): array
     {
         return [
-            'name' => $this->newScreen['title'] ?? '',
+            'name' => [
+                'label' => $this->newScreen['title'],
+                'link' => route('screen-builder.edit', ['screen' => $this->newScreen['id']]),
+            ],
             'description' => $this->newScreen['description'] ?? '',
             'type' => $this->newScreen['type'] ?? '',
-            'screen_category_id' => $this->newScreen['screen_category_id'] ?? '',
+            'Screen Categories' => $this->newScreen['screen_category'] ?? '',
+            'created_at' => $this->newScreen['created_at'] ?? Carbon::now(),
         ];
     }
 

ProcessMaker/Events/ScreenDeleted.php

@@ -31,8 +31,8 @@ public function __construct(Screen $screen)
     public function getData(): array
     {
         return [
-            'title' => $this->screen->getAttributes()['title'],
-            'description' => $this->screen->getAttributes()['description'],
+            'name' => $this->screen->getAttribute('title'),
+            'description' => $this->screen->getAttribute('description'),
             'deleted_at' => Carbon::now(),
         ];
     }

ProcessMaker/Events/ScreenUpdated.php

@@ -4,6 +4,7 @@
 
 use Illuminate\Foundation\Events\Dispatchable;
 use ProcessMaker\Contracts\SecurityLogEventInterface;
+use ProcessMaker\Helpers\ArrayHelper;
 use ProcessMaker\Models\Screen;
 use ProcessMaker\Models\ScreenCategory;
 use ProcessMaker\Traits\FormatSecurityLogChanges;
@@ -19,6 +20,11 @@ class ScreenUpdated implements SecurityLogEventInterface
 
     private array $original;
 
+    public const REMOVE_KEYS = [
+        'screen_category_id',
+        'tmp_screen_category_id'
+    ];
+
     /**
      * Create a new event instance.
      *
@@ -31,10 +37,12 @@ public function __construct(Screen $screen, array $changes, array $original)
         $this->original = $original;
 
         // Get category name
-        $this->original['screen_category'] = isset($original['screen_category_id']) ? ScreenCategory::getNamesByIds($this->original['screen_category_id']) : '';
+        $this->original['screen_category'] = isset($original['screen_category_id'])
+        ? ScreenCategory::getNamesByIds($this->original['screen_category_id']) : '';
         unset($this->original['screen_category_id']);
-        $this->changes['screen_category'] = isset($changes['screen_category_id']) ? ScreenCategory::getNamesByIds($this->changes['screen_category_id']) : '';
-        unset($this->changes['screen_category_id']);
+        $this->changes['screen_category'] = isset($changes['tmp_screen_category_id'])
+        ? ScreenCategory::getNamesByIds($this->changes['tmp_screen_category_id']) : '';
+        $this->changes = array_diff_key($this->changes, array_flip($this::REMOVE_KEYS));
     }
 
     /**
@@ -44,14 +52,19 @@ public function __construct(Screen $screen, array $changes, array $original)
      */
     public function getData(): array
     {
+        $basic = [
+            'name' => [
+                'label' => $this->screen->getAttribute('title'),
+                'link' => route('screen-builder.edit', ['screen' => $this->screen->getAttribute('id')]),
+            ],
+            'last_modified' => $this->screen->getAttribute('updated_at'),
+        ];
         if (array_key_exists('config', $this->changes)) {
-            return array_merge([
-                'last_modified' => $this->screen->getAttribute('updated_at'),
-            ]);
+            return $basic;
         } else {
             return array_merge([
                 'last_modified' => $this->screen->getAttribute('updated_at'),
-            ], $this->formatChanges($this->changes, $this->original));
+            ], ArrayHelper::getArrayDifferencesWithFormat($this->changes, $this->original));
         }
     }