PS-907 fix rendition and attribute definitions forms (#621)

Author: 4rthem

databox/api/fixtures/Newspaper.yaml

@@ -505,11 +505,14 @@ App\Entity\Integration\WorkspaceIntegration:
         title: ToastUI
         integration: tui.photo-editor
         workspace: '@w_newspaper'
+        ownerId: <keycloakUser("phrasea-admin", ["phrasea-admin"])>
     wi_renditions:
         integration: core.rendition
         workspace: '@w_newspaper'
+        ownerId: <keycloakUser("phrasea-admin", ["phrasea-admin"])>
     wi_watermark:
         integration: core.watermark
+        ownerId: <keycloakUser("phrasea-admin", ["phrasea-admin"])>
         workspace: '@w_newspaper'
         config:
             attributeName: watermark
@@ -518,13 +521,15 @@ App\Entity\Integration\WorkspaceIntegration:
         needs:
             - '@wi_renditions'
     wi_blurhash:
+        ownerId: <keycloakUser("phrasea-admin", ["phrasea-admin"])>
         integration: blurhash
         workspace: '@w_newspaper'
         config:
             rendition: thumbnail
         needs:
             - '@wi_renditions'
     wi_expose:
+        ownerId: <keycloakUser("phrasea-admin", ["phrasea-admin"])>
         title: Expose Basket
         integration: phrasea.expose
         config:

databox/api/src/Api/Model/Output/AttributeDefinitionOutput.php

@@ -48,7 +48,7 @@ class AttributeDefinitionOutput extends AbstractUuidOutput
     #[Groups([AttributeDefinition::GROUP_LIST])]
     public ?string $fileType = null;
 
-    #[Groups([AttributeDefinition::GROUP_LIST, Asset::GROUP_LIST, Share::GROUP_PUBLIC_READ])]
+    #[Groups([AttributeDefinition::GROUP_LIST, Asset::GROUP_LIST, Asset::GROUP_READ, Share::GROUP_PUBLIC_READ])]
     public string $fieldType = TextAttributeType::NAME;
 
     #[Groups([AttributeDefinition::GROUP_LIST])]

databox/api/src/Entity/Core/AttributeDefinition.php

@@ -43,17 +43,17 @@
     shortName: 'attribute-definition',
     operations: [
         new Get(
-            security: 'is_granted("'.AbstractVoter::READ.'", object)'
+            security: 'is_granted("'.AbstractVoter::READ.'", object)',
         ),
         new Delete(security: 'is_granted("DELETE", object)'),
         new Put(
-            normalizationContext: [
-                'groups' => [self::GROUP_READ],
-            ],
             security: 'is_granted("'.AbstractVoter::EDIT.'", object)'
         ),
         new Patch(security: 'is_granted("'.AbstractVoter::EDIT.'", object)'),
         new GetCollection(
+            normalizationContext: [
+                'groups' => [AttributeDefinition::GROUP_LIST],
+            ],
             parameters: [
                 'searchable' => new QueryParameter(
                     schema: ['type' => 'boolean'],
@@ -62,12 +62,9 @@
                 'workspaceId' => new QueryParameter(
                     schema: ['type' => 'string'],
                 ),
-            ]
+            ],
         ),
         new Post(
-            normalizationContext: [
-                'groups' => [self::GROUP_READ],
-            ],
             security: 'is_granted("'.JwtUser::IS_AUTHENTICATED_FULLY.'")',
             securityPostDenormalize: 'is_granted("CREATE", object)',
         ),
@@ -96,7 +93,7 @@
         ),
     ],
     normalizationContext: [
-        'groups' => [AttributeDefinition::GROUP_LIST],
+        'groups' => [AttributeDefinition::GROUP_LIST, AttributeDefinition::GROUP_READ],
     ],
     input: AttributeDefinitionInput::class,
     output: AttributeDefinitionOutput::class,

databox/api/src/Entity/Core/RenditionPolicy.php

@@ -63,7 +63,7 @@ class RenditionPolicy extends AbstractUuidEntity implements \Stringable
     #[Assert\NotNull]
     protected ?Workspace $workspace = null;
 
-    #[Groups([RenditionPolicy::GROUP_LIST, RenditionPolicy::GROUP_READ])]
+    #[Groups([RenditionPolicy::GROUP_LIST, RenditionPolicy::GROUP_READ, RenditionDefinition::GROUP_LIST, RenditionDefinition::GROUP_READ])]
     #[ORM\Column(type: Types::STRING, length: 80)]
     #[Assert\NotNull]
     private ?string $name = null;

databox/client/src/components/Dialog/Workspace/AttributeDefinitionManager.tsx

@@ -443,7 +443,12 @@ export default function AttributeDefinitionManager({
 function normalizeData(data: AttributeDefinition) {
     return {
         ...data,
-        policy: data.policy ? (data.policy as AttributePolicy)['@id'] : null,
+        policy:
+            typeof data.policy === 'string'
+                ? data.policy
+                : data.policy
+                  ? (data.policy as AttributePolicy)['@id']
+                  : null,
         entityList: data.entityList
             ? (data.entityList as EntityList)['@id']
             : null,

databox/client/src/components/Dialog/Workspace/RenditionDefinitionManager.tsx

@@ -262,7 +262,7 @@ function normalizeData(data: RenditionDefinition) {
     return {
         ...data,
         buildMode: data.buildMode?.toString(),
-        class:
+        policy:
             typeof data.policy === 'string'
                 ? data.policy
                 : data.policy