Adjusted version number

.github/workflows/test-runner.yml

@@ -1,39 +0,0 @@
-name: Tests
-
-on:
-    push:
-        branches: [ master ]
-    pull_request:
-
-jobs:
-    build:
-
-        runs-on: ubuntu-latest
-
-        steps:
-            - uses: actions/checkout@v2
-
-            - name: Validate composer.json and composer.lock
-              run: composer validate
-
-            - name: Cache Composer packages
-              id: composer-cache
-              uses: actions/cache@v2
-              with:
-                  path: vendor
-                  key: ${{ runner.os }}-php-${{ hashFiles('**/composer.lock') }}
-                  restore-keys: |
-                      ${{ runner.os }}-php-
-
-            - name: Install dependencies
-              if: steps.composer-cache.outputs.cache-hit != 'true'
-              run: composer install --prefer-dist --no-progress --no-suggest
-
-            - name: Run test suite
-              run: vendor/bin/phpunit --stop-on-failure
-
-        # Add a test script to composer.json, for instance: "test": "vendor/bin/phpunit"
-        # Docs: https://getcomposer.org/doc/articles/scripts.md
-
-        # - name: Run test suite
-        #   run: composer run-script test

.github/workflows/update-package-index.yml

@@ -1,39 +0,0 @@
-name: Package push
-
-on:
-    push:
-        branches: [ master ]
-    release:
-        types: [ created ]
-
-jobs:
-    build:
-
-        runs-on: ubuntu-latest
-
-        steps:
-            -   name: Setup PHP with PECL extension
-                uses: shivammathur/setup-php@v2
-                with:
-                    php-version: '7.4'
-
-            - name: Set git credentials
-              run: git config --global user.name "Vassyli" && git config --global user.email "basilius.sauter+automated@gmail.com" && git config --global user.password "${{ secrets.GH_TOKEN }}"
-
-            - name: Set composer token
-              run: composer config -g github-oauth.github.com ${{ secrets.GH_TOKEN }}
-
-            - name: "Checkouts satis and packages"
-              run: git clone https://github.com/composer/satis && git clone https://github.com/lotgd/packages
-
-            - name: "Install satis"
-              run: cd satis && composer install && cd ..
-
-            - name: "Build satis"
-              run: satis/bin/satis build packages/satis.json packages/build
-
-            - name: Set git url with credentials for pushing
-              run: cd packages && git config remote.origin.url 'https://Vassyli:${{ secrets.GH_TOKEN }}@github.com/lotgd/packages.git' && cd ..
-
-            - name: "Commit changes and push to origin"
-              run: cd packages && git add build/* && git commit -a -m "Updated packages" && git push origin master && cd ..

.gitignore

@@ -228,8 +228,3 @@ pip-log.txt
 #Mr Developer
 .mr.developer.cfg
 dbconfig.php
-composer.phar
-clover.xml
-.phpunit.result.cache
-
-.php_cs.cache

.php_cs.dist

@@ -1,142 +0,0 @@
-<?php
-
-return PhpCsFixer\Config::create()
-    ->setRiskyAllowed(true)
-    ->setRules([
-        'align_multiline_comment' => true,
-        'array_indentation' => true,
-        'array_syntax' => ['syntax' => 'short'],
-        'binary_operator_spaces' => true,
-        'blank_line_after_namespace' => true,
-        'braces' => [
-            'position_after_control_structures' => 'same',
-            'position_after_functions_and_oop_constructs' => 'next',
-        ],
-        'cast_spaces' => ['space' => 'none'],
-        'class_definition' => true,
-        'class_keyword_remove' => false,
-        'combine_consecutive_issets' => true,
-        'combine_consecutive_unsets' => true,
-        'compact_nullable_typehint' => true,
-        'declare_strict_types' => true,
-        'declare_equal_normalize' => true,
-        'dir_constant' => true,
-        'doctrine_annotation_array_assignment' => true,
-        'doctrine_annotation_braces' => true,
-        'doctrine_annotation_indentation' => true,
-        'doctrine_annotation_spaces' => [
-            'after_array_assignments_colon' => false,
-            'after_array_assignments_equals' => false,
-            'before_argument_assignments' => false,
-            'before_array_assignments_colon' => false,
-            'before_array_assignments_equals' => false,
-        ],
-        'elseif' => true,
-        'encoding' => true,
-        'ereg_to_preg' => true,
-        'escape_implicit_backslashes' => true,
-        'explicit_indirect_variable' => true,
-        'explicit_string_variable' => true,
-        'full_opening_tag' => true,
-        'fully_qualified_strict_types' => true,
-        'function_declaration' => true,
-        'function_typehint_space' => true,
-        'general_phpdoc_annotation_remove' => true,
-        'indentation_type' => true,
-        'line_ending' => true,
-        'linebreak_after_opening_tag' => true,
-        'list_syntax' => ['syntax' => 'short'],
-        'lowercase_cast' => true,
-        'lowercase_constants' => true,
-        'lowercase_static_reference' => true,
-        'magic_constant_casing' => true,
-        'magic_method_casing' => true,
-        'method_argument_space' => true,
-        'method_chaining_indentation' => true,
-        'multiline_comment_opening_closing' => true,
-        'multiline_whitespace_before_semicolons' => ['strategy' => 'new_line_for_chained_calls'],
-        'native_constant_invocation' => true,
-        'native_function_invocation' => true,
-        'native_function_type_declaration_casing' => true,
-        'new_with_braces' => true,
-        'no_alias_functions' => true,
-        'no_alternative_syntax' => true,
-        'no_blank_lines_after_phpdoc' => true,
-        'no_break_comment' => true,
-        'no_closing_tag' => true,
-        'no_empty_comment' => true,
-        'no_empty_phpdoc' => true,
-        'no_empty_statement' => true,
-        'no_extra_blank_lines' => true,
-        'no_homoglyph_names' => true,
-        'no_leading_import_slash' => true,
-        'no_leading_namespace_whitespace' => true,
-        'no_mixed_echo_print' => ["use" => "print"],
-        'no_multiline_whitespace_around_double_arrow' => true,
-        'no_php4_constructor' => true,
-        'no_short_bool_cast' => true,
-        'no_singleline_whitespace_before_semicolons' => true,
-        'no_spaces_after_function_name' => true,
-        'no_spaces_around_offset' => true,
-        'no_spaces_inside_parenthesis' => true,
-        'no_trailing_comma_in_list_call' => true,
-        'no_trailing_comma_in_singleline_array' => true,
-        'no_trailing_whitespace' => true,
-        'no_trailing_whitespace_in_comment' => true,
-        'no_unneeded_control_parentheses' => true,
-        'no_unneeded_curly_braces' => true,
-        'no_unneeded_final_method' => true,
-        'no_unreachable_default_argument_value' => true,
-        'no_unused_imports' => true,
-        'no_useless_else' => true,
-        'no_useless_return' => true,
-        'no_whitespace_before_comma_in_array' => true,
-        'non_printable_character' => true,
-        'normalize_index_brace' => true,
-        'object_operator_without_whitespace' => true,
-        'ordered_imports' => true,
-        'phpdoc_add_missing_param_annotation' => true,
-        'phpdoc_indent' => true,
-        'phpdoc_no_access' => true,
-        'phpdoc_no_alias_tag' => true,
-        'phpdoc_no_empty_return' => true,
-        'phpdoc_no_package' => true,
-        'phpdoc_no_useless_inheritdoc' => true,
-        'phpdoc_order' => true,
-        'phpdoc_scalar' => true,
-        'phpdoc_single_line_var_spacing' => true,
-        'phpdoc_summary' => true,
-        'phpdoc_to_return_type' => true,
-        'phpdoc_trim_consecutive_blank_line_separation' => true,
-        'phpdoc_types' => true,
-        'phpdoc_types_order' => ['null_adjustment' => 'always_last'],
-        'phpdoc_var_annotation_correct_order' => true,
-        'pow_to_exponentiation' => true,
-        'psr4' => true,
-        'random_api_migration' => true,
-        'return_type_declaration' => true,
-        'self_accessor' => true,
-        'semicolon_after_instruction' => true,
-        'set_type_to_cast' => true,
-        'short_scalar_cast' => true,
-        'single_blank_line_at_eof' => true,
-        'single_blank_line_before_namespace' => true,
-        'single_class_element_per_statement' => true,
-        'single_import_per_statement' => true,
-        'single_line_after_imports' => true,
-        'standardize_not_equals' => true,
-        'standardize_increment' => true,
-        'switch_case_semicolon_to_colon' => true,
-        'switch_case_space' => true,
-        'ternary_to_null_coalescing' => true,
-        'trailing_comma_in_multiline_array' => true,
-        'trim_array_spaces' => false,
-        'unary_operator_spaces' => true,
-        'visibility_required' => true,
-        'whitespace_after_comma_in_array' => true,
-        'yoda_style' => false,
-    ])
-    ->setFinder(
-        PhpCsFixer\Finder::create()
-            ->in(__DIR__."/src")
-    );

.travis.yml

@@ -0,0 +1,9 @@
+sudo: false
+language: php
+php:
+  - '7.1'
+  - '7.2'
+install:
+  - composer install
+script:
+  - ./t

README.md

@@ -1,6 +1,6 @@
 # Legend of the Green Dragon (Core)
 
-![Tests](https://github.com/lotgd/core/workflows/Tests/badge.svg)
+[![Build Status](https://travis-ci.org/lotgd/core.svg?branch=master)](https://travis-ci.org/lotgd/core)
 
 Legend of the Green Dragon is a text-based RPG originally developed by Eric Stevens and JT Traub as a remake of and homage to the classic BBS Door game,
 Legend of the Red Dragon, by Seth Able Robinson. You can play it at numerous sites, including http://www.lotgd.net/.

composer.json

@@ -1,7 +1,8 @@
 {
     "name": "lotgd/core",
     "description": "Core functionality for Legend of the Green Dragon, a text-based RPG game.",
-    "license": "AGPL-3.0-or-later",
+    "license": "AGPL-3.0",
+    "version": "0.4.1",
     "autoload": {
         "psr-4": {
             "LotGD\\Core\\": "src/",
@@ -12,18 +13,13 @@
         "bin/daenerys"
     ],
     "require": {
-        "php": "^8.0",
-        "ext-pdo": "*",
-        "composer/composer": "^1.10|^2.0",
-        "gedmo/doctrine-extensions": "^2.3|^3.0",
-        "doctrine/orm": "^2.8",
-        "doctrine/common": "^3.0",
-        "monolog/monolog": "^2.0",
-        "symfony/console": "^5.0",
-        "symfony/yaml": "^5.0",
-        "twig/twig": "^3.0",
-        "ramsey/uuid-doctrine": "^1.5",
-        "jetbrains/phpstorm-attributes": "^1.0",
+        "php": "^7.1.0",
+        "composer/composer": "*",
+        "gedmo/doctrine-extensions": "*",
+        "doctrine/orm": "2.5.*",
+        "monolog/monolog": "^1.12",
+        "symfony/console": "^3.0",
+        "symfony/yaml": "^3.0",
         "d11wtq/boris": "^1.0"
     },
     "repositories": [
@@ -34,8 +30,8 @@
 
     ],
     "require-dev": {
-        "phpunit/phpunit": "*",
-        "phpunit/php-code-coverage": "*",
-        "friendsofphp/php-cs-fixer": "*"
+        "phpunit/phpunit": "^5.0",
+        "phpunit/dbunit": "^2.0",
+        "block8/php-docblock-checker": "2.0.0"
     }
 }

config/lotgd.yml

@@ -3,7 +3,6 @@ database:
     name: daenerys
     user: root
     password:
-    disableAutoSchemaUpdate: false
 game:
     epoch: 2016-07-01 00:00:00.0 -8
     offsetSeconds: 0

phpunit.xml

@@ -1,7 +1,6 @@
 <phpunit bootstrap="bootstrap/bootstrap.php">
     <php>
         <env name="LOTGD_TESTS_CONFIG_PATH" value="./config/lotgd.yml"/>
-        <env name="COLUMNS" value="120" />
     </php>
     <testsuites>
         <testsuite name="All">

src/Action.php

@@ -9,20 +9,23 @@ namespace LotGD\Core;
  */
 class Action
 {
-    protected string $id;
+    protected $id;
+    protected $destinationSceneId;
+    protected $title = null;
+    protected $parameters = [];
 
     /**
      * Construct a new action with the specified Scene as its destination.
-     * @param string $destinationSceneId
+     * @param int $destinationSceneId
      * @param string|null $title
      * @param array $parameters
      */
-    public function __construct(
-        protected string $destinationSceneId,
-        protected ?string $title = null,
-        protected array $parameters = []
-    ) {
-        $this->id = \bin2hex(\random_bytes(8));
+    public function __construct(int $destinationSceneId, ?string $title = null, array $parameters = [])
+    {
+        $this->id = bin2hex(random_bytes(8));
+        $this->destinationSceneId = $destinationSceneId;
+        $this->title = $title;
+        $this->parameters = $parameters;
     }
 
     /**
@@ -38,15 +41,15 @@ class Action
     /**
      * Return the database ID of the destination scene, where the user will
      * go if they take this action.
-     * @return string
+     * @return int
      */
-    public function getDestinationSceneId(): string
+    public function getDestinationSceneId(): int
     {
         return $this->destinationSceneId;
     }
 
     /**
-     * @return string|null
+     * @return null|string
      */
     public function getTitle(): ?string
     {
@@ -54,15 +57,15 @@ class Action
     }
 
     /**
-     * @param string|null $title
+     * @return null|string
      */
-    public function setTitle(?string $title): void
+    public function setTitle(?string $title)
     {
         $this->title = $title;
     }
 
     /**
-     * Returns all parameters for this action.
+     * Returns all parameters for this action
      * @return array
      */
     public function getParameters(): array
@@ -71,7 +74,7 @@ class Action
     }
 
     /**
-     * Sets all parameters for this action.
+     * Sets all parameters for this action
      * @param array $parameters
      */
     public function setParameters(array $parameters): void

src/ActionGroup.php

@@ -1,5 +1,5 @@
 <?php
-declare(strict_types=1);
+declare (strict_types=1);
 
 namespace LotGD\Core;
 
@@ -11,6 +11,9 @@ class ActionGroup implements \Countable
     const DefaultGroup = 'lotgd/core/default';
     const HiddenGroup = 'lotgd/core/hidden';
 
+    private $id;
+    private $title;
+    private $sortKey;
     private $actions;
 
     /**
@@ -19,11 +22,11 @@ class ActionGroup implements \Countable
      * @param string $title Title to display to the end user. Empty string means no title.
      * @param int $sortKey Navigation menus are displayed in the order sorted by this integer.
      */
-    public function __construct(
-        private string $id,
-        private string $title,
-        private int $sortKey
-    ) {
+    public function __construct(string $id, string $title, int $sortKey)
+    {
+        $this->id = $id;
+        $this->title = $title;
+        $this->sortKey = $sortKey;
         $this->actions = [];
     }
 
@@ -33,7 +36,7 @@ class ActionGroup implements \Countable
      */
     public function count(): int
     {
-        return \count($this->actions);
+        return count($this->actions);
     }
 
     /**
@@ -67,7 +70,7 @@ class ActionGroup implements \Countable
 
     /**
      * Returns the ordered array of actions.
-     * @return Action[]
+     * @return array<Action>
      */
     public function getActions(): array
     {
@@ -76,7 +79,7 @@ class ActionGroup implements \Countable
 
     /**
      * Sets the ordered array of actions.
-     * @param Action[] $actions
+     * @param array<Action> $actions
      */
     public function setActions(array $actions)
     {

src/Attachment.php

@@ -3,31 +3,24 @@ declare(strict_types=1);
 
 namespace LotGD\Core;
 
-use Exception;
-use LotGD\Core\Models\Scene;
-
 /**
  * An attachment to a scene. This is desigend to be subclasses by modules to
  * provide functinoality like forms or maybe image attachments to go along with a scene.
  */
-abstract class Attachment implements AttachmentInterface
+abstract class Attachment
 {
-    protected string $id;
+    protected $id;
+    protected $type;
 
     /**
      * Construct a new attachment of the given type. Randomly assigns it an ID.
-     * @param Game $game
-     * @param Scene $scene
-     * @throws Exception
+     * @param string $type Type of this attachment, in the vendor/module/type format.
+     * @return Attachment
      */
-    public function __construct(Game $game, Scene $scene)
+    public function __construct(string $type)
     {
-        $this->id = \bin2hex(\random_bytes(8));
-    }
-
-    public function __toString(): string
-    {
-        return "<Attachment#{$this->id} '{". static::class . "}'>";
+        $this->id = bin2hex(random_bytes(8));
+        $this->type = $type;
     }
 
     /**
@@ -39,4 +32,13 @@ abstract class Attachment implements AttachmentInterface
     {
         return $this->id;
     }
+
+    /**
+     * Returns the type of this attachment, in vendor/module/type format.
+     * @return string
+     */
+    public function getType(): string
+    {
+        return $this->type;
+    }
 }

src/AttachmentInterface.php

@@ -1,24 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core;
-
-use LotGD\Core\Models\Scene;
-
-interface AttachmentInterface
-{
-    /**
-     * AttachmentInterface constructor.
-     * @param Game $g Should not be saved internally.
-     * @param Scene $scene Should not be saved internally.
-     */
-    public function __construct(Game $g, Scene $scene);
-    public function __toString(): string;
-    public function getId(): string;
-
-    /**
-     * Returns an array with attachment-specific fields.
-     * @return array
-     */
-    public function getData(): array;
-}

src/Battle.php

@@ -3,17 +3,21 @@ declare(strict_types=1);
 
 namespace LotGD\Core;
 
-use Doctrine\Common\Collections\ArrayCollection;
-use Doctrine\Common\Collections\Collection;
+use Doctrine\Common\Collections\{
+    ArrayCollection,
+    Collection
+};
 
-use LotGD\Core\Exceptions\ArgumentException;
-use LotGD\Core\Exceptions\BattleIsOverException;
-use LotGD\Core\Exceptions\BattleNotOverException;
-use LotGD\Core\Models\BattleEvents\CriticalHitEvent;
-use LotGD\Core\Models\BattleEvents\DamageEvent;
-use LotGD\Core\Models\BattleEvents\DeathEvent;
-use LotGD\Core\Models\Buff;
-use LotGD\Core\Models\FighterInterface;
+use LotGD\Core\{
+    DiceBag,
+    Exceptions\ArgumentException,
+    Exceptions\BattleIsOverException,
+    Exceptions\BattleNotOverException,
+    Models\FighterInterface
+};
+use LotGD\Core\Models\{
+    Buff, BattleEvents\BuffMessageEvent, BattleEvents\CriticalHitEvent, BattleEvents\DamageEvent, BattleEvents\DeathEvent, Scene
+};
 
 /**
  * Class for managing and running battles between 2 participants.
@@ -30,10 +34,18 @@ class Battle
     const RESULT_PLAYERDEATH = 1;
     const RESULT_MONSTERDEATH = 2;
     
-    protected ArrayCollection $events;
-    protected int $result = 0;
-    protected int $round = 0;
-    protected array $configuration = [
+    protected $player;
+    protected $monster;
+    protected $game;
+    protected $events;
+    protected $result = 0;
+    protected $round = 0;
+    
+    /**
+     * Battle Configuration
+     * @var array
+     */
+    protected $configuration = [
         "riposteEnabled" => true,
         "levelAdjustementEnabled" => true,
         "criticalHitEnabled" => true,
@@ -41,15 +53,15 @@ class Battle
     
     /**
      * Takes a game object and two participants (Player and Monster) to fight a battle.
-     * @param Game $game
+     * @param \LotGD\Core\Game $game
      * @param FighterInterface $player
-     * @param FighterInterface|null $monster
+     * @param FighterInterface $monster
      */
-    public function __construct(
-        protected Game $game,
-        protected FighterInterface $player,
-        protected ?FighterInterface $monster
-    ) {
+    public function __construct(Game $game, FighterInterface $player, ?FighterInterface $monster)
+    {
+        $this->game = $game;
+        $this->player = $player;
+        $this->monster = $monster;
         $this->events = new ArrayCollection();
     }
 
@@ -59,7 +71,7 @@ class Battle
      */
     public function serialize(): string
     {
-        return \serialize([
+        return serialize([
             "monster" => $this->monster,
             "result" => $this->result,
             "round" => $this->round,
@@ -71,14 +83,14 @@ class Battle
      * @param Game $game
      * @param FighterInterface $player
      * @param string $serialized
-     * @return self
+     * @return Battle
      */
     public static function unserialize(Game $game, FighterInterface $player, string $serialized): self
     {
         $battle = new self($game, $player, null);
-        $unserialized = \unserialize($serialized);
+        $unserialized = unserialize($serialized);
 
-        $battle->monster = $unserialized["monster"];
+        $battle->monster  = $unserialized["monster"];
         $battle->result = $unserialized["result"];
         $battle->round = $unserialized["round"];
         $battle->configuration = $unserialized["configuration"];
@@ -87,7 +99,7 @@ class Battle
     }
     
     /**
-     * Returns a list of all battle events.
+     * Returns a list of all battle events
      * @return Collection
      */
     public function getEvents(): Collection
@@ -96,23 +108,23 @@ class Battle
     }
     
     /**
-     * Disables ripostes.
+     * Disables ripostes
      */
-    public function disableRiposte(): void
+    public function disableRiposte()
     {
         $this->configuration["riposteEnabled"] = false;
     }
     
     /**
-     * Enables ripostes.
+     * Enables ripostes
      */
-    public function enableRiposte(): void
+    public function enableRiposte()
     {
         $this->configuration["riposteEnabled"] = true;
     }
     
     /**
-     * Returns true if ripostes are enabled.
+     * Returns true if ripostes are enabled
      * @return bool
      */
     public function isRiposteEnabled(): bool
@@ -121,23 +133,23 @@ class Battle
     }
     
     /**
-     * Enables level adjustement.
+     * Enables level adjustement
      */
-    public function enableLevelAdjustement(): void
+    public function enableLevelAdjustement()
     {
         $this->configuration["levelAdjustementEnabled"] = true;
     }
     
     /**
-     * Disables level adjustement.
+     * Disables level adjustement
      */
-    public function disableLevelAdjustement(): void
+    public function disableLevelAdjustement()
     {
         $this->configuration["levelAdjustementEnabled"] = false;
     }
     
     /**
-     * Returns true if level adjustements are enabled.
+     * Returns true if level adjustements are enabled
      * @return bool
      */
     public function isLevelAdjustementEnabled(): bool
@@ -146,7 +158,7 @@ class Battle
     }
     
     /**
-     * Returns true if critical hit events are enabled.
+     * Returns true if critical hit events are enabled
      * @return bool
      */
     public function isCriticalHitEnabled(): bool
@@ -155,17 +167,17 @@ class Battle
     }
     
     /**
-     * Disable critical hits.
+     * Disable critical hits
      */
-    public function disableCriticalHit(): void
+    public function disableCriticalHit()
     {
         $this->configuration["criticalHitEnabled"] = false;
     }
     
     /**
-     * enables critical hits.
+     * enables critical hits
      */
-    public function enableCriticalHit(): void
+    public function enableCriticalHit()
     {
         $this->configuration["criticalHitEnabled"] = true;
     }
@@ -180,7 +192,7 @@ class Battle
     }
     
     /**
-     * Returns the player instance.
+     * Returns the player instance
      * @return FighterInterface
      */
     public function getPlayer(): FighterInterface
@@ -189,7 +201,7 @@ class Battle
     }
     
     /**
-     * Returns the montser instance.
+     * Returns the montser instance
      * @return FighterInterface
      */
     public function getMonster(): FighterInterface
@@ -198,9 +210,9 @@ class Battle
     }
     
     /**
-     * Returns the winner of this fight.
-     * @throws BattleNotOverException if battle is not over.
+     * Returns the winner of this fight
      * @return FighterInterface
+     * @throws BattleNotOverException if battle is not over.
      */
     public function getWinner(): FighterInterface
     {
@@ -212,9 +224,9 @@ class Battle
     }
     
     /**
-     * Returns the loser of this fight.
-     * @throws BattleNotOverException if battle is not over.
+     * Returns the loser of this fight
      * @return FighterInterface
+     * @throws BattleNotOverException if battle is not over.
      */
     public function getLoser(): FighterInterface
     {
@@ -229,10 +241,10 @@ class Battle
      * Fights the number of rounds given by the parameter $n and returns the number
      * of actual rounds fought.
      * @param int $n
-     * @param int $firstDamageRound Which damage rounds are calculated. Cannot be 0.
+     * @param bool $firstDamageRound Which damage rounds are calculated. Cannot be 0.
+     * @return int Number of fights fought.
      * @throws ArgumentException if firstDamageRound is 0.
      * @throws BattleIsOverException
-     * @return int Number of fights fought.
      */
     public function fightNRounds(int $n = 1, int $firstDamageRound = self::DAMAGEROUND_BOTH): int
     {
@@ -253,14 +265,14 @@ class Battle
             }
         }
         
-        return $count + 1;
+        return $count+1;
     }
     
     /**
-     * Fights exactly 1 round.
+     * Fights exactly 1 round
      * @param int $firstDamageRound
      */
-    protected function fightOneRound(int $firstDamageRound): void
+    protected function fightOneRound(int $firstDamageRound)
     {
         $damageHasBeenDone = false;
         
@@ -273,7 +285,7 @@ class Battle
         $offenseTurnEvents = $firstDamageRound & self::DAMAGEROUND_PLAYER ? $this->turn($this->player, $this->monster) : new ArrayCollection();
         $defenseTurnEvents = $firstDamageRound & self::DAMAGEROUND_MONSTER ? $this->turn($this->monster, $this->player) : new ArrayCollection();
 
-        $events = new ArrayCollection([...$offenseTurnEvents->toArray(), ...$defenseTurnEvents->toArray()]);
+        $events = new ArrayCollection(array_merge($offenseTurnEvents->toArray(), $defenseTurnEvents->toArray()));
         $eventsToAdd = new ArrayCollection();
 
         foreach ($events as $event) {
@@ -303,17 +315,17 @@ class Battle
         $monsterBuffExpiringEvents = $this->monster->getBuffs()->expireOneRound();
         
         $this->events = new ArrayCollection(
-            [
-                ...$this->events->toArray(),
-                ...$playerBuffStartEvents->toArray(),
-                ...$monsterBuffStartEvents->toArray(),
-                ...$eventsToAdd->toArray(),
-                ...$playerBuffEndEvents->toArray(),
-                ...$monsterBuffEndEvents->toArray(),
-                ...$playerBuffExpiringEvents->toArray(),
-                ...$monsterBuffExpiringEvents->toArray(),
-                ...isset($deathEvent) ? [$deathEvent] : [],
-            ]
+            array_merge(
+                $this->events->toArray(),
+                $playerBuffStartEvents->toArray(),
+                $monsterBuffStartEvents->toArray(),
+                $eventsToAdd->toArray(),
+                $playerBuffEndEvents->toArray(),
+                $monsterBuffEndEvents->toArray(),
+                $playerBuffExpiringEvents->toArray(),
+                $monsterBuffExpiringEvents->toArray(),
+                isset($deathEvent) ? [$deathEvent] : []
+            )
         );
     }
     
@@ -321,7 +333,6 @@ class Battle
      * Runs one turn.
      * @param FighterInterface $attacker
      * @param FighterInterface $defender
-     * @return ArrayCollection
      */
     protected function turn(FighterInterface $attacker, FighterInterface $defender): ArrayCollection
     {
@@ -373,8 +384,8 @@ class Battle
         }
         
         // Conversion from float to int, since the random number generator takes int values.
-        $attackersAttack = (int)\round($attackersAttack, 0);
-        $defendersDefense = (int)\round($defendersDefense, 0);
+        $attackersAttack = (int) round($attackersAttack, 0);
+        $defendersDefense = (int) round($defendersDefense, 0);
         
         // Lets roll the
         $attackersAtkRoll = $this->game->getDiceBag()->pseudoBell(0, $attackersAttack);
@@ -402,10 +413,10 @@ class Battle
             $damage = 0;
         } elseif ($attackerIsInvulnurable) {
             // Attaker is invulnurable, damage is always > 0 (there is no riposte)
-            $damage = \abs($damage);
+            $damage = abs($damage);
         } elseif ($defenderIsInvulnurable) {
             // Defender is invulnurable, damage is always < 0 (defender always ripostes)
-            $damage = -\abs($damage);
+            $damage = - abs($damage);
         }
         
         if ($damage < 0) {
@@ -427,7 +438,7 @@ class Battle
         }
         
         // Round the damage value and convert to int.
-        $damage = (int)\round($damage, 0);
+        $damage = (int)round($damage, 0);
         
         // Add the damage event
         $events->add(new DamageEvent($attacker, $defender, $damage));
@@ -443,15 +454,15 @@ class Battle
         $defendersDamageDependentBuffEvents = $defendersBuffs->processDamageDependentBuffs(Buff::ACTIVATE_DEFENSE, -$damage, $this->game, $defender, $attacker);
         
         return new ArrayCollection(
-            [
-                ...$attackersBuffStartEvents->toArray(),
-                ...$attackersDirectBuffEvents->toArray(),
-                ...$defendersBuffStartEvents->toArray(),
-                ...$defendersDirectBuffEvents->toArray(),
-                ...$events->toArray(),
-                ...$attackersDamageDependentBuffEvents->toArray(),
-                ...$defendersDamageDependentBuffEvents->toArray(),
-            ]
+            array_merge(
+                $attackersBuffStartEvents->toArray(),
+                $attackersDirectBuffEvents->toArray(),
+                $defendersBuffStartEvents->toArray(),
+                $defendersDirectBuffEvents->toArray(),
+                $events->toArray(),
+                $attackersDamageDependentBuffEvents->toArray(),
+                $defendersDamageDependentBuffEvents->toArray()
+            )
         );
     }
 }

src/Bootstrap.php

@@ -4,21 +4,27 @@ declare(strict_types=1);
 namespace LotGD\Core;
 
 use Doctrine\Common\Annotations\AnnotationRegistry;
-use Doctrine\DBAL\DBALException;
-use Doctrine\DBAL\Types\Type;
-use Doctrine\ORM\EntityManager;
-use Doctrine\ORM\EntityManagerInterface;
+use Doctrine\Common\EventManager as DoctrineEventManager;
+use Doctrine\Common\Util\Debug;
 use Doctrine\ORM\Events as DoctrineEvents;
-use Doctrine\ORM\Tools\SchemaTool;
-use Doctrine\ORM\Tools\Setup;
-use LotGD\Core\Doctrine\EntityPostLoadEventListener;
-use LotGD\Core\Exceptions\InvalidConfigurationException;
-use Monolog\Handler\RotatingFileHandler;
-use Monolog\Logger;
-
+use Doctrine\ORM\ {
+    EntityManager,
+    EntityManagerInterface,
+    Mapping\AnsiQuoteStrategy,
+    Tools\Setup,
+    Tools\SchemaTool
+};
+use Monolog\ {
+    Logger,
+    Handler\RotatingFileHandler
+};
 use Psr\Log\LoggerInterface;
 use Symfony\Component\Console\Application;
 
+use LotGD\Core\{
+    ComposerManager, BootstrapInterface, Doctrine\EntityPostLoadEventListener, Exceptions\InvalidConfigurationException
+};
+
 /**
  * The entry point for constructing a properly configured LotGD Game object.
  */
@@ -26,24 +32,24 @@ class Bootstrap
 {
     private $logger;
     private $game;
-    /** @var LibraryConfigurationManager */
+    /** @var  LibraryConfigurationManager */
     private $libraryConfigurationManager;
     private $annotationDirectories = [];
 
     /**
      * Create a new Game object, with all the necessary configuration.
-     * @param string|null $cwd
+     * @param string $cwd
      * @return Game The newly created Game object.
      */
     public static function createGame(string $cwd = null): Game
     {
         $game = new self();
-        $cwd = $cwd ?? \getcwd();
+        $cwd = $cwd ?? getcwd();
         return $game->getGame($cwd);
     }
 
     /**
-     * Starts the game kernel with the most important classes and returns the object.
+     * Starts the game kernel with the most important classes and returns the object
      * @param string $cwd
      * @return Game
      */
@@ -57,17 +63,16 @@ class Bootstrap
         $composer = $this->createComposerManager($cwd);
         $this->libraryConfigurationManager = $this->createLibraryConfigurationManager($composer, $cwd);
 
-        [$dsn, $user, $password] = $config->getDatabaseConnectionDetails($cwd);
+        list($dsn, $user, $password) = $config->getDatabaseConnectionDetails($cwd);
         $pdo = $this->connectToDatabase($dsn, $user, $password);
-        $entityManager = $this->createEntityManager($pdo, $config);
+        $entityManager = $this->createEntityManager($pdo);
 
         $this->game = (new GameBuilder())
             ->withConfiguration($config)
             ->withLogger($this->logger)
             ->withEntityManager($entityManager)
             ->withCwd($cwd)
-            ->create()
-        ;
+            ->create();
 
         // Add Event listener to entity manager
         $dem = $entityManager->getEventManager();
@@ -80,10 +85,10 @@ class Bootstrap
     }
 
     /**
-     * Creates a library configuration manager.
+     * Creates a library configuration manager
      * @param ComposerManager $composerManager
      * @param string $cwd
-     * @return LibraryConfigurationManager
+     * @return \LotGD\Core\LibraryConfigurationManager
      */
     protected function createLibraryConfigurationManager(
         ComposerManager $composerManager,
@@ -93,10 +98,8 @@ class Bootstrap
     }
 
     /**
-     * Connects to a database using pdo.
-     * @param string $dsn
-     * @param string $user
-     * @param string $password
+     * Connects to a database using pdo
+     * @param \LotGD\Core\Configuration $config
      * @return \PDO
      */
     protected function connectToDatabase(string $dsn, string $user, string $password): \PDO
@@ -105,7 +108,7 @@ class Bootstrap
     }
 
     /**
-     * Creates and returns an instance of ComposerManager.
+     * Creates and returns an instance of ComposerManager
      * @param string $cwd
      * @return ComposerManager
      */
@@ -118,16 +121,16 @@ class Bootstrap
     /**
      * Returns a configuration object reading from the file located at the path stored in $cwd/config/lotgd.yml.
      * @param string $cwd
+     * @return \LotGD\Core\Configuration
      * @throws InvalidConfigurationException
-     * @return Configuration
      */
     protected function createConfiguration(string $cwd): Configuration
     {
         if (empty($configFilePath)) {
-            $configFilePath = \implode(\DIRECTORY_SEPARATOR, [$cwd, "config", "lotgd.yml"]);
+            $configFilePath = implode(DIRECTORY_SEPARATOR, [$cwd, "config", "lotgd.yml"]);
         }
 
-        if ($configFilePath === false || \strlen($configFilePath) == 0 || \is_file($configFilePath) === false) {
+        if ($configFilePath === false || strlen($configFilePath) == 0 || is_file($configFilePath) === false) {
             throw new InvalidConfigurationException("Invalid or missing configuration file: {$configFilePath}.");
         }
 
@@ -136,7 +139,7 @@ class Bootstrap
     }
 
     /**
-     * Returns a logger instance.
+     * Returns a logger instance
      * @param Configuration $config
      * @param string $name
      * @return LoggerInterface
@@ -145,41 +148,35 @@ class Bootstrap
     {
         $logger = new Logger($name);
         // Add lotgd as the prefix for the log filenames.
-        $logger->pushHandler(new RotatingFileHandler($config->getLogPath() . \DIRECTORY_SEPARATOR . $name, 14));
+        $logger->pushHandler(new RotatingFileHandler($config->getLogPath() . DIRECTORY_SEPARATOR . $name, 14));
 
         return $logger;
     }
 
     /**
-     * Creates the EntityManager using the pdo connection given in it's argument.
+     * Creates the EntityManager using the pdo connection given in it's argument
      * @param \PDO $pdo
-     * @param Configuration
      * @return EntityManagerInterface
      */
-    protected function createEntityManager(\PDO $pdo, Configuration $config): EntityManagerInterface
+    protected function createEntityManager(\PDO $pdo): EntityManagerInterface
     {
         $this->annotationDirectories = $this->generateAnnotationDirectories();
-        $this->logger->debug("Adding annotation directories:");
+        $this->logger->addDebug("Adding annotation directories:");
         foreach ($this->annotationDirectories as $d) {
-            $this->logger->debug("  {$d}");
+            $this->logger->addDebug("  {$d}");
         }
         $configuration = Setup::createAnnotationMetadataConfiguration($this->annotationDirectories, true);
 
+        // Set a quote
+        $configuration->setQuoteStrategy(new AnsiQuoteStrategy());
+
         // Create entity manager
         $entityManager = EntityManager::create(["pdo" => $pdo], $configuration);
 
-        // Register uuid type
-        try {
-            Type::addType('uuid', 'Ramsey\Uuid\Doctrine\UuidType');
-        } catch (DBALException $e) {
-        }
-
         // Create Schema and update database if needed
-        if ($config->getDatabaseAutoSchemaUpdate()) {
-            $metaData = $entityManager->getMetadataFactory()->getAllMetadata();
-            $schemaTool = new SchemaTool($entityManager);
-            $schemaTool->updateSchema($metaData);
-        }
+        $metaData = $entityManager->getMetadataFactory()->getAllMetadata();
+        $schemaTool = new SchemaTool($entityManager);
+        $schemaTool->updateSchema($metaData);
 
         return $entityManager;
     }
@@ -191,12 +188,12 @@ class Bootstrap
     protected function generateAnnotationDirectories(): array
     {
         // Read db annotations from our own model files.
-        $directories = [__DIR__ . \DIRECTORY_SEPARATOR . 'Models'];
+        $directories = [__DIR__ . DIRECTORY_SEPARATOR . 'Models'];
 
         // Get additional annotation directories from library configs.
         $libraryDirectories = $this->libraryConfigurationManager->getEntityDirectories();
 
-        return \array_merge($directories, $libraryDirectories);
+        return array_merge($directories, $libraryDirectories);
     }
 
     /**

src/BuffList.php

@@ -3,58 +3,73 @@ declare(strict_types=1);
 
 namespace LotGD\Core;
 
-use Doctrine\Common\Collections\ArrayCollection;
-use Doctrine\Common\Collections\Collection;
+use Doctrine\Common\Collections\{
+    ArrayCollection,
+    Collection
+};
+
+use LotGD\Core\Exceptions\{
+    ArgumentException,
+    BuffListAlreadyActivatedException
+};
+use LotGD\Core\Models\{
+    Buff,
+    Character,
+    FighterInterface,
+    BattleEvents\BuffMessageEvent,
+    BattleEvents\DamageLifetapEvent,
+    BattleEvents\DamageReflectionEvent,
+    BattleEvents\RegenerationBuffEvent,
+    BattleEvents\MinionDamageEvent
+};
 
-use LotGD\Core\Exceptions\ArgumentException;
-use LotGD\Core\Exceptions\BuffListAlreadyActivatedException;
-use LotGD\Core\Exceptions\BuffSlotOccupiedException;
-use LotGD\Core\Models\BattleEvents\BuffMessageEvent;
-use LotGD\Core\Models\BattleEvents\DamageLifetapEvent;
-use LotGD\Core\Models\BattleEvents\DamageReflectionEvent;
-use LotGD\Core\Models\BattleEvents\MinionDamageEvent;
-use LotGD\Core\Models\BattleEvents\RegenerationBuffEvent;
-use LotGD\Core\Models\Buff;
-use LotGD\Core\Models\FighterInterface;
 
 /**
- * Description of BuffList.
+ * Description of BuffList
  */
 class BuffList
 {
-    protected array $buffsBySlot;
-    protected array $activeBuffs = [];
-    protected ArrayCollection $usedBuffs;
+    protected $buffs;
+    protected $buffsBySlot;
+    protected $activeBuffs = [];
+    /** @var Doctrine\Common\Collections\ArrayCollection */
+    protected $usedBuffs;
 
-    /** True of the modifiers have already been calculated */
-    protected bool $modifiersCalculated = false;
-    /** True if the badguy is invulnurable */
-    protected bool $badguyInvulnurable = false;
+    /** @var boolean True of the modifiers have already been calculated */
+    protected $modifiersCalculated = false;
+    /** @var boolean True if the badguy is invulnurable */
+    protected $badguyInvulnurable = false;
+    /** @var float */
+    protected $badguyDamageModifier = 1.;
+    /** @var float */
+    protected $badguyAttackModifier = 1.;
+    /** @var float */
+    protected $badguyDefenseModifier = 1.;
+    /** @var boolean True if the goodguy is invulnurable */
+    protected $goodguyInvulnurable = false;
+    /** @var float */
+    protected $goodguyDamageModifier = 1.;
+    /** @var float */
+    protected $goodguyAttackModifier = 1.;
+    /** @var float */
+    protected $goodguyDefenseModifier = 1.;
 
-    protected float $badguyDamageModifier = 1.;
-    protected float $badguyAttackModifier = 1.;
-    protected float $badguyDefenseModifier = 1.;
-    protected bool $goodguyInvulnurable = false;
-    protected float $goodguyDamageModifier = 1.;
-    protected float $goodguyAttackModifier = 1.;
-    protected float $goodguyDefenseModifier = 1.;
-
-    protected ArrayCollection $events;
-    protected bool $loaded = false;
+    protected $events;
+    protected $loaded = false;
 
     /**
-     * Initiates some variables.
+     * Initiates some variables
      * @param Collection $buffs
      */
-    public function __construct(
-        protected Collection $buffs
-    ) {
+    public function __construct(Collection $buffs)
+    {
+        $this->buffs = $buffs;
         $this->events = new ArrayCollection();
         $this->usedBuffs = new ArrayCollection();
     }
 
     /**
-     * Loads all buffs (since it's a lazy correlation).
+     * Loads all buffs (since it's a lazy correlation)
      */
     public function loadBuffs()
     {
@@ -82,7 +97,7 @@ class BuffList
     }
 
     /**
-     * Marks the given buff as used.
+     * Marks the given buff as used
      * @param Buff $buff
      */
     protected function useBuff(Buff $buff)
@@ -91,7 +106,7 @@ class BuffList
     }
 
     /**
-     * Returns the buff's start or round message.
+     * Returns the buff's start or round message
      * @param Buff $buff
      * @return string
      */
@@ -110,7 +125,7 @@ class BuffList
     }
 
     /**
-     * Resets the buff usage for a new round.
+     * Resets the buff usage for a new round
      */
     public function resetBuffUsage()
     {
@@ -121,23 +136,22 @@ class BuffList
 
     /**
      * Returns whether any buffs are in use.
-     * @return bool
      */
     public function hasBuffsInUse(): bool
     {
-        return \count($this->usedBuffs) > 0;
+        return count($this->usedBuffs) > 0 ? true : false;
     }
 
     /**
      * Activates all buffs that activate upon the given activation parameter.
      * @param int $activation
+     * @return Collection
      * @throws ArgumentException
      * @throws BuffListAlreadyActivatedException
-     * @return Collection
      */
     public function activate(int $activation): Collection
     {
-        if ($activation % 2 !== 0 && $activation !== 1) {
+        if ($activation%2 !== 0 && $activation !== 1) {
             throw new ArgumentException("You can only activate one activation type at a time.");
         }
 
@@ -172,7 +186,7 @@ class BuffList
     }
 
     /**
-     * Decreases the rounds left on all used buffs.
+     * Decreases the rounds left on all used buffs
      * @return Collection A Collection containing expire messages (if there are any)
      */
     public function expireOneRound(): Collection
@@ -247,9 +261,10 @@ class BuffList
     }
 
     /**
-     * Calculates all total modifiers.
+     * Calculates all total modifiers
+     * @return type
      */
-    protected function calculateModifiers(): void
+    protected function calculateModifiers()
     {
         if ($this->modifiersCalculated === true) {
             return;
@@ -287,13 +302,14 @@ class BuffList
             // Only look at buffs that are activated in battle.
             if ($buff->getsActivatedAt(Buff::ACTIVATE_NONE)) {
                 continue;
+            } else {
+                yield $buff;
             }
-            yield $buff;
         }
     }
 
     /**
-     * Returns the badguy attack modifier calculated over the whole bufflist.
+     * Returns the badguy attack modifier calculated over the whole bufflist
      * @return float
      */
     public function getBadguyAttackModifier(): float
@@ -302,8 +318,8 @@ class BuffList
         return $this->badguyAttackModifier;
     }
 
-    /**
-     * Returns the badguy defense modifier calculated over the whole bufflist.
+     /**
+     * Returns the badguy defense modifier calculated over the whole bufflist
      * @return float
      */
     public function getBadguyDefenseModifier(): float
@@ -313,7 +329,7 @@ class BuffList
     }
 
     /**
-     * Returns the badguy damage modifier calculated over the whole bufflist.
+     * Returns the badguy damage modifier calculated over the whole bufflist
      * @return float
      */
     public function getBadguyDamageModifier(): float
@@ -323,7 +339,7 @@ class BuffList
     }
 
     /**
-     * Returns true if the badguy is invulnurable.
+     * Returns true if the badguy is invulnurable
      * @return bool
      */
     public function badguyIsInvulnurable(): bool
@@ -333,7 +349,7 @@ class BuffList
     }
 
     /**
-     * Returns the badguy attack modifier calculated over the whole bufflist.
+     * Returns the badguy attack modifier calculated over the whole bufflist
      * @return float
      */
     public function getGoodguyAttackModifier(): float
@@ -342,8 +358,8 @@ class BuffList
         return $this->goodguyAttackModifier;
     }
 
-    /**
-     * Returns the badguy defense modifier calculated over the whole bufflist.
+     /**
+     * Returns the badguy defense modifier calculated over the whole bufflist
      * @return float
      */
     public function getGoodguyDefenseModifier(): float
@@ -352,8 +368,8 @@ class BuffList
         return $this->goodguyDefenseModifier;
     }
 
-    /**
-     * Returns the badguy damage modifier calculated over the whole bufflist.
+     /**
+     * Returns the badguy damage modifier calculated over the whole bufflist
      * @return float
      */
     public function getGoodguyDamageModifier(): float
@@ -363,7 +379,7 @@ class BuffList
     }
 
     /**
-     * Returns true if the goodguy is invulnurable.
+     * Returns true if the goodguy is invulnurable
      * @return bool
      */
     public function goodguyIsInvulnurable(): bool
@@ -373,9 +389,9 @@ class BuffList
     }
 
     /**
-     * Processes buffs that do direct damage or regeneration.
+     * Processes buffs that do direct damage or regeneration
      * @param int $activation
-     * @param Game $game
+     * @param \LotGD\Core\Game $game
      * @param FighterInterface $goodguy
      * @param FighterInterface $badguy
      * @return Collection
@@ -461,7 +477,7 @@ class BuffList
 
                     $events[] = new MinionDamageEvent(
                         $target,
-                        (int)\round($damage, 0),
+                        (int)round($damage, 0),
                         $message
                     );
                 }
@@ -474,10 +490,10 @@ class BuffList
     }
 
     /**
-     * Processes buffs that are dependant on the damage done in one round.
+     * Processes buffs that are dependant on the damage done in one round
      * @param int $activation
      * @param int $damage Positive damage is applied to the badguy, negative damage is applied to the goodguy
-     * @param Game $game
+     * @param \LotGD\Core\Game $game
      * @param FighterInterface $goodguy
      * @param FighterInterface $badguy
      * @return Collection
@@ -502,7 +518,7 @@ class BuffList
                     $reflectedDamage = 0;
                     $message = $buff->getNoEffectMessage();
                 } else {
-                    $reflectedDamage = (int)\round($buff->getGoodguyDamageReflection() * $damage * -1, 0);
+                    $reflectedDamage = (int)round($buff->getGoodguyDamageReflection() * $damage * -1, 0);
                     if ($reflectedDamage === 0) {
                         $message = $buff->getNoEffectMessage();
                     } else {
@@ -520,7 +536,7 @@ class BuffList
             if ($buff->getBadguyDamageReflection() !== 0.) {
                 if ($damage > 0) {
                     // Damage is > 0, so badguy takes damage, we can normally reflect
-                    $reflectedDamage = (int)\round($buff->getBadguyDamageReflection() * $damage, 0);
+                    $reflectedDamage = (int)round($buff->getBadguyDamageReflection() * $damage, 0);
                     if ($reflectedDamage === 0) {
                         $message = $buff->getNoEffectMessage();
                     } else {
@@ -549,7 +565,7 @@ class BuffList
                     $message = $buff->getEffectFailsMessage();
                 } elseif ($damage < 0) {
                     // Damage is < 0, goodguy takes damage. We act upon this.
-                    $healAmount = (int)\round($damage * -$buff->getBadguyLifetap(), 0);
+                    $healAmount = (int)round($damage * -$buff->getBadguyLifetap(), 0);
                     if ($healAmount === 0) {
                         $message = $buff->getNoEffectMessage();
                     } else {
@@ -570,7 +586,7 @@ class BuffList
             if ($buff->getBadguyLifetap() !== 0.) {
                 if ($damage > 0) {
                     // Damage is > 0, badguy takes damage. We act upon this to heal goodguy.
-                    $healAmount = (int)\round($damage * $buff->getBadguyLifetap(), 0);
+                    $healAmount = (int)round($damage * $buff->getBadguyLifetap(), 0);
                     if ($healAmount === 0) {
                         $message = $buff->getNoEffectMessage();
                     } else {

src/ComposerManager.php

@@ -3,29 +3,36 @@ declare(strict_types=1);
 
 namespace LotGD\Core;
 
-use Composer\Composer;
-use Composer\Factory;
-use Composer\IO\NullIO;
-use Composer\Package\CompletePackageInterface;
-use Composer\Package\PackageInterface;
-use Exception;
+use Composer\{
+    Composer,
+    Factory,
+    IO\NullIO,
+    Package\CompletePackageInterface
+};
+use Monolog\Logger;
 
-use LotGD\Core\Exceptions\InvalidConfigurationException;
-use LotGD\Core\Exceptions\LibraryDoesNotExistException;
+use LotGD\Core\{
+    Exceptions\InvalidConfigurationException,
+    Exceptions\LibraryDoesNotExistException
+};
 
 /**
  * Helps perform tasks with the composer configuration.
  */
 class ComposerManager
 {
-    private ?Composer $composer = null;
+    private $composer;
+    private $cwd;
 
     /**
      * Construct a manager with an optional working directory where composer.json
      * lives.
      * @param string $cwd
      */
-    public function __construct(private string $cwd) {}
+    public function __construct(string $cwd)
+    {
+        $this->cwd = $cwd;
+    }
 
     /**
      * Returns a Composer instance to perform underlying operations on. Be careful.
@@ -35,8 +42,8 @@ class ComposerManager
     {
         if ($this->composer === null) {
             // Verify location of composer.json.
-            $composerConfigPath = $this->cwd . \DIRECTORY_SEPARATOR . "composer.json";
-            if (!\file_exists($composerConfigPath)) {
+            $composerConfigPath = $this->cwd . DIRECTORY_SEPARATOR . "composer.json";
+            if (!file_exists($composerConfigPath)) {
                 throw new InvalidConfigurationException("composer.json cannot be found at {$composerConfigPath}.");
             }
 
@@ -49,9 +56,8 @@ class ComposerManager
 
     /**
      * Return the Composer package for the corresponding library, in vendor/module format.
-     * @param string $library
+     * @return PackageInterface Package corresponding to this library.
      * @throws LibraryDoesNotExistException
-     * @return CompletePackageInterface Package corresponding to this library.
      */
     public function getPackageForLibrary(string $library): CompletePackageInterface
     {
@@ -67,27 +73,27 @@ class ComposerManager
 
     /**
      * Return all the packages installed in the current setup.
-     * @return PackageInterface[]
+     * @return array<Composer\PackageInterface>
      */
     public function getPackages(): array
     {
-        return [
-            $this->getComposer()->getPackage(),
-            ...$this->getComposer()->getRepositoryManager()->getLocalRepository()->getPackages(),
-        ];
+        return array_merge(
+            [$this->getComposer()->getPackage()],
+            $this->getComposer()->getRepositoryManager()->getLocalRepository()->getPackages()
+        );
     }
 
     /**
      * Return a list of the configured packages which are LotGD modules (type = 'lotgd-module').
-     * @return PackageInterface[]
+     * @return array Array of \Composer\PackageInterface.
      */
     public function getModulePackages(): array
     {
-        $result = [];
+        $result = array();
         $packages = $this->getComposer()->getRepositoryManager()->getLocalRepository()->getPackages();
         foreach ($packages as $p) {
             if ($p->getType() === 'lotgd-module') {
-                \array_push($result, $p);
+                array_push($result, $p);
             }
         }
 
@@ -100,7 +106,7 @@ class ComposerManager
      * @return string|null Path representing $namespace or null if $namespace
      * cannot be found or if the path does not exist.
      */
-    public function translateNamespaceToPath(string $namespace): ?string
+    public function translateNamespaceToPath(string $namespace)
     {
         // Find the directory for this namespace by using the autoloader's
         // classmap.
@@ -110,32 +116,32 @@ class ComposerManager
         // Standardize the namespace to remove any leading \ and add a trailing \
         $n = $namespace;
         if ('\\' == $n[0]) {
-            $n = \substr($n, 1);
+            $n = substr($n, 1);
         }
-        if (\strlen($n) > 0 && '\\' != $n[\strlen($n) - 1]) {
+        if (strlen($n) > 0 && '\\' != $n[strlen($n) - 1]) {
             $n .= '\\';
         }
 
-        $split = \explode('\\', $n);
-        $suffix = \array_splice($split, -1, 1); // starts with ['']
+        $split = explode('\\', $n);
+        $suffix = array_splice($split, -1, 1); // starts with ['']
         $path = null;
         while (!empty($split)) {
-            $key = \implode('\\', $split) . '\\';
-            $dir = \implode(\DIRECTORY_SEPARATOR, $suffix);
+            $key = implode('\\', $split) . '\\';
+            $dir = implode(DIRECTORY_SEPARATOR, $suffix);
             // Prefix to directory mappings are arrays in Composer's
             // ClassLoader object. Not sure why. This might break in
             // some unforseen case.
-            if (isset($prefixes[$key]) && \is_dir($prefixes[$key][0] .  \DIRECTORY_SEPARATOR . $dir)) {
-                $path = $prefixes[$key][0] .  \DIRECTORY_SEPARATOR . $dir;
+            if (isset($prefixes[$key]) && is_dir($prefixes[$key][0] .  DIRECTORY_SEPARATOR . $dir)) {
+                $path = $prefixes[$key][0] .  DIRECTORY_SEPARATOR . $dir;
                 break;
             }
-            $suffix = \array_merge($suffix, \array_splice($split, -1, 1));
+            $suffix = array_merge($suffix, array_splice($split, -1, 1));
         }
 
         if ($path == null) {
             return null;
         }
-        $path = \realpath($path);
+        $path = realpath($path);
         if ($path == false) {
             return null;
         }
@@ -145,20 +151,19 @@ class ComposerManager
     /**
      * Returns a path (could be relative) to the proper autoload.php file in
      * the current setup.
-     * @return string
      */
     public function findAutoloader(): string
     {
         // Dance to find the autoloader.
         // TOOD: change this to open up the Composer config and use $c['config']['vendor-dir'] instead of "vendor"
         $order = [
-            \implode(\DIRECTORY_SEPARATOR, [$this->cwd, "vendor", "autoload.php"]),
-            \implode(\DIRECTORY_SEPARATOR, [__DIR__, "..", "vendor", "autoload.php"]),
-            \implode(\DIRECTORY_SEPARATOR, [__DIR__, "..", "autoload.php"]),
+            implode(DIRECTORY_SEPARATOR, [$this->cwd, "vendor", "autoload.php"]),
+            implode(DIRECTORY_SEPARATOR, [__DIR__, "..", "vendor", "autoload.php"]),
+            implode(DIRECTORY_SEPARATOR, [__DIR__, "..", "autoload.php"]),
         ];
 
         foreach ($order as $path) {
-            if (\file_exists($path)) {
+            if (file_exists($path)) {
                 return $path;
             }
         }

src/Configuration.php

@@ -5,26 +5,25 @@ namespace LotGD\Core;
 
 use DateTime;
 
-use LotGD\Core\Exceptions\InvalidConfigurationException;
 use Symfony\Component\Yaml\Exception\ParseException;
-
 use Symfony\Component\Yaml\Yaml;
 
+use LotGD\Core\Exceptions\InvalidConfigurationException;
+
 /**
  * The configuration information for a LotGD game. Configuration is read from
  * a YAML file, who's path is specified when you construct the object.
  */
 class Configuration
 {
-    private string $databaseDSN;
-    private string $databaseName;
-    private string $databaseUser;
-    private string $databasePassword;
-    private bool $databaseAutoSchemaUpdate;
-    private string $logPath;
-    private DateTime $gameEpoch;
-    private int $gameOffsetSeconds;
-    private int $gameDaysPerDay;
+    private $databaseDSN;
+    private $databaseName;
+    private $databaseUser;
+    private $databasePassword;
+    private $logPath;
+    private $gameEpoch;
+    private $gameOffsetSeconds;
+    private $gameDaysPerDay;
 
     /**
      * Create the configuration object, reading from the specified path.
@@ -42,8 +41,8 @@ class Configuration
 
         // Log dir path is relative to config directory.
         $logPath = $rawConfig['logs']['path'] ?? '';
-        $realLogPath = \dirname($configFilePath) . \DIRECTORY_SEPARATOR . $logPath;
-        if ($realLogPath === false || \strlen($realLogPath) == 0 || \is_dir($realLogPath) === false) {
+        $realLogPath = dirname($configFilePath) . DIRECTORY_SEPARATOR . $logPath;
+        if ($realLogPath === false || strlen($realLogPath) == 0 || is_dir($realLogPath) === false) {
             throw new InvalidConfigurationException("Invalid or missing log path: {$realLogPath}");
         }
         $this->logPath = $realLogPath;
@@ -53,10 +52,10 @@ class Configuration
         $passwd = $rawConfig['database']['password'] ?? '';
         $name = $rawConfig['database']['name'] ?? '';
 
-        if ($dsn === false || \strlen($dsn) == 0) {
+        if ($dsn === false || strlen($dsn) == 0) {
             throw new InvalidConfigurationException("Invalid or missing data source name: {$dsn}");
         }
-        if ($user === false || \strlen($user) == 0) {
+        if ($user === false || strlen($user) == 0) {
             throw new InvalidConfigurationException("Invalid or missing database user: {$user}");
         }
         if ($passwd === false) {
@@ -71,12 +70,6 @@ class Configuration
         $this->databasePassword = $passwd;
         $this->databaseName = $name;
 
-        if (empty($rawConfig['database']['disableAutoSchemaUpdate'])) {
-            $this->databaseAutoSchemaUpdate = true;
-        } else {
-            $this->databaseAutoSchemaUpdate = false;
-        }
-
         $gameEpoch = $rawConfig['game']['epoch'];
         $gameOffsetSeconds = $rawConfig['game']['offsetSeconds'];
         $gameDaysPerDay = $rawConfig['game']['daysPerDay'];
@@ -105,12 +98,12 @@ class Configuration
      */
     protected function retrieveRawConfig(string $configFilePath): array
     {
-        return Yaml::parse(\file_get_contents($configFilePath));
+        return Yaml::parse(file_get_contents($configFilePath));
     }
     
     /**
      * Returns database connection details needed for pdo to establish a connection.
-     *
+     * 
      * This function takes optionally replaces the string %cwd% in the database dsn and
      * replaces it with the first parameter. This is important to normalize the database location
      * across different working directories. Alternatively, SQLite databse names can also directly
@@ -121,7 +114,7 @@ class Configuration
     public function getDatabaseConnectionDetails(string $cwd = ""): array
     {
         return [
-            \str_replace("%cwd%", $cwd . \DIRECTORY_SEPARATOR, $this->getDatabaseDSN()),
+            str_replace("%cwd%", $cwd . DIRECTORY_SEPARATOR, $this->getDatabaseDSN()),
             $this->getDatabaseUser(),
             $this->getDatabasePassword(),
         ];
@@ -164,15 +157,6 @@ class Configuration
         return $this->databasePassword;
     }
 
-    /**
-     * True if doctrine should not auto update.
-     * @return bool
-     */
-    public function getDatabaseAutoSchemaUpdate(): bool
-    {
-        return $this->databaseAutoSchemaUpdate;
-    }
-
     /**
      * Return the path to the directory to store log files.
      * @return string The configured log directory path.
@@ -213,7 +197,6 @@ class Configuration
     /**
      * Generate a textual representation of the configuration, for debugging
      * purposes.
-     * @return string
      */
     public function __toString(): string
     {

src/Console/Command/BaseCommand.php

@@ -3,17 +3,16 @@ declare(strict_types=1);
 
 namespace LotGD\Core\Console\Command;
 
-use LotGD\Core\Game;
-use Monolog\Logger;
 use Symfony\Component\Console\Command\Command;
 
+use LotGD\Core\Game;
+
 /**
  * Parent class for daenerys tool commands.
  */
 abstract class BaseCommand extends Command
 {
-    protected Game $game;
-    protected ?string $namespace = null;
+    protected $game;
 
     /**
      * Construct the command, using the provided Game.
@@ -24,22 +23,4 @@ abstract class BaseCommand extends Command
         parent::__construct();
         $this->game = $game;
     }
-
-    /**
-     * Returns a cloned logger with a different context name.
-     * @return Logger
-     */
-    public function getCliLogger(): Logger
-    {
-        return $this->game->getLogger()->withName("daenerys-cli");
-    }
-
-    protected function namespaced(string $command): string
-    {
-        if ($this->namespace) {
-            return "{$this->namespace}:{$command}";
-        } else {
-            return $command;
-        }
-    }
 }

src/Console/Command/Character/CharacterAddCommand.php

@@ -1,105 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Character;
-
-use Exception;
-use LotGD\Core\Models\Character;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class CharacterAddCommand extends CharacterBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("add"))
-            ->setDescription("Add a character.")
-            ->setDefinition(
-                new InputDefinition([
-                    new InputArgument(
-                        "name",
-                        mode: InputArgument::REQUIRED,
-                        description: "Character name",
-                    ),
-                    new InputOption(
-                        "level",
-                        mode: InputOption::VALUE_OPTIONAL,
-                        description: "Character level",
-                        default: 1,
-                    ),
-                    new InputOption(
-                        "maxHealth",
-                        mode: InputOption::VALUE_OPTIONAL,
-                        description: "Maximum health of the character. 10*level if not given.",
-                        default: null,
-                    ),
-                ])
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $logger = $this->getCliLogger();
-        $io = new SymfonyStyle($input, $output);
-
-        $name = $input->getArgument("name");
-        $level = intval($input->getOption("level"));
-        $maxHealth = $input->getOption("maxHealth");
-
-        if ($level <= 0) {
-            $io->error("Level must at least be 1.");
-            return Command::FAILURE;
-        }
-
-        // Set maxHealth in dependence of the level if not given.
-        if ($maxHealth === null) {
-            $maxHealth = $level*10;
-        } else {
-            $maxHealth = intval($maxHealth);
-        }
-
-        if ($maxHealth < 0) {
-            $io->error("Maximum health must be at least 0.");
-            return Command::FAILURE;
-        } elseif ($maxHealth === 0) {
-            $io->warning("The character will have 0 max health and will be permanently dead.");
-        }
-
-        $character = Character::createAtFullHealth([
-            "name" => $name,
-            "level" => $level,
-            "maxHealth" => $maxHealth,
-        ]);
-
-        try {
-            $em->persist($character);
-
-            // Commit changes
-            $em->flush();
-        } catch (Exception $e) {
-            $io->error("Creating the character was not possible. Reason: {$e->getMessage()}.");
-            return Command::FAILURE;
-        }
-
-        $io->success("{$character} was successfully created.");
-        $logger->info("Character was created.", ["character" => $character]);
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Character/CharacterBaseCommand.php

@@ -1,36 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Character;
-
-use LotGD\Core\Console\Command\BaseCommand;
-use LotGD\Core\Models\Character;
-use Symfony\Component\Console\Input\InputArgument;
-
-class CharacterBaseCommand extends BaseCommand
-{
-    protected ?string $namespace = "character";
-
-    /**
-     * @return InputArgument
-     */
-    protected function getCharacterIdArgumentDefinition(): InputArgument
-    {
-        return new InputArgument(
-            name: "id",
-            mode: InputArgument::REQUIRED,
-            description: "Character ID",
-        );
-    }
-
-    /**
-     * @param string $id
-     * @return Character|null
-     */
-    protected function getCharacter(string $id): ?Character
-    {
-        /** @var Character|null $character */
-        $character = $this->game->getEntityManager()->getRepository(Character::class)->find($id);
-        return $character;
-    }
-}

src/Console/Command/Character/CharacterConfigListCommand.php

@@ -1,59 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Character;
-
-use LotGD\Core\Events\EventContextData;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class CharacterConfigListCommand extends CharacterBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("config:list"))
-            ->setDescription('List available settings for a character')
-            ->setDefinition([
-                $this->getCharacterIdArgumentDefinition(),
-            ])
-        ;
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output)
-    {
-        $io = new SymfonyStyle($input, $output);
-        $character = $this->getCharacter($input->getArgument("id"));
-
-        if (!$character) {
-            $io->error("Character was not found.");
-            return Command::FAILURE;
-        }
-
-        // Create hook
-        $context = EventContextData::create([
-            "character" => $character,
-            "io" => $io,
-            "settings" => [],
-        ]);
-        $newContext = $this->game->getEventManager()->publish(
-            event: "h/lotgd/core/cli/character-config-list",
-            contextData: $context
-        );
-        $settings = $newContext->get("settings");
-
-        $io->title("Character ".$character->getDisplayName());
-
-        if (count($settings) === 0) {
-            $io->note("There are no character settings available.");
-        } else {
-            $io->table(["setting", "value", "description"], $settings);
-        }
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Character/CharacterConfigResetCommand.php

@@ -1,68 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Character;
-
-use LotGD\Core\Events\EventContextData;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class CharacterConfigResetCommand extends CharacterBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("config:reset"))
-            ->setDescription('Reset a character setting')
-            ->setDefinition([
-                $this->getCharacterIdArgumentDefinition(),
-                new InputArgument(
-                    "setting",
-                    mode: InputArgument::REQUIRED,
-                    description: "Name of setting, see {$this->namespaced('config:list')}.",
-                ),
-            ])
-        ;
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output)
-    {
-        $logger = $this->getCliLogger();
-        $io = new SymfonyStyle($input, $output);
-        $character = $this->getCharacter($input->getArgument("id"));
-
-        if (!$character) {
-            $io->error("Module was not found.");
-            return Command::FAILURE;
-        }
-
-        $io->title("Character {$character->getDisplayName()}");
-
-        // Create hook
-        $context = EventContextData::create([
-            "character" => $character,
-            "io" => $io,
-            "setting" => $input->getArgument("setting"),
-            "return" => Command::FAILURE,
-            "reason" => "Setting does not exist.",
-        ]);
-
-        $newContext = $this->game->getEventManager()->publish(
-            event: "h/lotgd/core/cli/character-config-reset",
-            contextData: $context
-        );
-        if ($newContext->get("return") != Command::SUCCESS) {
-            $io->error($newContext->get("reason"));
-            return Command::FAILURE;
-        }
-
-        $this->game->getEntityManager()->flush();
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Character/CharacterConfigSetCommand.php

@@ -1,73 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Character;
-
-use LotGD\Core\Events\EventContextData;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class CharacterConfigSetCommand extends CharacterBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("config:set"))
-            ->setDescription('Change a character setting')
-            ->setDefinition([
-                $this->getCharacterIdArgumentDefinition(),
-                new InputArgument(
-                    "setting",
-                    mode: InputArgument::REQUIRED,
-                    description: "Name of setting, see {$this->namespaced('config:list')}.",
-                ),
-                new InputArgument(
-                    "value",
-                    InputArgument::REQUIRED,
-                    description: "New value for the given setting.",
-                ),
-            ])
-        ;
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output)
-    {
-        $logger = $this->getCliLogger();
-        $io = new SymfonyStyle($input, $output);
-        $character = $this->getCharacter($input->getArgument("id"));
-
-        if (!$character) {
-            $io->error("Module was not found.");
-            return Command::FAILURE;
-        }
-
-        $io->title("Character {$character->getDisplayName()}");
-
-        // Create hook
-        $context = EventContextData::create([
-            "character" => $character,
-            "io" => $io,
-            "setting" => $input->getArgument("setting"),
-            "value" => $input->getArgument("value"),
-            "return" => Command::FAILURE,
-            "reason" => "Setting does not exist.",
-        ]);
-        $newContext = $this->game->getEventManager()->publish(
-            event: "h/lotgd/core/cli/character-config-set",
-            contextData: $context
-        );
-        if ($newContext->get("return") != Command::SUCCESS) {
-            $io->error($newContext->get("reason"));
-            return Command::FAILURE;
-        }
-
-        $this->game->getEntityManager()->flush();
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Character/CharacterEditCommand.php

@@ -1,227 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Character;
-
-use Exception;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class CharacterEditCommand extends CharacterBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("edit"))
-            ->setDescription("Edit a given character.")
-            ->setDefinition(
-                new InputDefinition([
-                    $this->getCharacterIdArgumentDefinition(),
-                    new InputOption(
-                        "level",
-                        mode: InputOption::VALUE_REQUIRED,
-                        description: "Changes the level"
-                    ),
-                    new InputOption(
-                        "maxHealth",
-                        mode: InputOption::VALUE_REQUIRED,
-                        description: "Change maximum amount of health points. Health will be adjusted accordingly."
-                    ),
-                    new InputOption(
-                        "heal",
-                        mode: InputOption::VALUE_NONE,
-                        description: "Restores full health"
-                    ),
-                    new InputOption(
-                        "revive",
-                        mode: InputOption::VALUE_OPTIONAL,
-                        description: "Revives at full health. Give a number between 0 and 1 to revive fractionally.",
-                        default: false,
-                    ),
-                    new InputOption(
-                        "kill",
-                        mode: InputOption::VALUE_NONE,
-                        description: "Kills"
-                    ),
-                ])
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $logger = $this->getCliLogger();
-        $io = new SymfonyStyle($input, $output);
-
-        $id = $input->getArgument("id");
-        $level = $input->getOption("level");
-        $heal = $input->getOption("heal");
-        $revive = $input->getOption("revive");
-        $kill = $input->getOption("kill");
-        $maxHealth = $input->getOption("maxHealth");
-
-        $changed = false;
-
-        // Find character
-        $character = $this->getCharacter($id);
-
-        if (!$character) {
-            $io->error("The character with the id {$id} was not found.");
-            return Command::FAILURE;
-        }
-
-        // Change level
-        if ($level !== null) {
-            $level = intval($level);
-
-            if ($level <= 0) {
-                $io->error("Cannot set the level below 1.");
-                return Command::FAILURE;
-            }
-
-            // Only change level if necessary
-            if ($character->getLevel() !== $level) {
-                // Log
-                $logger->info("Character level changed", [
-                    "for" => $character,
-                    "from" => $character->getLevel(),
-                    "to" => $level
-                ]);
-
-                // Change
-                $character->setLevel($level);
-                $changed = true;
-            }
-        }
-
-        // Heal
-        if ($heal) {
-            if ($character->getHealth() >= $character->getMaxHealth()) {
-                $io->note("Character is already at full health.");
-            } elseif ($character->isAlive()) {
-                $oldHealth = $character->getHealth();
-
-                // Log
-                $logger->info("Character health changed", [
-                    "for" => $character,
-                    "from" => $oldHealth,
-                    "to" => $character->getMaxHealth()
-                ]);
-
-                // Change
-                $character->setHealth($character->getMaxHealth());
-                $io->success("Character was restored to full health ({$oldHealth} to {$character->getMaxHealth()}).");
-                $changed = true;
-            } else {
-                $io->error("Cannot heal a dead character. Use --revive instead.");
-                return Command::FAILURE;
-            }
-        }
-
-        // Revive the character
-        if ($revive !== false) {
-            // Make sure we revive between 0 and 1
-            if ($revive === null) {
-                $revive = 1;
-            } elseif (str_ends_with($revive, "%")) {
-                $revive = min(floatval($revive)/100, 1);
-            } else {
-                $revive = min(floatval($revive), 1);
-            }
-
-            if ($character->isAlive()) {
-                $io->error("Character is already alive. Use --heal instead.");
-                return Command::FAILURE;
-            } else {
-                // Make sure we heal at least by 1.
-                $reviveAmount = (int)round(max($revive * $character->getMaxHealth(), 1), 0);
-
-                // Log
-                $logger->info("Character was revived", [
-                    "for" => $character,
-                    "to" => $reviveAmount
-                ]);
-
-                // Change
-                $character->setHealth($reviveAmount);
-                $io->success("Character was revived with {$reviveAmount} of health points "
-                    ."(max: {$character->getMaxHealth()}).");
-                $changed = true;
-            }
-        }
-
-        if ($kill) {
-            if (!$character->isAlive()) {
-                $io->error("What is dead may never die.");
-                return Command::FAILURE;
-            } else {
-                // Log
-                $logger->info("Character was killed", ["for" => $character]);
-
-                // Change
-                $character->setHealth(0);
-                $io->success("Character was killed.");
-                $changed = true;
-            }
-        }
-
-        if ($maxHealth) {
-            $maxHealth = intval($maxHealth);
-            if ($maxHealth < 0) {
-                $io->error("Cannot set maximum health below 0.");
-                return Command::FAILURE;
-            }
-
-            if ($character->getMaxHealth() === 0) {
-                $healthProportion = 0;
-            } else {
-                $healthProportion = $character->getHealth() / $character->getMaxHealth();
-            }
-
-            // Log
-            $logger->info("Character maxHealth changed", [
-                "for" => $character,
-                "from" => $character->getMaxHealth(),
-                "to" => $maxHealth
-            ]);
-
-            // Change
-            $character->setMaxHealth($maxHealth);
-            $character->setHealth((int)round($healthProportion*$maxHealth, 0));
-            $io->success("Character has new maximum health of {$maxHealth} (current health is {$character->getHealth()}).");
-            $changed = true;
-        }
-
-        // Save changes
-        if ($changed) {
-            try {
-                $em->flush();
-                $io->success("The character was successfully changed.");
-
-                // Log
-                $logger->info("Changed committed.");
-            } catch (Exception $e) {
-                $io->error("Character could not be saved. Reason: {$e}");
-                $logger->error("Changes rolled back.", ["exception" => $e]);
-                return Command::FAILURE;
-            }
-        } else {
-            $io->note("Nothing was changed.");
-        }
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Character/CharacterListCommand.php

@@ -1,81 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Character;
-
-use LotGD\Core\Models\Character;
-use LotGD\Core\Models\Repositories\CharacterRepository;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Command to list all characters.
- */
-class CharacterListCommand extends CharacterBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("list"))
-            ->setDescription('Lists all characters')
-            ->setDefinition(
-                new InputDefinition([
-                    new InputOption(
-                        "includeSoftDeleted",
-                        mode: InputOption::VALUE_NONE,
-                        description: "Includes soft-deleted characters",
-                    ),
-                    new InputOption(
-                        "onlySoftDeleted",
-                        mode: InputOption::VALUE_NONE,
-                        description: "Displays only soft-deleted characters",
-                    ),
-                ])
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $io = new SymfonyStyle($input, $output);
-
-        /** @var CharacterRepository $repository */
-        $repository = $em->getRepository(Character::class);
-
-        $marker = "";
-        if ($input->getOption("includeSoftDeleted")) {
-            $io->writeln("* marks soft-deleted characters");
-
-            $marker = "*";
-            $characters = $repository->findAll(CharacterRepository::INCLUDE_SOFTDELETED);
-        } elseif ($input->getOption("onlySoftDeleted")) {
-            $io->writeln("Only soft-deleted characters are shown");
-            $characters = $repository->findAll(CharacterRepository::ONLY_SOFTDELETED);
-        } else {
-            $characters = $repository->findAll();
-        }
-
-        $table = [["id", "name", "level"], []];
-        foreach ($characters as $character) {
-            $table[1][] = [
-                $character->getId(),
-                $marker.$character->getName(),
-                $character->getLevel(),
-            ];
-        }
-
-        $io->table(...$table);
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Character/CharacterRemoveCommand.php

@@ -1,95 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Character;
-
-use Exception;
-use LotGD\Core\Models\Character;
-use LotGD\Core\Models\Repositories\CharacterRepository;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class CharacterRemoveCommand extends CharacterBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("remove"))
-            ->setDescription("Definitely removes a character (no soft delete).")
-            ->setDefinition(
-                new InputDefinition([
-                    $this->getCharacterIdArgumentDefinition(),
-                    new InputOption(
-                        name: "soft",
-                        mode: InputOption::VALUE_NONE,
-                        description: "Only removes the character softly (soft delete)."
-                    ),
-                ])
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        /** @var CharacterRepository $characterRepository */
-        $characterRepository = $em->getRepository(Character::class);
-        $logger = $this->getCliLogger();
-
-        $io = new SymfonyStyle($input, $output);
-
-        $id = $input->getArgument("id");
-
-        // Find character
-        /** @var Character $character */
-        $character = $characterRepository->findWithSoftDeleted($id);
-
-        if (!$character) {
-            $io->error("The character with the id {$id} was not found.");
-            return Command::FAILURE;
-        }
-
-        if ($character->isDeleted()) {
-            $io->info("Character was marked as soft-deleted.");
-        }
-
-        try {
-            if ($input->getOption("soft")) {
-                // Only soft-delete if requested
-                $character->delete($em);
-
-                // Commit changes
-                $em->flush();
-
-                $io->success("{$character} was successfully soft-deleted.");
-                $logger->info("Character was soft-deleted.", ["character" => $character]);
-            } else {
-                $em->remove($character);
-
-                // Commit changes
-                $em->flush();
-
-                $io->success("{$character} was successfully removed.");
-                $logger->info("Character was removed.", ["character" => $character]);
-            }
-        } catch (Exception $e) {
-            $io->error("Removing {$character} was not possible. Reason: {$e}.");
-            return Command::FAILURE;
-        }
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Character/CharacterResetViewpointCommand.php

@@ -1,73 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Character;
-
-use Exception;
-use LotGD\Core\Models\Character;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class CharacterResetViewpointCommand extends CharacterBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("resetViewpoint"))
-            ->setDescription("Resets the viewpoint of a given character.")
-            ->setDefinition(
-                new InputDefinition([
-                    $this->getCharacterIdArgumentDefinition(),
-                ]),
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $logger = $this->getCliLogger();
-
-        $io = new SymfonyStyle($input, $output);
-        $id = $input->getArgument("id");
-
-        /* @var $character Character */
-        $character = $em->getRepository(Character::class)->find($id);
-
-        if ($character === null) {
-            $io->error("Character not found.");
-            return Command::FAILURE;
-        }
-
-        if ($character->getViewpoint() === null) {
-            $io->info("Character does not have a viewpoint yet.");
-        } else {
-            try {
-                $em->remove($character->getViewpoint());
-                $character->setViewpoint(null);
-
-                $io->success("Viewpoint was successfully reset.");
-
-                # Save
-                $em->flush();
-            } catch (Exception $e) {
-                $io->error("Resetting the viewpoint was not possible. Reason: {$e}");
-                return Command::FAILURE;
-            }
-        }
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Character/CharacterShowCommand.php

@@ -1,113 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Character;
-
-use Exception;
-use LotGD\Core\Console\Command\BaseCommand;
-use LotGD\Core\Models\Character;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Helper\TableSeparator;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class CharacterShowCommand extends CharacterBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("show"))
-            ->setDescription("Shows details about character.")
-            ->setDefinition(
-                new InputDefinition([
-                    $this->getCharacterIdArgumentDefinition(),
-                    new InputOption(
-                        "onlyViewpoint",
-                        mode: InputOption::VALUE_NONE,
-                        description: "Set to only display viewpoint",
-                    )
-                ])
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $logger = $this->game->getLogger();
-
-        $io = new SymfonyStyle($input, $output);
-
-        $id = $input->getArgument("id");
-        $onlyViewpoint = $input->getOption("onlyViewpoint");
-
-        // Find character
-        /** @var Character $character */
-        $character = $em->getRepository(Character::class)->find($id);
-
-        if (!$character) {
-            $io->error("The character with the id {$id} was not found.");
-            return Command::FAILURE;
-        }
-
-        if (!$onlyViewpoint) {
-            $io->title("About Character {$character->getName()}");
-
-            $io->listing([
-                "ID: {$character->getId()}",
-                "Display name: {$character->getDisplayName()}",
-                "Level: {$character->getLevel()}",
-                "Health: {$character->getHealth()}/{$character->getMaxHealth()}",
-                "Alive: ".($character->isAlive()?"yes":"no"),
-                "Attack: {$character->getAttack()}",
-                "Defense: {$character->getDefense()}",
-            ]);
-
-            $io->section("Viewpoint");
-        } else {
-            $io->title("Viewpoint of {$character->getName()}");
-        }
-
-        $viewpoint = $character->getViewpoint();
-
-        if (!$viewpoint) {
-            $io->text("No viewpoint yet");
-        } else {
-            $io->text($viewpoint->getTitle() . "\n");
-            $io->text($viewpoint->getDescription());
-
-            $io->section("Viewpoint actions");
-            $actionGroups = $viewpoint->getActionGroups();
-
-            $rows = [];
-
-            foreach ($actionGroups as $actionGroup) {
-                $rows[] = [$actionGroup->getId(), $actionGroup->getTitle(), "", "", ""];
-
-                foreach ($actionGroup->getActions() as $action) {
-                    $rows[] = ["", "", $action->getId(), $action->getTitle(), $action->getDestinationSceneId()];
-                }
-
-                if (count($actionGroup->getActions())) {
-                    $rows[] = new TableSeparator();
-                }
-            }
-
-            $io->table(["Group id", "Group name", "Action id", "Action name", "Destination"], $rows);
-        }
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/ConsoleCommand.php

@@ -3,8 +3,6 @@ declare(strict_types=1);
 
 namespace LotGD\Core\Console\Command;
 
-use LotGD\Core\Game;
-use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
@@ -19,29 +17,26 @@ class ConsoleCommand extends BaseCommand
     protected function configure()
     {
         $this->setName('console')
-            ->setDescription('Start a shell to interact with the game')
-        ;
+             ->setDescription('Start a shell to interact with the game');
     }
 
     /**
      * @inheritDoc
      */
-    protected function execute(InputInterface $input, OutputInterface $output): int
+    protected function execute(InputInterface $input, OutputInterface $output)
     {
-        print "Daenerys console, the dragon prompt. lotgd/core " . Game::getVersion() . ".\n";
-        print "Enter some PHP, but be careful, this is live and attached to your currently configured setup:\n\n";
-        print $this->game->getConfiguration();
+        print("Daenerys console, the dragon prompt. lotgd/core " . \LotGD\Core\Game::getVersion() . ".\n");
+        print("Enter some PHP, but be careful, this is live and attached to your currently configured setup:\n\n");
+        print($this->game->getConfiguration());
 
-        print "\n";
-        print "Try things like `\$g::getVersion()`. To quit, ^D or `exit();`.\n";
-        print "\n";
+        print("\n");
+        print("Try things like `\$g::getVersion()`. To quit, ^D or `exit();`.\n");
+        print("\n");
 
         $boris = new \Boris\Boris('🐲 > ');
-        $boris->setLocal([
-            'g' => $this->game,
-        ]);
+        $boris->setLocal(array(
+            'g' => $this->game
+        ));
         $boris->start();
-
-        return Command::SUCCESS;
     }
 }

src/Console/Command/Database/DatabaseSchemaUpdateCommand.php

@@ -1,41 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Database;
-
-use LotGD\Core\Console\Command\BaseCommand;
-use Doctrine\ORM\Tools\SchemaTool;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-
-/**
- * Danerys command to initiate the database with default values.
- */
-class DatabaseSchemaUpdateCommand extends BaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName('database:schemaUpdate')
-            ->setDescription('Updates the database schema manually.')
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $entityManager = $this->game->getEntityManager();
-        $metaData = $entityManager->getMetadataFactory()->getAllMetadata();
-        $schemaTool = new SchemaTool($entityManager);
-        $schemaTool->updateSchema($metaData);
-
-        $entityManager->flush();
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/DatabaseInitCommand.php

@@ -1,13 +1,15 @@
 <?php
 declare(strict_types=1);
 
-namespace LotGD\Core\Console\Command\Database;
+namespace LotGD\Core\Console\Command;
 
-use LotGD\Core\Console\Command\BaseCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
+use LotGD\Core\Console\Main;
+use LotGD\Core\Game;
+
 /**
  * Danerys command to initiate the database with default values.
  */
@@ -19,17 +21,14 @@ class DatabaseInitCommand extends BaseCommand
     protected function configure()
     {
         $this->setName('database:init')
-            ->setDescription('Initiates database with default values.')
-        ;
+             ->setDescription('Initiates database with default values.');
     }
 
     /**
      * @inheritDoc
      */
-    protected function execute(InputInterface $input, OutputInterface $output): int
+    protected function execute(InputInterface $input, OutputInterface $output)
     {
         $this->game->getEntityManager()->flush();
-
-        return Command::SUCCESS;
     }
 }

src/Console/Command/Module/ModuleBaseCommand.php

@@ -1,37 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Module;
-
-use Doctrine\Persistence\ObjectRepository;
-use LotGD\Core\Console\Command\BaseCommand;
-use LotGD\Core\Models\Module;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputInterface;
-
-class ModuleBaseCommand extends BaseCommand
-{
-    protected ?string $namespace = "module";
-
-    /**
-     * @return InputArgument
-     */
-    protected function getModuleNameArgumentDefinition(): InputArgument
-    {
-        return new InputArgument(
-            name: "moduleName",
-            mode: InputArgument::REQUIRED,
-            description: "Name of the module, in vendor/package format",
-        );
-    }
-
-    protected function getModuleRepository(): ObjectRepository
-    {
-        return $this->game->getEntityManager()->getRepository(Module::class);
-    }
-
-    protected function getModuleModel(InputInterface $input): ?Module
-    {
-        return $this->game->getModuleManager()->getModule($input->getArgument("moduleName"));
-    }
-}

src/Console/Command/Module/ModuleConfigListCommand.php

@@ -1,59 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Module;
-
-use LotGD\Core\Events\EventContextData;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class ModuleConfigListCommand extends ModuleBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("config:list"))
-            ->setDescription('List available configuration option for a module')
-            ->setDefinition([
-                $this->getModuleNameArgumentDefinition(),
-            ])
-        ;
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output)
-    {
-        $io = new SymfonyStyle($input, $output);
-        $module = $this->getModuleModel($input);
-
-        if (!$module) {
-            $io->error("Module was not found.");
-            return Command::FAILURE;
-        }
-
-        // Create hook
-        $context = EventContextData::create([
-            "module" => $module,
-            "io" => $io,
-            "settings" => [],
-        ]);
-        $newContext = $this->game->getEventManager()->publish(
-            event: "h/lotgd/core/cli/module-config-list/".$module->getLibrary(),
-            contextData: $context
-        );
-        $settings = $newContext->get("settings");
-
-        $io->title("Module ".$module->getLibrary());
-
-        if (count($settings) === 0) {
-            $io->note("This module does not provide any settings.");
-        } else {
-            $io->table(["setting", "value", "description"], $settings);
-        }
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Module/ModuleConfigResetCommand.php

@@ -1,68 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Module;
-
-use LotGD\Core\Events\EventContextData;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class ModuleConfigResetCommand extends ModuleBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("config:reset"))
-            ->setDescription('Reset a module setting')
-            ->setDefinition([
-                $this->getModuleNameArgumentDefinition(),
-                new InputArgument(
-                    "setting",
-                    mode: InputArgument::REQUIRED,
-                    description: "Name of setting, see {$this->namespaced('config:list')}.",
-                ),
-            ])
-        ;
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output)
-    {
-        $logger = $this->getCliLogger();
-        $io = new SymfonyStyle($input, $output);
-        $module = $this->getModuleModel($input);
-
-        if (!$module) {
-            $io->error("Module was not found.");
-            return Command::FAILURE;
-        }
-
-        $io->title("Module {$module->getLibrary()}");
-
-        // Create hook
-        $context = EventContextData::create([
-            "module" => $module,
-            "io" => $io,
-            "setting" => $input->getArgument("setting"),
-            "return" => Command::FAILURE,
-            "reason" => "Setting does not exist.",
-        ]);
-
-        $newContext = $this->game->getEventManager()->publish(
-            event: "h/lotgd/core/cli/module-config-reset/{$module->getLibrary()}",
-            contextData: $context
-        );
-        if ($newContext->get("return") != Command::SUCCESS) {
-            $io->error($newContext->get("reason"));
-            return Command::FAILURE;
-        }
-
-        $this->game->getEntityManager()->flush();
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Module/ModuleConfigSetCommand.php

@@ -1,73 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Module;
-
-use LotGD\Core\Events\EventContextData;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class ModuleConfigSetCommand extends ModuleBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("config:set"))
-            ->setDescription('Change a module setting')
-            ->setDefinition([
-                $this->getModuleNameArgumentDefinition(),
-                new InputArgument(
-                    "setting",
-                    mode: InputArgument::REQUIRED,
-                    description: "Name of setting, see {$this->namespaced('config:list')}.",
-                ),
-                new InputArgument(
-                    "value",
-                    InputArgument::REQUIRED,
-                    description: "New value for the given setting.",
-                ),
-            ])
-        ;
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output)
-    {
-        $logger = $this->getCliLogger();
-        $io = new SymfonyStyle($input, $output);
-        $module = $this->getModuleModel($input);
-
-        if (!$module) {
-            $io->error("Module was not found.");
-            return Command::FAILURE;
-        }
-
-        $io->title("Module {$module->getLibrary()}");
-
-        // Create hook
-        $context = EventContextData::create([
-            "module" => $module,
-            "io" => $io,
-            "setting" => $input->getArgument("setting"),
-            "value" => $input->getArgument("value"),
-            "return" => Command::FAILURE,
-            "reason" => "Setting does not exist.",
-        ]);
-        $newContext = $this->game->getEventManager()->publish(
-            event: "h/lotgd/core/cli/module-config-set/{$module->getLibrary()}",
-            contextData: $context
-        );
-        if ($newContext->get("return") != Command::SUCCESS) {
-            $io->error($newContext->get("reason"));
-            return Command::FAILURE;
-        }
-
-        $this->game->getEntityManager()->flush();
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Module/ModuleListCommand.php

@@ -1,48 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Module;
-
-use LotGD\Core\Models\Module;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class ModuleListCommand extends ModuleBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("list"))
-            ->setDescription('List all installed modules.')
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $io = new SymfonyStyle($input, $output);
-        /** @var Module[] $modules */
-        $modules = $this->game->getModuleManager()->getModules();
-
-        $io->title("Installed modules");
-
-        if (count($modules) > 0) {
-            $listing = [];
-            foreach ($modules as $module) {
-                $listing[] = [$module->getLibrary() => $module->getCreatedAt()->format("d M Y, H:i")];
-            }
-
-            $io->definitionList(...$listing);
-        } else {
-            $io->note("No modules installed.");
-        }
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Module/ModuleRegisterCommand.php

@@ -1,125 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Module;
-
-use Exception;
-use LotGD\Core\Console\Command\BaseCommand;
-use LotGD\Core\Exceptions\ClassNotFoundException;
-use LotGD\Core\Exceptions\InvalidConfigurationException;
-use LotGD\Core\Exceptions\ModuleAlreadyExistsException;
-use LotGD\Core\Exceptions\WrongTypeException;
-use LotGD\Core\LibraryConfiguration;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Danerys command to register and initiate any newly installed modules.
- */
-class ModuleRegisterCommand extends ModuleBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("register"))
-            ->setDescription('Register and initialize any newly installed modules')
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $io = new SymfonyStyle($input, $output);
-        $modules = $this->game->getComposerManager()->getModulePackages();
-
-        $globalFlawless = true;
-        $registered = [];
-        foreach ($modules as $p) {
-            $flawless = $this->registerModule($p->getName(), $io, $registered);
-
-            $globalFlawless &= $flawless;
-        }
-
-        if (!$globalFlawless) {
-            $io->warning("Some module were not registered properly.");
-            return Command::FAILURE;
-        }
-
-        return Command::SUCCESS;
-    }
-
-    /**
-     * Register a given package as a module if it is of type lotdg-module. Resolves dependencies and skips already registered packages.
-     * @param string $packageName
-     * @param SymfonyStyle $io
-     * @param array $registered
-     * @return bool True if registering was flawless
-     * @throws InvalidConfigurationException
-     * @throws WrongTypeException
-     * @throws Exception
-     */
-    protected function registerModule(
-        string $packageName,
-        SymfonyStyle $io,
-        array &$registered
-    ): bool {
-        $composerRepository = $this->game->getComposerManager()->getComposer()
-            ->getRepositoryManager()->getLocalRepository();
-
-        $package = $composerRepository->findPackage($packageName, "*");
-
-        # Skip if not a lotgd-module
-        if ($package->getType() !== "lotgd-module") {
-            return true;
-        }
-
-        # Skip if already registered
-        if (!empty($registered[$packageName])) {
-            return true;
-        }
-
-        $io->text("Reading module {$packageName} {$package->getPrettyVersion()}");
-
-        # Try to load module configuration ($moduleRoot/lotgd.yml)
-        try {
-            $library = new LibraryConfiguration($this->game->getComposerManager(), $package, $this->game->getCWD());
-        } catch (InvalidConfigurationException) {
-            $io->error("\tModule {$packageName} does not have a valid lotgd.yml in its root.");
-            return false;
-        }
-
-        # Register dependencies first.
-        $dependencyFlawless = true;
-        $dependencies = $package->getRequires();
-        foreach ($dependencies as $dependency) {
-            $dependencyFlawless &= $this->registerModule($dependency->getTarget(), $io, $registered);
-        }
-
-        # If $dependencyFlawless is not true anymore (as true & false == 0), we should abort as a dependency was not met.
-        if (!$dependencyFlawless) {
-            $io->warning("\t{$packageName} was not completely installed, as one of its dependencies had an "
-                ."error during registration.");
-            return false;
-        }
-
-        try {
-            $this->game->getModuleManager()->register($library);
-            $io->success("\tRegistered new module {$packageName}");
-        } catch (ModuleAlreadyExistsException $e) {
-            $io->note("\tSkipping already registered module {$packageName}");
-        } catch (ClassNotFoundException $e) {
-            $io->error("\tError installing module {$packageName}: {$e->getMessage()}");
-            return false;
-        }
-
-        $registered[$packageName] = true;
-
-        return true;
-    }
-}

src/Console/Command/ModuleRegisterCommand.php

@@ -0,0 +1,49 @@
+<?php
+declare(strict_types=1);
+
+namespace LotGD\Core\Console\Command;
+
+use Symfony\Component\Console\Input\InputInterface;
+use Symfony\Component\Console\Output\OutputInterface;
+
+use LotGD\Core\Exceptions\ClassNotFoundException;
+use LotGD\Core\Exceptions\ModuleAlreadyExistsException;
+use LotGD\Core\LibraryConfiguration;
+
+/**
+ * Danerys command to register and initiate any newly installed modules.
+ */
+class ModuleRegisterCommand extends BaseCommand
+{
+    /**
+     * @inheritDoc
+     */
+    protected function configure()
+    {
+        $this->setName('module:register')
+             ->setDescription('Register and initialize any newly installed modules');
+    }
+
+    /**
+     * @inheritDoc
+     */
+    protected function execute(InputInterface $input, OutputInterface $output)
+    {
+        $modules = $this->game->getComposerManager()->getModulePackages();
+
+        foreach ($modules as $p) {
+            $library = new LibraryConfiguration($this->game->getComposerManager(), $p, $this->game->getCWD());
+            $name = $library->getName();
+
+            try {
+                $this->game->getModuleManager()->register($library);
+
+                $output->writeln("<info>Registered new module {$name}</info>");
+            } catch (ModuleAlreadyExistsException $e) {
+                $output->writeln("Skipping already registered module {$name}");
+            } catch (ClassNotFoundException $e) {
+                $output->writeln("<error>Error installing module {$name}: " . $e->getMessage() . "</error>");
+            }
+        }
+    }
+}

src/Console/Command/ModuleValidateCommand.php

@@ -1,44 +1,40 @@
 <?php
 declare(strict_types=1);
 
-namespace LotGD\Core\Console\Command\Module;
+namespace LotGD\Core\Console\Command;
 
-use LotGD\Core\Console\Command\BaseCommand;
-use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
 /**
  * Danerys command to validate installed modules.
  */
-class ModuleValidateCommand extends ModuleBaseCommand
+class ModuleValidateCommand extends BaseCommand
 {
     /**
      * @inheritDoc
      */
     protected function configure()
     {
-        $this->setName($this->namespaced("validate"))
-            ->setDescription('Validate installed modules')
-        ;
+        $this->setName('module:validate')
+             ->setDescription('Validate installed modules');
     }
 
     /**
      * @inheritDoc
      */
-    protected function execute(InputInterface $input, OutputInterface $output): int
+    protected function execute(InputInterface $input, OutputInterface $output)
     {
         $results = $this->game->getModuleManager()->validate();
 
-        if (\count($results) > 0) {
+        if (count($results) > 0) {
             foreach ($results as $r) {
                 $output->writeln($r);
             }
-
-            return Command::FAILURE;
+            return 1;
+        } else {
+            $output->writeln("<info>LotGD modules validated</info>");
+            return 0;
         }
-        $output->writeln("<info>LotGD modules validated</info>");
-
-        return Command::SUCCESS;
     }
 }

src/Console/Command/Scene/SceneAddCommand.php

@@ -1,99 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use Exception;
-use LotGD\Core\Models\Scene;
-use LotGD\Core\Models\SceneTemplate;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class SceneAddCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("add"))
-            ->setDescription("Add a scene.")
-            ->setDefinition(
-                new InputDefinition([
-                    new InputArgument(
-                        "title",
-                        mode: InputArgument::REQUIRED,
-                        description: "Scene title",
-                    ),
-                    new InputArgument(
-                        "description",
-                        mode: InputArgument::OPTIONAL,
-                        description: "Scene description",
-                        default: "",
-                    ),
-                    new InputOption(
-                        "template",
-                        mode: InputOption::VALUE_OPTIONAL,
-                        description: "A valid, user-assignable scene template. Check sceneTemplate:list to get all available scenes.",
-                        default: null,
-                    )
-                ])
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $logger = $this->getCliLogger();
-
-        $io = new SymfonyStyle($input, $output);
-
-        $title = $input->getArgument("title");
-        $description = $input->getArgument("description");
-        $templateClass = $input->getOption("template");
-
-        /* @var $template SceneTemplate */
-        if ($templateClass) {
-            $template = $em->getRepository(SceneTemplate::class)->find($templateClass);
-
-            if (!$template) {
-                $io->warning("Template '$template' has not been found. Set to NULL instead.");
-            }
-        } else {
-            $template = $templateClass;
-        }
-
-        $scene = new Scene(
-            title: $title,
-            description: $description,
-            template: $template
-        );
-
-        try {
-            $em->persist($scene);
-
-            // Commit changes
-            $em->flush();
-        } catch (Exception $e) {
-            $io->error("Persisting of the scene was not possible. Reason: {$e->getMessage()}.");
-            return Command::FAILURE;
-        }
-
-        $io->success("Scene was successfully created. ID: {$scene->getId()}.");
-        $logger->info("{$scene} was created.");
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Scene/SceneAddConnectionGroupCommand.php

@@ -1,85 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use Exception;
-use LotGD\Core\Exceptions\ArgumentException;
-use LotGD\Core\Models\Scene;
-use LotGD\Core\Models\SceneConnectionGroup;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class SceneAddConnectionGroupCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced('addConnectionGroup'))
-            ->setDescription("Add a connection group to an existing scene.")
-            ->setDefinition(
-                new InputDefinition([
-                    $this->getSceneIdArgumentDefinition(),
-                    new InputArgument("groupName", InputArgument::REQUIRED, "Internal id of the group."),
-                    new InputArgument("groupTitle", InputArgument::REQUIRED, "Title of the group (what the character can see"),
-                ]),
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $logger = $this->getCliLogger();
-
-        $io = new SymfonyStyle($input, $output);
-
-        $sceneId = $input->getArgument("id");
-        $groupName = $input->getArgument("groupName");
-        $groupTitle = $input->getArgument("groupTitle");
-
-        // Search scene
-        /** @var ?Scene $scene */
-        $scene = $em->getRepository(Scene::class)->find($sceneId);
-
-        if (!$scene) {
-            $io->error("The requested scene with the ID {$sceneId} was not found.");
-            return Command::FAILURE;
-        }
-
-        // Make scene connection group
-        $connectionGroup = new SceneConnectionGroup($groupName, $groupTitle);
-
-        // Add
-        try {
-            $scene->addConnectionGroup($connectionGroup);
-
-            // Commit changes
-            $em->flush();
-        } catch(ArgumentException $e) {
-            // Catches the error if a group already exists.
-            $io->error($e->getMessage());
-            return Command::FAILURE;
-        } catch (Exception $e) {
-            $io->error("An unknown error occurred: {$e->getMessage()}");
-            return Command::FAILURE;
-        }
-
-        $io->success("{$connectionGroup} successfully added.");
-        $logger->info("{$connectionGroup} was added to {$scene}.");
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Scene/SceneBaseCommand.php

@@ -1,53 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use LotGD\Core\Console\Command\BaseCommand;
-use LotGD\Core\Models\Scene;
-use LotGD\Core\SceneTemplates\SceneTemplateInterface;
-use Symfony\Component\Console\Input\InputArgument;
-
-class SceneBaseCommand extends BaseCommand
-{
-    protected ?string $namespace = "scene";
-
-    /**
-     * @return InputArgument
-     */
-    protected function getSceneIdArgumentDefinition(): InputArgument
-    {
-        return new InputArgument(
-            name: "id",
-            mode: InputArgument::REQUIRED,
-            description: "Scene ID",
-        );
-    }
-
-    /**
-     * @param string $id
-     * @return Scene|null
-     */
-    protected function getScene(string $id): ?Scene
-    {
-        /** @var Scene|null $scene */
-        $scene = $this->game->getEntityManager()->getRepository(Scene::class)->find($id);
-        return $scene;
-    }
-
-    /**
-     * @param Scene $scene
-     * @return string
-     */
-    protected function getSceneTemplatePath(Scene $scene)
-    {
-        $sceneTemplate = "no-template";
-        if ($scene->getTemplate()) {
-            /** @var SceneTemplateInterface $templateClass */
-            $templateClass = $scene->getTemplate()->getClass();
-            $sceneTemplate = $templateClass::getNavigationEvent();
-        }
-
-        return $sceneTemplate;
-    }
-}

src/Console/Command/Scene/SceneConfigListCommand.php

@@ -1,62 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use LotGD\Core\Events\EventContextData;
-use LotGD\Core\SceneTemplates\SceneTemplateInterface;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class SceneConfigListCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("config:list"))
-            ->setDescription('List available settings for a scene')
-            ->setDefinition([
-                $this->getSceneIdArgumentDefinition(),
-            ])
-        ;
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output)
-    {
-        $io = new SymfonyStyle($input, $output);
-        $scene = $this->getScene($input->getArgument("id"));
-
-        if (!$scene) {
-            $io->error("Scene was not found.");
-            return Command::FAILURE;
-        }
-
-        $sceneTemplate = $this->getSceneTemplatePath($scene);
-
-        // Create hook
-        $context = EventContextData::create([
-            "scene" => $scene,
-            "io" => $io,
-            "settings" => [],
-        ]);
-        $newContext = $this->game->getEventManager()->publish(
-            event: "h/lotgd/core/cli/scene-config-list/$sceneTemplate",
-            contextData: $context
-        );
-        $settings = $newContext->get("settings");
-
-        $io->title("Scene ".$scene->getTitle());
-
-        if (count($settings) === 0) {
-            $io->note("There are no scene settings available.");
-        } else {
-            $io->table(["setting", "value", "description"], $settings);
-        }
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Scene/SceneConfigResetCommand.php

@@ -1,71 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use LotGD\Core\Events\EventContextData;
-use LotGD\Core\SceneTemplates\SceneTemplateInterface;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class SceneConfigResetCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("config:reset"))
-            ->setDescription('Reset a scene setting')
-            ->setDefinition([
-                $this->getSceneIdArgumentDefinition(),
-                new InputArgument(
-                    "setting",
-                    mode: InputArgument::REQUIRED,
-                    description: "Name of setting, see {$this->namespaced('config:list')}.",
-                ),
-            ])
-        ;
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output)
-    {
-        $logger = $this->getCliLogger();
-        $io = new SymfonyStyle($input, $output);
-        $scene = $this->getScene($input->getArgument("id"));
-
-        if (!$scene) {
-            $io->error("Scene was not found.");
-            return Command::FAILURE;
-        }
-
-        $sceneTemplate = $this->getSceneTemplatePath($scene);
-
-        $io->title("Scene {$scene->getTitle()}");
-
-        // Create hook
-        $context = EventContextData::create([
-            "scene" => $scene,
-            "io" => $io,
-            "setting" => $input->getArgument("setting"),
-            "return" => Command::FAILURE,
-            "reason" => "Setting does not exist.",
-        ]);
-
-        $newContext = $this->game->getEventManager()->publish(
-            event: "h/lotgd/core/cli/character-config-reset/{$sceneTemplate}",
-            contextData: $context
-        );
-        if ($newContext->get("return") != Command::SUCCESS) {
-            $io->error($newContext->get("reason"));
-            return Command::FAILURE;
-        }
-
-        $this->game->getEntityManager()->flush();
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Scene/SceneConfigSetCommand.php

@@ -1,75 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use LotGD\Core\Events\EventContextData;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class SceneConfigSetCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("config:set"))
-            ->setDescription('Change a scene setting')
-            ->setDefinition([
-                $this->getSceneIdArgumentDefinition(),
-                new InputArgument(
-                    "setting",
-                    mode: InputArgument::REQUIRED,
-                    description: "Name of setting, see {$this->namespaced('config:list')}.",
-                ),
-                new InputArgument(
-                    "value",
-                    InputArgument::REQUIRED,
-                    description: "New value for the given setting.",
-                ),
-            ])
-        ;
-    }
-
-    protected function execute(InputInterface $input, OutputInterface $output)
-    {
-        $logger = $this->getCliLogger();
-        $io = new SymfonyStyle($input, $output);
-        $scene = $this->getScene($input->getArgument("id"));
-
-        if (!$scene) {
-            $io->error("Scene was not found.");
-            return Command::FAILURE;
-        }
-
-        $sceneTemplate = $this->getSceneTemplatePath($scene);
-
-        $io->title("Scene {$scene->getTitle()}");
-
-        // Create hook
-        $context = EventContextData::create([
-            "scene" => $scene,
-            "io" => $io,
-            "setting" => $input->getArgument("setting"),
-            "value" => $input->getArgument("value"),
-            "return" => Command::FAILURE,
-            "reason" => "Setting does not exist.",
-        ]);
-        $newContext = $this->game->getEventManager()->publish(
-            event: "h/lotgd/core/cli/character-config-set/{$sceneTemplate}",
-            contextData: $context
-        );
-        if ($newContext->get("return") != Command::SUCCESS) {
-            $io->error($newContext->get("reason"));
-            return Command::FAILURE;
-        }
-
-        $this->game->getEntityManager()->flush();
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Scene/SceneConnectCommand.php

@@ -1,155 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use Exception;
-use LotGD\Core\Exceptions\ArgumentException;
-use LotGD\Core\Models\Scene;
-use LotGD\Core\Models\SceneConnectable;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Input\InputOption;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class SceneConnectCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("connect"))
-            ->setDescription("Connects two scenes.")
-            ->setDefinition(
-                new InputDefinition([
-                    new InputArgument(
-                        "outgoing",
-                        mode: InputArgument::REQUIRED,
-                        description: "Outgoing scene ID",
-                    ),
-                    new InputArgument(
-                        "incoming",
-                        mode: InputArgument::REQUIRED,
-                        description: "Incoming scene ID",
-                    ),
-                    new InputOption(
-                        "outgoingGroupName",
-                        shortcut: "o",
-                        mode: InputOption::VALUE_OPTIONAL,
-                        description: "A valid, user-assignable scene template. Check sceneTemplate:list to get all available scenes.",
-                        default: null,
-                    ),
-                    new InputOption(
-                        "incomingGroupName",
-                        shortcut: "i",
-                        mode: InputOption::VALUE_OPTIONAL,
-                        description: "A valid, user-assignable scene template. Check sceneTemplate:list to get all available scenes.",
-                        default: null,
-                    ),
-                    new InputOption(
-                        "directionality",
-                        shortcut: "d",
-                        mode: InputOption::VALUE_OPTIONAL,
-                        description: "0 for bidirectional, 1 for unidirectional (outgoing->incoming)",
-                        default: 0,
-                    )
-                ])
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $logger = $this->getCliLogger();
-        $sceneRepository = $em->getRepository(Scene::class);
-
-        $io = new SymfonyStyle($input, $output);
-
-        /** @var ?Scene $outgoingScene */
-        $outgoingScene = $sceneRepository->find($input->getArgument("outgoing"));
-        /** @var ?Scene $incomingScene */
-        $incomingScene = $sceneRepository->find($input->getArgument("incoming"));
-
-        // Check of scenes actually exist
-        if (!$outgoingScene) {
-            $io->error("The outgoing scene was not found.");
-            return Command::FAILURE;
-        }
-
-        if (!$incomingScene) {
-            $io->error("The incoming scene was not found");
-            return Command::FAILURE;
-        }
-
-        // Get group names
-        $outgoingGroupName = $input->getOption("outgoingGroupName");
-        $incomingGroupName = $input->getOption("incomingGroupName");
-
-        /** @var SceneConnectable $outgoing */
-        $outgoing = null;
-        /** @var SceneConnectable $outgoing */
-        $incoming = null;
-
-        // Determine the outgoing Connectable
-        if ($outgoingGroupName) {
-            if (!$outgoingScene->hasConnectionGroup($outgoingGroupName)) {
-                $io->error("The outgoing scene does not have a connection group with the id {$outgoingGroupName}");
-                return Command::FAILURE;
-            } else {
-                $outgoing = $outgoingScene->getConnectionGroup($outgoingGroupName);
-            }
-        } else {
-            $outgoing = $outgoingScene;
-        }
-
-        // Determine the incoming Connectable
-        if ($incomingGroupName) {
-            if (!$incomingScene->hasConnectionGroup($incomingGroupName)) {
-                $io->error("The incoming scene does not have a connection group with the id {$incomingGroupName}");
-                return Command::FAILURE;
-            } else {
-                $incoming = $incomingScene->getConnectionGroup($incomingGroupName);
-            }
-        } else {
-            $incoming = $incomingScene;
-        }
-
-        // Get directionality
-        $directionality = intval($input->getOption("directionality"));
-
-        if ($directionality < 0 or $directionality > 1) {
-            $io->warning("Directionality was not either 0 or 1. It was forced to 0.");
-            $directionality = 0;
-        }
-
-        // Connect the connectables
-        try {
-            $outgoing->connect($incoming, $directionality);
-
-            // Commit changes
-            $em->flush();
-        } catch (ArgumentException $e) {
-            $io->error("Scenes were not connected. Reason: {$e->getMessage()}.");
-            return Command::FAILURE;
-        } catch (Exception $e) {
-            $io->error("An unknown error occurred: {$e}");
-            return Command::FAILURE;
-        }
-
-        $io->success("The two scenes were successfully connected.");
-        $logger->info("Connected {$outgoingScene} to {$incomingScene}.");
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Scene/SceneDisconnectCommand.php

@@ -1,91 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use Exception;
-use LotGD\Core\Models\Scene;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class SceneDisconnectCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("disconnect"))
-            ->setDescription("Disconnects two scenes.")
-            ->setDefinition(
-                new InputDefinition([
-                    new InputArgument(
-                        "scene1",
-                        mode: InputArgument::REQUIRED,
-                        description: "One scene ID",
-                    ),
-                    new InputArgument(
-                        "scene2",
-                        mode: InputArgument::REQUIRED,
-                        description: "The other scene ID",
-                    ),
-                ])
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $logger = $this->getCliLogger();
-        $sceneRepository = $em->getRepository(Scene::class);
-
-        $io = new SymfonyStyle($input, $output);
-
-        /** @var Scene $scene1 */
-        $scene1 = $sceneRepository->find($input->getArgument("scene1"));
-        /** @var Scene $scene2 */
-        $scene2 = $sceneRepository->find($input->getArgument("scene2"));
-
-        if (!$scene1) {
-            $io->error("Scene with id {$input->getArgument('scene1')} was not found.");
-            return Command::FAILURE;
-        }
-
-        if (!$scene2) {
-            $io->error("Scene with id {$input->getArgument('scene2')} was not found.");
-            return Command::FAILURE;
-        }
-
-        $connection = $scene1->getConnectionTo($scene2);
-
-        if (!$connection) {
-            $io->error("The given scenes do not share a connection.");
-            return Command::FAILURE;
-        }
-
-        try {
-            // Commit changes
-            $em->remove($connection);
-            $em->flush();
-        } catch (Exception $e) {
-            $io->error("An unknown error occurred: {$e->getMessage()}");
-            return Command::FAILURE;
-        }
-
-        $io->success("The connections between the two given scenes was removed.");
-        $logger->info("Disconnected {$connection->getOutgoingScene()} and {$connection->getIncomingScene()}.");
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Scene/SceneListCommand.php

@@ -1,50 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use LotGD\Core\Console\Command\BaseCommand;
-use LotGD\Core\Models\Scene;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class SceneListCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("list"))
-            ->setDescription("Lists all scenes")
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $io = new SymfonyStyle($input, $output);
-
-        /** @var Scene[] $scenes */
-        $scenes = $em->getRepository(Scene::class)->findAll();
-
-        $table = [["id", "title", "connections", "template"], []];
-        foreach ($scenes as $scene) {
-            $table[1][] = [
-                $scene->getId(),
-                $scene->getTitle(),
-                count($scene->getConnectedScenes()),
-                $scene->getTemplate()?->getClass(),
-            ];
-        }
-
-        $io->table(...$table);
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Scene/SceneRemoveCommand.php

@@ -1,71 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use LotGD\Core\Models\Scene;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class SceneRemoveCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("remove"))
-            ->setDescription("Removes a scene.")
-            ->setDefinition(
-                new InputDefinition([
-                    $this->getSceneIdArgumentDefinition(),
-                ])
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-
-        $io = new SymfonyStyle($input, $output);
-
-        $sceneId = $input->getArgument("id");
-
-        // Get scene
-        /** @var Scene $scene */
-        $scene = $em->getRepository(Scene::class)->find($sceneId);
-
-        if (!$scene) {
-            $io->error("The scene with the ID {$sceneId} was not found.");
-            return Command::FAILURE;
-        }
-
-        if (!$scene->isRemovable()) {
-            $io->error("The scene with the ID {$sceneId} was marked as not removable. Please remove the responsible module instead.");
-            return Command::FAILURE;
-        }
-
-        // Mark for removal and flush
-        try {
-            $em->remove($scene);
-            $em->flush();
-        } catch (\Exception $e) {
-            $io->error("Removal of {$sceneId} was not possible: {$e->getMessage()}");
-            return Command::FAILURE;
-        }
-
-        $io->success("{$scene} was successfully removed.");
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Scene/SceneRemoveConnectionGroupCommand.php

@@ -1,95 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use LotGD\Core\Models\Scene;
-use LotGD\Core\Models\SceneConnection;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputArgument;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class SceneRemoveConnectionGroupCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("removeConnectionGroup"))
-            ->setDescription("Removes a connection group from an existing scene.")
-            ->setDefinition(
-                new InputDefinition([
-                    $this->getSceneIdArgumentDefinition(),
-                    new InputArgument("groupName", InputArgument::REQUIRED, "Internal id of the group."),
-                ]),
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $logger = $this->getCliLogger();
-
-        $io = new SymfonyStyle($input, $output);
-
-        $sceneId = $input->getArgument("id");
-        $groupName = $input->getArgument("groupName");
-
-        // Search scene
-        /** @var ?Scene $scene */
-        $scene = $em->getRepository(Scene::class)->find($sceneId);
-
-        if (!$scene) {
-            $io->error("The scene with the ID {$sceneId} was not found.");
-            return Command::FAILURE;
-        }
-
-        if (!$scene->hasConnectionGroup($groupName)) {
-            $io->error("The scene {$sceneId} does not have a connection group with the name {$groupName}");
-            return Command::FAILURE;
-        }
-
-        $connectionGroup = $scene->getConnectionGroup($groupName);
-
-        # Mark for removal
-        $em->remove($connectionGroup);
-
-        # Update outgoing connections if they refer to the deleted connectionGroup
-        $connections = $scene->getConnections();
-        /** @var SceneConnection $connection */
-        foreach ($connections as $connection) {
-            if ($connection->getIncomingScene() === $scene and $connection->getIncomingConnectionGroupName() === $groupName) {
-                $connection->setIncomingConnectionGroupName(null);
-                $io->comment("Updated connection to {$connection->getOutgoingScene()->getTitle()}");
-            }
-
-            if ($connection->getOutgoingScene() === $scene and $connection->getOutgoingConnectionGroupName() === $groupName) {
-                $connection->setOutgoingConnectionGroupName(null);
-                $io->comment("Updated connection to {$connection->getIncomingScene()->getTitle()}");
-            }
-        }
-
-        try {
-            $em->flush();
-        } catch (\Exception $e) {
-            $io->error("An unknown error occurred: {$e->getMessage()}");
-            return Command::FAILURE;
-        }
-
-        $io->success("{$connectionGroup} was successfully removed");
-        $logger->info("Removed {$connectionGroup} from {$scene}.");
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/Scene/SceneShowCommand.php

@@ -1,121 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\Scene;
-
-use LotGD\Core\Models\Scene;
-use LotGD\Core\Models\SceneConnectable;
-use LotGD\Core\Models\SceneConnection;
-use LotGD\Core\Models\SceneConnectionGroup;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputDefinition;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-/**
- * Resets the viewpoint of a given character.
- */
-class SceneShowCommand extends SceneBaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName($this->namespaced("show"))
-            ->setDescription("Show details about a specific scene.")
-            ->setDefinition(
-                new InputDefinition([
-                    $this->getSceneIdArgumentDefinition(),
-                ])
-            )
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-
-        $io = new SymfonyStyle($input, $output);
-        $id = $input->getArgument("id");
-
-        /* @var $scene Scene */
-        $scene = $em->getRepository(Scene::class)->find($id);
-
-        if ($scene === null) {
-            $io->error("Scene not found.");
-            return Command::FAILURE;
-        }
-
-        $io->title("About scene '{$scene->getTitle()}'");
-        $io->listing([
-            "ID: {$scene->getId()}",
-            "Title: {$scene->getTitle()}",
-            "Template: {$scene->getTemplate()?->getClass()}",
-        ]);
-
-        $io->text($scene->getDescription());
-
-        $io->section("Connection groups");
-
-        /** @var SceneConnectionGroup[] $connectionGroups */
-        $connectionGroups = $scene->getConnectionGroups();
-        $list = [];
-        foreach ($connectionGroups as $connectionGroup) {
-            $list[] = "{$connectionGroup->getTitle()} (id={$connectionGroup->getName()})";
-        }
-        $io->listing($list);
-
-        $io->section("Connected Scenes");
-
-        /** @var SceneConnection[] $connections */
-        $connections = $scene->getConnections();
-        $list = [];
-        foreach ($connections as $connection) {
-            # Get formatting for outgoing scene connection group name
-            $outgoingSceneConnectionGroup = $connection->getOutgoingConnectionGroupName();
-            if ($outgoingSceneConnectionGroup) {
-                $outgoingSceneConnectionGroup = " (on $outgoingSceneConnectionGroup)";
-            } else {
-                $outgoingSceneConnectionGroup = "";
-            }
-
-            # Get formatting for incoming scene connection group name
-            $incomingSceneConnectionGroup = $connection->getIncomingConnectionGroupName();
-            if ($incomingSceneConnectionGroup) {
-                $incomingSceneConnectionGroup = " (on $incomingSceneConnectionGroup)";
-            } else {
-                $incomingSceneConnectionGroup = " ";
-            }
-
-            # Treat outgoing and incoming connections slightly differently
-            if ($connection->getOutgoingScene() === $scene) {
-                $other = $connection->getIncomingScene();
-
-                # Check if the connection is bidirectional (only out (this)->in)
-                if ($connection->isDirectionality(SceneConnectable::Bidirectional)) {
-                    $list[] = "this$outgoingSceneConnectionGroup <=> {$other->getTitle()}$incomingSceneConnectionGroup(id={$other->getId()})";
-                } else {
-                    $list[] = "this$outgoingSceneConnectionGroup => {$other->getTitle()}$incomingSceneConnectionGroup(id={$other->getId()})";
-                }
-            } else {
-                $other = $connection->getOutgoingScene();
-
-                # Check if the connection is bidirectional (only out->in (this))
-                if ($connection->isDirectionality(SceneConnectable::Bidirectional)) {
-                    $list[] = "this$incomingSceneConnectionGroup <=> {$other->getTitle()}$outgoingSceneConnectionGroup (id={$other->getId()})";
-                } else {
-                    $list[] = "this$incomingSceneConnectionGroup <= {$other->getTitle()}$outgoingSceneConnectionGroup (id={$other->getId()})";
-                }
-            }
-        }
-
-        $io->listing($list);
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Command/SceneTemplates/SceneTemplateListCommand.php

@@ -1,51 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Console\Command\SceneTemplates;
-
-use LotGD\Core\Console\Command\BaseCommand;
-use LotGD\Core\Models\SceneTemplate;
-use Symfony\Component\Console\Command\Command;
-use Symfony\Component\Console\Input\InputInterface;
-use Symfony\Component\Console\Output\OutputInterface;
-use Symfony\Component\Console\Style\SymfonyStyle;
-
-class SceneTemplateListCommand extends BaseCommand
-{
-    /**
-     * @inheritDoc
-     */
-    protected function configure()
-    {
-        $this->setName('sceneTemplate:list')
-            ->setDescription('Lists all registered scene templates')
-        ;
-    }
-
-    /**
-     * @inheritDoc
-     */
-    protected function execute(InputInterface $input, OutputInterface $output): int
-    {
-        $em = $this->game->getEntityManager();
-        $io = new SymfonyStyle($input, $output);
-
-        /** @var SceneTemplate[] $templates */
-        $templates = $em->getRepository(SceneTemplate::class)->findAll();
-
-        $table = [["class", "module", "assignable", "# scenes", "# viewpoints"], []];
-        foreach ($templates as $template) {
-            $table[1][] = [
-                $template->getClass(),
-                $template->getModule(),
-                $template->isUserAssignable()?"X":"-",
-                count($template->getOwningScenes()),
-                count($template->getOwningViewpoints()),
-            ];
-        }
-
-        $io->table(...$table);
-
-        return Command::SUCCESS;
-    }
-}

src/Console/Main.php

@@ -3,49 +3,25 @@ declare(strict_types=1);
 
 namespace LotGD\Core\Console;
 
-use LotGD\Core\Bootstrap;
-
-use LotGD\Core\Console\Command\Character\CharacterAddCommand;
-use LotGD\Core\Console\Command\Character\CharacterConfigListCommand;
-use LotGD\Core\Console\Command\Character\CharacterConfigResetCommand;
-use LotGD\Core\Console\Command\Character\CharacterConfigSetCommand;
-use LotGD\Core\Console\Command\Character\CharacterEditCommand;
-use LotGD\Core\Console\Command\Character\CharacterListCommand;
-use LotGD\Core\Console\Command\Character\CharacterRemoveCommand;
-use LotGD\Core\Console\Command\Character\CharacterResetViewpointCommand;
-use LotGD\Core\Console\Command\Character\CharacterShowCommand;
-use LotGD\Core\Console\Command\ConsoleCommand;
-use LotGD\Core\Console\Command\Database\DatabaseInitCommand;
-use LotGD\Core\Console\Command\Database\DatabaseSchemaUpdateCommand;
-use LotGD\Core\Console\Command\Module\ModuleConfigListCommand;
-use LotGD\Core\Console\Command\Module\ModuleConfigResetCommand;
-use LotGD\Core\Console\Command\Module\ModuleConfigSetCommand;
-use LotGD\Core\Console\Command\Module\ModuleListCommand;
-use LotGD\Core\Console\Command\Module\ModuleRegisterCommand;
-use LotGD\Core\Console\Command\Module\ModuleValidateCommand;
-use LotGD\Core\Console\Command\Scene\SceneConfigListCommand;
-use LotGD\Core\Console\Command\Scene\SceneConfigResetCommand;
-use LotGD\Core\Console\Command\Scene\SceneConfigSetCommand;
-use LotGD\Core\Console\Command\SceneTemplates\SceneTemplateListCommand;
-use LotGD\Core\Console\Command\Scene\SceneAddCommand;
-use LotGD\Core\Console\Command\Scene\SceneAddConnectionGroupCommand;
-use LotGD\Core\Console\Command\Scene\SceneConnectCommand;
-use LotGD\Core\Console\Command\Scene\SceneDisconnectCommand;
-use LotGD\Core\Console\Command\Scene\SceneListCommand;
-use LotGD\Core\Console\Command\Scene\SceneRemoveCommand;
-use LotGD\Core\Console\Command\Scene\SceneRemoveConnectionGroupCommand;
-use LotGD\Core\Console\Command\Scene\SceneShowCommand;
-use LotGD\Core\Game;
 use Symfony\Component\Console\Application;
 
+use LotGD\Core\Bootstrap;
+use LotGD\Core\Game;
+use LotGD\Core\Console\Command\{
+    DatabaseInitCommand,
+    ModuleValidateCommand,
+    ModuleRegisterCommand,
+    ConsoleCommand
+};
+
 /**
  * Main execution class for the daenerys tool.
  */
 class Main
 {
-    private Application $application;
-    private Bootstrap $bootstrap;
-    private Game $game;
+    private $application;
+    private $bootstrap;
+    private $game;
 
     /**
      * Construct a new daenerys tool instance.
@@ -55,7 +31,7 @@ class Main
         $this->application = new Application();
 
         $this->application->setName("daenerys 🐲 ");
-        $this->application->setVersion("lotgd/core version " . Game::getVersion() . "");
+        $this->application->setVersion("0.0.1 (lotgd/core version " . \LotGD\Core\Game::getVersion() . ")");
     }
 
     /**
@@ -63,62 +39,23 @@ class Main
      */
     protected function addCommands()
     {
-        $this->application->add(new DatabaseInitCommand($this->game));
-        $this->application->add(new DatabaseSchemaUpdateCommand($this->game));
-
-        $this->application->add(new ConsoleCommand($this->game));
-
-        // Module commands
-        $this->application->add(new ModuleConfigListCommand($this->game));
-        $this->application->add(new ModuleConfigResetCommand($this->game));
-        $this->application->add(new ModuleConfigSetCommand($this->game));
-        $this->application->add(new ModuleListCommand($this->game));
-        $this->application->add(new ModuleRegisterCommand($this->game));
         $this->application->add(new ModuleValidateCommand($this->game));
-
-        // Character commands
-        $this->application->add(new CharacterAddCommand($this->game));
-        $this->application->add(new CharacterConfigListCommand($this->game));
-        $this->application->add(new CharacterConfigResetCommand($this->game));
-        $this->application->add(new CharacterConfigSetCommand($this->game));
-        $this->application->add(new CharacterEditCommand($this->game));
-        $this->application->add(new CharacterListCommand($this->game));
-        $this->application->add(new CharacterRemoveCommand($this->game));
-        $this->application->add(new CharacterResetViewpointCommand($this->game));
-        $this->application->add(new CharacterShowCommand($this->game));
-
-        // Scene commands
-        $this->application->add(new SceneAddCommand($this->game));
-        $this->application->add(new SceneConfigListCommand($this->game));
-        $this->application->add(new SceneConfigResetCommand($this->game));
-        $this->application->add(new SceneConfigSetCommand($this->game));
-        $this->application->add(new SceneListCommand($this->game));
-        $this->application->add(new SceneRemoveCommand($this->game));
-        $this->application->add(new SceneShowCommand($this->game));
-
-        // Scene connections
-        $this->application->add(new SceneConnectCommand($this->game));
-        $this->application->add(new SceneDisconnectCommand($this->game));
-
-        // Scene connection group
-        $this->application->add(new SceneAddConnectionGroupCommand($this->game));
-        $this->application->add(new SceneRemoveConnectionGroupCommand($this->game));
-
-        // Scene templates
-        $this->application->add(new SceneTemplateListCommand($this->game));
+        $this->application->add(new ModuleRegisterCommand($this->game));
+        $this->application->add(new DatabaseInitCommand($this->game));
+        $this->application->add(new ConsoleCommand($this->game));
 
         // Add additional ones
         $this->bootstrap->addDaenerysCommands($this->application);
     }
 
     /**
-     * Run the Daenerys tool.
+     * Run the danerys tool.
      */
     public function run()
     {
         // Bootstrap application
         $this->bootstrap = new Bootstrap();
-        $this->game = $this->bootstrap->getGame(\getcwd());
+        $this->game = $this->bootstrap->getGame(getcwd());
 
         // Add commands
         $this->addCommands();

src/DiceBag.php

@@ -28,7 +28,7 @@ class DiceBag
      */
     public function uniform(float $min, float $max): float
     {
-        return (\mt_rand(0, 100) / 100.0) * ($max - $min) + $min;
+        return (mt_rand(0, 100) / 100.0) * ($max - $min) + $min;
     }
 
     /**
@@ -49,7 +49,7 @@ class DiceBag
             $max = $a;
         }
 
-        return \mt_rand($min, $max);
+        return mt_rand($min, $max);
     }
 
     /**
@@ -71,9 +71,9 @@ class DiceBag
         $mean = ($max - $min) / 2;
         $r = 0;
         do {
-            $u1 = \mt_rand() / \mt_getrandmax();
-            $u2 = \mt_rand() / \mt_getrandmax();
-            $r = \sqrt(-2 * \log($u1)) * \cos(2 * \pi() * $u2) + $mean;
+            $u1 = mt_rand() / mt_getrandmax();
+            $u2 = mt_rand() / mt_getrandmax();
+            $r = sqrt(-2 * log($u1)) * cos(2 * pi() * $u2) + $mean;
         } while ($r < $min || $r > $max);
 
         return $r;
@@ -82,30 +82,31 @@ class DiceBag
     /**
      * This function has uniform distribution except for the extreme values, which are
      * half as likely to happen.
-     * The code for this function was taken from LotGD in version 0.9.7.
+     * The code for this function was taken from LotGD in version 0.9.7
      * @author MightyE, JT
-     * @param int|null $min
-     * @param int|null $max
+     * @param int $min
+     * @param int $max
      * @return int
      */
     public function pseudoBell(int $min = null, int $max = null): int
     {
-        if (\is_null($min)) {
-            return \mt_rand();
+        if (is_null($min)) {
+            return mt_rand();
         }
 
         $min *= 1000;
 
-        if (\is_null($max)) {
-            return (int)\round(\mt_rand($min) / 1000, 0);
+        if (is_null($max)) {
+            return (int)round(mt_rand($min)/1000, 0);
         }
         $max *= 1000;
 
         if ($min === $max) {
-            return (int)\round($min / 1000, 0);
+            return (int)round($min/1000, 0);
         } elseif ($min < $max) {
-            return (int)\round(\mt_rand($min, $max) / 1000, 0);
+            return (int)round(mt_rand($min, $max)/1000, 0);
+        } else {
+            return (int)round(mt_rand($max, $min)/1000, 0);
         }
-        return (int)\round(\mt_rand($max, $min) / 1000, 0);
     }
 }

src/Doctrine/Annotations/Extension.php

@@ -4,37 +4,38 @@ declare(strict_types=1);
 namespace LotGD\Core\Doctrine\Annotations;
 
 use Doctrine\Common\Annotations\Annotation;
-use Doctrine\Common\Annotations\Annotation\Attribute;
 use Doctrine\Common\Annotations\Annotation\Attributes;
+use Doctrine\Common\Annotations\Annotation\Attribute;
 use LotGD\Core\Exceptions\ArgumentException;
 use LotGD\Core\Models\ExtendableModelInterface;
 
 /**
  * Annotation that is used to flag which entity a class extends.
+ * @package LotGD\Core\Doctrine
  * @Annotation
  * @Target("CLASS")
  * @Attributes({
- *     @Attribute("of", type="string")
+ *  @Attribute("of", type = "string")
  * })
  */
 class Extension
 {
-    private string $modelClass;
+    /** @var string */
+    private $modelClass;
 
     /**
      * Extension constructor.
      * @param array $attributes
      * @throws ArgumentException
      */
-    public function __construct(array $attributes)
-    {
+    public function __construct(array $attributes) {
         $this->modelClass = $attributes["of"];
 
-        if (!\class_exists($this->modelClass)) {
+        if (!class_exists($this->modelClass)) {
             throw new ArgumentException("The class given in of must be a valid class.");
         }
 
-        if (!\in_array(ExtendableModelInterface::class, \class_implements($this->modelClass))) {
+        if (!in_array(ExtendableModelInterface::class, class_implements($this->modelClass))) {
             throw new ArgumentException("The class given in of must implement the ExtendableModelInterface.");
         }
     }
@@ -47,4 +48,4 @@ class Extension
     {
         return $this->modelClass;
     }
-}
+}

src/Doctrine/Annotations/ExtensionMethod.php

@@ -4,36 +4,37 @@ declare(strict_types=1);
 namespace LotGD\Core\Doctrine\Annotations;
 
 use Doctrine\Common\Annotations\Annotation;
-use Doctrine\Common\Annotations\Annotation\Attribute;
 use Doctrine\Common\Annotations\Annotation\Attributes;
+use Doctrine\Common\Annotations\Annotation\Attribute;
 use LotGD\Core\Exceptions\ArgumentException;
 
 /**
  * Annotation that is used to link a static method to a model entity.
+ * @package LotGD\Core\Doctrine\Annotations
  * @Annotation
  * @Target("METHOD")
  * @Attributes({
- *     @Attribute("as", type="string")
+ *  @Attribute("as", type = "string")
  * })
  */
 class ExtensionMethod
 {
-    private string $methodName = "";
+    /** @var string */
+    private $methodName = "";
 
     /**
      * ExtensionMethod constructor.
      * @param array $attributes
      * @throws ArgumentException
      */
-    public function __construct(array $attributes)
-    {
+    public function __construct(array $attributes) {
         $this->methodName = $attributes["as"];
 
-        if (!\is_string($this->methodName)) {
+        if (!is_string($this->methodName)) {
             throw new ArgumentException("Property 'as' must be a string.");
         }
 
-        if (\strlen($this->methodName) == 0) {
+        if (strlen($this->methodName) == 0) {
             throw new ArgumentException("Property 'as' must not be an empty string.");
         }
     }
@@ -46,4 +47,4 @@ class ExtensionMethod
     {
         return $this->methodName;
     }
-}
+}

src/Doctrine/EntityPostLoadEventListener.php

@@ -3,21 +3,28 @@ declare(strict_types=1);
 
 namespace LotGD\Core\Doctrine;
 
+use Doctrine\Common\Util\Debug;
 use Doctrine\ORM\Event\LifecycleEventArgs;
 use LotGD\Core\Game;
 use LotGD\Core\GameAwareInterface;
 
 /**
- * Class EntityPostLoadEventListener.
+ * Class EntityPostLoadEventListener
+ * @package LotGD\Core\Doctrine
  */
 class EntityPostLoadEventListener
 {
+    /** @var Game $game */
+    private $game;
 
     /**
      * EntityPostLoadEventListener constructor.
-     * @param Game $game
+     * @param Game $g
      */
-    public function __construct(private Game $game) {}
+    public function __construct(Game $g)
+    {
+        $this->game = $g;
+    }
 
     /**
      * Called upon event postLoad.
@@ -30,4 +37,4 @@ class EntityPostLoadEventListener
             $entity->setGame($this->game);
         }
     }
-}
+}

src/EventHandler.php

@@ -2,7 +2,6 @@
 declare(strict_types=1);
 
 namespace LotGD\Core;
-
 use LotGD\Core\Events\EventContext;
 
 interface EventHandler
@@ -11,8 +10,9 @@ interface EventHandler
      * Called when an event is published that is handled by this class.
      *
      * @param Game $g The game.
-     * @param EventContext $context Arbitrary dictionary representing context around this event.
-     * @return EventContext Return an array if you want to make changes to the $context before
+     * @param string $event Name of this event.
+     * @param array $context Arbitrary dictionary representing context around this event.
+     * @return array|null Return an array if you want to make changes to the $context before
      * the next handler is called. Otherwise, return null. Any changes made will be propogated
      * to the event publisher as well.
      */

src/EventManager.php

@@ -3,23 +3,31 @@ declare(strict_types=1);
 
 namespace LotGD\Core;
 
+use Doctrine\ORM\EntityManagerInterface;
+
 use LotGD\Core\Events\EventContext;
-use LotGD\Core\Events\EventContextData;
+use LotGD\Core\Models\EventSubscription;
+use LotGD\Core\EventHandler;
 use LotGD\Core\Exceptions\ClassNotFoundException;
 use LotGD\Core\Exceptions\SubscriptionNotFoundException;
 use LotGD\Core\Exceptions\WrongTypeException;
-use LotGD\Core\Models\EventSubscription;
+use LotGD\Core\Events\EventContextData;
 
 /**
  * Manages a simple publish/subscribe system based on regular expressions
- * matching event names and running a fixed.
+ * matching event names and running a fixed
  */
 class EventManager
 {
+    private $g;
+
     /**
      * @param Game $g The game.
      */
-    public function __construct(private Game $g) {}
+    public function __construct(Game $g)
+    {
+        $this->g = $g;
+    }
 
     /**
      * Publish an event. Will immediately cause handleEvent() to be called on all
@@ -38,13 +46,13 @@ class EventManager
         // TODO: Add an in-memory cache here. Will likely only be in the 1000s of
         // patterns, so no need to go to the remote key-value store.
 
-        $this->g->getLogger()->debug("Publishing event {$event}.");
+        $this->g->getLogger()->addDebug("Publishing event {$event}.");
 
         $subscriptions = $this->getSubscriptions();
         foreach ($subscriptions as $s) {
-            if (\preg_match($s->getPattern(), $event)) {
+            if (preg_match($s->getPattern(), $event)) {
                 $class = $s->getClass();
-                $this->g->getLogger()->debug("  Handling with {$class}.");
+                $this->g->getLogger()->addDebug("  Handling with {$class}.");
 
                 $eventContext = new EventContext($event, $s->getPattern(), $contextData);
 
@@ -74,7 +82,7 @@ class EventManager
      */
     public function subscribe(string $pattern, string $class, string $library)
     {
-        $this->g->getLogger()->debug("Subscribing event pattern={$pattern} class={$class} library={$library}");
+        $this->g->getLogger()->addDebug("Subscribing event pattern={$pattern} class={$class} library={$library}");
 
         try {
             // Can we resolve this class?
@@ -94,17 +102,16 @@ class EventManager
         }
 
         // Validate the regular expression.
-        if (@\preg_match($pattern, '') === false) {
+        if (@preg_match($pattern, '') === false) {
             throw new WrongTypeException("Invalid regular expression: {$pattern}");
         }
 
         $e = EventSubscription::create([
             'pattern' => $pattern,
             'class' => $class,
-            'library' => $library,
+            'library' => $library
         ]);
-
-        $this->g->getEntityManager()->persist($e);
+        $e->save($this->g->getEntityManager());
     }
 
     /**
@@ -114,16 +121,15 @@ class EventManager
      * @param string $pattern Regular expression, in PHP format, to match against
      * published event names.
      * @param string $class Fully qualified class name.
-     * @param string $library
      * @throws SubscriptionNotFoundException if the specified subscription does not exist.
      */
     public function unsubscribe(string $pattern, string $class, string $library)
     {
-        $e = $this->g->getEntityManager()->getRepository(EventSubscription::class)->find([
+        $e = $this->g->getEntityManager()->getRepository(EventSubscription::class)->find(array(
             'pattern' => $pattern,
             'class' => $class,
-            'library' => $library,
-        ]);
+            'library' => $library
+        ));
         if (!$e) {
             throw new SubscriptionNotFoundException("Subscription not found with pattern={$pattern} class={$class} library={$library}.");
         }

src/Events/CharacterEventData.php

@@ -1,28 +1,19 @@
 <?php
-declare(strict_types=1);
 
 namespace LotGD\Core\Events;
 
-use JetBrains\PhpStorm\ArrayShape;
+
+use LotGD\Core\Exceptions\ArgumentException;
 use LotGD\Core\Models\Character;
 
 /**
- * Class CharacterEventData.
+ * Class CharacterEventData
+ * @package LotGD\Core\Events
  */
 class CharacterEventData extends EventContextData
 {
-    #[ArrayShape([
-        "character" => [
-            "type" => Character::class,
-            "required" => "bool",
-        ],
-        "value" => [
-            "type" => "mixed",
-            "required" => "bool",
-        ],
-    ])]
-    protected static ?array $argumentConfig = [
+    protected static $argumentConfig = [
         "character" => ["type" => Character::class, "required" => true],
-        "value" => ["type" => "mixed", "required" => false],
+        "value" => ["type" => "mixed", "required" => false]
     ];
-}
+}

src/Events/EventContext.php

@@ -3,12 +3,18 @@ declare(strict_types=1);
 
 namespace LotGD\Core\Events;
 
+
 /**
- * Class EventContext.
+ * Class EventContext
+ * @package LotGD\Core
  * @immutable
  */
 class EventContext
 {
+    private $matchingPattern;
+    private $event;
+    private $data;
+
     /**
      * EventContext constructor.
      * @param string $event The published event
@@ -16,10 +22,13 @@ class EventContext
      * @param EventContextData $data
      */
     public function __construct(
-        private string $event,
-        private string $matchingPattern,
-        private EventContextData $data
+        string $event,
+        string $matchingPattern,
+        EventContextData $data
     ) {
+        $this->event = $event;
+        $this->matchingPattern = $matchingPattern;
+        $this->data = $data;
     }
 
     /**
@@ -60,7 +69,7 @@ class EventContext
     }
 
     /**
-     * Returns a data field.
+     * Returns a data field
      * @param $field
      * @return mixed
      */
@@ -70,7 +79,7 @@ class EventContext
     }
 
     /**
-     * Sets a data field.
+     * Sets a data field
      * @param $field
      * @param $value
      */
@@ -97,4 +106,4 @@ class EventContext
     {
         return $this->data === $originalData ? false : true;
     }
-}
+}

src/Events/EventContextData.php

@@ -9,18 +9,12 @@ use LotGD\Core\Exceptions\ArgumentException;
  * EventContextData to provide a basic structure for managing contextual data of an event.
  *
  * This class must be immutable and returns always a new instance of itself for any change.
+ * @package LotGD\Core\Events
  * @immutable
  */
 class EventContextData
 {
-    protected static ?array $argumentConfig = null;
-
-    /**
-     * protected constructor..
-     * @see self::create
-     * @param array $data
-     */
-    protected function __construct(private array $data) {}
+    private $data;
 
     /**
      * Creates a new instance of a data container.
@@ -40,53 +34,53 @@ class EventContextData
 
     /**
      * Checks a field configuration given in self::$argumentConfig.
-     * @param array $data
+     * @param $data
      * @throws ArgumentException
      */
-    public static function checkConfiguration(array $data)
+    public static function checkConfiguration($data)
     {
         $configuration = static::$argumentConfig;
         $types = [
-            "mixed" => function ($x) {
-                return true;
-            },
-            "int" => function ($x) {
-                return \is_int($x);
-            },
-            "float" => function ($x) {
-                return \is_float($x);
-            },
-            "numeric" => function ($x) {
-                return \is_numeric($x);
-            },
-            "string" => function ($x) {
-                return \is_string($x);
-            },
+            "mixed" => function ($x) {return true;},
+            "int" => function ($x) {return is_int($x);},
+            "float" => function ($x) {return is_float($x);},
+            "numeric" => function($x) {return is_numeric($x);},
+            "string" => function($x) {return is_string($x);},
         ];
 
-        $keys = \array_keys($data);
+        $keys = array_keys($data);
         foreach ($keys as $key) {
             if (!isset($configuration[$key])) {
-                throw new ArgumentException(\sprintf("%s does not accept a field called %s", static::class, $key));
+                throw new ArgumentException(sprintf("%s does not accept a field called %s", static::class, $key));
             }
         }
         foreach ($configuration as $key => $config) {
             if ($config["required"] === true and !isset($data[$key])) {
-                throw new ArgumentException(\sprintf("%s must have a field called %s.", static::class, $key));
+                throw new ArgumentException(sprintf("%s must have a field called %s.", static::class, $key));
             }
 
             if (isset($types[$config["type"]])) {
                 if ($types[$config["type"]]($data[$key]) === false) {
-                    throw new ArgumentException(\sprintf("The field %s of %s must be of type %s.", $key, static::class, $config["type"]));
+                    throw new ArgumentException(sprintf("The field %s of %s must be of type %s.", $key, static::class, $config["type"]));
                 }
             } else {
                 if (!$data[$key] instanceof $config["type"]) {
-                    throw new ArgumentException(\sprintf("The field %s of %s must be of type %s", $key, static::class, $config["type"]));
+                    throw new ArgumentException(sprintf("The field %s of %s must be of type %s", $key, static::class, $config["type"]));
                 }
             }
         }
     }
 
+    /**
+     * protected constructor..
+     * @see self::create
+     * @param array $data
+     */
+    protected function __construct(array $data)
+    {
+        $this->data = $data;
+    }
+
     /**
      * Returns true if container has a certain field.
      * @param string $field
@@ -94,7 +88,7 @@ class EventContextData
      */
     public function has(string $field): bool
     {
-        return \array_key_exists($field, $this->data);
+        return array_key_exists($field, $this->data);
     }
 
     /**
@@ -106,29 +100,31 @@ class EventContextData
     {
         if ($this->has($field)) {
             return $this->data[$field];
+        } else {
+            $this->throwException($field);
         }
-        $this->throwException($field);
     }
 
     /**
-     * Sets a field to a new value and returns a new data container.
+     * Sets a field to a new value and returns a new data container
      * @param string $field
-     * @param mixed $value
+     * @param $value
      * @return EventContextData
      */
-    public function set(string $field, mixed $value): self
+    public function set(string $field, $value): self
     {
         if ($this->has($field)) {
             $data = $this->data;
             $data[$field] = $value;
 
             return new static($data);
+        } else {
+            $this->throwException($field);
         }
-        $this->throwException($field);
     }
 
     /**
-     * Sets multiple fields at once.
+     * Sets multiple fields at once
      * @param array $data array of $field=>$value pairs
      * @return EventContextData
      */
@@ -148,12 +144,12 @@ class EventContextData
     }
 
     /**
-     * Returns a list of fields in this context.
+     * Returns a list of fields in this context
      * @return array
      */
     private function getListOfFields(): array
     {
-        return \array_keys($this->data);
+        return array_keys($this->data);
     }
 
     /**
@@ -162,8 +158,8 @@ class EventContextData
      */
     private function getFormattedListOfFields(): string
     {
-        return \substr(
-            \implode(", ", $this->getListOfFields()),
+        return substr(
+            implode(", ", $this->getListOfFields()),
             0,
             -2
         );

src/Events/NavigateToSceneData.php

@@ -3,11 +3,15 @@ declare(strict_types=1);
 
 namespace LotGD\Core\Events;
 
-use JetBrains\PhpStorm\ArrayShape;
+
+use Doctrine\Common\Util\Debug;
+use Doctrine\DBAL\Schema\View;
 use LotGD\Core\Exceptions\ArgumentException;
+use LotGD\Core\Models\Character;
 use LotGD\Core\Models\Scene;
 use LotGD\Core\Models\Viewpoint;
 
+
 /**
  * NavigateToScene data container which can be used for navigational events.
  *
@@ -17,6 +21,7 @@ use LotGD\Core\Models\Viewpoint;
  *  scene       Scene
  *  parameters  array
  *  redirect    Scene|null
+ * @package LotGD\Core\Events
  */
 class NavigateToSceneData extends EventContextData
 {
@@ -25,49 +30,40 @@ class NavigateToSceneData extends EventContextData
      * @param array $data Must contain fields referrer, viewpoint, scene, parameters and redirect; none more.
      * @throws ArgumentException
      */
-    protected function __construct(
-        #[ArrayShape([
-            "referrer" => Scene::class . "|null",
-            "viewpoint" => Viewpoint::class,
-            "scene" => Scene::class,
-            "parameters" => "array",
-            "reddirect" => Scene::class . "|null",
-        ])]
-        array $data,
-    ) {
+    protected function __construct(array $data)
+    {
         $mustHaveForm = ["referrer", "viewpoint", "scene", "parameters", "redirect"];
-        $doesHaveForm = \array_keys($data);
-        \sort($mustHaveForm);
-        \sort($doesHaveForm);
+        $doesHaveForm = array_keys($data);
+        sort($mustHaveForm); sort($doesHaveForm);
 
         if ($doesHaveForm !== $mustHaveForm) {
             throw new ArgumentException("A new NavigateToScene event must have referrer, viewpoint, scene, parameters and redirect.");
         }
 
         if ($data["referrer"] instanceof Scene === false and $data["referrer"] !== null) {
-            throw new ArgumentException(\sprintf(
+            throw new ArgumentException(sprintf(
                 "data[referrer] must be an instance of %s, %s given.",
                 Scene::class,
-                \get_class($data["referrer"])
+                get_class($data["referrer"])
             ));
         }
 
         if ($data["scene"] instanceof Scene === false) {
-            throw new ArgumentException(\sprintf(
+            throw new ArgumentException(sprintf(
                 "data[scene] must be an instance of %s, %s given.",
                 Scene::class,
-                \get_class($data["scene"])
+                get_class($data["scene"])
             ));
         }
 
         if ($data["viewpoint"] instanceof Viewpoint === false) {
-            throw new ArgumentException(\sprintf(
+            throw new ArgumentException(sprintf(
                 "data[viewpoint] must be an instance of %s, %s given.",
                 Viewpoint::class,
-                \get_class($data["viewpoint"])
+                get_class($data["viewpoint"])
             ));
         }
 
         parent::__construct($data);
     }
-}
+}

src/Events/NewViewpointData.php

@@ -3,7 +3,7 @@ declare(strict_types=1);
 
 namespace LotGD\Core\Events;
 
-use JetBrains\PhpStorm\ArrayShape;
+
 use LotGD\Core\Exceptions\ArgumentException;
 use LotGD\Core\Models\Character;
 use LotGD\Core\Models\Scene;
@@ -14,6 +14,7 @@ use LotGD\Core\Models\Scene;
  * Fields are:
  *  character   Character
  *  scene       Scene|null
+ * @package LotGD\Core\Events
  */
 class NewViewpointData extends EventContextData
 {
@@ -22,33 +23,28 @@ class NewViewpointData extends EventContextData
      * @param array $data
      * @throws ArgumentException In case $data contains invalid data.
      */
-    protected function __construct(
-        #[ArrayShape([
-            "character" => Character::class,
-            "scene" => Scene::class . "|null",
-        ])]
-        array $data,
-    ) {
-        if (\array_keys($data) !== ["character", "scene"]) {
+    protected function __construct(array $data)
+    {
+        if (array_keys($data) !== ["character", "scene"]) {
             throw new ArgumentException("A NewViewpoint event must have only character and scene.");
         }
 
         if (!$data["character"] instanceof Character) {
-            throw new ArgumentException(\sprintf(
+            throw new ArgumentException(sprintf(
                 "NewViewpoint data[character] must be an instance of %s, %s given.",
                 Character::class,
-                \get_class($data)
+                get_class($data)
             ));
         }
 
         if ($data["scene"] !== null and !$data["scene"] instanceof Scene) {
-            throw new ArgumentException(\sprintf(
+            throw new ArgumentException(sprintf(
                 "NewViewpoint data[scene] must be an instance of %s or null, %s given.",
                 Scene::class,
-                \get_class($data)
+                get_class($data)
             ));
         }
 
         parent::__construct($data);
     }
-}
+}

src/Events/ViewpointDecorationEventData.php

@@ -3,21 +3,16 @@ declare(strict_types=1);
 
 namespace LotGD\Core\Events;
 
-use JetBrains\PhpStorm\ArrayShape;
+use LotGD\Core\Exceptions\ArgumentException;
 use LotGD\Core\Models\Viewpoint;
 
 /**
- * Class ViewpointDecorationEventData.
+ * Class ViewpointDecorationEventData
+ * @package LotGD\Core\Events
  */
 class ViewpointDecorationEventData extends EventContextData
 {
-    #[ArrayShape([
-        "viewpoint" => [
-            "type" => Viewpoint::class,
-            "required" => "bool",
-        ],
-    ])]
-    protected static ?array $argumentConfig = [
-        "viewpoint" => ["type" => Viewpoint::class, "required" => true],
+    protected static $argumentConfig = [
+        "viewpoint" => ["type" => Viewpoint::class, "required" => true]
     ];
-}
+}

src/Exceptions/ActionNotFoundException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class ActionNotFoundException extends CoreException
 {
+
 }

src/Exceptions/ArgumentEmptyException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class ArgumentEmptyException extends ArgumentException
 {
+    
 }

src/Exceptions/ArgumentException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class ArgumentException extends CoreException
 {
+    
 }

src/Exceptions/AttributeMissingException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class AttributeMissingException extends CoreException
 {
+    
 }

src/Exceptions/BattleEventException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class BattleEventException extends BattleException
 {
+    
 }

src/Exceptions/BattleException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class BattleException extends CoreException
 {
+    
 }

src/Exceptions/BattleIsOverException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class BattleIsOverException extends BattleException
 {
+    
 }

src/Exceptions/BattleNotOverException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class BattleNotOverException extends BattleException
 {
+    
 }

src/Exceptions/BuffListAlreadyActivatedException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class BuffListAlreadyActivatedException extends CoreException
 {
+    
 }

src/Exceptions/BuffSlotOccupiedException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class BuffSlotOccupiedException extends CoreException
 {
+    
 }

src/Exceptions/BuilderException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a builder is missing an argument.
+ * Exception if a builder is missing an argument
  */
 class BuilderException extends CoreException
 {
+    
 }

src/Exceptions/CharacterNotFoundException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class CharacterNotFoundException extends CoreException
 {
+
 }

src/Exceptions/CharacterStatException.php

@@ -1,11 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Exceptions;
-
-/**
- * Class CharacterStatException.
- */
-class CharacterStatException extends CoreException
-{
-}

src/Exceptions/CharacterStatExistsException.php

@@ -1,11 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Exceptions;
-
-/**
- * Class CharacterStatExistsException.
- */
-class CharacterStatExistsException extends CharacterStatException
-{
-}

src/Exceptions/CharacterStatGroupExistsException.php

@@ -1,11 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Exceptions;
-
-/**
- * Class CharacterStatGroupExistsException.
- */
-class CharacterStatGroupExistsException extends CharacterStatException
-{
-}

src/Exceptions/CharacterStatGroupNotFoundException.php

@@ -1,11 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Exceptions;
-
-/**
- * Class CharacterStatGroupNotFoundException.
- */
-class CharacterStatGroupNotFoundException extends CharacterStatException
-{
-}

src/Exceptions/CharacterStatNotFoundException.php

@@ -1,11 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Exceptions;
-
-/**
- * Class CharacterStatNotFoundException.
- */
-class CharacterStatNotFoundException extends CharacterStatException
-{
-}

src/Exceptions/ClassNotFoundException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class ClassNotFoundException extends CoreException
 {
+    
 }

src/Exceptions/CoreException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class CoreException extends \Exception
 {
+    
 }

src/Exceptions/EntityAlreadyExistsException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class EntityAlreadyExistsException extends EntityException
 {
+
 }

src/Exceptions/EntityDoesNotExistException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class EntityDoesNotExistException extends EntityException
 {
+
 }

src/Exceptions/EntityException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * A basic entity exception.
+ * A basic entity exception
  */
 class EntityException extends CoreException
 {
+
 }

src/Exceptions/InsecureTwigTemplateError.php

@@ -1,8 +0,0 @@
-<?php
-declare(strict_types=1);
-
-namespace LotGD\Core\Exceptions;
-
-class InsecureTwigTemplateError extends CoreException
-{
-}

src/Exceptions/InvalidConfigurationException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class InvalidConfigurationException extends CoreException
 {
+
 }

src/Exceptions/InvalidModelException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class InvalidModelException extends CoreException
 {
+    
 }

src/Exceptions/IsNullException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class IsNullException extends CoreException
 {
+    
 }

src/Exceptions/KeyNotFoundException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class KeyNotFoundException extends CoreException
 {
+    
 }

src/Exceptions/LibraryDoesNotExistException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class LibraryDoesNotExistException extends CoreException
 {
+
 }

src/Exceptions/ModuleAlreadyExistsException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class ModuleAlreadyExistsException extends CoreException
 {
+
 }

src/Exceptions/ModuleDoesNotExistException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class ModuleDoesNotExistException extends CoreException
 {
+
 }

src/Exceptions/NotImplementedException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class NotImplementedException extends CoreException
 {
+    
 }

src/Exceptions/ParentAlreadySetException.php

@@ -4,8 +4,9 @@ declare(strict_types=1);
 namespace LotGD\Core\Exceptions;
 
 /**
- * Exception if a specific, required argument is missing.
+ * Exception if a specific, required argument is missing
  */
 class ParentAlreadySetException extends CoreException
 {
+    
 }

src/Exceptions/PermissionAlreadyExistsException.php

@@ -8,4 +8,5 @@ namespace LotGD\Core\Exceptions;
  */
 class PermissionAlreadyExistsException extends EntityAlreadyExistsException
 {
+
 }