ScramblePrivateProperty::beforeTraverse() - Code Metrics - naneau/php-obfuscator - Measure and Improve Code Quality continuously with Scrutinizer

ScramblePrivateProperty::beforeTraverse() A
last analyzed 2016-12-19 15:58 UTC

↳ Parent: ScramblePrivateProperty

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	8
rs	9.4285
cc	1
eloc	5
nc	1
nop	1

<?php
/**
 * ScramblePrivateProperty.php
 *
 * @category        Naneau
 * @package         Obfuscator
 * @subpackage      NodeVisitor
 */

namespace Naneau\Obfuscator\Node\Visitor;

use Naneau\Obfuscator\Node\Visitor\TrackingRenamerTrait;
use Naneau\Obfuscator\Node\Visitor\SkipTrait;

use Naneau\Obfuscator\Node\Visitor\Scrambler as ScramblerVisitor;
use Naneau\Obfuscator\StringScrambler;

use PhpParser\Node;

use PhpParser\Node\Stmt\Class_ as ClassNode;
use PhpParser\Node\Stmt\Property;

use PhpParser\Node\Expr\PropertyFetch;

use PhpParser\Node\Expr\Variable;

/**
 * ScramblePrivateProperty
 *
 * Renames private properties
 *
 * WARNING
 *
 * See warning for private method scrambler
 *
 * @category        Naneau
 * @package         Obfuscator
 * @subpackage      NodeVisitor
 */
class ScramblePrivateProperty extends ScramblerVisitor
{
    use TrackingRenamerTrait;
    use SkipTrait;

    /**
     * Constructor
     *
     * @param  StringScrambler $scrambler
     * @return void

     **/
    public function __construct(StringScrambler $scrambler)
    {
        parent::__construct($scrambler);
    }

    /**
     * Before node traversal
     *
     * @param  Node[] $nodes
     * @return array
     **/
    public function beforeTraverse(array $nodes)
    {
        $this
            ->resetRenamed()
            ->scanPropertyDefinitions($nodes);

        return $nodes;
    }

    /**
     * Check all variable nodes
     *
     * @param  Node $node
     * @return void
     **/
    public function enterNode(Node $node)
    {
        if ($node instanceof PropertyFetch) {

            if (!is_string($node->name)) {
                return;
            }

            if ($this->isRenamed($node->name)) {
                $node->name = $this->getNewName($node->name);
                return $node;
            }
        }
    }

    /**
     * Recursively scan for private method definitions and rename them
     *
     * @param  Node[] $nodes
     * @return void
     **/
    private function scanPropertyDefinitions(array $nodes)

    {
        foreach ($nodes as $node) {
            // Scramble the private method definitions
            if ($node instanceof Property && ($node->type & ClassNode::MODIFIER_PRIVATE)) {
                foreach($node->props as $property) {

                    // Record original name and scramble it
                    $originalName = $property->name;
                    $this->scramble($property);

                    // Record renaming
                    $this->renamed($originalName, $property->name);
                }

            }

            // Recurse over child nodes
            if (isset($node->stmts) && is_array($node->stmts)) {
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
                $this->scanPropertyDefinitions($node->stmts);
interface SomeInterface { }
class SomeClass implements SomeInterface {
    public $a;
}

function someFunction(SomeInterface $object) {
    if ($object instanceof SomeClass) {
        $a = $object->a;
    }
}
            }
        }
    }
}


1		<?php
2		/**
3		* ScramblePrivateProperty.php
4		*
5		* @category Naneau
6		* @package Obfuscator
7		* @subpackage NodeVisitor
8		*/
9
10		namespace Naneau\Obfuscator\Node\Visitor;
11
12		use Naneau\Obfuscator\Node\Visitor\TrackingRenamerTrait;
13		use Naneau\Obfuscator\Node\Visitor\SkipTrait;
14
15		use Naneau\Obfuscator\Node\Visitor\Scrambler as ScramblerVisitor;
16		use Naneau\Obfuscator\StringScrambler;
17
18		use PhpParser\Node;
19
20		use PhpParser\Node\Stmt\Class_ as ClassNode;
21		use PhpParser\Node\Stmt\Property;
22
23		use PhpParser\Node\Expr\PropertyFetch;
24
25		use PhpParser\Node\Expr\Variable;
26
27		/**
28		* ScramblePrivateProperty
29		*
30		* Renames private properties
31		*
32		* WARNING
33		*
34		* See warning for private method scrambler
35		*
36		* @category Naneau
37		* @package Obfuscator
38		* @subpackage NodeVisitor
39		*/
40		class ScramblePrivateProperty extends ScramblerVisitor
41		{
42		use TrackingRenamerTrait;
43		use SkipTrait;
44
45		/**
46		* Constructor
47		*
48		* @param StringScrambler $scrambler
49		* @return void
		0 ignored issues – show Comprehensibility Best Practice introduced 2015-12-10 00:13 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
50		**/
51		public function __construct(StringScrambler $scrambler)
52		{
53		parent::__construct($scrambler);
54		}
55
56		/**
57		* Before node traversal
58		*
59		* @param Node[] $nodes
60		* @return array
61		**/
62		public function beforeTraverse(array $nodes)
63		{
64		$this
65		->resetRenamed()
66		->scanPropertyDefinitions($nodes);
67
68		return $nodes;
69		}
70
71		/**
72		* Check all variable nodes
73		*
74		* @param Node $node
75		* @return void
76		**/
77		public function enterNode(Node $node)
78		{
79		if ($node instanceof PropertyFetch) {
80
81		if (!is_string($node->name)) {
82		return;
83		}
84
85		if ($this->isRenamed($node->name)) {
86		$node->name = $this->getNewName($node->name);
87		return $node;
88		}
89		}
90		}
91
92		/**
93		* Recursively scan for private method definitions and rename them
94		*
95		* @param Node[] $nodes
96		* @return void
97		**/
98	View Code Duplication	private function scanPropertyDefinitions(array $nodes)
		0 ignored issues – show Duplication introduced 2015-12-10 00:13 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
99		{
100		foreach ($nodes as $node) {
101		// Scramble the private method definitions
102		if ($node instanceof Property && ($node->type & ClassNode::MODIFIER_PRIVATE)) {
103		foreach($node->props as $property) {
104
105		// Record original name and scramble it
106		$originalName = $property->name;
107		$this->scramble($property);
108
109		// Record renaming
110		$this->renamed($originalName, $property->name);
111		}
112
113		}
114
115		// Recurse over child nodes
116		if (isset($node->stmts) && is_array($node->stmts)) {
		0 ignored issues – show Bug introduced 2015-12-10 00:13 UTC by Report Bug Copy Issue Report Accessing `stmts` on the interface `PhpParser\Node` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
117		$this->scanPropertyDefinitions($node->stmts);
		0 ignored issues – show Bug introduced 2015-12-10 00:13 UTC by Report Bug Copy Issue Report Accessing `stmts` on the interface `PhpParser\Node` suggest that you code against a concrete implementation. How about adding an `instanceof` check? If you access a property on an interface, you most likely code against a concrete implementation of the interface. Available Fixes Adding an additional type check: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeInterface $object) { if ($object instanceof SomeClass) { $a = $object->a; } } Changing the type hint: interface SomeInterface { } class SomeClass implements SomeInterface { public $a; } function someFunction(SomeClass $object) { $a = $object->a; } Loading history...
118		}
119		}
120		}
121		}
122

naneau / php-obfuscator

GitHub Access Token became invalid

ScramblePrivateProperty::beforeTraverse() A last analyzed 2016-12-19 15:58 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

ScramblePrivateProperty::beforeTraverse() A
last analyzed 2016-12-19 15:58 UTC