Revert "Version 3.x : No live component and multi upload"

Author: maximehuran

.github/copilot-instructions.md

@@ -0,0 +1,21 @@
+# Copilot Instructions for SyliusMediaManagerPlugin
+
+## General Guidelines
+- Follow the Sylius 2.x and Symfony 7.x best practices for PHP code.
+- Use strict types and type hints wherever possible.
+- All new code must be PSR-12 compliant and pass PHPStan and ECS checks.
+- Use dependency injection for all services and avoid using service locators.
+- Write clear, concise, and well-documented code.
+- Use English for all code comments, variable names, and documentation.
+
+## Commit Messages
+- Use clear, descriptive commit messages in English.
+- Reference related issues or tickets when applicable.
+
+## Pull Requests
+- Ensure all checks pass before requesting review.
+- Provide a clear description of the changes and testing steps.
+
+---
+
+For more details, refer to the project README and Sylius/Symfony documentation.

.github/workflows/recipe.yaml

@@ -2,7 +2,7 @@ name: Flex Recipe
 
 on:
   push:
-    branches: [ $default-branch ]
+    branches: [ master ]
   pull_request:
 
 jobs:
@@ -14,7 +14,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: ['8.2', '8.3']
+        php: [ '8.2', '8.3' ]
         sylius: [ '~2.0.0', '~2.1.0' ]
 
     steps:

.github/workflows/tests.yaml

@@ -2,7 +2,7 @@ name: Tests
 
 on:
   push:
-    branches: [ $default-branch ]
+    branches: [ master ]
   pull_request:
 
 jobs:
@@ -14,8 +14,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [ '8.2', '8.3' ]
-        node: [ '20' ]
+        php: ['8.2', '8.3']
+        node: ['20']
 
     env:
       SYMFONY_ARGS: --no-tls

.php-cs-fixer.dist.php

@@ -34,9 +34,9 @@
     ->setRiskyAllowed(true)
     ->setRules([
         '@DoctrineAnnotation' => true,
-        '@PHP7x1Migration' => true,
-        '@PHP7x1Migration:risky' => true,
-        '@PHPUnit6x0Migration:risky' => true,
+        '@PHP71Migration' => true,
+        '@PHP71Migration:risky' => true,
+        '@PHPUnit60Migration:risky' => true,
         '@Symfony' => true,
         '@Symfony:risky' => true,
         'align_multiline_comment' => [
@@ -254,7 +254,7 @@
         'trim_array_spaces' => true,
         'type_declaration_spaces' => true,
         'unary_operator_spaces' => true,
-        'modifier_keywords' => [
+        'visibility_required' => [
             'elements' => [
                 'const',
                 'property',

Makefile

@@ -72,12 +72,17 @@ setup_application:
 	(cd ${APP_DIR} && ${COMPOSER} config --no-plugins allow-plugins true)
 	(cd ${APP_DIR} && ${COMPOSER} config --no-plugins --json extra.symfony.endpoint '["https://api.github.com/repos/monsieurbiz/symfony-recipes/contents/index.json?ref=flex/master","flex://defaults"]')
 	(cd ${APP_DIR} && ${COMPOSER} require --no-install --no-scripts --no-progress sylius/sylius="~${SYLIUS_VERSION}") # Make sure to install the required version of sylius because the sylius-standard has a soft constraint
+	# Temporary fix for Sylius 2.1
+	(cd ${APP_DIR} && ${COMPOSER} require --no-install --no-scripts --no-progress api-platform/state)
+	# End Temporary fix for Sylius 2.1
 	$(MAKE) ${APP_DIR}/.php-version
 	$(MAKE) ${APP_DIR}/php.ini
 	(cd ${APP_DIR} && ${COMPOSER} install --no-interaction)
 	$(MAKE) apply_dist
 	(cd ${APP_DIR} && ${COMPOSER} require --no-progress monsieurbiz/${PLUGIN_NAME}="*@dev")
 	rm -rf ${APP_DIR}/var/cache
+	mkdir -p ${APP_DIR}/public/media/gallery
+	cp -r fixtures/gallery/* ${APP_DIR}/public/media/gallery
 
 
 ${APP_DIR}/docker-compose.yaml:
@@ -136,13 +141,13 @@ test.container: ## Lint the symfony container
 	${CONSOLE} lint:container
 
 test.yaml: ## Lint the symfony Yaml files
-	${CONSOLE} lint:yaml ../../src/Resources/config --parse-tags
+	${CONSOLE} lint:yaml ../../config --parse-tags
 
 test.schema: ## Validate MySQL Schema
 	${CONSOLE} doctrine:schema:validate
 
 test.twig: ## Validate Twig templates
-	${CONSOLE} lint:twig --no-debug templates/ ../../src/Resources/views/
+	${CONSOLE} lint:twig --no-debug templates/ ../../templates/
 
 ###
 ### SYLIUS

README.md

@@ -9,15 +9,17 @@
 
 This plugin adds a media manager to your images, videos and other files type fields in Sylius.
 
-![Demo of the media manager](docs/images/demo.gif)
+![Example of field display](docs/images/media-manager-product-medias.png)
+![Example of image listing](docs/images/media-manager-images.png)
+![Example of PDF listing](docs/images/media-manager-pdfs.png)
 
 ## Compatibility
 
-| Sylius Version | PHP Version     |
-|----------------|-----------------|
-| 2.0, 2.1       | 8.2 - 8.3       |
+| Sylius Version | PHP Version |
+|----------------|-------------|
+| 2.0, 2,1       | 8.2 - 8.3   |
 
-ℹ️ For Sylius 1.x, see our [1.x branch](/tree/1.x) and all 1.x releases.
+ℹ️ For Sylius 1.x, see our [1.x branch](https://github.com/monsieurbiz/SyliusMediaManagerPlugin/tree/1.x) and all 1.x releases.
 
 ## Installation
 
@@ -48,7 +50,7 @@ return [
 ];
 ```
 
-Copy the plugin configuration files in your `config` folder: https://github.com/monsieurbiz/symfony-recipes/tree/master/monsieurbiz/sylius-media-manager-plugin/3.0/config
+Copy the plugin configuration files in your `config` folder: https://github.com/monsieurbiz/symfony-recipes/tree/master/monsieurbiz/sylius-media-manager-plugin/1.0/config
 
 Add these variables to your `.env` :
 
@@ -97,6 +99,10 @@ Use `MonsieurBiz\SyliusMediaManagerPlugin\Form\Type\FaviconType`
 
 Use `MonsieurBiz\SyliusMediaManagerPlugin\Form\Type\AudioType`
 
+### All the previous file types
+
+Use `MonsieurBiz\SyliusMediaManagerPlugin\Form\Type\FileType`
+
 ## Configuration
 
 You can specify the default maximum file size allowed for the media manager by setting the `MONSIEURBIZ_SYLIUS_MEDIA_MANAGER_MAX_FILE_SIZE` environment variable.

UPGRADE-2.0.md

@@ -0,0 +1,43 @@
+# UPGRADE FROM `1.2` TO `2.0`
+
+**Replace form type option name**
+
+If you use one of the following options with Media Manager form type (`AudioType`, `FaviconType`, `ImageType`, `PdfType` or `VideoType`), 
+you had to rename it as follows:
+
+* `["file-type" => "TYPE"]` became `["file_type" => "TYPE"]`
+* `["filter-width" => INT]` became `["filter_width" => INT]]`
+
+**Use new services**
+
+`FileHelper` service has been split into dedicated services:
+* `FileRepositoryInterface` to list file from a folder or get one file by it path
+* `DirectoryOperatorInterface` where all directory operations takes place
+* `FileMetadataResolverInterface` for file types resolution
+* `FilePathResolverInterface` for absolute and relative path resolution
+* `FileValidatorInterface` for file validation
+
+If you use `FileHelper` service, you need to inject one of the service above instead.
+
+**Config**
+
+Change the file `config/packages/monsieurbiz_media_manager_plugin.yaml` with this content : 
+
+```yaml
+imports:
+    - { resource: "@MonsieurBizSyliusMediaManagerPlugin/config/config.yaml" }
+
+liip_imagine:
+    loaders:
+        sylius_image:
+            filesystem:
+                data_root:
+                    - "%sylius_core.public_dir%/media" # Add media folder
+
+twig:
+    form_themes: ['@MonsieurBizSyliusMediaManagerPlugin/form/_theme.html.twig']
+```
+
+**Remove old route file**
+
+Remove the file `config/routes/monsieurbiz_media_manager_plugin.yaml`

composer.json

@@ -5,8 +5,9 @@
     "description": "Add a media manager to your Sylius.",
     "license": "MIT",
     "require": {
-        "php": "^8.1",
-        "sylius/sylius": "~2.0"
+        "php": "^8.2",
+        "sylius/sylius": "~2.0",
+        "phpdocumentor/reflection-docblock": "^5.6"
     },
     "require-dev": {
         "friendsofphp/php-cs-fixer": "^3.16",
@@ -36,7 +37,7 @@
     },
     "extra": {
         "branch-alias": {
-            "dev-master": "3.0-dev"
+            "dev-master": "2.0-dev"
         },
         "symfony": {
             "docker": false,

config/config.yaml

@@ -0,0 +1,7 @@
+imports:
+    - { resource: 'images.yaml' }
+    - { resource: 'twig_hooks.yaml' }
+
+parameters:
+    env(MONSIEURBIZ_SYLIUS_MEDIA_MANAGER_PAGINATION_ITEMS_PER_PAGE): 20
+    monsieurbiz_sylius_media_manager.pagination.items_per_page: '%env(int:MONSIEURBIZ_SYLIUS_MEDIA_MANAGER_PAGINATION_ITEMS_PER_PAGE)%'