Merge pull request #89 from netgen/admin-issues-fixes

Various improvements in admin

Author: iherak

bundle/RemoteMedia/Helper.php

@@ -47,10 +47,10 @@ public function formatBrowseItem(Value $value)
 
         if ($mediaType === Value::TYPE_IMAGE) {
             $browseUrl = $this->provider->buildVariation($value, 'admin', $thumbOptions)->url;
-            $previewUrl = $value->secure_url;
+            $previewUrl = $this->provider->buildVariation($value, 'default', 'admin_preview')->url;
         } elseif ($mediaType === Value::TYPE_VIDEO) {
             $browseUrl = $this->provider->getVideoThumbnail($value, $thumbOptions);
-            $previewUrl = $this->provider->getVideoThumbnail($value);
+            $previewUrl = $this->provider->getVideoThumbnail($value, ['content_type_identifier' => 'default', 'variation_name' => 'admin_preview']);
         } else {
             $browseUrl = '';
             $previewUrl = '';

bundle/RemoteMedia/Provider/Cloudinary/CloudinaryProvider.php

@@ -284,6 +284,12 @@ public function updateResourceContext(string $resourceId, string $resourceType,
      */
     public function getVideoThumbnail(Value $value, $options = []): string
     {
+        if (array_key_exists('content_type_identifier', $options) && array_key_exists('variation_name', $options)) {
+            $options['transformation'] = $this->processConfiguredVariation($value, $options['variation_name'], $options['content_type_identifier']);
+            unset($options['content_type_identifier']);
+            unset($options['variation_name']);
+        }
+
         if (count($options) === 0 || !array_key_exists('resource_type', $options)) {
             $options['resource_type'] = 'video';
         }

bundle/Resources/config/default_settings.yml

@@ -4,7 +4,7 @@ system:
             default:
                 admin_preview:
                     transformations:
-                        - { name: limit, params: [600,600] }
+                        - { name: limit, params: [400,400] }
             ng_article:
                 test:
                     transformations:

bundle/Resources/public/css/remotemedia.css

@@ -6,10 +6,10 @@
         type: "{% if field_value %}{{ field_value.resourceType|default('image') }}{% else %}image{% endif %}",
         {% if not field_value or field_value.mediaType == 'image' %}
             mediaType: "image",
-            previewUrl: "{% if field_value %}{{ field_value.secure_url }}{% endif %}",
+            previewUrl: "{% if field_value %}{{ netgen_remote_media(field_value, 'admin_preview').url }}{% endif %}",
         {% elseif field_value.mediaType == 'video' %}
             mediaType: "video",
-            previewUrl: "{{ netgen_remote_video_thumbnail(field_value) }}",
+            previewUrl: "{{ netgen_remote_video_thumbnail(field_value, {'content_type_identifier': 'default', 'variation_name': 'admin_preview'}) }}",
         {% else %}
             mediaType: "other",
             previewUrl: "",

bundle/Resources/public/js/remotemedia.js

@@ -5,8 +5,8 @@
 }
 
 {if $value.resourceId}
+    {def $format = 'admin_preview'}
     {if $value.mediaType|eq('image')}
-        {def $format = 'admin_preview'}
         {def $variation = ngremotemedia($value, $attribute.object.class_identifier, $format)}
 
         {if not(is_set($alt_text))}
@@ -29,7 +29,7 @@
 
         <br/>
 
-        {def $thumbnail = videoThumbnail($value)}
+        {def $thumbnail = videoThumbnail($value, hash('content_type_identifier', $attribute.object.class_identifier, 'variation_name', $format))}
         <img src="{$thumbnail}"/>
     {else}
         <i class="fa fa-book" aria-hidden="true"></i>

bundle/Resources/views/ezadminui/parts/edit/ngrm_field.html.twig

@@ -104,6 +104,10 @@ function namedParameterList()
                 'value' => array(
                     'type' => 'Value',
                     'required' => true
+                ),
+                'options' => array(
+                    'type' => 'array',
+                    'required' => false
                 )
             ),
             'ngremotevideo' => array(
@@ -146,7 +150,7 @@ function modify(
         } elseif ($operatorName === 'ng_remote_croppable') {
             $operatorValue = $this->isCroppable($namedParameters['class_identifier']);
         } elseif ($operatorName === 'videoThumbnail') {
-            $operatorValue = $this->videoThumbnail($namedParameters['value']);
+            $operatorValue = $this->videoThumbnail($namedParameters['value'], $namedParameters['options']);
         } elseif ($operatorName === 'ngremotevideo') {
             $operatorValue = $this->getvideoTag(
                 $namedParameters['value'],
@@ -228,12 +232,12 @@ function isCroppable($class_identifier)
         return !empty($variationResolver->getCroppbableVariations($class_identifier));
     }
 
-    function videoThumbnail($value)
+    function videoThumbnail($value, $options = [])
     {
         $container = ezpKernel::instance()->getServiceContainer();
         $provider = $container->get( 'netgen_remote_media.provider' );
 
-        return $provider->getVideoThumbnail($value);
+        return $provider->getVideoThumbnail($value, $options);
     }
 
     function getVideoTag($value, $availableFormats, $format)