Merge remote-tracking branch 'origin/feature/is_imageslider-simplified' into feature/is_imageslider-simplified

.gitignore

@@ -0,0 +1,2 @@
+
+is_imageslider/vendor/

is_imageslider/config/admin/services.yml

@@ -1,7 +1,6 @@
 imports:
     - { resource: ../common.yml }
 
-
 services:
     _defaults:
         public: true
@@ -11,7 +10,7 @@ services:
         public: true
         class: Oksydan\IsImageslider\Controller\IsImagesliderController
         arguments:
-      - '@oksydan.is_imageslider.handler.file_eraser'
+            - "@oksydan.is_imageslider.handler.file_eraser"
             - "@=service('prestashop.adapter.legacy.context').getLanguages(false)"
             - "@oksydan.is_imageslider.cache.template_cache"
 
@@ -19,144 +18,146 @@ services:
     oksydan.is_imageslider.image_slider_installer:
         class: 'Oksydan\IsImageslider\Installer\ImageSliderInstaller'
         arguments:
-      - '@doctrine.dbal.default_connection'
+            - "@doctrine.dbal.default_connection"
-      - '@oksydan.is_imageslider.installer.database_yml_parser'
+            - "@oksydan.is_imageslider.installer.database_yml_parser"
             - "@=service('prestashop.adapter.legacy.context').getContext()"
 
     oksydan.is_imageslider.installer.database_yml_parser:
         class: 'Oksydan\IsImageslider\Installer\DatabaseYamlParser'
         arguments:
-      - '@oksydan.is_imageslider.installer.database_yml_parser_provider'
+            - "@oksydan.is_imageslider.installer.database_yml_parser_provider"
 
     oksydan.is_imageslider.installer.database_yml_parser_provider:
         class: 'Oksydan\IsImageslider\Installer\Provider\DatabaseYamlProvider'
         arguments:
-      - '@oksydan.is_imageslider.module'
+            - "@oksydan.is_imageslider.module"
 
     oksydan.is_imageslider.installer.action_databse_create_table:
         class: 'Oksydan\IsImageslider\Installer\ActionDatabaseCrateTable'
-    parent: 'oksydan.is_imageslider.installer.abstract_action_databse'
+        parent: "oksydan.is_imageslider.installer.abstract_action_databse"
         public: true
 
     oksydan.is_imageslider.installer.action_databse_drop_table:
         class: 'Oksydan\IsImageslider\Installer\ActionDatabaseDropTable'
-    parent: 'oksydan.is_imageslider.installer.abstract_action_databse'
+        parent: "oksydan.is_imageslider.installer.abstract_action_databse"
         public: true
 
     oksydan.is_imageslider.installer.abstract_action_databse:
         class: 'Oksydan\IsImageslider\Installer\ActionDatabaseAbstract'
         abstract: true
         arguments:
-      - '@doctrine.dbal.default_connection'
+            - "@doctrine.dbal.default_connection"
-      - '%database_prefix%'
+            - "%database_prefix%"
 
     oksydan.is_imageslider.handler.file_uploader:
         class: 'Oksydan\IsImageslider\Handler\FileUploader'
         arguments:
-      $targetDirectory: '%is_imageslider_images_directory%'
+            $targetDirectory: "%is_imageslider_images_directory%"
 
     oksydan.is_imageslider.handler.file_eraser:
         class: 'Oksydan\IsImageslider\Handler\FileEraser'
         arguments:
-      $targetDirectory: '%is_imageslider_images_directory%'
+            $targetDirectory: "%is_imageslider_images_directory%"
-
 
     #creation/edition form
     oksydan.is_imageslider.form.identifiable_object.data_provider.image_slider_form_data_provider:
         class: 'Oksydan\IsImageslider\Form\Provider\ImageSliderFormDataProvider'
         arguments:
-      - '@oksydan.is_imageslider.repository.image_slider'
+            - "@oksydan.is_imageslider.repository.image_slider"
-      - '@oksydan.is_imageslider.provider.image_provider'
+            - "@oksydan.is_imageslider.provider.image_provider"
-      - '@prestashop.core.admin.lang.repository'
+            - "@prestashop.core.admin.lang.repository"
-      - '%is_imageslider_placeholder_img%'
+            - "%is_imageslider_placeholder_img%"
-      - '@prestashop.adapter.shop.context'
+            - "@prestashop.adapter.shop.context"
 
     oksydan.is_imageslider.form.identifiable_object.builder.image_slider_form_builder:
         class: 'PrestaShop\PrestaShop\Core\Form\IdentifiableObject\Builder\FormBuilder'
-    factory: 'prestashop.core.form.builder.form_builder_factory:create'
+        factory:
+            ["@prestashop.core.form.builder.form_builder_factory", "create"]
         arguments:
             - 'Oksydan\IsImageslider\Form\ImageSliderType'
-      - '@oksydan.is_imageslider.form.identifiable_object.data_provider.image_slider_form_data_provider'
+            - "@oksydan.is_imageslider.form.identifiable_object.data_provider.image_slider_form_data_provider"
 
     oksydan.is_imageslider.form.identifiable_object.data_handler.image_slider_form_data_handler:
         class: 'Oksydan\IsImageslider\Form\DataHandler\ImageSliderFormDataHandler'
         arguments:
-      - '@oksydan.is_imageslider.repository.image_slider'
+            - "@oksydan.is_imageslider.repository.image_slider"
-      - '@prestashop.core.admin.lang.repository'
+            - "@prestashop.core.admin.lang.repository"
-      - '@doctrine.orm.default_entity_manager'
+            - "@doctrine.orm.default_entity_manager"
-      - '@oksydan.is_imageslider.handler.file_uploader'
+            - "@oksydan.is_imageslider.handler.file_uploader"
-      - '@oksydan.is_imageslider.handler.file_eraser'
+            - "@oksydan.is_imageslider.handler.file_eraser"
             - "@=service('prestashop.adapter.legacy.context').getLanguages()"
 
     oksydan.is_imageslider.form.identifiable_object.handler.image_slider_form_handler:
         class: 'PrestaShop\PrestaShop\Core\Form\IdentifiableObject\Handler\FormHandler'
-    factory: 'prestashop.core.form.identifiable_object.handler.form_handler_factory:create'
+        factory:
+            [
+                "@prestashop.core.form.identifiable_object.handler.form_handler_factory",
+                "create",
+            ]
         arguments:
-      - '@oksydan.is_imageslider.form.identifiable_object.data_handler.image_slider_form_data_handler'
+            - "@oksydan.is_imageslider.form.identifiable_object.data_handler.image_slider_form_data_handler"
 
     #grid services
     oksydan.is_imageslider.grid.definition.factory.image_sliders:
         class: 'Oksydan\IsImageslider\Grid\Definition\Factory\ImageSliderGridDefinitionFactory'
-    parent: 'prestashop.core.grid.definition.factory.abstract_grid_definition'
+        parent: "prestashop.core.grid.definition.factory.abstract_grid_definition"
         public: true
 
     oksydan.is_imageslider.grid.query.image_slider_query_builder:
         class: 'Oksydan\IsImageslider\Grid\Query\ImageSliderQueryBuilder'
-    parent: 'prestashop.core.grid.abstract_query_builder'
+        parent: "prestashop.core.grid.abstract_query_builder"
         public: true
         arguments:
-      - '@prestashop.adapter.shop.context'
+            - "@prestashop.adapter.shop.context"
             - "@=service('prestashop.adapter.legacy.context').getContext().language.id"
 
     oksydan.is_imageslider.grid.image_slider_grid_factory:
         class: 'PrestaShop\PrestaShop\Core\Grid\GridFactory'
         arguments:
-      - '@oksydan.is_imageslider.grid.definition.factory.image_sliders'
+            - "@oksydan.is_imageslider.grid.definition.factory.image_sliders"
-      - '@oksydan.is_imageslider.grid.data.factory.image_slider_decorator'
+            - "@oksydan.is_imageslider.grid.data.factory.image_slider_decorator"
-      - '@prestashop.core.grid.filter.form_factory'
+            - "@prestashop.core.grid.filter.form_factory"
-      - '@prestashop.core.hook.dispatcher'
+            - "@prestashop.core.hook.dispatcher"
-
 
     oksydan.is_imageslider.grid.data.factory.image_slider_data_factory:
-    class: '%prestashop.core.grid.data.factory.doctrine_grid_data_factory%'
+        class: "%prestashop.core.grid.data.factory.doctrine_grid_data_factory%"
         arguments:
-      - '@oksydan.is_imageslider.grid.query.image_slider_query_builder'
+            - "@oksydan.is_imageslider.grid.query.image_slider_query_builder"
-      - '@prestashop.core.hook.dispatcher'
+            - "@prestashop.core.hook.dispatcher"
-      - '@prestashop.core.grid.query.doctrine_query_parser'
+            - "@prestashop.core.grid.query.doctrine_query_parser"
-      - 'is_imageslider'
+            - "is_imageslider"
-
 
     oksydan.is_imageslider.grid.data.factory.image_slider_decorator:
         class: 'Oksydan\IsImageslider\Grid\Data\Factory\ImageSliderGridDataFactory'
         arguments:
-      - '@oksydan.is_imageslider.grid.data.factory.image_slider_data_factory'
+            - "@oksydan.is_imageslider.grid.data.factory.image_slider_data_factory"
-      - '@oksydan.is_imageslider.provider.image_provider'
+            - "@oksydan.is_imageslider.provider.image_provider"
 
     # config form
     oksydan.is_imageslider.image_slider.configuration:
         class: 'Oksydan\IsImageslider\Form\DataConfiguration\ImageSliderDataConfiguration'
         arguments:
-      - '@prestashop.adapter.legacy.configuration'
+            - "@prestashop.adapter.legacy.configuration"
-      - '@prestashop.adapter.shop.context'
+            - "@prestashop.adapter.shop.context"
-      - '@prestashop.adapter.multistore_feature'
+            - "@prestashop.adapter.multistore_feature"
 
     oksydan.is_imageslider.image_slider_configuration.form_provider:
         class: 'Oksydan\IsImageslider\Form\Provider\ImageSliderConfigurationFormDataProvider'
         arguments:
-      - '@oksydan.is_imageslider.image_slider.configuration'
+            - "@oksydan.is_imageslider.image_slider.configuration"
 
     oksydan.is_imageslider.image_slider_configuration.form_handler:
         class: 'PrestaShop\PrestaShop\Core\Form\Handler'
         arguments:
-      - '@form.factory'
+            - "@form.factory"
-      - '@prestashop.core.hook.dispatcher'
+            - "@prestashop.core.hook.dispatcher"
-      - '@oksydan.is_imageslider.image_slider_configuration.form_provider'
+            - "@oksydan.is_imageslider.image_slider_configuration.form_provider"
             - 'Oksydan\IsImageslider\Form\ImageSliderConfigurationType'
-      - 'ConfigurationForm'
+            - "ConfigurationForm"
 
     oksydan.is_imageslider.form.image_slider_type:
         class: 'Oksydan\IsImageslider\Form\ImageSliderType'
-    parent: 'form.type.translatable.aware'
+        parent: "form.type.translatable.aware"
         arguments:
             - '@=service("prestashop.adapter.multistore_feature").isUsed()'
         public: true
@@ -165,27 +166,32 @@ services:
 
     oksydan.is_imageslider.form.image_slider_configuration_type:
         class: 'Oksydan\IsImageslider\Form\ImageSliderConfigurationType'
-    parent: 'form.type.translatable.aware'
+        parent: "form.type.translatable.aware"
         public: true
         tags:
             - { name: form.type }
 
     Oksydan\IsImageslider\Type\TranslatableFile:
         class: 'Oksydan\IsImageslider\Type\TranslatableFile'
-    parent: 'form.type.common.translatable'
+        parent: "form.type.translatable.aware"
         public: true
         tags:
             - { name: form.type }
+        arguments:
+            $availableLocales: "@=service('prestashop.adapter.legacy.context').getLanguages()"
+            $saveFormLocaleChoice: false
+            $defaultFormLanguageId: "@=service('prestashop.adapter.legacy.context').getLanguage().id"
+            $defaultShopLanguageId: "@=service('prestashop.adapter.legacy.context').getLanguage().id"
 
     oksydan.is_imageslider.grid.position_definition:
         class: PrestaShop\PrestaShop\Core\Grid\Position\PositionDefinition
         arguments:
-      - 'image_slider'
+            - "image_slider"
-      - 'id_slide'
+            - "id_slide"
-      - 'position'
+            - "position"
 
     oksydan.is_imageslider.provider.image_provider:
         class: 'Oksydan\IsImageslider\Provider\ImageProvider'
         public: true
         arguments:
-      - '%is_imageslider_images_uri%'
+            - "%is_imageslider_images_uri%"

is_imageslider/config/common.yml

@@ -1,8 +1,8 @@
 parameters:
-  is_imageslider_images_directory: '%kernel.project_dir%/modules/is_imageslider/img/'
+    is_imageslider_images_directory: "%kernel.project_dir%/modules/is_imageslider/img/"
     is_imageslider_module_uri: !php/const _MODULE_DIR_
-  is_imageslider_images_uri: '%is_imageslider_module_uri%is_imageslider/img/'
+    is_imageslider_images_uri: "%is_imageslider_module_uri%is_imageslider/img/"
-  is_imageslider_placeholder_img: '/modules/is_imageslider/views/img/placeholder.jpeg'
+    is_imageslider_placeholder_img: "/modules/is_imageslider/views/img/placeholder.jpeg"
 
 services:
     _defaults:
@@ -10,15 +10,15 @@ services:
 
     oksydan.is_imageslider.module:
         class: Is_imageslider
-    factory: [ 'Module', 'getInstanceByName' ]
+        factory: ["Module", "getInstanceByName"]
         public: false
         arguments:
-      - 'is_imageslider'
+            - "is_imageslider"
 
     oksydan.is_imageslider.repository.image_slider:
         class: Oksydan\IsImageslider\Repository\ImageSliderRepository
         public: true
-    factory: ['@doctrine.orm.default_entity_manager', getRepository]
+        factory: ["@doctrine.orm.default_entity_manager", getRepository]
         arguments:
             - Oksydan\IsImageslider\Entity\ImageSlider
 
@@ -26,15 +26,15 @@ services:
         class: Oksydan\IsImageslider\Repository\HookModuleRepository
         public: true
         arguments:
-      - '@doctrine.dbal.default_connection'
+            - "@doctrine.dbal.default_connection"
-      - '%database_prefix%'
+            - "%database_prefix%"
 
     oksydan.is_imageslider.presenter.image_slide_presenter:
         class: Oksydan\IsImageslider\Presenter\ImageSlidePresenter
         public: true
         arguments:
-      - '%is_imageslider_images_uri%'
+            - "%is_imageslider_images_uri%"
-      - '%is_imageslider_images_directory%'
+            - "%is_imageslider_images_directory%"
             - "@=service('prestashop.adapter.legacy.context').getContext()"
 
     oksydan.is_imageslider.configuration.slider_configuration:
@@ -45,47 +45,52 @@ services:
         class: Oksydan\IsImageslider\Hook\AbstractHook
         public: true
         arguments:
-      - '@oksydan.is_imageslider.module'
+            - "@oksydan.is_imageslider.module"
             - "@=service('prestashop.adapter.legacy.context').getContext()"
 
     oksydan.is_imageslider.hook.abstract_display_hook:
         class: Oksydan\IsImageslider\Hook\AbstractDisplayHook
-    parent: 'oksydan.is_imageslider.hook.abstract_hook'
+        parent: "oksydan.is_imageslider.hook.abstract_hook"
         public: true
         arguments:
-      - '@oksydan.is_imageslider.configuration.slider_configuration'
+            - "@oksydan.is_imageslider.configuration.slider_configuration"
 
     oksydan.is_imageslider.hook.abstract_cacheable_display_hook:
         class: Oksydan\IsImageslider\Hook\AbstractCacheableDisplayHook
-    parent: 'oksydan.is_imageslider.hook.abstract_display_hook'
+        parent: "oksydan.is_imageslider.hook.abstract_display_hook"
         public: true
         arguments:
-      - '@oksydan.is_imageslider.repository.image_slider'
+            - "@oksydan.is_imageslider.repository.image_slider"
-      - '@oksydan.is_imageslider.presenter.image_slide_presenter'
+            - "@oksydan.is_imageslider.presenter.image_slide_presenter"
-      - '@oksydan.is_imageslider.cache.template_cache'
+            - "@oksydan.is_imageslider.cache.template_cache"
 
     oksydan.is_imageslider.hook.display_home:
         class: Oksydan\IsImageslider\Hook\DisplayHome
-    parent: 'oksydan.is_imageslider.hook.abstract_cacheable_display_hook'
+        parent: "oksydan.is_imageslider.hook.abstract_cacheable_display_hook"
+        public: true
+
+    oksydan.is_imageslider.hook.display_hero_slider:
+        class: Oksydan\IsImageslider\Hook\DisplayHome
+        parent: "oksydan.is_imageslider.hook.abstract_cacheable_display_hook"
         public: true
 
     oksydan.is_imageslider.hook.display_header:
         class: Oksydan\IsImageslider\Hook\DisplayHeader
-    parent: 'oksydan.is_imageslider.hook.abstract_cacheable_display_hook'
+        parent: "oksydan.is_imageslider.hook.abstract_cacheable_display_hook"
         public: true
 
     # Modification of the module into a widget
     oksydan.is_imageslider.hook.widget_capability:
         class: Oksydan\IsImageslider\Hook\WidgetCapability
-    parent: 'oksydan.is_imageslider.hook.abstract_cacheable_display_hook'
+        parent: "oksydan.is_imageslider.hook.abstract_cacheable_display_hook"
         public: true
 
     oksydan.is_imageslider.cache.template_cache:
         class: Oksydan\IsImageslider\Cache\TemplateCache
         public: true
         arguments:
-      - '@oksydan.is_imageslider.module'
+            - "@oksydan.is_imageslider.module"
             - "@=service('prestashop.adapter.legacy.context').getContext()"
-      - '@oksydan.is_imageslider.repository.hook_module_repository'
+            - "@oksydan.is_imageslider.repository.hook_module_repository"
-      - '@prestashop.adapter.legacy.configuration'
+            - "@prestashop.adapter.legacy.configuration"
-      - '@oksydan.is_imageslider.repository.image_slider'
+            - "@oksydan.is_imageslider.repository.image_slider"

is_imageslider/is_imageslider.php

@@ -10,6 +10,7 @@ if (file_exists(__DIR__ . '/vendor/autoload.php')) {
     require_once __DIR__ . '/vendor/autoload.php';
 }
 
+use Symfony\Component\HttpFoundation\RedirectResponse;
 use Oksydan\IsImageslider\Hook\HookInterface;
 use Oksydan\IsImageslider\Installer\ImageSliderInstaller;
 use PrestaShop\PrestaShop\Adapter\SymfonyContainer;
@@ -55,6 +56,7 @@ class Is_imageslider extends Module implements WidgetInterface
             parent::install()
             && $this->registerHook('displayHeader')
             && $this->registerHook('displayHome')
+            && $this->registerHook('displayHeroSlider')
             && $this->getInstaller()->createTables();
     }
 
@@ -66,11 +68,11 @@ class Is_imageslider extends Module implements WidgetInterface
         return $this->getInstaller()->dropTables() && parent::uninstall();
     }
 
-    public function getContent(): void
+    public function getContent()
     {
-        \Tools::redirectAdmin(SymfonyContainer::getInstance()->get('router')->generate('is_imageslider_controller'));
+        $url = SymfonyContainer::getInstance()->get('router')->generate('is_imageslider_controller');
+        return new RedirectResponse($url);
     }
-
     /**
      * @template T
      *

is_imageslider/src/Form/DataHandler/ImageSliderFormDataHandler.php

@@ -79,6 +79,9 @@ class ImageSliderFormDataHandler implements FormDataHandlerInterface
         $imageSlide->setPosition($this->imageSliderRepository->getHighestPosition() + 1);
         $this->addAssociatedShops($imageSlide, $data['shop_association'] ?? null);
 
+        $uploadedImage = !empty($data['image']) ? $this->uploadFile($data['image']) : null;
+        $uploadedImageMobile = !empty($data['image_mobile']) ? $this->uploadFile($data['image_mobile']) : null;
+
         foreach ($this->languages as $language) {
             $langId = (int) $language['id_lang'];
             $lang = $this->langRepository->findOneById($langId);
@@ -87,16 +90,16 @@ class ImageSliderFormDataHandler implements FormDataHandlerInterface
             $imageSliderLang
                 ->setLang($lang)
                 ->setTitle($data['title'][$langId] ?? '')
-                ->setUrl($data['url'][$langId] ?? '')
+                ->setUrl($data['url'] ?? '')
                 ->setLegend($data['legend'][$langId] ?? '')
                 ->setDescription($data['description'][$langId] ?? '');
 
-            if (!empty($data['image'][$langId])) {
+            if ($uploadedImage !== null) {
-                $imageSliderLang->setImage($this->uploadFile($data['image'][$langId]));
+                $imageSliderLang->setImage($uploadedImage);
             }
 
-            if (!empty($data['image_mobile'][$langId])) {
+            if ($uploadedImageMobile !== null) {
-                $imageSliderLang->setImageMobile($this->uploadFile($data['image_mobile'][$langId]));
+                $imageSliderLang->setImageMobile($uploadedImageMobile);
             }
 
             $imageSlide->addImageSliderLang($imageSliderLang);
@@ -122,6 +125,9 @@ class ImageSliderFormDataHandler implements FormDataHandlerInterface
         $imageSlide->setDisplayTo($data['display_to'] ?? new \DateTime());
         $this->addAssociatedShops($imageSlide, $data['shop_association'] ?? null);
 
+        $uploadedImage = !empty($data['image']) ? $this->uploadFile($data['image']) : null;
+        $uploadedImageMobile = !empty($data['image_mobile']) ? $this->uploadFile($data['image_mobile']) : null;
+
         foreach ($this->languages as $language) {
             $langId = (int) $language['id_lang'];
             $imageSliderLang = $imageSlide->getImageSliderLangByLangId($langId);
@@ -136,22 +142,22 @@ class ImageSliderFormDataHandler implements FormDataHandlerInterface
 
             $imageSliderLang
                 ->setTitle($data['title'][$langId] ?? '')
-                ->setUrl($data['url'][$langId] ?? '')
+                ->setUrl($data['url'] ?? '')
                 ->setLegend($data['legend'][$langId] ?? '')
                 ->setDescription($data['description'][$langId] ?? '');
 
-            if (!empty($data['image'][$langId])) {
+            if ($uploadedImage !== null) {
                 if ($imageSliderLang->getImage() !== null) {
                     $this->eraseFile($imageSliderLang->getImage());
                 }
-                $imageSliderLang->setImage($this->uploadFile($data['image'][$langId]));
+                $imageSliderLang->setImage($uploadedImage);
             }
 
-            if (!empty($data['image_mobile'][$langId])) {
+            if ($uploadedImageMobile !== null) {
-                if ($imageSliderLang->getImage() !== null) {
+                if ($imageSliderLang->getImageMobile() !== null) {
                     $this->eraseFile($imageSliderLang->getImageMobile());
                 }
-                $imageSliderLang->setImageMobile($this->uploadFile($data['image_mobile'][$langId]));
+                $imageSliderLang->setImageMobile($uploadedImageMobile);
             }
 
             if ($newImageSliderLang) {

is_imageslider/src/Form/ImageSliderConfigurationType.php

@@ -43,8 +43,7 @@ class ImageSliderConfigurationType extends TranslatorAwareType
                         'min' => $minTime,
                         'max' => $maxTime,
                         'invalidMessage' => $rangeInvalidMessage,
-                        'maxMessage' => $rangeInvalidMessage,
+                        'notInRangeMessage' => $rangeInvalidMessage
-                        'minMessage' => $rangeInvalidMessage,
                     ]),
                 ],
             ])

is_imageslider/src/Form/ImageSliderType.php

@@ -5,7 +5,6 @@ declare(strict_types=1);
 namespace Oksydan\IsImageslider\Form;
 
 use Oksydan\IsImageslider\Translations\TranslationDomains;
-use Oksydan\IsImageslider\Type\TranslatableFile;
 use PrestaShopBundle\Form\Admin\Type\FormattedTextareaType;
 use PrestaShopBundle\Form\Admin\Type\ImagePreviewType;
 use PrestaShopBundle\Form\Admin\Type\ShopChoiceTreeType;
@@ -16,7 +15,7 @@ use Symfony\Component\Form\Extension\Core\Type\DateTimeType;
 use Symfony\Component\Form\Extension\Core\Type\FileType;
 use Symfony\Component\Form\Extension\Core\Type\TextType;
 use Symfony\Component\Form\FormBuilderInterface;
-use Symfony\Component\Translation\TranslatorInterface;
+use Symfony\Contracts\Translation\TranslatorInterface;
 use Symfony\Component\Validator\Constraints\File;
 use Symfony\Component\Validator\Constraints\NotBlank;
 
@@ -63,37 +62,25 @@ class ImageSliderType extends TranslatorAwareType
         }
 
         $builder
-            ->add('image_preview', TranslatableType::class, [
+            ->add('image_preview', ImagePreviewType::class, [
-                'type' => ImagePreviewType::class,
                 'label' => $this->trans('Image preview', TranslationDomains::TRANSLATION_DOMAIN_ADMIN),
-                'locales' => $this->locales,
                 'required' => false,
             ])
-            ->add('image', TranslatableFile::class, [
+            ->add('image', FileType::class, [
-                'type' => FileType::class,
                 'label' => $this->trans('Image', TranslationDomains::TRANSLATION_DOMAIN_ADMIN),
-                'locales' => $this->locales,
-                'options' => [
                 'data_class' => null,
                 'constraints' => $imageConstrains,
-                ],
+                'required' => !$isEdit,
-                'required' => true,
             ])
-            ->add('image_mobile_preview', TranslatableType::class, [
+            ->add('image_mobile_preview', ImagePreviewType::class, [
-                'type' => ImagePreviewType::class,
                 'label' => $this->trans('Image mobile preview', TranslationDomains::TRANSLATION_DOMAIN_ADMIN),
-                'locales' => $this->locales,
                 'required' => false,
             ])
-            ->add('image_mobile', TranslatableFile::class, [
+            ->add('image_mobile', FileType::class, [
-                'type' => FileType::class,
                 'label' => $this->trans('Image mobile', TranslationDomains::TRANSLATION_DOMAIN_ADMIN),
-                'locales' => $this->locales,
-                'options' => [
                 'data_class' => null,
                 'constraints' => $imageConstrains,
-                ],
+                'required' => false,
-                'required' => true,
             ])
             ->add('title', TranslatableType::class, [
                 'type' => TextType::class,
@@ -107,16 +94,12 @@ class ImageSliderType extends TranslatorAwareType
                 'locales' => $this->locales,
                 'required' => false,
             ])
-            ->add('url', TranslatableType::class, [
+            ->add('url', TextType::class, [
-                'type' => TextType::class,
                 'label' => $this->trans('Link', TranslationDomains::TRANSLATION_DOMAIN_ADMIN),
-                'locales' => $this->locales,
                 'required' => true,
-                'options' => [
                 'constraints' => [
                     new NotBlank(),
                 ],
-                ],
             ])
             ->add('description', TranslatableType::class, [
                 'type' => FormattedTextareaType::class,

is_imageslider/src/Form/Provider/ImageSliderFormDataProvider.php

@@ -77,16 +77,20 @@ class ImageSliderFormDataProvider implements FormDataProviderInterface
         $slideData['display_from'] = $imageSlide->getDisplayFrom();
         $slideData['display_to'] = $imageSlide->getDisplayTo();
 
+        $firstLang = true;
         foreach ($imageSlide->getSliderLangs() as $imageSlideLang) {
             $slideData['title'][$imageSlideLang->getLang()->getId()] = $imageSlideLang->getTitle();
             $slideData['legend'][$imageSlideLang->getLang()->getId()] = $imageSlideLang->getLegend();
-            $slideData['url'][$imageSlideLang->getLang()->getId()] = $imageSlideLang->getUrl();
             $slideData['description'][$imageSlideLang->getLang()->getId()] = $imageSlideLang->getDescription();
-            $slideData['image'][$imageSlideLang->getLang()->getId()] = $imageSlideLang->getImage();
-            $slideData['image_mobile'][$imageSlideLang->getLang()->getId()] = $imageSlideLang->getImageMobile();
 
-            $slideData['image_preview'][$imageSlideLang->getLang()->getId()] = $this->imagesliderImageThumbProvider->getPath($imageSlideLang->getImage()) ?? $this->placeholderImage;
+            if ($firstLang) {
-            $slideData['image_mobile_preview'][$imageSlideLang->getLang()->getId()] = $this->imagesliderImageThumbProvider->getPath($imageSlideLang->getImageMobile()) ?? $this->placeholderImage;
+                $slideData['url'] = $imageSlideLang->getUrl();
+                $slideData['image'] = $imageSlideLang->getImage();
+                $slideData['image_mobile'] = $imageSlideLang->getImageMobile();
+                $slideData['image_preview'] = $this->imagesliderImageThumbProvider->getPath($imageSlideLang->getImage()) ?? $this->placeholderImage;
+                $slideData['image_mobile_preview'] = $this->imagesliderImageThumbProvider->getPath($imageSlideLang->getImageMobile()) ?? $this->placeholderImage;
+                $firstLang = false;
+            }
         }
 
         return $slideData;
@@ -113,11 +117,11 @@ class ImageSliderFormDataProvider implements FormDataProviderInterface
     public function getDefaultData(): array
     {
         return [
-            'image_preview' => $this->getImagePreviewPlaceholder(),
+            'image_preview' => $this->placeholderImage,
-            'image_mobile_preview' => $this->getImagePreviewPlaceholder(),
+            'image_mobile_preview' => $this->placeholderImage,
             'title' => [],
             'legend' => [],
-            'url' => [],
+            'url' => '',
             'description' => [],
             'active' => false,
             'display_from' => new \DateTime(),

is_imageslider/src/Grid/Query/ImageSliderQueryBuilder.php

@@ -57,8 +57,8 @@ final class ImageSliderQueryBuilder extends AbstractDoctrineQueryBuilder
             $searchCriteria->getOrderBy(),
             $searchCriteria->getOrderWay()
         )
-            ->setFirstResult($searchCriteria->getOffset())
+            ->setFirstResult($searchCriteria->getOffset() ?? 0)
-            ->setMaxResults($searchCriteria->getLimit());
+            ->setMaxResults($searchCriteria->getLimit() ?? 10);
 
         $qb->orderBy('position');
 

is_imageslider/vendor/composer/autoload_classmap.php

@@ -31,6 +31,7 @@ return array(
     'Oksydan\\IsImageslider\\Hook\\AbstractHook' => $baseDir . '/src/Hook/AbstractHook.php',
     'Oksydan\\IsImageslider\\Hook\\DisplayHeader' => $baseDir . '/src/Hook/DisplayHeader.php',
     'Oksydan\\IsImageslider\\Hook\\DisplayHome' => $baseDir . '/src/Hook/DisplayHome.php',
+    'Oksydan\\IsImageslider\\Hook\\DisplayHeroSlider' => $baseDir . '/src/Hook/DisplayHeroSlider.php',
     'Oksydan\\IsImageslider\\Hook\\HookInterface' => $baseDir . '/src/Hook/HookInterface.php',
     'Oksydan\\IsImageslider\\Hook\\WidgetCapability' => $baseDir . '/src/Hook/WidgetCapability.php',
     'Oksydan\\IsImageslider\\Installer\\ActionDatabaseAbstract' => $baseDir . '/src/Installer/ActionDatabaseAbstract.php',

is_imageslider/vendor/composer/autoload_static.php

@@ -46,6 +46,7 @@ class ComposerStaticInit9ba08c95d0af82fb7a45da2b6d8d15e0
         'Oksydan\\IsImageslider\\Hook\\AbstractHook' => __DIR__ . '/../..' . '/src/Hook/AbstractHook.php',
         'Oksydan\\IsImageslider\\Hook\\DisplayHeader' => __DIR__ . '/../..' . '/src/Hook/DisplayHeader.php',
         'Oksydan\\IsImageslider\\Hook\\DisplayHome' => __DIR__ . '/../..' . '/src/Hook/DisplayHome.php',
+        'Oksydan\\IsImageslider\\Hook\\DisplayHeroSlider' => __DIR__ . '/../..' . '/src/Hook/DisplayHeroSlider.php',
         'Oksydan\\IsImageslider\\Hook\\HookInterface' => __DIR__ . '/../..' . '/src/Hook/HookInterface.php',
         'Oksydan\\IsImageslider\\Hook\\WidgetCapability' => __DIR__ . '/../..' . '/src/Hook/WidgetCapability.php',
         'Oksydan\\IsImageslider\\Installer\\ActionDatabaseAbstract' => __DIR__ . '/../..' . '/src/Installer/ActionDatabaseAbstract.php',