add contract interface (#5)

* add contract interface

* misc

Author: TomasVotruba

src/ClassPropertyExtension/AbstractClassPropertyExtension.php

@@ -8,9 +8,10 @@
 use PHPStan\Reflection\PropertiesClassReflectionExtension;
 use PHPStan\Reflection\PropertyReflection;
 use PHPStan\Reflection\ReflectionProvider;
+use PHPStanCakePHP2\Contract\PropertyNameExtensionInterface;
 use PHPStanCakePHP2\Reflection\PublicReadOnlyPropertyReflection;
 
-abstract class AbstractClassPropertyExtension implements PropertiesClassReflectionExtension
+abstract class AbstractClassPropertyExtension implements PropertiesClassReflectionExtension, PropertyNameExtensionInterface
 {
     private ReflectionProvider $reflectionProvider;
 
@@ -42,26 +43,4 @@ public function getProperty(
 
         return new PublicReadOnlyPropertyReflection($correctedPropertyName, $classReflection);
     }
-
-    /**
-     * @todo use constract instead to separate
-     * Get the class name of the type of property.
-     */
-    abstract protected function getPropertyParentClassName(): string;
-
-    /**
-     * @todo use constract instead to separate
-     * Get the class names which can contain the property.
-     *
-     * @return array<string>
-     */
-    abstract protected function getContainingClassNames(): array;
-
-    /**
-     * @todo use constract instead to separate
-     * Return the class name from the property name.
-     */
-    abstract protected function getClassNameFromPropertyName(
-        string $propertyName
-    ): string;
 }

src/ClassPropertyExtension/ClassModelsPropertyExtension.php

@@ -9,15 +9,15 @@
  */
 final class ClassModelsPropertyExtension extends AbstractClassPropertyExtension
 {
-    protected function getPropertyParentClassName(): string
+    public function getPropertyParentClassName(): string
     {
         return 'Model';
     }
 
     /**
      * @return array<string>
      */
-    protected function getContainingClassNames(): array
+    public function getContainingClassNames(): array
     {
         return [
             'Controller',
@@ -26,7 +26,7 @@ protected function getContainingClassNames(): array
         ];
     }
 
-    protected function getClassNameFromPropertyName(
+    public function getClassNameFromPropertyName(
         string $propertyName
     ): string {
         return $propertyName;

src/ClassPropertyExtension/ShellClassPropertyExtension.php

@@ -11,20 +11,20 @@
  */
 final class ShellClassPropertyExtension extends AbstractClassPropertyExtension
 {
-    protected function getPropertyParentClassName(): string
+    public function getPropertyParentClassName(): string
     {
         return 'Shell';
     }
 
     /**
      * @return array<string>
      */
-    protected function getContainingClassNames(): array
+    public function getContainingClassNames(): array
     {
         return ['Shell'];
     }
 
-    protected function getClassNameFromPropertyName(
+    public function getClassNameFromPropertyName(
         string $propertyName
     ): string {
         return $propertyName . 'Task';

src/Contract/PropertyNameExtensionInterface.php

@@ -0,0 +1,25 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PHPStanCakePHP2\Contract;
+
+interface PropertyNameExtensionInterface
+{
+    /**
+     * Get the class name of the type of property.
+     */
+    public function getPropertyParentClassName(): string;
+
+    /**
+     * Get the class names which can contain the property.
+     *
+     * @return string[]
+     */
+    public function getContainingClassNames(): array;
+
+    /**
+     * Return the class name from the property name.
+     */
+    public function getClassNameFromPropertyName(string $propertyName): string;
+}

tests/config/phpstan.neon

@@ -2,7 +2,6 @@ parameters:
     behaviorPaths:
         - tests/Source/Model/Behavior/*.php
 
-    # @todo add exists validatoin to avoid misspaths
     schemaPaths:
         - tests/Source/Config/Schema/*.php