Merge pull request #14 from pimcore/11.1-from-7113486479

herbertroth · web-flow · commit 7706414eb5a0 · 2023-12-06T13:49:26.000+01:00
Automated PR
diff --git a/bundles/CoreBundle/src/Command/ClassesDefinitionsBuildCommand.php b/bundles/CoreBundle/src/Command/ClassesDefinitionsBuildCommand.php
@@ -16,6 +16,7 @@
 
 namespace Pimcore\Bundle\CoreBundle\Command;
 
+use Pimcore\Cache;
 use Pimcore\Console\AbstractCommand;
 use Pimcore\DataObject\ClassBuilder\PHPClassDumperInterface;
 use Pimcore\DataObject\ClassBuilder\PHPFieldCollectionClassDumperInterface;
@@ -49,15 +50,13 @@ public function __construct(
 
     protected function execute(InputInterface $input, OutputInterface $output): int
     {
-        $cacheStatus = \Pimcore\Cache::isEnabled();
-        \Pimcore\Cache::disable();
+        $cacheStatus = Cache::isEnabled();
+        Cache::disable();
         $objectClassesFolders = array_unique([PIMCORE_CLASS_DEFINITION_DIRECTORY, PIMCORE_CUSTOM_CONFIGURATION_CLASS_DEFINITION_DIRECTORY]);
 
         foreach ($objectClassesFolders as $objectClassesFolder) {
             $files = glob($objectClassesFolder.'/*.php');
 
-            $changes = [];
-
             foreach ($files as $file) {
                 $class = include $file;
 
@@ -84,7 +83,7 @@ protected function execute(InputInterface $input, OutputInterface $output): int
         }
 
         if ($cacheStatus) {
-            \Pimcore\Cache::enable();
+            Cache::enable();
         }
 
         return 0;
diff --git a/bundles/CoreBundle/src/DependencyInjection/Compiler/WorkflowPass.php b/bundles/CoreBundle/src/DependencyInjection/Compiler/WorkflowPass.php
@@ -181,6 +181,7 @@ public function process(ContainerBuilder $container): void
             if (isset($markingStoreDefinition)) {
                 $workflowDefinition->replaceArgument(1, $markingStoreDefinition);
             }
+            $workflowDefinition->setPublic(true);
             $workflowDefinition->replaceArgument(3, $workflowName);
             $workflowDefinition->replaceArgument(4, $workflowConfig['events_to_dispatch'] ?? null);
 
diff --git a/composer.json b/composer.json
@@ -137,9 +137,8 @@
   "require-dev": {
     "codeception/codeception": "^5.0.3",
     "codeception/module-symfony": "^3.1.0",
-    "codeception/phpunit-wrapper": "^9",
     "ergebnis/phpstan-rules": "^2.0",
-    "phpstan/phpstan": "1.10.43",
+    "phpstan/phpstan": "1.10.46",
     "phpstan/phpstan-symfony": "^1.3.5",
     "phpunit/phpunit": "^9.3",
     "gotenberg/gotenberg-php": "^1.1",
diff --git a/doc/01_Getting_Started/00_Installation/00_Docker_Based_Installation.md b/doc/01_Getting_Started/00_Installation/00_Docker_Based_Installation.md
@@ -43,9 +43,9 @@ If you choose to install backend search (which is installed by default), you mus
 :::
 
 5. :heavy_check_mark: DONE - You can now visit your pimcore instance:
-    * The frontend: <http://localhost>
+    * The frontend: [localhost](http://localhost)
     * The admin interface, using the credentials you have chosen above:
-      <http://localhost/admin>
+      [Admin interface](http://localhost/admin)
 
 
 ## Caching
diff --git a/doc/02_MVC/04_Routing_and_URLs/02_Custom_Routes.md b/doc/02_MVC/04_Routing_and_URLs/02_Custom_Routes.md
@@ -113,7 +113,7 @@ to do so just embrace the part with curly brackets { } (see example below).
 
 | Name     | Pattern                | Reverse          | Controller                                    | Variables | Defaults     | Site IDs | Priority     | Methods     |
 |----------|------------------------|------------------|-----------------------------------------------|-----------|--------------|----------|--------------|-------------|
-| news category  | /\\/news-category\\/([^_]+)_([0-9]+)(_category_)?([0-9]+)?/ |  /news-category/%text_%id{_category_%categoryId}    | App\Controller\NewsController::listingAction  | text,id,text2,categoryId   |              |        | 1              |       |
+| news category  | /\\/news-category\\/([^_]+)_([0-9]+)(_category_)?([0-9]+)?/ |  /news-category/%text_%id\{_category_%categoryId\}    | App\Controller\NewsController::listingAction  | text,id,text2,categoryId   |              |        | 1              |       |
   
 ![Grid with the new route](../../img/Routing_grid2.png)
 
diff --git a/doc/03_Documents/01_Editables/02_Areablock/02_Bricks.md b/doc/03_Documents/01_Editables/02_Areablock/02_Bricks.md
@@ -92,7 +92,7 @@ as registered on the areabrick manager (see below).
 | Location | Path                                                                                                                                                                                       |
 |----------|--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
 | global   | `templates/areas/<brickId>/`                                                                                                                                                               |
-| bundle   | `<bundlePath>/Resources/views/areas/<brickId>/` for legacy (Symfony <= 4) bundle structure<br/>or<br/>`<bundlePath>/templates/areas/<brickId>/` for modern (Symfony >= 5) bundle structure |
+| bundle   | `<bundlePath>/Resources/views/areas/<brickId>/` for legacy (Symfony \<= 4) bundle structure<br/>or<br/>`<bundlePath>/templates/areas/<brickId>/` for modern (Symfony >= 5) bundle structure |
 
 Depending on the template location, the following files will be used. You can always completely control locations by 
 implementing the methods for templates and icon yourself (see `AreabrickInterface`):
diff --git a/doc/03_Documents/01_Editables/02_Areablock/README.md b/doc/03_Documents/01_Editables/02_Areablock/README.md
@@ -70,7 +70,7 @@ And you can see the effect, below:
 | `manual`            | bool   | Forces the manual mode, which enables a complete free implementation for areablocks, for example using real `<table>` elements... example see below                                          |
 | `reload`            | bool   | Set to `true`, to force a reload in editmode after reordering items (default: `false`)                                                                                                       |
 | `limit`             | int    | Limit the amount of elements                                                                                                                                                                 |
-| `limits`            | array  | An array of area-ID's with count to limit the amount of certain elements e.g. {"iframe": 1, "teasers": 2}                                                                     |
+| `limits`            | array  | An array of area-ID's with count to limit the amount of certain elements e.g. \{"iframe": 1, "teasers": 2\}                                                                     |
 | `areablock_toolbar` | array  | Array with option that allows you to configure the toolbar. Possible options are `width`, `buttonWidth` and `buttonMaxCharacters`                                                            |
 | `controlsAlign`     | string | The position of the control button bar. Options are: `top`, `right` and `left`.                                                                                                              |
 | `controlsTrigger`   | string | Options are: `hover`(default) and `fixed` .                                                                                                              |
diff --git a/doc/03_Documents/01_Editables/40_WYSIWYG.md b/doc/03_Documents/01_Editables/40_WYSIWYG.md
@@ -54,5 +54,5 @@ framework:
 ```
 If you want to adapt this configuration please have a look at the [symfony documentation](https://symfony.com/doc/current/html_sanitizer.html). Add your custom configuration to you project, e.g. to `config/packages/html_sanitizer.yaml`
 
-> Note: When using API to set WYSIWYG data, please pass encoded characters for html entities e.g. <,>, & etc.
-> The data is encoded by the sanitizer before persisting into db and the same encoded data will be returned by the API.
+> Note: When using API to set WYSIWYG data, please pass encoded characters for html entities e.g. `<`,`>`, `&` etc.
+> The data is encoded by the sanitizer before persisting into db and the same encoded data will be returned by the API.
diff --git a/doc/19_Development_Tools_and_Details/07_Logging.md b/doc/19_Development_Tools_and_Details/07_Logging.md
@@ -44,7 +44,7 @@ pimcore:
 | 2                                                                                | User-ID                                   |
 | Pimcore\Bundle\AdminBundle\Controller\Admin\Document\PageController::saveAction  | Module\Controller::Action                 |
 | pimcore_admin_document_page_save                                                 | Route name                                |
-| {"task":"pub ....                                                                | Request Parameters (shortened & censored) |
+| \{"task":"pub .... \}                                                                | Request Parameters (shortened & censored) |
 
 ## redirect.log
 Sometimes it's necessary to debug redirects, for example when a redirect ends in an infinite loop. 
diff --git a/doc/19_Development_Tools_and_Details/40_Custom_Admin_Login_Entry_Point.md b/doc/19_Development_Tools_and_Details/40_Custom_Admin_Login_Entry_Point.md
@@ -9,7 +9,7 @@ pimcore_admin:
 ``` 
 > Please note:   
 > custom_admin_path_identifier should be at least 20 characters long
-> and must not start with `/admin` if Pimcore version <= 6.0.5 ! 
+> and must not start with `/admin` if Pimcore version \<= 6.0.5 ! 
 
 Add custom entry for `PimcoreCoreBundle:PublicServices:customAdminEntryPoint` in your routing.yaml:  
 ```yaml
diff --git a/doc/20_Extending_Pimcore/17_Custom_Persistent_Models.md b/doc/20_Extending_Pimcore/17_Custom_Persistent_Models.md
@@ -12,7 +12,7 @@ Pimcore provides 2 possible ways of working with custom entities namely Doctrine
 
 ## Option 1: Use Doctrine ORM
 Pimcore comes already with the Doctrine bundle, so you can easily create your own entities.
-Please check <https://symfony.com/doc/current/doctrine.html> for more details.
+Please check [https://symfony.com/doc/current/doctrine.html](https://symfony.com/doc/current/doctrine.html) for more details.
 
 ## Option 2: Working with Pimcore Data Access Objects (Dao)
 
diff --git a/doc/21_Deployment/01_Version_Control_Systems.md b/doc/21_Deployment/01_Version_Control_Systems.md
@@ -4,4 +4,4 @@ Since Pimcore creates lots of temporary files during runtime it's recommended to
 
 If using Git for your project we recommend the following `.gitignore`: 
 
-* <https://github.com/pimcore/demo/blob/11.x/.gitignore>
+* [https://github.com/pimcore/demo/blob/11.x/.gitignore](https://github.com/pimcore/demo/blob/11.x/.gitignore)
diff --git a/doc/21_Deployment/03_Configuration_Environments.md b/doc/21_Deployment/03_Configuration_Environments.md
@@ -13,8 +13,8 @@ or (e.g. in *prod* environments) you can configure everything via real environme
 
 In addition to Symfony configurations, Pimcore also supports environment specific configs for: 
 
-* <https://github.com/pimcore/demo/tree/11.x/config/pimcore> 
-* <https://github.com/pimcore/demo/tree/11.x/var/config>
+* [https://github.com/pimcore/demo/tree/11.x/config/pimcore](https://github.com/pimcore/demo/tree/11.x/config/pimcore) 
+* [https://github.com/pimcore/demo/tree/11.x/var/config](https://github.com/pimcore/demo/tree/11.x/var/config)
 
 
 ## Configuration Storage Locations & Fallbacks
diff --git a/doc/21_Deployment/05_Deployment_Tools.md b/doc/21_Deployment/05_Deployment_Tools.md
@@ -9,9 +9,9 @@ All Pimcore configurations are saved as YAML or PHP files on the file system. As
 [multi environment feature](03_Configuration_Environments.md) different configuration files for different deployment stages
 can be defined.
 
-* <https://github.com/pimcore/demo/tree/11.x/config>
-* <https://github.com/pimcore/demo/tree/11.x/config/pimcore>
-* <https://github.com/pimcore/demo/tree/11.x/var/config>
+* [https://github.com/pimcore/demo/tree/11.x/config](https://github.com/pimcore/demo/tree/11.x/config)
+* [https://github.com/pimcore/demo/tree/11.x/config/pimcore](https://github.com/pimcore/demo/tree/11.x/config/pimcore)
+* [https://github.com/pimcore/demo/tree/11.x/var/config](https://github.com/pimcore/demo/tree/11.x/var/config)
 
 
 ## Pimcore Class Definitions
diff --git a/doc/23_Installation_and_Upgrade/01_System_Requirements.md b/doc/23_Installation_and_Upgrade/01_System_Requirements.md
@@ -16,7 +16,7 @@ For production, we highly recommend a *nix based system.
 - Nginx
 
 
-### PHP >=8.1 <8.3
+### PHP >=8.1 \<8.3
 Both **mod_php** and **FCGI (FPM)** are supported.  
 
 #### Required Settings and Modules & Extensions
diff --git a/doc/23_Installation_and_Upgrade/07_Updating_Pimcore/10_V6_to_V10.md b/doc/23_Installation_and_Upgrade/07_Updating_Pimcore/10_V6_to_V10.md
@@ -1,11 +1,11 @@
 # Upgrading Pimcore from Version 6.x to Version 10
 
 ## System Requirement changes
- - PHP >=8.0 <8.2
+ - PHP >=8.0 \<8.2
  - Apache >= 2.4
  - Composer >= 2.0
 
-> As Pimcore 6.x works only with PHP < 8.0 and Pimcore 10 works only with PHP >=8.0 <8.2, a switch of PHP version is needed during the upgrade process.
+> As Pimcore 6.x works only with PHP < 8.0 and Pimcore 10 works only with PHP >=8.0 \<8.2, a switch of PHP version is needed during the upgrade process.
 
 ## Database Requirement changes
 - MariaDB >= 10.3
@@ -163,10 +163,10 @@ mv var/config/custom-logo.* var/admin/custom-logo.image
 
 ## SWITCH COMPOSER AND PHP VERSION, AND DO THE UPDATE!
 - Ensure Composer is updated to version >= 2.0
-- Switch PHP version to PHP >=8.0 <8.2
+- Switch PHP version to PHP >=8.0 \<8.2
 - Run composer update (`COMPOSER_MEMORY_LIMIT=-1 composer update`)
 
-## CHANGES TO DO AFTER THE UPDATE! (TO DO WITH PHP >=8.0 <8.2)
+## CHANGES TO DO AFTER THE UPDATE! (TO DO WITH PHP >=8.0 \<8.2)
 
 ### Verify migrations execution
 By default, Composer update runs Pimcore migrations automatically, however it is better to check if all migrations are executed for smooth transition.
diff --git a/doc/23_Installation_and_Upgrade/09_Upgrade_Notes/README.md b/doc/23_Installation_and_Upgrade/09_Upgrade_Notes/README.md
@@ -174,7 +174,7 @@ The tokens for password reset are now stored in the DB and are one time use only
 
 -  Enabled Content Security Policy by default.
 -  Implemented Symfony HTML sanitizer for WYSIWYG editors. Please make sure to sanitize your persisted data with help of this [script](https://gist.github.com/dvesh3/0e585a16dfbf546bc17a9eef1c5640b3).
-Also, when using API to set WYSIWYG data, please pass encoded characters for html entities <,>, & etc.
+Also, when using API to set WYSIWYG data, please pass encoded characters for html entities `<`,`>`, `&` etc.
 The data is encoded by the sanitizer before persisting into db and the same encoded data will be returned by the API.
 
 
diff --git a/lib/Event/Workflow/GlobalActionEvent.php b/lib/Event/Workflow/GlobalActionEvent.php
@@ -18,14 +18,14 @@
 
 use Pimcore\Event\Traits\ArgumentsAwareTrait;
 use Pimcore\Workflow\GlobalAction;
-use Symfony\Component\Workflow\Workflow;
+use Symfony\Component\Workflow\WorkflowInterface;
 use Symfony\Contracts\EventDispatcher\Event;
 
 class GlobalActionEvent extends Event
 {
     use ArgumentsAwareTrait;
 
-    protected Workflow $workflow;
+    protected WorkflowInterface $workflow;
 
     protected mixed $subject = null;
 
@@ -35,15 +35,19 @@ class GlobalActionEvent extends Event
      * DocumentEvent constructor.
      *
      */
-    public function __construct(Workflow $workflow, mixed $subject, GlobalAction $globalAction, array $arguments = [])
+    public function __construct(
+        WorkflowInterface $workflow,
+        mixed $subject,
+        GlobalAction $globalAction,
+        array $arguments = [])
     {
         $this->workflow = $workflow;
         $this->subject = $subject;
         $this->globalAction = $globalAction;
         $this->arguments = $arguments;
     }
 
-    public function getWorkflow(): Workflow
+    public function getWorkflow(): WorkflowInterface
     {
         return $this->workflow;
     }
diff --git a/lib/Model/DataObject/ClassDefinition/ClassDefinitionManager.php b/lib/Model/DataObject/ClassDefinition/ClassDefinitionManager.php
@@ -29,6 +29,8 @@ class ClassDefinitionManager
 
     /**
      * Delete all classes from db
+     *
+     * @return list<array{string, string, string}>
      */
     public function cleanUpDeletedClassDefinitions(): array
     {
@@ -58,16 +60,16 @@ public function cleanUpDeletedClassDefinitions(): array
 
     /**
      * Updates all classes from PIMCORE_CLASS_DEFINITION_DIRECTORY
+     *
+     * @return list<array{string, string, string}>
      */
     public function createOrUpdateClassDefinitions(): array
     {
         $objectClassesFolders = array_unique([PIMCORE_CLASS_DEFINITION_DIRECTORY, PIMCORE_CUSTOM_CONFIGURATION_CLASS_DEFINITION_DIRECTORY]);
+        $changes = [];
 
         foreach ($objectClassesFolders as $objectClassesFolder) {
             $files = glob($objectClassesFolder.'/*.php');
-
-            $changes = [];
-
             foreach ($files as $file) {
                 $class = include $file;
 
diff --git a/lib/Templating/Renderer/IncludeRenderer.php b/lib/Templating/Renderer/IncludeRenderer.php
@@ -91,7 +91,7 @@ public function render(mixed $include, array $params = [], bool $editmode = fals
             });
 
             // TODO is this enough for cache or should we disable caching completely?
-            if (method_exists($include, 'getUseTargetGroup') && $include->getUseTargetGroup()) {
+            if (is_object($include) && method_exists($include, 'getUseTargetGroup') && $include->getUseTargetGroup()) {
                 $cacheParams['target_group'] = $include->getUseTargetGroup();
             }
 
diff --git a/lib/Tool/Authentication.php b/lib/Tool/Authentication.php
@@ -160,9 +160,7 @@ public static function verifyPassword(User $user, string $password): bool
             return false;
         }
 
-        $password = self::preparePlainTextPassword($user->getName(), $password);
-
-        if (!password_verify($password, $user->getPassword())) {
+        if (!password_verify(self::preparePlainTextPassword($user->getName(), $password), $user->getPassword())) {
             return false;
         }
 
diff --git a/lib/Workflow/GlobalAction.php b/lib/Workflow/GlobalAction.php
@@ -19,7 +19,7 @@
 use Pimcore\Workflow\Notes\CustomHtmlServiceInterface;
 use Pimcore\Workflow\Notes\NotesAwareInterface;
 use Pimcore\Workflow\Notes\NotesAwareTrait;
-use Symfony\Component\Workflow\Workflow;
+use Symfony\Component\Workflow\WorkflowInterface;
 
 class GlobalAction implements NotesAwareInterface
 {
@@ -80,7 +80,7 @@ public function getGuard(): ?string
         return $this->options['guard'] ?? null;
     }
 
-    public function isGuardValid(Workflow $workflow, object $subject): bool
+    public function isGuardValid(WorkflowInterface $workflow, object $subject): bool
     {
         if (empty($this->getGuard())) {
             return true;
diff --git a/lib/Workflow/Manager.php b/lib/Workflow/Manager.php
@@ -32,7 +32,7 @@
 use Symfony\Component\Workflow\Exception\LogicException;
 use Symfony\Component\Workflow\Marking;
 use Symfony\Component\Workflow\Registry;
-use Symfony\Component\Workflow\Workflow;
+use Symfony\Component\Workflow\WorkflowInterface;
 use Symfony\Contracts\EventDispatcher\EventDispatcherInterface;
 
 class Manager
@@ -116,7 +116,7 @@ public function getPlaceConfig(string $workflowName, string $place): ?PlaceConfi
      *
      * @return PlaceConfig[];
      */
-    public function getOrderedPlaceConfigs(Workflow $workflow, Marking $marking = null): array
+    public function getOrderedPlaceConfigs(WorkflowInterface $workflow, Marking $marking = null): array
     {
         if (is_null($marking)) {
             return $this->placeConfigs[$workflow->getName()] ?? [];
@@ -170,7 +170,7 @@ public function getWorkflowConfig(string $workflowName): WorkflowConfig
 
     /**
      *
-     * @return Workflow[]
+     * @return WorkflowInterface[]
      */
     public function getAllWorkflowsForSubject(object $subject): array
     {
@@ -189,7 +189,7 @@ public function getAllWorkflowsForSubject(object $subject): array
         return $workflows;
     }
 
-    public function getWorkflowIfExists(object $subject, string $workflowName): ?Workflow
+    public function getWorkflowIfExists(object $subject, string $workflowName): ?WorkflowInterface
     {
         try {
             $workflow = $this->workflowRegistry->get($subject, $workflowName);
@@ -214,8 +214,13 @@ public function getWorkflowByName(string $workflowName): ?object
      * @throws ValidationException
      * @throws \Exception
      */
-    public function applyWithAdditionalData(Workflow $workflow, Asset|PageSnippet|Concrete $subject, string $transition, array $additionalData, bool $saveSubject = false): Marking
-    {
+    public function applyWithAdditionalData(
+        WorkflowInterface $workflow,
+        Asset|PageSnippet|Concrete $subject,
+        string $transition,
+        array $additionalData,
+        bool $saveSubject = false
+    ): Marking {
         $this->notesSubscriber->setAdditionalData($additionalData);
 
         $marking = $workflow->apply($subject, $transition, $additionalData);
@@ -241,8 +246,13 @@ public function applyWithAdditionalData(Workflow $workflow, Asset|PageSnippet|Co
      *
      * @throws \Exception
      */
-    public function applyGlobalAction(Workflow $workflow, object $subject, string $globalAction, array $additionalData, bool $saveSubject = false): Marking
-    {
+    public function applyGlobalAction(
+        WorkflowInterface $workflow,
+        object $subject,
+        string $globalAction,
+        array $additionalData,
+        bool $saveSubject = false
+    ): Marking {
         $globalActionObj = $this->getGlobalAction($workflow->getName(), $globalAction);
         if (!$globalActionObj) {
             throw new LogicException(sprintf('global action %s not found', $globalAction));
@@ -345,7 +355,7 @@ public function ensureInitialPlace(string $workflowName, object $subject): bool
         return false;
     }
 
-    public function getInitialPlacesForWorkflow(Workflow $workflow): array
+    public function getInitialPlacesForWorkflow(WorkflowInterface $workflow): array
     {
         $definition = $workflow->getDefinition();
 
diff --git a/lib/Workflow/Place/PlaceConfig.php b/lib/Workflow/Place/PlaceConfig.php
diff --git a/models/DataObject/ClassDefinition/Data/Password.php b/models/DataObject/ClassDefinition/Data/Password.php
diff --git a/models/Document/Editable/Link.php b/models/Document/Editable/Link.php
diff --git a/models/Element/Service.php b/models/Element/Service.php

Original file line number	Diff line number	Diff line change
`@@ -181,6 +181,7 @@ public function process(ContainerBuilder $container): void`
`181`	`181`	`if (isset($markingStoreDefinition)) {`
`182`	`182`	`$workflowDefinition->replaceArgument(1, $markingStoreDefinition);`
`183`	`183`	`}`
	`184`	`+ $workflowDefinition->setPublic(true);`
`184`	`185`	`$workflowDefinition->replaceArgument(3, $workflowName);`
`185`	`186`	`$workflowDefinition->replaceArgument(4, $workflowConfig['events_to_dispatch'] ?? null);`
`186`	`187`