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,191 +1,197 @@
 imports:
-  - { resource: ../common.yml }
-
+    - { resource: ../common.yml }
 
 services:
-  _defaults:
-    public: true
+    _defaults:
+        public: true
 
-  # Controller
-  Oksydan\IsImageslider\Controller\IsImagesliderController:
-    public: true
-    class: Oksydan\IsImageslider\Controller\IsImagesliderController
-    arguments:
-      - '@oksydan.is_imageslider.handler.file_eraser'
-      - "@=service('prestashop.adapter.legacy.context').getLanguages(false)"
-      - "@oksydan.is_imageslider.cache.template_cache"
+    # Controller
+    Oksydan\IsImageslider\Controller\IsImagesliderController:
+        public: true
+        class: Oksydan\IsImageslider\Controller\IsImagesliderController
+        arguments:
+            - "@oksydan.is_imageslider.handler.file_eraser"
+            - "@=service('prestashop.adapter.legacy.context').getLanguages(false)"
+            - "@oksydan.is_imageslider.cache.template_cache"
 
-  # installer
-  oksydan.is_imageslider.image_slider_installer:
-    class: 'Oksydan\IsImageslider\Installer\ImageSliderInstaller'
-    arguments:
-      - '@doctrine.dbal.default_connection'
-      - '@oksydan.is_imageslider.installer.database_yml_parser'
-      - "@=service('prestashop.adapter.legacy.context').getContext()"
+    # installer
+    oksydan.is_imageslider.image_slider_installer:
+        class: 'Oksydan\IsImageslider\Installer\ImageSliderInstaller'
+        arguments:
+            - "@doctrine.dbal.default_connection"
+            - "@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:
+        class: 'Oksydan\IsImageslider\Installer\DatabaseYamlParser'
+        arguments:
+            - "@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.installer.database_yml_parser_provider:
+        class: 'Oksydan\IsImageslider\Installer\Provider\DatabaseYamlProvider'
+        arguments:
+            - "@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'
-    public: true
+    oksydan.is_imageslider.installer.action_databse_create_table:
+        class: 'Oksydan\IsImageslider\Installer\ActionDatabaseCrateTable'
+        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'
-    public: true
+    oksydan.is_imageslider.installer.action_databse_drop_table:
+        class: 'Oksydan\IsImageslider\Installer\ActionDatabaseDropTable'
+        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'
-      - '%database_prefix%'
+    oksydan.is_imageslider.installer.abstract_action_databse:
+        class: 'Oksydan\IsImageslider\Installer\ActionDatabaseAbstract'
+        abstract: true
+        arguments:
+            - "@doctrine.dbal.default_connection"
+            - "%database_prefix%"
 
-  oksydan.is_imageslider.handler.file_uploader:
-    class: 'Oksydan\IsImageslider\Handler\FileUploader'
-    arguments:
-      $targetDirectory: '%is_imageslider_images_directory%'
+    oksydan.is_imageslider.handler.file_uploader:
+        class: 'Oksydan\IsImageslider\Handler\FileUploader'
+        arguments:
+            $targetDirectory: "%is_imageslider_images_directory%"
 
-  oksydan.is_imageslider.handler.file_eraser:
-    class: 'Oksydan\IsImageslider\Handler\FileEraser'
-    arguments:
-      $targetDirectory: '%is_imageslider_images_directory%'
+    oksydan.is_imageslider.handler.file_eraser:
+        class: 'Oksydan\IsImageslider\Handler\FileEraser'
+        arguments:
+            $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.provider.image_provider"
+            - "@prestashop.core.admin.lang.repository"
+            - "%is_imageslider_placeholder_img%"
+            - "@prestashop.adapter.shop.context"
 
-  #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.provider.image_provider'
-      - '@prestashop.core.admin.lang.repository'
-      - '%is_imageslider_placeholder_img%'
-      - '@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"]
+        arguments:
+            - 'Oksydan\IsImageslider\Form\ImageSliderType'
+            - "@oksydan.is_imageslider.form.identifiable_object.data_provider.image_slider_form_data_provider"
 
-  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'
-    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_handler.image_slider_form_data_handler:
+        class: 'Oksydan\IsImageslider\Form\DataHandler\ImageSliderFormDataHandler'
+        arguments:
+            - "@oksydan.is_imageslider.repository.image_slider"
+            - "@prestashop.core.admin.lang.repository"
+            - "@doctrine.orm.default_entity_manager"
+            - "@oksydan.is_imageslider.handler.file_uploader"
+            - "@oksydan.is_imageslider.handler.file_eraser"
+            - "@=service('prestashop.adapter.legacy.context').getLanguages()"
 
-  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'
-      - '@prestashop.core.admin.lang.repository'
-      - '@doctrine.orm.default_entity_manager'
-      - '@oksydan.is_imageslider.handler.file_uploader'
-      - '@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",
+            ]
+        arguments:
+            - "@oksydan.is_imageslider.form.identifiable_object.data_handler.image_slider_form_data_handler"
 
-  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'
-    arguments:
-      - '@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"
+        public: true
 
-  #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'
-    public: true
+    oksydan.is_imageslider.grid.query.image_slider_query_builder:
+        class: 'Oksydan\IsImageslider\Grid\Query\ImageSliderQueryBuilder'
+        parent: "prestashop.core.grid.abstract_query_builder"
+        public: true
+        arguments:
+            - "@prestashop.adapter.shop.context"
+            - "@=service('prestashop.adapter.legacy.context').getContext().language.id"
 
-  oksydan.is_imageslider.grid.query.image_slider_query_builder:
-    class: 'Oksydan\IsImageslider\Grid\Query\ImageSliderQueryBuilder'
-    parent: 'prestashop.core.grid.abstract_query_builder'
-    public: true
-    arguments:
-      - '@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.data.factory.image_slider_decorator"
+            - "@prestashop.core.grid.filter.form_factory"
+            - "@prestashop.core.hook.dispatcher"
 
-  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.data.factory.image_slider_decorator'
-      - '@prestashop.core.grid.filter.form_factory'
-      - '@prestashop.core.hook.dispatcher'
+    oksydan.is_imageslider.grid.data.factory.image_slider_data_factory:
+        class: "%prestashop.core.grid.data.factory.doctrine_grid_data_factory%"
+        arguments:
+            - "@oksydan.is_imageslider.grid.query.image_slider_query_builder"
+            - "@prestashop.core.hook.dispatcher"
+            - "@prestashop.core.grid.query.doctrine_query_parser"
+            - "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.provider.image_provider"
 
-  oksydan.is_imageslider.grid.data.factory.image_slider_data_factory:
-    class: '%prestashop.core.grid.data.factory.doctrine_grid_data_factory%'
-    arguments:
-      - '@oksydan.is_imageslider.grid.query.image_slider_query_builder'
-      - '@prestashop.core.hook.dispatcher'
-      - '@prestashop.core.grid.query.doctrine_query_parser'
-      - 'is_imageslider'
+    # config form
+    oksydan.is_imageslider.image_slider.configuration:
+        class: 'Oksydan\IsImageslider\Form\DataConfiguration\ImageSliderDataConfiguration'
+        arguments:
+            - "@prestashop.adapter.legacy.configuration"
+            - "@prestashop.adapter.shop.context"
+            - "@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.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.provider.image_provider'
+    oksydan.is_imageslider.image_slider_configuration.form_handler:
+        class: 'PrestaShop\PrestaShop\Core\Form\Handler'
+        arguments:
+            - "@form.factory"
+            - "@prestashop.core.hook.dispatcher"
+            - "@oksydan.is_imageslider.image_slider_configuration.form_provider"
+            - 'Oksydan\IsImageslider\Form\ImageSliderConfigurationType'
+            - "ConfigurationForm"
 
-  # config form
-  oksydan.is_imageslider.image_slider.configuration:
-    class: 'Oksydan\IsImageslider\Form\DataConfiguration\ImageSliderDataConfiguration'
-    arguments:
-      - '@prestashop.adapter.legacy.configuration'
-      - '@prestashop.adapter.shop.context'
-      - '@prestashop.adapter.multistore_feature'
+    oksydan.is_imageslider.form.image_slider_type:
+        class: 'Oksydan\IsImageslider\Form\ImageSliderType'
+        parent: "form.type.translatable.aware"
+        arguments:
+            - '@=service("prestashop.adapter.multistore_feature").isUsed()'
+        public: true
+        tags:
+            - { name: form.type }
 
-  oksydan.is_imageslider.image_slider_configuration.form_provider:
-    class: 'Oksydan\IsImageslider\Form\Provider\ImageSliderConfigurationFormDataProvider'
-    arguments:
-      - '@oksydan.is_imageslider.image_slider.configuration'
+    oksydan.is_imageslider.form.image_slider_configuration_type:
+        class: 'Oksydan\IsImageslider\Form\ImageSliderConfigurationType'
+        parent: "form.type.translatable.aware"
+        public: true
+        tags:
+            - { name: form.type }
 
-  oksydan.is_imageslider.image_slider_configuration.form_handler:
-    class: 'PrestaShop\PrestaShop\Core\Form\Handler'
-    arguments:
-      - '@form.factory'
-      - '@prestashop.core.hook.dispatcher'
-      - '@oksydan.is_imageslider.image_slider_configuration.form_provider'
-      - 'Oksydan\IsImageslider\Form\ImageSliderConfigurationType'
-      - 'ConfigurationForm'
+    Oksydan\IsImageslider\Type\TranslatableFile:
+        class: 'Oksydan\IsImageslider\Type\TranslatableFile'
+        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.form.image_slider_type:
-    class: 'Oksydan\IsImageslider\Form\ImageSliderType'
-    parent: 'form.type.translatable.aware'
-    arguments:
-      - '@=service("prestashop.adapter.multistore_feature").isUsed()'
-    public: true
-    tags:
-      - { name: form.type }
+    oksydan.is_imageslider.grid.position_definition:
+        class: PrestaShop\PrestaShop\Core\Grid\Position\PositionDefinition
+        arguments:
+            - "image_slider"
+            - "id_slide"
+            - "position"
 
-  oksydan.is_imageslider.form.image_slider_configuration_type:
-    class: 'Oksydan\IsImageslider\Form\ImageSliderConfigurationType'
-    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'
-    public: true
-    tags:
-      - { name: form.type }
-
-  oksydan.is_imageslider.grid.position_definition:
-    class: PrestaShop\PrestaShop\Core\Grid\Position\PositionDefinition
-    arguments:
-      - 'image_slider'
-      - 'id_slide'
-      - 'position'
-
-  oksydan.is_imageslider.provider.image_provider:
-    class: 'Oksydan\IsImageslider\Provider\ImageProvider'
-    public: true
-    arguments:
-      - '%is_imageslider_images_uri%'
+    oksydan.is_imageslider.provider.image_provider:
+        class: 'Oksydan\IsImageslider\Provider\ImageProvider'
+        public: true
+        arguments:
+            - "%is_imageslider_images_uri%"

is_imageslider/config/common.yml

@@ -1,91 +1,96 @@
 parameters:
-  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_placeholder_img: '/modules/is_imageslider/views/img/placeholder.jpeg'
+    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_placeholder_img: "/modules/is_imageslider/views/img/placeholder.jpeg"
 
 services:
-  _defaults:
-    public: true
+    _defaults:
+        public: true
 
-  oksydan.is_imageslider.module:
-    class: Is_imageslider
-    factory: [ 'Module', 'getInstanceByName' ]
-    public: false
-    arguments:
-      - 'is_imageslider'
+    oksydan.is_imageslider.module:
+        class: Is_imageslider
+        factory: ["Module", "getInstanceByName"]
+        public: false
+        arguments:
+            - "is_imageslider"
 
-  oksydan.is_imageslider.repository.image_slider:
-    class: Oksydan\IsImageslider\Repository\ImageSliderRepository
-    public: true
-    factory: ['@doctrine.orm.default_entity_manager', getRepository]
-    arguments:
-      - Oksydan\IsImageslider\Entity\ImageSlider
+    oksydan.is_imageslider.repository.image_slider:
+        class: Oksydan\IsImageslider\Repository\ImageSliderRepository
+        public: true
+        factory: ["@doctrine.orm.default_entity_manager", getRepository]
+        arguments:
+            - Oksydan\IsImageslider\Entity\ImageSlider
 
-  oksydan.is_imageslider.repository.hook_module_repository:
-    class: Oksydan\IsImageslider\Repository\HookModuleRepository
-    public: true
-    arguments:
-      - '@doctrine.dbal.default_connection'
-      - '%database_prefix%'
+    oksydan.is_imageslider.repository.hook_module_repository:
+        class: Oksydan\IsImageslider\Repository\HookModuleRepository
+        public: true
+        arguments:
+            - "@doctrine.dbal.default_connection"
+            - "%database_prefix%"
 
-  oksydan.is_imageslider.presenter.image_slide_presenter:
-    class: Oksydan\IsImageslider\Presenter\ImageSlidePresenter
-    public: true
-    arguments:
-      - '%is_imageslider_images_uri%'
-      - '%is_imageslider_images_directory%'
-      - "@=service('prestashop.adapter.legacy.context').getContext()"
+    oksydan.is_imageslider.presenter.image_slide_presenter:
+        class: Oksydan\IsImageslider\Presenter\ImageSlidePresenter
+        public: true
+        arguments:
+            - "%is_imageslider_images_uri%"
+            - "%is_imageslider_images_directory%"
+            - "@=service('prestashop.adapter.legacy.context').getContext()"
 
-  oksydan.is_imageslider.configuration.slider_configuration:
-    class: Oksydan\IsImageslider\Configuration\SliderConfiguration
-    public: true
+    oksydan.is_imageslider.configuration.slider_configuration:
+        class: Oksydan\IsImageslider\Configuration\SliderConfiguration
+        public: true
 
-  oksydan.is_imageslider.hook.abstract_hook:
-    class: Oksydan\IsImageslider\Hook\AbstractHook
-    public: true
-    arguments:
-      - '@oksydan.is_imageslider.module'
-      - "@=service('prestashop.adapter.legacy.context').getContext()"
+    oksydan.is_imageslider.hook.abstract_hook:
+        class: Oksydan\IsImageslider\Hook\AbstractHook
+        public: true
+        arguments:
+            - "@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'
-    public: true
-    arguments:
-      - '@oksydan.is_imageslider.configuration.slider_configuration'
+    oksydan.is_imageslider.hook.abstract_display_hook:
+        class: Oksydan\IsImageslider\Hook\AbstractDisplayHook
+        parent: "oksydan.is_imageslider.hook.abstract_hook"
+        public: true
+        arguments:
+            - "@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'
-    public: true
-    arguments:
-      - '@oksydan.is_imageslider.repository.image_slider'
-      - '@oksydan.is_imageslider.presenter.image_slide_presenter'
-      - '@oksydan.is_imageslider.cache.template_cache'
+    oksydan.is_imageslider.hook.abstract_cacheable_display_hook:
+        class: Oksydan\IsImageslider\Hook\AbstractCacheableDisplayHook
+        parent: "oksydan.is_imageslider.hook.abstract_display_hook"
+        public: true
+        arguments:
+            - "@oksydan.is_imageslider.repository.image_slider"
+            - "@oksydan.is_imageslider.presenter.image_slide_presenter"
+            - "@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'
-    public: true
+    oksydan.is_imageslider.hook.display_home:
+        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'
-    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
 
-  # 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'
-    public: true
+    oksydan.is_imageslider.hook.display_header:
+        class: Oksydan\IsImageslider\Hook\DisplayHeader
+        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'
-      - "@=service('prestashop.adapter.legacy.context').getContext()"
-      - '@oksydan.is_imageslider.repository.hook_module_repository'
-      - '@prestashop.adapter.legacy.configuration'
-      - '@oksydan.is_imageslider.repository.image_slider'
+    # 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"
+        public: true
+
+    oksydan.is_imageslider.cache.template_cache:
+        class: Oksydan\IsImageslider\Cache\TemplateCache
+        public: true
+        arguments:
+            - "@oksydan.is_imageslider.module"
+            - "@=service('prestashop.adapter.legacy.context').getContext()"
+            - "@oksydan.is_imageslider.repository.hook_module_repository"
+            - "@prestashop.adapter.legacy.configuration"
+            - "@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])) {
-                $imageSliderLang->setImage($this->uploadFile($data['image'][$langId]));
+            if ($uploadedImage !== null) {
+                $imageSliderLang->setImage($uploadedImage);
             }
 
-            if (!empty($data['image_mobile'][$langId])) {
-                $imageSliderLang->setImageMobile($this->uploadFile($data['image_mobile'][$langId]));
+            if ($uploadedImageMobile !== null) {
+                $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 ($imageSliderLang->getImage() !== null) {
+            if ($uploadedImageMobile !== 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,
-                        'minMessage' => $rangeInvalidMessage,
+                        'notInRangeMessage' => $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, [
-                'type' => ImagePreviewType::class,
+            ->add('image_preview', ImagePreviewType::class, [
                 'label' => $this->trans('Image preview', TranslationDomains::TRANSLATION_DOMAIN_ADMIN),
-                'locales' => $this->locales,
                 'required' => false,
             ])
-            ->add('image', TranslatableFile::class, [
-                'type' => FileType::class,
+            ->add('image', FileType::class, [
                 'label' => $this->trans('Image', TranslationDomains::TRANSLATION_DOMAIN_ADMIN),
-                'locales' => $this->locales,
-                'options' => [
-                    'data_class' => null,
-                    'constraints' => $imageConstrains,
-                ],
-                'required' => true,
+                'data_class' => null,
+                'constraints' => $imageConstrains,
+                'required' => !$isEdit,
             ])
-            ->add('image_mobile_preview', TranslatableType::class, [
-                'type' => ImagePreviewType::class,
+            ->add('image_mobile_preview', ImagePreviewType::class, [
                 'label' => $this->trans('Image mobile preview', TranslationDomains::TRANSLATION_DOMAIN_ADMIN),
-                'locales' => $this->locales,
                 'required' => false,
             ])
-            ->add('image_mobile', TranslatableFile::class, [
-                'type' => FileType::class,
+            ->add('image_mobile', FileType::class, [
                 'label' => $this->trans('Image mobile', TranslationDomains::TRANSLATION_DOMAIN_ADMIN),
-                'locales' => $this->locales,
-                'options' => [
-                    'data_class' => null,
-                    'constraints' => $imageConstrains,
-                ],
-                'required' => true,
+                'data_class' => null,
+                'constraints' => $imageConstrains,
+                'required' => false,
             ])
             ->add('title', TranslatableType::class, [
                 'type' => TextType::class,
@@ -107,15 +94,11 @@ class ImageSliderType extends TranslatorAwareType
                 'locales' => $this->locales,
                 'required' => false,
             ])
-            ->add('url', TranslatableType::class, [
-                'type' => TextType::class,
+            ->add('url', TextType::class, [
                 'label' => $this->trans('Link', TranslationDomains::TRANSLATION_DOMAIN_ADMIN),
-                'locales' => $this->locales,
                 'required' => true,
-                'options' => [
-                    'constraints' => [
-                        new NotBlank(),
-                    ],
+                'constraints' => [
+                    new NotBlank(),
                 ],
             ])
             ->add('description', TranslatableType::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;
-            $slideData['image_mobile_preview'][$imageSlideLang->getLang()->getId()] = $this->imagesliderImageThumbProvider->getPath($imageSlideLang->getImageMobile()) ?? $this->placeholderImage;
+            if ($firstLang) {
+                $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_mobile_preview' => $this->getImagePreviewPlaceholder(),
+            'image_preview' => $this->placeholderImage,
+            '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())
-            ->setMaxResults($searchCriteria->getLimit());
+            ->setFirstResult($searchCriteria->getOffset() ?? 0)
+            ->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',