Ignore inaccessible class child nodes

Author: herndlm

src/NodeVisitor.php

@@ -11,6 +11,7 @@
 use PhpParser\Node\Scalar\String_;
 use PhpParser\Node\Stmt;
 use PhpParser\Node\Stmt\Class_;
+use PhpParser\Node\Stmt\ClassConst;
 use PhpParser\Node\Stmt\ClassLike;
 use PhpParser\Node\Stmt\ClassMethod;
 use PhpParser\Node\Stmt\Const_;
@@ -19,6 +20,7 @@
 use PhpParser\Node\Stmt\If_;
 use PhpParser\Node\Stmt\Interface_;
 use PhpParser\Node\Stmt\Namespace_;
+use PhpParser\Node\Stmt\Property;
 use PhpParser\Node\Stmt\Trait_;
 use PhpParser\NodeTraverser;
 use PhpParser\NodeVisitorAbstract;
@@ -238,6 +240,13 @@ public function leaveNode(Node $node, bool $preserveStack = false)
             // Implies `$parent instanceof ClassLike`, which means $node is a
             // either a method, property, or constant, or its part of the
             // declaration itself (e.g., `extends`).
+
+            if ($parent instanceof Class_ && ($node instanceof ClassMethod || $node instanceof ClassConst || $node instanceof Property)) {
+                if ($node->isPrivate() || ($parent->isFinal() && $node->isProtected())) {
+                    return NodeTraverser::REMOVE_NODE;
+                }
+            }
+
             return;
         }
 

test/files/classes.in.php

@@ -5,9 +5,14 @@ abstract class A extends B implements C
     /** doc */
     protected const A = 'B';
 
+    /** doc */
+    private const B = 'C';
+
     /** doc */
     public static $a = 'a';
 
+    private static $b = 'b';
+
     /** doc */
     public static function b($a): void
     {
@@ -16,6 +21,21 @@ public static function b($a): void
 
     /** doc */
     abstract public function c($a): string;
+
+    /** doc */
+    protected function d($a): void
+    {
+        return;
+    }
+
+    /** doc */
+    abstract protected function e($a): string;
+
+    /** doc */
+    private function f($a): void
+    {
+        return;
+    }
 }
 
 if (!class_exists('D')) {
@@ -27,3 +47,36 @@ public function a(string $a): string
         }
     }
 }
+
+final class E
+{
+    /** doc */
+    public $a = 'a';
+
+    /** doc */
+    protected $b = 'b';
+
+    /** doc */
+    public function a($a): void
+    {
+        return;
+    }
+
+    /** doc */
+    protected function b($a): void
+    {
+        return;
+    }
+}
+
+trait F
+{
+    /** doc */
+    private $a = 'a';
+
+    /** doc */
+    private function a($a): void
+    {
+        return;
+    }
+}

test/files/classes.out.php

@@ -15,6 +15,14 @@ public static function b($a): void
 
     /** doc */
     abstract public function c($a): string;
+
+    /** doc */
+    protected function d($a) : void
+    {
+    }
+
+    /** doc */
+    protected abstract function e($a) : string;
 }
 
 class D
@@ -23,3 +31,25 @@ public function a(string $a): string
     {
     }
 }
+
+final class E
+{
+    /** doc */
+    public $a = 'a';
+
+    /** doc */
+    public function a($a) : void
+    {
+    }
+}
+
+trait F
+{
+    /** doc */
+    private $a = 'a';
+
+    /** doc */
+    private function a($a) : void
+    {
+    }
+}