Returning custom field view permission (#13741)

* Returning custom field view permission * One more line got removed by accident * Setting in the test that the fields have 2 permissions
mautic · May 13, 2024 · f13b8a4 · f13b8a4
1 parent ca2bf45
commit f13b8a4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/app/bundles/LeadBundle/Security/Permissions/LeadPermissions.php b/app/bundles/LeadBundle/Security/Permissions/LeadPermissions.php
@@ -25,6 +25,7 @@ public function __construct($params)
         $this->permissions = [
             'fields' => [
                 'full' => 1024,
+                'view' => 1,
             ],
         ];
 
@@ -51,6 +52,7 @@ public function buildForm(FormBuilderInterface &$builder, array $options, array
             [
                 'choices' => [
                     'mautic.core.permissions.manage' => 'full',
+                    'mautic.core.permissions.view'   => 'view',
                 ],
                 'label'             => 'mautic.lead.permissions.fields',
                 'data'              => (!empty($data['fields']) ? $data['fields'] : []),

diff --git a/app/bundles/LeadBundle/Tests/Security/LeadPermissionsFunctionalTest.php b/app/bundles/LeadBundle/Tests/Security/LeadPermissionsFunctionalTest.php
@@ -31,7 +31,7 @@ public function testRolePageForPermissionAvailability(): void
         $this->assertEquals(8, $listsRole->count());
 
         $fieldsRole = $crawler->filter('input[name="role[permissions][lead:fields][]"]');
-        $this->assertEquals(1, $fieldsRole->count());
+        $this->assertEquals(2, $fieldsRole->count());
 
         $importsRole = $crawler->filter('input[name="role[permissions][lead:imports][]"]');
         $this->assertEquals(6, $importsRole->count());