Update usage doc (#11)

* add better usage doc

* add to bundles.php tipp

* * fix to command because of incompatibility
* fix for fieldDef handling

Author: mike4git

README.md

@@ -13,6 +13,17 @@ Before you can use the Bundle, you need to add the git repository to your compos
 composer require "mike4git/umlgeneration-bundle"
 ```
 
+and add it to your bundles.php:
+
+```php
+<?php declare(strict_types=1);
+
+return [
+    // mostly dev environment is enough
+    \UMLGenerationBundle\UMLGenerationBundle::class => ['dev' => true],
+];
+```
+
 Additionally, you'll have to install GraphViz (`dot` executable).
 Users of Debian/Ubuntu-based distributions may simply invoke:
 
@@ -38,6 +49,7 @@ Note that this will generate a myDotfileName.dot file
 ```bash
 $ dot -Tsvg myDotfileName.dot -o image.svg
 ```
+
 After that you should see something like this:
 
 ![Sample UML class diagram](doc/images/result.png)

src/Command/UMLGenerationCommand.php

@@ -17,10 +17,10 @@ class UMLGenerationCommand extends AbstractCommand
     private const COMMAND_NAME = 'uml:generate';
 
     public function __construct(
-        string $name = null,
         private ClassDefinitionsRepositoryInterface $classDefinitionsRepository,
         private ClassDefinition2UMLService $classDefinition2UMLService,
         private PrinterService $printerService,
+        string $name = null,
     ) {
         parent::__construct($name);
     }

src/Service/ClassDefinition2UMLService.php

@@ -106,20 +106,22 @@ public function getClasses(): array
     private function addRelation(mixed $fieldDefinition, Relation $relation, ClassDefinition $classDefinition): void
     {
         // TODO Check cases where $fieldDefinition->getClasses() has more than one item
-        /** @var string $class */
-        $class = $fieldDefinition->getClasses()[0]['classes'];
-        $relation->setSourceType($classDefinition->getName() ?? self::UNKNOWN)
-            ->setTargetType($class)
-            ->setSourceRolename($fieldDefinition->getTitle())
-            ->setMinimum($fieldDefinition->getMandatory() ? 1 : 0);
-
-        $relationsKey = sprintf('%s.%s - %s', $relation->getSourceType(), $fieldDefinition->getName(), $relation->getTargetType());
-
-        // if relation already exists it must be bidirectional
-        if (\array_key_exists($relationsKey, $this->relations)) {
-            $relation->setBidirectional(true);
+        if (!empty($fieldDefinition->getClasses())) {
+            /** @var string $class */
+            $class = $fieldDefinition->getClasses()[0]['classes'];
+            $relation->setSourceType($classDefinition->getName() ?? self::UNKNOWN)
+                ->setTargetType($class)
+                ->setSourceRolename($fieldDefinition->getTitle())
+                ->setMinimum($fieldDefinition->getMandatory() ? 1 : 0);
+
+            $relationsKey = sprintf('%s.%s - %s', $relation->getSourceType(), $fieldDefinition->getName(), $relation->getTargetType());
+
+            // if relation already exists it must be bidirectional
+            if (\array_key_exists($relationsKey, $this->relations)) {
+                $relation->setBidirectional(true);
+            }
+            $this->relations[$relationsKey] = $relation;
         }
-        $this->relations[$relationsKey] = $relation;
     }
 
     private function addReverseRelation(ClassDefinition\Data\ReverseObjectRelation $fieldDefinition, Relation $relation, ClassDefinition $classDefinition): void