Revert "Merge branch 'master' into 2.0"

This reverts commit 1529b88, reversing
changes made to c5da4fd.

Author: Bernhard Schmitt

.editorconfig

@@ -14,7 +14,7 @@ indent_size = 2
 [{package.json,.babelrc,.eslintrc,.stylelintrc}]
 indent_size = 2
 
-[*.{css,js,ts,neon}]
+[*.{css,js,ts}]
 indent_style = tab
 
 [Makefile]
@@ -24,4 +24,4 @@ indent_style = tab
 indent_style = tab
 
 [*.md]
-trim_trailing_whitespace = false
+trim_trailing_whitespace = false

.github/workflows/qa.yml

@@ -1,6 +1,6 @@
 name: CI
 
-on: [push, pull_request]
+on: [push]
 
 jobs:
   coding-standard:
@@ -51,11 +51,16 @@ jobs:
         run: |
           composer install
 
-      - name: Run phpstan
+      - name: Run phpstan on Tests/
         run: |
           bin/phpstan analyse \
+            --autoload-file Build/BuildEssentials/PhpUnit/UnitTestBootstrap.php \
             --level 8 \
-            Tests/Unit Classes
+            Tests/Unit
+
+      - name: Run phpstan on Classes/
+        run: |
+          bin/phpstan analyse --level 8 Classes
 
   unit-tests:
     runs-on: ubuntu-20.04

Classes/Command/ComponentCommandController.php

@@ -8,28 +8,24 @@
 use Neos\Flow\Annotations as Flow;
 use Neos\Flow\Cli\CommandController;
 use PackageFactory\AtomicFusion\PresentationObjects\Domain\Component\ComponentGenerator;
-use PackageFactory\AtomicFusion\PresentationObjects\Domain\Component\ComponentName;
-use PackageFactory\AtomicFusion\PresentationObjects\Domain\Enum\EnumGenerator;
-use PackageFactory\AtomicFusion\PresentationObjects\Domain\PackageKey;
-use PackageFactory\AtomicFusion\PresentationObjects\Domain\PackageResolver;
-use PackageFactory\AtomicFusion\PresentationObjects\Infrastructure\DefensiveConfirmationFileWriter;
+use PackageFactory\AtomicFusion\PresentationObjects\Domain\Value\ValueGenerator;
 
 /**
- * The command controller for kick-starting PresentationObject components
+ * The command controller for kickstarting PresentationObject components
  */
 class ComponentCommandController extends CommandController
 {
     /**
      * @Flow\Inject
-     * @var PackageResolver
+     * @var ComponentGenerator
      */
-    protected $packageResolver;
+    protected $componentGenerator;
 
     /**
-     * @Flow\InjectConfiguration(path="componentGeneration.colocate")
-     * @var bool
+     * @Flow\Inject
+     * @var ValueGenerator
      */
-    protected $colocate;
+    protected $valueGenerator;
 
     /**
      * Create a new PresentationObject component and factory
@@ -45,35 +41,20 @@ class ComponentCommandController extends CommandController
      * The following values are allowed for types:
      *
      * * string, int, float, bool
-     * * slot
      * * Value class names created with <u>component:kickstartvalue</u> in the same
      *   component namespace
      * * Component class names created with <u>component:kickstart</u> in the same
      *   package
      * * ImageSource
      * * Uri
-     * * array<...> with any of the above as an argument
      *
      * @param string $name The name of the new component
-     * @param bool $listable If set, an additional list type will be generated
-     * @param bool $yes If set, no confirmation is going to be required for overwriting files
+     * @param null|string $packageKey Package key of an optional target package, if not set the configured default package or the first available site package will be used
      * @return void
-     * @throws \Neos\Utility\Exception\FilesException
      */
-    public function kickStartCommand(string $name, bool $listable = false, bool $yes = false): void
+    public function kickStartCommand(string $name, ?string $packageKey = null): void
     {
-        $componentGenerator = new ComponentGenerator(
-            new DefensiveConfirmationFileWriter($this->output, $yes)
-        );
-        $package = $this->packageResolver->resolvePackage();
-
-        $componentGenerator->generateComponent(
-            ComponentName::fromInput($name, PackageKey::fromPackage($package)),
-            $this->request->getExceedingArguments(),
-            $package->getPackagePath(),
-            $this->colocate,
-            $listable
-        );
+        $this->componentGenerator->generateComponent($name, $this->request->getExceedingArguments(), $packageKey);
     }
 
     /**
@@ -90,25 +71,12 @@ public function kickStartCommand(string $name, bool $listable = false, bool $yes
      * @param string $componentName The name of the component the new pseudo-enum belongs to
      * @param string $name The name of the new pseudo-enum
      * @param string $type The type of the new pseudo-enum (must be one of: "string", "int")
-     * @param array|string[] $values A comma-separated colon list of names:values for the new pseudo-enum, e.g. a,b,c , a:1,b:2,c:3 or a:1.2,b:2.4,c:3.6
-     * @param bool $yes If set, no confirmation is going to be required for overwriting files
+     * @param array|string[] $values A comma-separated list of values for the new pseudo-enum
+     * @param null|string $packageKey Package key of an optional target package, if not set the configured default package or the first available site package will be used
      * @return void
      */
-    public function kickStartEnumCommand(string $componentName, string $name, string $type, array $values = [], bool $yes = false): void
+    public function kickStartValueCommand(string $componentName, string $name, string $type, array $values = [], ?string $packageKey = null): void
     {
-        $enumGenerator = new EnumGenerator(
-            new \DateTimeImmutable(),
-            new DefensiveConfirmationFileWriter($this->output, $yes)
-        );
-        $package = $this->packageResolver->resolvePackage();
-
-        $enumGenerator->generateEnum(
-            ComponentName::fromInput($componentName, PackageKey::fromPackage($package)),
-            $name,
-            $type,
-            $values,
-            $package->getPackagePath(),
-            $this->colocate
-        );
+        $this->valueGenerator->generateValue($componentName, $name, $type, $values, $packageKey);
     }
 }