fix scramble private methods

src/Naneau/Obfuscator/Node/Visitor/ScramblePrivateMethod.php

@@ -75,8 +75,7 @@ public function enterNode(Node $node)
         }
 
         // Scramble calls
-        if ($node instanceof MethodCall || $node instanceof StaticCall) {
-
+        if (($node instanceof MethodCall && $node->var->name === 'this') || ($node instanceof StaticCall && $node->class instanceof Node\Name && $node->class->toString() === 'self')) {
             // Node wasn't renamed
             if (!$this->isRenamed($node->name)) {
                 return;

src/Naneau/Obfuscator/Node/Visitor/ScramblePrivateProperty.php

@@ -78,7 +78,7 @@ public function enterNode(Node $node)
     {
         if ($node instanceof PropertyFetch) {
 
-            if (!is_string($node->name)) {
+            if (!is_string($node->name) || $node->var->name !== "this") {
                 return;
             }
 

tests/before/MultipleClasses.php

@@ -29,10 +29,32 @@ public function publicMethod() {
         echo "This is public method of second class";
         $this->_privateProperty = parent::$publicProperty;
     }
+
+    static public function anotherPublicMethod() {
+    }
 }
 
 class ThirdClass {
+
+    private $publicProperty;
+
+    static private function anotherPublicMethod() {
+
+    }
+
     public function __construct(SecondClass $secondObject) {
         $secondObject->publicMethod();
+        $secondObject::anotherPublicMethod();
+        $secondObject->publicProperty = 'test';
+    }
+
+    private function publicMethod() {
+        echo 'test';
+    }
+
+    protected function someFunc() {
+        $this->publicProperty = 'test';
+        $this->publicMethod();
+        self::anotherPublicMethod();
     }
 }

tests/expected/MultipleClasses.php

@@ -1,2 +1,2 @@
 <?php
-class FirstClass { protected $_protectedProperty; public $publicProperty; protected function _protectedMethod() { echo 'This is protected method of first class'; } public function publicMethod() { echo 'This is public method of first class'; } } class SecondClass extends FirstClass { private $sp8839d9; protected function _protectedMethod() { parent::_protectedMethod(); echo 'This is protected method of second class'; $this->sp8839d9 = parent::$_protectedProperty; } public function publicMethod() { parent::publicMethod(); echo 'This is public method of second class'; $this->sp8839d9 = parent::$publicProperty; } } class ThirdClass { public function __construct(SecondClass $spb91639) { $spb91639->publicMethod(); } }
+class FirstClass { protected $_protectedProperty; public $publicProperty; protected function _protectedMethod() { echo 'This is protected method of first class'; } public function publicMethod() { echo 'This is public method of first class'; } } class SecondClass extends FirstClass { private $sp8839d9; protected function _protectedMethod() { parent::_protectedMethod(); echo 'This is protected method of second class'; $this->sp8839d9 = parent::$_protectedProperty; } public function publicMethod() { parent::publicMethod(); echo 'This is public method of second class'; $this->sp8839d9 = parent::$publicProperty; } public static function anotherPublicMethod() { } } class ThirdClass { private $spa36ab6; private static function spe81a11() { } public function __construct(SecondClass $spb91639) { $spb91639->publicMethod(); $spb91639::anotherPublicMethod(); $spb91639->publicProperty = 'test'; } private function sp70ab23() { echo 'test'; } protected function someFunc() { $this->spa36ab6 = 'test'; $this->sp70ab23(); self::spe81a11(); } }