Complete README with new rules (#160)

* readme

* improve docs

Author: TomasVotruba

README.md

@@ -264,6 +264,55 @@ abstract class ParentClass
 
 <br>
 
+### ForbiddenNewArgumentRule
+
+Type "%s" is forbidden to be created manually with `new X()`. Use service and constructor injection instead
+
+```yaml
+services:
+    -
+        class: Symplify\PHPStanRules\Rules\ForbiddenNewArgumentRule
+        tag: [phpstan.rules.rule]
+        arguments:
+            forbiddenTypes:
+                - RepositoryService
+```
+
+↓
+
+```php
+class SomeService
+{
+    public function run()
+    {
+        $repositoryService = new RepositoryService();
+        $item = $repositoryService->get(1);
+    }
+}
+```
+
+:x:
+
+<br>
+
+```php
+class SomeService
+{
+    public function __construct(private RepositoryService $repositoryService)
+    {
+    }
+
+    public function run()
+    {
+        $item = $this->repositoryService->get(1);
+    }
+}
+```
+
+:+1:
+
+<br>
+
 ### ForbiddenFuncCallRule
 
 Function `"%s()"` cannot be used/left in the code
@@ -963,6 +1012,17 @@ final class SomeClass
 
 ## 2. Doctrine-specific Rules
 
+### RequireQueryBuilderOnRepositoryRule
+
+Prevents using `$entityManager->createQueryBuilder('...')`,  use `$repository->createQueryBuilder()` as safer.
+
+```yaml
+rules:
+    - Symplify\PHPStanRules\Rules\Doctrine\RequireQueryBuilderOnRepositoryRule
+```
+
+<br>
+
 ### NoGetRepositoryOutsideServiceRule
 
 Instead of getting repository from EntityManager, use constructor injection and service pattern to keep code clean
@@ -1082,6 +1142,29 @@ final class SomeFixture extends AbstractFixture
 
 ## 3. Symfony-specific Rules
 
+### NoGetDoctrineInControllerRule
+
+Prevents using `$this->getDoctrine()` in controllers, to promote dependency injection.
+
+```yaml
+rules:
+    - Symplify\PHPStanRules\Rules\Symfony\NoGetDoctrineInControllerRule
+```
+
+<br>
+
+### NoGetInControllerRule
+
+Prevents using `$this->get(...)` in controllers, to promote dependency injection.
+
+```yaml
+rules:
+    - Symplify\PHPStanRules\Rules\Symfony\NoGetInControllerRule
+```
+
+<br>
+
+
 ### NoAbstractControllerConstructorRule
 
 Abstract controller should not have constructor, as it can lead to tight coupling. Use @required annotation instead

config/doctrine-rules.neon

@@ -2,3 +2,6 @@ rules:
     - Symplify\PHPStanRules\Rules\Doctrine\NoGetRepositoryOutsideServiceRule
     - Symplify\PHPStanRules\Rules\Doctrine\NoParentRepositoryRule
     - Symplify\PHPStanRules\Rules\Doctrine\NoRepositoryCallInDataFixtureRule
+
+    # test fixtures
+    - Symplify\PHPStanRules\Rules\Doctrine\RequireQueryBuilderOnRepositoryRule

src/Rules/Doctrine/NoEntityMockingRule.php

@@ -6,7 +6,6 @@
 
 use PhpParser\Node;
 use PhpParser\Node\Expr\MethodCall;
-use PhpParser\Node\Identifier;
 use PHPStan\Analyser\Scope;
 use PHPStan\Reflection\ReflectionProvider;
 use PHPStan\Rules\Rule;