Fixed bug that the private property which injected by parent class does not exists in class.

Author: limingxinleo

src/Aop/PropertyHandlerTrait.php

@@ -54,7 +54,7 @@ protected function __handlePropertyHandler(string $className)
         $parentReflectionClass = $reflectionClass;
         while ($parentReflectionClass = $parentReflectionClass->getParentClass()) {
             $parentClassProperties = ReflectionManager::reflectPropertyNames($parentReflectionClass->getName());
-            $parentClassProperties = array_filter($parentClassProperties, static function($property) use ($reflectionClass){
+            $parentClassProperties = array_filter($parentClassProperties, static function ($property) use ($reflectionClass) {
                 return $reflectionClass->hasProperty($property);
             });
             $parentClassProperties = array_diff($parentClassProperties, $handled);

tests/InjectTest.php

@@ -36,9 +36,11 @@
 use HyperfTest\Di\Stub\Inject\Origin3Class;
 use HyperfTest\Di\Stub\Inject\Origin4Class;
 use HyperfTest\Di\Stub\Inject\Origin5Class;
+use HyperfTest\Di\Stub\Inject\Origin6Class;
 use HyperfTest\Di\Stub\Inject\OriginClass;
 use HyperfTest\Di\Stub\Inject\Parent2Class;
 use HyperfTest\Di\Stub\Inject\Parent3Class;
+use HyperfTest\Di\Stub\Inject\Parent4Class;
 use HyperfTest\Di\Stub\Inject\ParentClass;
 use HyperfTest\Di\Stub\Inject\Tar;
 use Mockery;
@@ -222,6 +224,31 @@ public function testInjectParentNoRunParent()
         $this->assertInstanceOf(Foo::class, $origin->getFoo());
     }
 
+    public function testInjectParentPrivateProperty()
+    {
+        $this->getContainer();
+        $ast = new Ast();
+        $classes = [
+            Parent4Class::class,
+            Origin6Class::class,
+        ];
+
+        if (! is_dir($dir = BASE_PATH . '/runtime/container/proxy/')) {
+            mkdir($dir, 0777, true);
+        }
+
+        foreach ($classes as $class) {
+            $code = $ast->proxy($class);
+            $id = md5($class);
+            file_put_contents($file = $dir . $id . '.proxy.php', $code);
+            require_once $file;
+        }
+
+        $origin = new Origin6Class();
+        $this->assertInstanceOf(Foo::class, $origin->getFoo());
+        $this->assertInstanceOf(Bar::class, $origin->getBar());
+    }
+
     public function testInjectException()
     {
         $this->getContainer();
@@ -283,6 +310,8 @@ protected function getContainer()
             Origin5Class::class,
             Parent2Class::class,
             Parent3Class::class,
+            Parent4Class::class,
+            Origin6Class::class,
         ];
 
         foreach ($classes as $class) {

tests/Stub/Inject/Origin6Class.php

@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+/**
+ * This file is part of Hyperf.
+ *
+ * @link     https://www.hyperf.io
+ * @document https://hyperf.wiki
+ * @contact  [email protected]
+ * @license  https://github.com/hyperf/hyperf/blob/master/LICENSE
+ */
+namespace HyperfTest\Di\Stub\Inject;
+
+use Hyperf\Di\Annotation\Inject;
+
+class Origin6Class extends Parent4Class
+{
+    /**
+     * @Inject
+     * @var Bar
+     */
+    private $bar;
+
+    public function getBar()
+    {
+        return $this->bar;
+    }
+}

tests/Stub/Inject/Parent4Class.php

@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+/**
+ * This file is part of Hyperf.
+ *
+ * @link     https://www.hyperf.io
+ * @document https://hyperf.wiki
+ * @contact  [email protected]
+ * @license  https://github.com/hyperf/hyperf/blob/master/LICENSE
+ */
+namespace HyperfTest\Di\Stub\Inject;
+
+use Hyperf\Di\Annotation\Inject;
+
+class Parent4Class
+{
+    /**
+     * @Inject
+     * @var Foo
+     */
+    private $foo;
+
+    public function getFoo()
+    {
+        return $this->foo;
+    }
+}