[FEATURE]: Add rules for container

Author: sabbelasichon

README.md

@@ -143,3 +143,14 @@ $myIntAttribute = $site->getAttribute('myIntAttribute');
 // PHPStan will now know that $myStringAttribute is of type string
 $myStringAttribute = $site->getAttribute('myStringAttribute');
 ```
+
+### Check for private Services
+You have to provide a path to App_KernelDevelopmentDebugContainer.xml or similar XML file describing your container.
+This is generated by [ssch/typo3-debug-dump-pass](https://github.com/sabbelasichon/typo3-debug-dump-pass) in your /var/cache/{TYPO3_CONTEXT}/ folder.
+
+```NEON
+parameters:
+    typo3:
+        containerXmlPath: var/cache/development/App_KernelDevelopmentDebugContainer.xml
+```
+

build.xml

@@ -60,6 +60,7 @@
 			passthru="true"
 			checkreturn="true"
 		>
+			<arg line="--memory-limit=-1"/>
 		</exec>
 	</target>
 

extension.neon

@@ -1,3 +1,6 @@
+includes:
+    - rules.neon
+
 services:
     -
         class: SaschaEgerer\PhpstanTypo3\Reflection\RepositoryFindMethodsClassReflectionExtension
@@ -51,10 +54,6 @@ services:
             siteGetAttributeMapping: %typo3.siteGetAttributeMapping%
         tags:
             - phpstan.rules.rule
-    -
-        class: SaschaEgerer\PhpstanTypo3\Rule\ValidatorResolverOptionsRule
-        tags:
-            - phpstan.rules.rule
     -
         class: SaschaEgerer\PhpstanTypo3\Type\RepositoryQueryDynamicReturnTypeExtension
         tags:
@@ -105,6 +104,16 @@ services:
         class: SaschaEgerer\PhpstanTypo3\Type\DateTimeAspectGetDynamicReturnTypeExtension
         tags:
             - phpstan.broker.dynamicMethodReturnTypeExtension
+    -
+    	class: SaschaEgerer\PhpstanTypo3\Service\PrivateServiceAnalyzer
+    # service map
+    typo3.serviceMapFactory:
+        class: SaschaEgerer\PhpstanTypo3\Contract\ServiceMapFactory
+        factory: SaschaEgerer\PhpstanTypo3\Service\XmlServiceMapFactory
+        arguments:
+             containerXmlPath: %typo3.containerXmlPath%
+    -
+        factory: @typo3.serviceMapFactory::create()
 parameters:
     bootstrapFiles:
         - phpstan.bootstrap.php

phpstan.neon

@@ -1,5 +1,6 @@
 includes:
     - vendor/phpstan/phpstan-strict-rules/rules.neon
+    - vendor/phpstan/phpstan-phpunit/extension.neon
     - extension.neon
     - tests/Unit/Type/data/context-get-aspect-return-types.neon
     - tests/Unit/Type/data/request-get-attribute-return-types.neon

rules.neon

@@ -0,0 +1,4 @@
+rules:
+   - SaschaEgerer\PhpstanTypo3\Rule\ValidatorResolverOptionsRule
+   - SaschaEgerer\PhpstanTypo3\Rule\ContainerInterfacePrivateServiceRule
+   - SaschaEgerer\PhpstanTypo3\Rule\GeneralUtilityMakeInstancePrivateServiceRule

src/Contract/ServiceMapInterface.php renamed to src/Contract/ServiceMap.php

@@ -1,16 +1,21 @@
-<?php
-declare(strict_types=1);
+<?php declare(strict_types = 1);
 
 namespace SaschaEgerer\PhpstanTypo3\Contract;
 
+use PhpParser\Node\Expr;
+use PHPStan\Analyser\Scope;
 use SaschaEgerer\PhpstanTypo3\Service\ServiceDefinition;
 
-interface ServiceMapInterface
+interface ServiceMap
 {
+
 	/**
 	 * @return ServiceDefinition[]
 	 */
 	public function getServiceDefinitions(): array;
 
 	public function getServiceDefinitionById(string $id): ?ServiceDefinition;
+
+	public function getServiceIdFromNode(Expr $node, Scope $scope): ?string;
+
 }

src/Contract/ServiceMapFactory.php

@@ -0,0 +1,10 @@
+<?php declare(strict_types = 1);
+
+namespace SaschaEgerer\PhpstanTypo3\Contract;
+
+interface ServiceMapFactory
+{
+
+	public function create(): ServiceMap;
+
+}

src/Contract/ServiceMapFactoryInterface.php

@@ -0,0 +1,64 @@
+<?php declare(strict_types = 1);
+
+namespace SaschaEgerer\PhpstanTypo3\Rule;
+
+use PhpParser\Node;
+use PhpParser\Node\Expr\MethodCall;
+use PHPStan\Analyser\Scope;
+use PHPStan\Rules\Rule;
+use PHPStan\Type\ObjectType;
+use SaschaEgerer\PhpstanTypo3\Service\PrivateServiceAnalyzer;
+
+/**
+ * @implements Rule<MethodCall>
+ */
+final class ContainerInterfacePrivateServiceRule implements Rule
+{
+
+	private PrivateServiceAnalyzer $privateServiceAnalyzer;
+
+	public function __construct(PrivateServiceAnalyzer $privateServiceAnalyzer)
+	{
+		$this->privateServiceAnalyzer = $privateServiceAnalyzer;
+	}
+
+	public function getNodeType(): string
+	{
+		return MethodCall::class;
+	}
+
+	public function processNode(Node $node, Scope $scope): array
+	{
+		if ($this->shouldSkip($node, $scope)) {
+			return [];
+		}
+
+		return $this->privateServiceAnalyzer->analyze($node, $scope);
+	}
+
+	private function shouldSkip(MethodCall $node, Scope $scope): bool
+	{
+		if (!$node->name instanceof Node\Identifier) {
+			return true;
+		}
+
+		$methodCallArguments = $node->getArgs();
+
+		if (!isset($methodCallArguments[0])) {
+			return true;
+		}
+
+		$methodCallName = $node->name->name;
+
+		if ($methodCallName !== 'get') {
+			return true;
+		}
+
+		$argType = $scope->getType($node->var);
+
+		$isPsrContainerType = (new ObjectType('Psr\Container\ContainerInterface'))->isSuperTypeOf($argType);
+
+		return !$isPsrContainerType->yes();
+	}
+
+}

src/Rule/ContainerInterfacePrivateServiceRule.php

@@ -0,0 +1,68 @@
+<?php declare(strict_types = 1);
+
+namespace SaschaEgerer\PhpstanTypo3\Rule;
+
+use PhpParser\Node;
+use PhpParser\Node\Expr\StaticCall;
+use PHPStan\Analyser\Scope;
+use PHPStan\Rules\Rule;
+use SaschaEgerer\PhpstanTypo3\Service\PrivateServiceAnalyzer;
+use TYPO3\CMS\Core\Utility\GeneralUtility;
+
+/**
+ * @implements Rule<StaticCall>
+ */
+final class GeneralUtilityMakeInstancePrivateServiceRule implements Rule
+{
+
+	private PrivateServiceAnalyzer $privateServiceAnalyzer;
+
+	public function __construct(PrivateServiceAnalyzer $privateServiceAnalyzer)
+	{
+		$this->privateServiceAnalyzer = $privateServiceAnalyzer;
+	}
+
+	public function getNodeType(): string
+	{
+		return StaticCall::class;
+	}
+
+	public function processNode(Node $node, Scope $scope): array
+	{
+		if ($this->shouldSkip($node)) {
+			return [];
+		}
+
+		return $this->privateServiceAnalyzer->analyze($node, $scope);
+	}
+
+	private function shouldSkip(StaticCall $node): bool
+	{
+		if (!$node->name instanceof Node\Identifier) {
+			return true;
+		}
+
+		$methodCallArguments = $node->getArgs();
+
+		if (!isset($methodCallArguments[0])) {
+			return true;
+		}
+
+		$methodCallName = $node->name->name;
+
+		if ($methodCallName !== 'makeInstance') {
+			return true;
+		}
+
+		if (count($methodCallArguments) > 1) {
+			return true;
+		}
+
+		if (!$node->class instanceof Node\Name\FullyQualified) {
+			return true;
+		}
+
+		return $node->class->toString() !== GeneralUtility::class;
+	}
+
+}