Richard

The ruleset force to add PHPDoc blocks for all methods

12

The Symfony coding strandard [documentation](http://symfony.com/doc/current/contributing/code/standards.html#documentation) states: > Add PHPDoc blocks for all classes, methods, and functions (**though you may be asked to remove PHPDoc that do not add value**); For...

Precision should be without decimal while it's set to 0

1

We can't set no decimal at all for the precision (to use plain integers for the steps). For example if we set `precision=0`, the precision will be the default value...

According to Ocramius in [this post](https://github.com/webdevilopers/php-ddd/issues/13#issuecomment-299641329), we should never flush an entity, like in [DoctrineRepository.php](https://github.com/CodelyTV/php-ddd-example/blob/6da760a2c904898a62a5cc1ee3ce7a3cee3eb0b2/src/Shared/Infrastructure/Persistence/Doctrine/DoctrineRepository.php). This param [will be deprecated in Doctrine 3.0](https://github.com/doctrine/orm/blob/old-prototype-3.x/UPGRADE.md#bc-break-removed-entitymanagerflushentity-and-entitymanagerflushentities). Related issues in Doctrine: - https://github.com/doctrine/orm/issues/6118 -...

We can see a `flush` method at [DoctrineRepository.php#L31](https://github.com/CodelyTV/php-ddd-example/blob/main/src/Shared/Infrastructure/Persistence/Doctrine/DoctrineRepository.php#L31). According to [this post](https://www.thinktocode.com/2019/01/24/doctrine-repositories-should-be-collections-without-flush/), Doctrine repositories should be collections without flush.

Suggest app/ directory by default instead of ./ for shared volume

2

Currently the shared volume is set to `./`, however we could suggest by default a directory like _app/_. Same apply for _docker-sync.yml_. That would show that not everything is supposed...

Operation not permitted with chmod as root on settings.php

4

***Codebase*** Mounted codebase **Describe your issue** Operation not permitted on chmod 777 settings.php, even as root, from inside the container: ``` $ docker-compose exec php bash [email protected]:/var/www/html $ sudo chmod...

Ability to load fixtures once before test class

2

Currently we can load fixtures: - before each test case using PHPUnit `setup()` - once before all tests by customizing Symfony bootstrap process like I suggested in liip/LiipFunctionalTestBundle#303 Only the...

UML class diagram feature

1

Hi, It seems mdp allows sequence diagram but not class diagram. Do you plan to implement UML class diagram as well?

Naming convention of data provider methods

1

As we can see in the [data providers section](https://phpunit.de/manual/6.5/en/writing-tests-for-phpunit.html#writing-tests-for-phpunit.data-providers) the data provider method is named `additionProvider`. Since a method should start with a verb, it's confusing. So, to avoid a...