Merge pull request #6 from AmazeeLabs/entity-type-agnostic

refactor: make the frontify field widget entity type agnostic

Author: colorfield

css/frontify_media_form.css renamed to css/frontify_entity_form.css

@@ -17,13 +17,13 @@ frontify_media_library:
     - core/drupal
     - core/drupalSettings
     - frontify/frontify_finder
-frontify_media_form:
+frontify_entity_form:
   version: 1.x
   js:
-    js/frontify_media_form.js: { }
+    js/frontify_entity_form.js: { }
   css:
     component:
-      css/frontify_media_form.css: { }
+      css/frontify_entity_form.css: { }
   dependencies:
     - core/drupal
     - frontify/frontify_finder

frontify.libraries.yml

@@ -6,7 +6,6 @@
 (function () {
   Drupal.frontifyMediaForm = {
     handleFinder(el) {
-      console.log('Frontify media form');
       el.addEventListener('click', async element => {
         element.currentTarget.disabled = true;
         // Handle unlimited fields.
@@ -31,9 +30,9 @@
             domain: drupalSettings.Frontify.api_url,
             options: {
               permanentDownloadUrls: true,
-              // @todo configure extensions based on the media bundle.
-              //   this should be done as a third party setting of the Media provider
-              //   and selected based on its type.
+              // @todo configure extensions based on the field, so no need
+              //   to bind this to a specific media entity, and the field widget
+              //   can be used independently.
               filters: [
                 {
                   key: 'ext',
@@ -76,8 +75,10 @@
             $fieldItem.querySelector('textarea.frontify-asset-metadata').value =
               JSON.stringify(assets[0]);
 
-            // Media name
-            document.querySelector('#edit-name-wrapper input').value = assets[0].title;
+            // Media specifics: set the Media name from the Frontify one.
+            if (drupalSettings.Frontify.parent_entity_type_id === 'media') {
+              document.querySelector('#edit-name-wrapper input').value = assets[0].title;
+            }
 
             $field
               .querySelector('.frontify-wrapper-finder-overlay')

js/frontify_media_form.js renamed to js/frontify_entity_form.js

@@ -13,7 +13,7 @@
           case 'media_library':
             Drupal.frontifyMediaLibrary.handleFinder(el);
             break;
-          case 'media_form':
+          case 'entity_form':
             Drupal.frontifyMediaForm.handleFinder(el);
             break;
           default:

js/frontify_once.js

@@ -2,6 +2,7 @@
 
 namespace Drupal\frontify\Plugin\Field\FieldWidget;
 
+use Drupal\Core\Entity\Plugin\DataType\EntityAdapter;
 use Drupal\Core\Field\Attribute\FieldWidget;
 use Drupal\Core\Field\FieldItemListInterface;
 use Drupal\Core\Form\FormStateInterface;
@@ -92,6 +93,10 @@ public function formElement(
       ],
     ];
 
+
+    $parent = $items->getParent();
+    $entityTypeId = $parent instanceof EntityAdapter ? $parent->getEntity()->getEntityTypeId() : '';
+
     $element['open'] = [
       '#type' => 'button',
       '#value' => $this->t('Open Frontify'),
@@ -102,14 +107,15 @@ public function formElement(
       '#attached' => [
         'library' => [
           'frontify/frontify_once',
-          'frontify/frontify_media_form',
+          'frontify/frontify_entity_form',
         ],
         'drupalSettings' => [
           'Frontify' => [
-            'context' => 'media_form',
+            'context' => 'entity_form',
             'api_url' => $config->get('frontify_api_url'),
             'debug_mode' => $config->get('debug_mode'),
             'preview_image_width' => self::PREVIEW_IMAGE_WIDTH,
+            'parent_entity_type_id' => $entityTypeId,
           ],
         ],
       ],