Function __construct has a Cognitive Complexity of 108 (exceeds 5 allowed). Consider refactoring.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up

Method __construct has 105 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

The class PS has an overall complexity of 61 which is very high. The configured complexity threshold is 50.

Open

class PS extends PSI_Plugin
{
    /**
     * variable, which holds the content of the command
     * @var array

• Exclude checks
  • Disable engine
  • Disable check

Function execute has a Cognitive Complexity of 22 (exceeds 5 allowed). Consider refactoring.

Open

    public function execute()
    {
        if (empty($this->_filecontent)) {
            return;
        }

• Read up
  Read up

Function _addchild has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.

Open

    private function _addchild($child, SimpleXMLExtended $xml, &$positions)
    {
        foreach ($child as $key=>$value) {
            $xmlnode = $xml->addChild("Process");
            if (isset($value[0])) {

• Read up
  Read up

Method execute has 37 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function execute()
    {
        if (empty($this->_filecontent)) {
            return;
        }

Method _addchild has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    private function _addchild($child, SimpleXMLExtended $xml, &$positions)
    {
        foreach ($child as $key=>$value) {
            $xmlnode = $xml->addChild("Process");
            if (isset($value[0])) {

Consider simplifying this complex logical expression.

Open

                if ((PSI_OS !== 'WINNT') &&
                    ((($parentid === 1) && (!defined('PSI_PLUGIN_PS_SHOW_PID1CHILD_EXPANDED') || (PSI_PLUGIN_PS_SHOW_PID1CHILD_EXPANDED === false)))
                    || ((!defined('PSI_PLUGIN_PS_SHOW_KTHREADD_EXPANDED') || (PSI_PLUGIN_PS_SHOW_KTHREADD_EXPANDED === false)) && ($value[4] === "[kthreadd]")))) {
                    $xmlnode->addAttribute('Expanded', 0);
                }

Avoid deeply nested control flow statements.

Open

                            for ($i = 0; $i < $total; $i++) {
                                if (CommonFunctions::rfts($process[$i], $buf, 0, 4096, false)) {

                                    if (($totalmem != 0) && (preg_match('/^VmRSS:\s+(\d+)\s+kB/m', $buf, $tmppmem))) {
                                        $pmem = round(100 * $tmppmem[1] / $totalmem, 1);

The method __construct() has 124 lines of code. Current threshold is set to 100. Avoid really long methods.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Exclude checks
  • Disable engine
  • Disable check

The method __construct() has an NPath complexity of 831. The configured NPath complexity threshold is 200.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    function bar() {
        // lots of complicated code
    }
}

The method _addchild() has a Cyclomatic Complexity of 15. The configured cyclomatic complexity threshold is 10.

Open

    private function _addchild($child, SimpleXMLExtended $xml, &$positions)
    {
        foreach ($child as $key=>$value) {
            $xmlnode = $xml->addChild("Process");
            if (isset($value[0])) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The method execute() has a Cyclomatic Complexity of 11. The configured cyclomatic complexity threshold is 10.

Open

    public function execute()
    {
        if (empty($this->_filecontent)) {
            return;
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

The method __construct() has a Cyclomatic Complexity of 33. The configured cyclomatic complexity threshold is 10.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

// Cyclomatic Complexity = 11
class Foo {
1   public function example() {
2       if ($a == $b) {
3           if ($a1 == $b1) {
                fiddle();
4           } elseif ($a2 == $b2) {
                fiddle();
            } else {
                fiddle();
            }
5       } elseif ($c == $d) {
6           while ($c == $d) {
                fiddle();
            }
7        } elseif ($e == $f) {
8           for ($n = 0; $n < $h; $n++) {
                fiddle();
            }
        } else {
            switch ($z) {
9               case 1:
                    fiddle();
                    break;
10              case 2:
                    fiddle();
                    break;
11              case 3:
                    fiddle();
                    break;
                default:
                    fiddle();
                    break;
            }
        }
    }
}

Avoid using undefined variables such as '$mbuf' which will lead to PHP notices.

Open

                    if (CommonFunctions::rfts('/proc/meminfo', $mbuf)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$namebuf' which will lead to PHP notices.

Open

                                    if (CommonFunctions::rfts(substr($process[$i], 0, strlen($process[$i])-6)."cmdline", $namebuf, 0, 4096, false)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$mbuf' which will lead to PHP notices.

Open

                        $bufe = preg_split("/\n/", $mbuf, -1, PREG_SPLIT_NO_EMPTY);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Avoid using undefined variables such as '$namebuf' which will lead to PHP notices.

Open

                                        $name = str_replace(chr(0), ' ', trim($namebuf));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    private function bar()
    {
        // $message is undefined
        echo $message;
    }
}

Missing class import via use statement (line '42', column '39').

Open

                    $objLocator = new COM('WbemScripting.SWbemLocator');

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

function make() {
    return new \stdClass();
}

Avoid using static access to class 'CommonFunctions' in method '__construct'.

Open

                    if (CommonFunctions::rfts('/proc/meminfo', $mbuf)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'CommonFunctions' in method '__construct'.

Open

                CommonFunctions::executeProgram("ps", "axo pid,ppid,pmem,pcpu,args", $buffer, PSI_DEBUG);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'CommonFunctions' in method '__construct'.

Open

                    $process_wmi = CommonFunctions::getWMI($wmi, 'Win32_Process', array('Caption', 'CommandLine', 'ProcessId', 'ParentProcessId', 'WorkingSetSize'));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method __construct uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                        } else {
                            $ps = trim($process['Caption']);
                        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'CommonFunctions' in method '__construct'.

Open

                                if (CommonFunctions::rfts($process[$i], $buf, 0, 4096, false)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid using static access to class 'CommonFunctions' in method '__construct'.

Open

                                    if (CommonFunctions::rfts(substr($process[$i], 0, strlen($process[$i])-6)."cmdline", $namebuf, 0, 4096, false)) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid assigning values to variables in if clauses and the like (line '64', column '30').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

The method __construct uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                                            } else {
                                                $args = "[".$args."]";
                                            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'CommonFunctions' in method '__construct'.

Open

                    $perf_wmi = CommonFunctions::getWMI($wmi, 'Win32_PerfFormattedData_PerfProc_Process', array('IDProcess', 'CreatingProcessID', 'PercentProcessorTime'));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method __construct uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                CommonFunctions::executeProgram("ps", "axo pid,ppid,pmem,pcpu,args", $buffer, PSI_DEBUG);
                if (((PSI_OS == 'Linux') || (PSI_OS == 'Android')) && (!preg_match("/^[^\n]+\n\s*\d+\s+\d+\s+[\d\.]+\s+[\d\.]+\s+.+/", $buffer))) { //alternative method if no data
                    if (CommonFunctions::rfts('/proc/meminfo', $mbuf)) {
                        $bufe = preg_split("/\n/", $mbuf, -1, PREG_SPLIT_NO_EMPTY);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method execute uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

        } else {
            $this->_result = array();
        }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

The method __construct uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                                    } else {
                                        $pmem = 0;
                                    }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using static access to class 'CommonFunctions' in method '__construct'.

Open

                    $os_wmi = CommonFunctions::getWMI($wmi, 'Win32_OperatingSystem', array('TotalVisibleMemorySize'));

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

Avoid assigning values to variables in if clauses and the like (line '99', column '57').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($foo = 'bar') { // possible typo
            // ...
        }
        if ($baz = 0) { // always false
            // ...
        }
    }
}

Avoid using static access to class 'CommonFunctions' in method '__construct'.

Open

            CommonFunctions::rfts(PSI_APP_ROOT."/data/ps.txt", $buffer);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar()
    {
        Bar::baz();
    }
}

The method __construct uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                $this->_filecontent = array();
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo
{
    public function bar($flag)
    {
        if ($flag) {
            // one branch
        } else {
            // another branch
        }
    }
}

Avoid using empty try-catch blocks in __construct.

Open

                } catch (Exception $e) {
                }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {

  public function bar()
  {
      try {
          // ...
      } catch (Exception $e) {} // empty catch block
  }
}

Avoid unused local variables such as '$key'.

Open

        foreach ($child as $key=>$value) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Foo {
    public function doSomething()
    {
        $i = 5; // Unused
    }
}

TODO found

Open

                            //=> subtrees of processes may be missing (WHAT TODO?!?)

• Exclude checks
  • Disable engine
  • Disable check

Each class must be in a namespace of at least one level (a top-level vendor name)

Open

class PS extends PSI_Plugin

• Exclude checks
  • Disable engine
  • Disable check

Avoid classes with short names like PS. Configured minimum length is 3.

Open

class PS extends PSI_Plugin
{
    /**
     * variable, which holds the content of the command
     * @var array

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Fo {

}

interface Fo {

}

Avoid variables with short names like $ps. Configured minimum length is 3.

Open

                            $ps = trim($process['CommandLine']);

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class Something {
    private $q = 15; // VIOLATION - Field
    public static function main( array $as ) { // VIOLATION - Formal
        $r = 20 + $this->q; // VIOLATION - Local
        for (int $i = 0; $i < 10; $i++) { // Not a Violation (inside FOR)
            $r += $this->q;
        }
    }
}

The property $_filecontent is not named in camelCase.

Open

class PS extends PSI_Plugin
{
    /**
     * variable, which holds the content of the command
     * @var array

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

The property $_result is not named in camelCase.

Open

class PS extends PSI_Plugin
{
    /**
     * variable, which holds the content of the command
     * @var array

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    protected $property_name;
}

Property name "$_result" should not be prefixed with an underscore to indicate visibility

Open

    private $_result = array();

• Exclude checks
  • Disable engine
  • Disable check

Property name "$_filecontent" should not be prefixed with an underscore to indicate visibility

Open

    private $_filecontent = array();

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at start of control structure

Open

                                if (CommonFunctions::rfts($process[$i], $buf, 0, 4096, false)) {

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at end of control structure

Open

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after "=>"; 0 found

Open

        foreach ($child as $key=>$value) {

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be indented to the same level as the CASE body

Open

            break;

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space before "=>"; 0 found

Open

        foreach ($child as $key=>$value) {

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space before "=>"; 0 found

Open

            foreach ($row as $key=>$val) {

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be indented to the same level as the CASE body

Open

            break;

• Exclude checks
  • Disable engine
  • Disable check

Expected 1 space after "=>"; 0 found

Open

            foreach ($row as $key=>$val) {

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be indented to the same level as the CASE body

Open

            break;

• Exclude checks
  • Disable engine
  • Disable check

Method name "_addchild" should not be prefixed with an underscore to indicate visibility

Open

    private function _addchild($child, SimpleXMLExtended $xml, &$positions)

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 133 characters

Open

                            //possible that ParentProcessId incorrectly refers to a process that reuses a process identifier. You can

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 131 characters

Open

                            //use the CreationDate property to determine whether the specified parent was created after the process

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 132 characters

Open

 * @license   http://opensource.org/licenses/gpl-2.0.php GNU General Public License version 2, or (at your option) any later version

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 165 characters

Open

                    $process_wmi = CommonFunctions::getWMI($wmi, 'Win32_Process', array('Caption', 'CommandLine', 'ProcessId', 'ParentProcessId', 'WorkingSetSize'));

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 181 characters

Open

                        $proccpu[trim($perf['IDProcess'])] = array('ParentProcessId'=>trim($perf['CreatingProcessID']), 'PercentProcessorTime'=>trim($perf['PercentProcessorTime']));

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 163 characters

Open

                    || ((!defined('PSI_PLUGIN_PS_SHOW_KTHREADD_EXPANDED') || (PSI_PLUGIN_PS_SHOW_KTHREADD_EXPANDED === false)) && ($value[4] === "[kthreadd]")))) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 171 characters

Open

                    $perf_wmi = CommonFunctions::getWMI($wmi, 'Win32_PerfFormattedData_PerfProc_Process', array('IDProcess', 'CreatingProcessID', 'PercentProcessorTime'));

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 149 characters

Open

                                    if (CommonFunctions::rfts(substr($process[$i], 0, strlen($process[$i])-6)."cmdline", $namebuf, 0, 4096, false)) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 133 characters

Open

                            //only identify a process for the lifetime of that process. It is possible that the process identified by

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 126 characters

Open

                            //ParentProcessId is terminated, so ParentProcessId may not refer to a running process. It is also

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 133 characters

Open

                            //Unique identifier of the process that creates a process. Process identifier numbers are reused, so they

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 148 characters

Open

                    ((($parentid === 1) && (!defined('PSI_PLUGIN_PS_SHOW_PID1CHILD_EXPANDED') || (PSI_PLUGIN_PS_SHOW_PID1CHILD_EXPANDED === false)))

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 24 spaces, found 20

Open

                    $perf_wmi = CommonFunctions::getWMI($wmi, 'Win32_PerfFormattedData_PerfProc_Process', array('IDProcess', 'CreatingProcessID', 'PercentProcessorTime'));

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 24 spaces, found 20

Open

                    foreach ($process_wmi as $process) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 32 spaces, found 28

Open

                            $this->_filecontent[] = $procid." ".$parentid." ".$memusage." ".$cpu." ".$ps;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 20 spaces, found 16

Open

                if (((PSI_OS == 'Linux') || (PSI_OS == 'Android')) && (!preg_match("/^[^\n]+\n\s*\d+\s+\d+\s+[\d\.]+\s+[\d\.]+\s+.+/", $buffer))) { //alternative method if no data

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 36 spaces, found 32

Open

                                $totalmem = $ar_buf[1];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 44 spaces, found 40

Open

                                        $pmem = round(100 * $tmppmem[1] / $totalmem, 1);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 24 spaces, found 20

Open

                    $memtotal = 0;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 24 spaces, found 20

Open

                    $process_wmi = CommonFunctions::getWMI($wmi, 'Win32_Process', array('Caption', 'CommandLine', 'ProcessId', 'ParentProcessId', 'WorkingSetSize'));

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 32 spaces, found 28

Open

                            $ps = trim($process['Caption']);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 20 spaces, found 16

Open

                try {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 24 spaces, found 20

Open

                    $os_wmi = CommonFunctions::getWMI($wmi, 'Win32_OperatingSystem', array('TotalVisibleMemorySize'));

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 24 spaces, found 20

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 24 spaces, found 20

Open

                    foreach ($perf_wmi as $perf) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 24 spaces, found 20

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 20 spaces, found 16

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 20 spaces, found 16

Open

                CommonFunctions::executeProgram("ps", "axo pid,ppid,pmem,pcpu,args", $buffer, PSI_DEBUG);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 28 spaces, found 24

Open

                        $bufe = preg_split("/\n/", $mbuf, -1, PREG_SPLIT_NO_EMPTY);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 28 spaces, found 24

Open

                        $memtotal = $os['TotalVisibleMemorySize'] * 1024;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 32 spaces, found 28

Open

                            $parentid  = trim($process['ParentProcessId']);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 28 spaces, found 24

Open

                        $totalmem = 0;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 28 spaces, found 24

Open

                        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 32 spaces, found 28

Open

                            for ($i = 0; $i < $total; $i++) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 36 spaces, found 32

Open

                                $cpu = $proccpu[$procid]['PercentProcessorTime'];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 32 spaces, found 28

Open

                            $process = array();

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 36

Open

                                    if (($totalmem != 0) && (preg_match('/^VmRSS:\s+(\d+)\s+kB/m', $buf, $tmppmem))) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 16 spaces, found 12

Open

            if (PSI_OS == 'WINNT') {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 24 spaces, found 20

Open

                    foreach ($os_wmi as $os) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 24 spaces, found 20

Open

                    $proccpu = array();

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 32 spaces, found 28

Open

                            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 28 spaces, found 24

Open

                        $processlist = glob('/proc/*/status', GLOB_NOSORT);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 48 spaces, found 40

Open

                                        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 24 spaces, found 20

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 32 spaces, found 28

Open

                            if (preg_match('/^MemTotal:\s+(\d+)\s*kB/i', $buf, $ar_buf)) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 28 spaces, found 24

Open

                        $buffer = "  PID  PPID %MEM %CPU COMMAND\n";

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 36 spaces, found 32

Open

                                if (CommonFunctions::rfts($process[$i], $buf, 0, 4096, false)) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 28 spaces, found 24

Open

                        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 28 spaces, found 24

Open

                        foreach ($bufe as $buf) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 32 spaces, found 28

Open

                            foreach ($processlist as $processitem) { //second sort

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 36

Open

                                    } else {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 36

Open

                                    if (CommonFunctions::rfts(substr($process[$i], 0, strlen($process[$i])-6)."cmdline", $namebuf, 0, 4096, false)) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 48 spaces, found 40

Open

                                        $buffer .= $pid." ".$ppid." ".$pmem." 0.0 ".$args."\n";

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 32 spaces, found 28

Open

                            if (isset($proccpu[$procid]) && ($proccpu[$procid]['ParentProcessId'] == $parentid)) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 16 spaces, found 12

Open

            } else {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 28 spaces, found 24

Open

                        } else {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 32 spaces, found 28

Open

                            $memusage = round(trim($process['WorkingSetSize']) * 100 / $memtotal, 1);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 52 spaces, found 44

Open

                                            } else {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 24 spaces, found 20

Open

                    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 12 spaces, found 8

Open

        case 'command':

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 24 spaces, found 20

Open

                    $objLocator = new COM('WbemScripting.SWbemLocator');

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 32 spaces, found 28

Open

                            $ps = trim($process['CommandLine']);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 28 spaces, found 24

Open

                        if (($procid = trim($process['ProcessId'])) != 0) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 20 spaces, found 16

Open

                } catch (Exception $e) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 24 spaces, found 20

Open

                    if (CommonFunctions::rfts('/proc/meminfo', $mbuf)) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 36 spaces, found 32

Open

                                break;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 32 spaces, found 28

Open

                            $buf = "";

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 48 spaces, found 40

Open

                                        $pid = $tmppid[1];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 52 spaces, found 44

Open

                                            if ($name !== "") {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 52 spaces, found 44

Open

                                            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 32 spaces, found 28

Open

                            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 20 spaces, found 16

Open

                }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 16 spaces, found 12

Open

            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 32 spaces, found 28

Open

                            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 32 spaces, found 28

Open

                            }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 44 spaces, found 40

Open

                                        $pmem = 0;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 40 spaces, found 36

Open

                                    $name = null;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 16 spaces, found 12

Open

            CommonFunctions::rfts(PSI_APP_ROOT."/data/ps.txt", $buffer);

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 36

Open

                                    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 16 spaces, found 12

Open

            $this->global_error->addConfigError("__construct()", "[ps] ACCESS");

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 28 spaces, found 24

Open

                        $proccpu[trim($perf['IDProcess'])] = array('ParentProcessId'=>trim($perf['CreatingProcessID']), 'PercentProcessorTime'=>trim($perf['PercentProcessorTime']));

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 28 spaces, found 24

Open

                        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 36 spaces, found 32

Open

                                $process[] = $processitem;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 36

Open

                                    if (preg_match('/^Pid:\s+(\d+)/m', $buf, $tmppid) &&

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 44 spaces, found 40

Open

                                        $name = str_replace(chr(0), ' ', trim($namebuf));

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 12 spaces, found 8

Open

        default:

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 32 spaces, found 28

Open

                            natsort($processlist); //first sort

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 48 spaces, found 40

Open

                                        $args = $tmpargs[1];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 36 spaces, found 32

Open

                                }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 24 spaces, found 20

Open

                    $wmi = $objLocator->ConnectServer('', 'root\CIMv2');

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 28 spaces, found 24

Open

                        break;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 28 spaces, found 24

Open

                        if (strlen(trim($process['CommandLine'])) > 0) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 32 spaces, found 28

Open

                            $cpu = 0;

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 28 spaces, found 24

Open

                        if (is_array($processlist) && (($total = count($processlist)) > 0)) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 36

Open

                                    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 48 spaces, found 40

Open

                                        if ($name !== null) {

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 48 spaces, found 40

Open

                                        $ppid = $tmpppid[1];

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 56 spaces, found 48

Open

                                                $args = "[".$args."]";

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 28 spaces, found 24

Open

                        }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 40 spaces, found 36

Open

                                    }

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected 12 spaces, found 8

Open

        case 'data':

• Exclude checks
  • Disable engine
  • Disable check

Line indented incorrectly; expected at least 56 spaces, found 48

Open

                                                $args = $name;

• Exclude checks
  • Disable engine
  • Disable check

The variable $process_wmi is not named in camelCase.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $perf_wmi is not named in camelCase.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ar_buf is not named in camelCase.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $os_wmi is not named in camelCase.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $perf_wmi is not named in camelCase.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $process_wmi is not named in camelCase.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $ar_buf is not named in camelCase.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The variable $os_wmi is not named in camelCase.

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        switch (strtolower(PSI_PLUGIN_PS_ACCESS)) {
        case 'command':

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function doSomething() {
        $data_module = new DataModule();
    }
}

The method _addchild is not named in camelCase.

Open

    private function _addchild($child, SimpleXMLExtended $xml, &$positions)
    {
        foreach ($child as $key=>$value) {
            $xmlnode = $xml->addChild("Process");
            if (isset($value[0])) {

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

class ClassName {
    public function get_name() {
    }
}