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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• Read up
  Read up

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

Open

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

• Read up
  Read up

File class.bat.inc.php has 710 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/**
 * BAT Plugin, which displays battery state
 *
 * @category  PHP

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

Open

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

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

Function xml has a Cognitive Complexity of 62 (exceeds 5 allowed). Consider refactoring.

Open

    public function xml()
    {
        foreach ($this->_result as $bat_item) {
            $xmlbat = $this->xml->addChild("Bat");
            if (isset($bat_item['name'])) {

• Read up
  Read up

Consider simplifying this complex logical expression.

Open

                if (preg_match('/^[dD]esign capacity:\s*(.*) (.*)$/', $roworig, $data)
                   || preg_match('/^energy-full-design:\s*(.*) (.*)$/', $roworig, $data)) {
                    $bat['design_capacity'] = str_replace(',', '.', $data[1])+0;
                    if (!isset($bat['capacity_unit'])) {
                        $bat['capacity_unit'] = trim($data[2]);

Method xml has 87 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function xml()
    {
        foreach ($this->_result as $bat_item) {
            $xmlbat = $this->xml->addChild("Bat");
            if (isset($bat_item['name'])) {

The class BAT has an overall complexity of 334 which is very high. The configured complexity threshold is 50.

Open

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

• Exclude checks
  • Disable engine
  • Disable check

Avoid deeply nested control flow statements.

Open

                            switch ($bufferWB[$bi]['Chemistry']) {
                                case 1: $techn = 'Other'; break;
                                case 2: $techn = 'Unknown'; break;
                                case 3: $techn = 'PbAc'; break;
                                case 4: $techn = 'NiCd'; break;

Avoid deeply nested control flow statements.

Open

                            if ($batstat != '') $buffer[$bi]['state'] .= 'POWER_SUPPLY_STATUS='.$batstat."\n";

Avoid deeply nested control flow statements.

Open

                            if (isset($bufferBS[$bi]['Voltage']) && ($bufferBS[$bi]['Voltage']>0)) {
                                $buffer[$bi]['state'] .= 'POWER_SUPPLY_VOLTAGE_NOW='.($bufferBS[$bi]['Voltage']*1000)."\n";
                                $hasvolt = true;
                            } elseif (isset($bufferWB[$bi]['DesignVoltage']) && ($bufferWB[$bi]['DesignVoltage']>0)) {
                                $buffer[$bi]['state'] .= 'POWER_SUPPLY_VOLTAGE_NOW='.($bufferWB[$bi]['DesignVoltage']*1000)."\n";

Avoid deeply nested control flow statements.

Open

                            if (CommonFunctions::rfts($batdevices[$i], $pbuffer, 1, 4096, false) && trim($pbuffer[0]==="1")) {
                                $infoitem = '';
                                $stateitem = '';
                                CommonFunctions::rfts(preg_replace('/\/present$/', '/uevent', $batdevices[$i]), $infoitem, 0, 4096, false);

Avoid deeply nested control flow statements.

Open

                            switch ($bufferWB[$bi]['BatteryStatus']) {
                                case  1: $batstat = 'Discharging'; break;
                                case  2: $batstat = 'AC connected'; break;
                                case  3: $batstat = 'Fully Charged'; break;
                                case  4: $batstat = 'Low'; break;

Avoid deeply nested control flow statements.

Open

                            switch ($bufferWPB[$bi]['Chemistry']) {
                                case 1: $techn = 'Other'; break;
                                case 2: $techn = 'Unknown'; break;
                                case 3: $techn = 'PbAc'; break;
                                case 4: $techn = 'NiCd'; break;

Avoid deeply nested control flow statements.

Open

                            if (($rfts_bi && ($infoitem!=='')) || ($rfts_bs && ($stateitem!==''))) {
                                if (preg_match('/^\/proc\/acpi\/battery\/(.+)\/info$/', $batdevices[$i], $batname)) {
                                    if ($infoitem!=='') {
                                        $infoitem .= 'POWER_SUPPLY_NAME='.$batname[1]."\n";
                                    } else {

Avoid deeply nested control flow statements.

Open

                            if (isset($bufferBS[$bi]['RemainingCapacity']) &&
                               (($bufferBS[$bi]['RemainingCapacity']>0) || ($hasvolt && ($bufferBS[$bi]['RemainingCapacity']==0)))) {
                                $buffer[$bi]['state'] .= 'POWER_SUPPLY_ENERGY_NOW='.($bufferBS[$bi]['RemainingCapacity']*1000)."\n";
                                $capacity = '';
                            }

Consider simplifying this complex logical expression.

Open

            if (defined('PSI_PLUGIN_BAT_SHOW_SERIAL') && PSI_PLUGIN_BAT_SHOW_SERIAL
               && isset($bat_item['serialnumber'])
               && ($bat_item['serialnumber'] !== "")
               && ($bat_item['serialnumber'] !== "0")
               && ($bat_item['serialnumber'] !== "0000")) {

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

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

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

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

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

The method xml() has an NPath complexity of 2795521. The configured NPath complexity threshold is 200.

Open

    public function xml()
    {
        foreach ($this->_result as $bat_item) {
            $xmlbat = $this->xml->addChild("Bat");
            if (isset($bat_item['name'])) {

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

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

The method execute() has an NPath complexity of 18534. The configured NPath complexity threshold is 200.

Open

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

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

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

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 xml() has a Cyclomatic Complexity of 43. The configured cyclomatic complexity threshold is 10.

Open

    public function xml()
    {
        foreach ($this->_result as $bat_item) {
            $xmlbat = $this->xml->addChild("Bat");
            if (isset($bat_item['name'])) {

• 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 156. 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;
            }
        }
    }
}

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

Open

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

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

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

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

Open

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

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

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

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 assigning values to variables in if clauses and the like (line '278', column '38').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 assigning values to variables in if clauses and the like (line '244', column '39').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 assigning values to variables in if clauses and the like (line '245', column '41').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 assigning values to variables in if clauses and the like (line '284', column '38').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 assigning values to variables in if clauses and the like (line '269', column '38').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 assigning values to variables in if clauses and the like (line '272', column '38').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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

                                if ((($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/voltage_min_design'))!==null)

• 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/voltage_max_design'))!==null) {

• 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/energy_now'))!==null) {

• 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/batt_vol'))!==null) {

• 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

                                   && (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/serial_number'))!==null)) {

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

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

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 {
                        $bat['present_voltage'] = round($data[1]/1000);
                    }

• 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 xml uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                if (isset($bat_item['full_capacity'])) {
                    if (isset($bat_item['design_capacity'])) {
                        $xmlbat->addAttribute("DesignCapacity", $bat_item['design_capacity']);
                    }

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

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

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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

                            $rfts_bs = CommonFunctions::rfts(preg_replace('/\/info$/', '/state', $batdevices[$i]), $stateitem, 0, 4096, false);

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

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

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 {
                        if (!isset($bat['capacity_unit'])) {
                            $bat['capacity_unit'] = "mWh";
                        } elseif ($bat['capacity_unit'] != "mWh") {
                            $bat['capacity_unit'] = "???";

• 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 xml uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

            } else {
                if (isset($bat_item['charging_state_i'])) {
                    $xmlbat->addAttribute("ChargingState", 'Charging');
                } elseif (!isset($bat_item['charging_state_e'])) {
                    $xmlbat->addAttribute("ChargingState", 'Discharging');

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

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

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 assigning values to variables in if clauses and the like (line '291', column '41').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 {
                            if ($capacity != '') $buffer[$bi]['state'] .= 'POWER_SUPPLY_CAPACITY='.$capacity."\n";
                        }

• 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 {
                        $bat['design_voltage_max'] = round($data[1]/1000);
                    }

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

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

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 assigning values to variables in if clauses and the like (line '281', column '38').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/health'))!==null) {

• 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/manufacturer'))!==null) {

• 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::rfts(PSI_APP_ROOT."/data/bat_info.txt", $info);

• 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::rfts(PSI_APP_ROOT."/data/bat_state.txt", $buffer[0]['state']);

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

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

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 {
                        $bat['present_voltage'] = round($data[1]/1000);
                    }

• 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

                CommonFunctions::executeProgram('ioreg', '-w0 -l -n AppleSmartBattery -r', $buffer[0]['info'], 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($batdevices[$i], $pbuffer, 1, 4096, false) && trim($pbuffer[0]==="1")) {

• 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/charge_full'))!==null) {

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

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

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 {
                        $bat['design_voltage'] = round($data[1]/1000);
                    }

• 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 {
                        $bat['present_voltage'] = round($data[1]/1000);
                    }

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

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

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 assigning values to variables in if clauses and the like (line '263', column '38').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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

                $bufferWB = CommonFunctions::getWMI($_cim, 'Win32_Battery', array('Caption', 'Name', 'EstimatedChargeRemaining', 'DesignVoltage', 'BatteryStatus', 'Chemistry'));

• 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('sysctl', 'hw.sensors.acpibat0', $buffer[0]['info'], 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

                            $rfts_bi = CommonFunctions::rfts($batdevices[$i], $infoitem, 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/capacity'))!==null) {

• 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/temp'))!==null) {

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

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

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 {
                        if (!isset($bat['capacity_unit'])) {
                            $bat['capacity_unit'] = "mWh";
                        } elseif ($bat['capacity_unit'] != "mWh") {
                            $bat['capacity_unit'] = "???";

• 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/charge_now'))!==null) {

• 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 '275', column '38').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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

                    $bufferBFCC = CommonFunctions::getWMI($_wmi, 'BatteryFullChargedCapacity', array('FullChargedCapacity'));

• 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 {
                $itemcount = 0;
                if ((PSI_OS == 'Linux') && defined('PSI_PLUGIN_BAT_UPOWER') && PSI_PLUGIN_BAT_UPOWER) {
                    $info = '';
                    CommonFunctions::executeProgram('upower', '-d', $info, false);

• 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 {
                                        $stateitem.= 'POWER_SUPPLY_NAME='.$batname[1]."\n";
                                    }

• 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 {
                        $bat['design_voltage'] = round($data[1]/1000);
                    }

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

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

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/voltage_now'))!==null) {

• 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/energy_full'))!==null) {

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

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

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 {
                        $bat['present_voltage'] = round($data[1]/1000);
                    }

• 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 xml uses an else expression. Else clauses are basically not necessary and you can simplify the code by not using them.

Open

                } else {
                    $xmlbat->addAttribute("ChargingState", 'Unknown state');
                }

• 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

                    CommonFunctions::executeProgram('upower', '-d', $info, false);

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

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

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 {
                        if (!isset($bat['capacity_unit'])) {
                            $bat['capacity_unit'] = "mWh";
                        } elseif ($bat['capacity_unit'] != "mWh") {
                            $bat['capacity_unit'] = "???";

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

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

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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 assigning values to variables in if clauses and the like (line '266', column '38').

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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::executeProgram('acpiconf', '-i batt', $buffer[0]['info'], 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/batt_temp'))!==null) {

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

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

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 {
                        $bat['design_voltage_max'] = round($data[1]/1000);
                    }

• 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

                                   || (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/voltage_max'))!==null)) {

• 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/technology'))!==null) {

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

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

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

Open

                } else {
                    $xmlbat->addAttribute("FullCapacity", 100);
                    $xmlbat->addAttribute("RemainingCapacity", $bat_item['capacity']);
                    $xmlbat->addAttribute("CapacityUnit", "%");
                }

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

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

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

Open

    public function __construct($enc)
    {
        parent::__construct(__CLASS__, $enc);
        $buffer = array();
        switch (strtolower(PSI_PLUGIN_BAT_ACCESS)) {

• 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

                    $bufferWPB = CommonFunctions::getWMI($_cim, 'Win32_PortableBattery', array('DesignVoltage', 'Chemistry', 'DesignCapacity', 'FullChargeCapacity', 'Manufacturer'));

• 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

                    $bufferBCC = CommonFunctions::getWMI($_wmi, 'BatteryCycleCount', array('CycleCount'));

• 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

                    $bufferBS = CommonFunctions::getWMI($_wmi, 'BatteryStatus', array('RemainingCapacity', 'Voltage'));

• 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::rfts(preg_replace('/\/present$/', '/uevent', $batdevices[$i]), $infoitem, 0, 4096, false);

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

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

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 {
                        $bat['present_voltage'] = round($data[1]/1000);
                    }

• 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 (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/status'))!==null) {

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

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

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 {
                        $bat['design_voltage'] = round($data[1]/1000);
                    }

• 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 sizeof() function in for loops.

Open

        for ($bi = 0; $bi < sizeof($buffer); $bi++) {
            if (isset($buffer[$bi]['info'])) {
                $this->_filecontent[$bi]['info'] = preg_split("/\n/", $buffer[$bi]['info'], -1, PREG_SPLIT_NO_EMPTY);
            }
            if (isset($buffer[$bi]['state'])) {

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

class Foo {

  public function bar()
  {
    $array = array();

    for ($i = 0; count($array); $i++) {
      // ...
    }
  }
}

Avoid using sizeof() function in for loops.

Open

        for ($bi = 0; $bi < sizeof($this->_filecontent); $bi++) {
            $bat = array();
            if (isset($this->_filecontent[$bi]['info'])) foreach ($this->_filecontent[$bi]['info'] as $roworig) {
                $roworig = trim($roworig);
                if (preg_match('/^[dD]esign capacity:\s*(.*) (.*)$/', $roworig, $data)

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

class Foo {

  public function bar()
  {
    $array = array();

    for ($i = 0; count($array); $i++) {
      // ...
    }
  }
}

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                } elseif (preg_match('/^POWER_SUPPLY_ENERGY_FULL=(.*)$/', $roworig, $data)) {
                    $bat['full_capacity'] = ($data[1]/1000);
                    if ($data[1]>=1000000000) { // µWh or nWh detection
                        if (!isset($bat['capacity_unit'])) {
                            $bat['capacity_unit'] = "µWh";

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                } elseif (preg_match('/^POWER_SUPPLY_ENERGY_FULL_DESIGN=(.*)$/', $roworig, $data)) {
                    $bat['design_capacity'] = ($data[1]/1000);
                    if ($data[1]>=1000000000) { // µWh or nWh detection
                        if (!isset($bat['capacity_unit'])) {
                            $bat['capacity_unit'] = "µWh";

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                } elseif (preg_match('/^POWER_SUPPLY_ENERGY_FULL=(.*)$/', $roworig, $data)) {
                    $bat['full_capacity'] = ($data[1]/1000);
                    if ($data[1]>=1000000000) { // µWh or nWh detection
                        if (!isset($bat['capacity_unit'])) {
                            $bat['capacity_unit'] = "µWh";

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                        if (isset($bufferWB[$bi]['Chemistry'])) {
                            switch ($bufferWB[$bi]['Chemistry']) {
                                case 1: $techn = 'Other'; break;
                                case 2: $techn = 'Unknown'; break;
                                case 3: $techn = 'PbAc'; break;

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                        if ((($techn == '') || ($techn == 'Unknown')) && isset($bufferWPB[$bi]['Chemistry'])) {
                            switch ($bufferWPB[$bi]['Chemistry']) {
                                case 1: $techn = 'Other'; break;
                                case 2: $techn = 'Unknown'; break;
                                case 3: $techn = 'PbAc'; break;

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                } elseif (preg_match('/^energy:\s*(.*) (.*)$/', $roworig, $data)) {
                    $bat['remaining_capacity'] = str_replace(',', '.', $data[1])+0;
                    if (!isset($bat['capacity_unit'])) {
                        $bat['capacity_unit'] = trim($data[2]);
                    } elseif ($bat['capacity_unit'] != trim($data[2])) {

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                } elseif (preg_match('/^[lL]ast full capacity:\s*(.*) (.*)$/', $roworig, $data)
                   || preg_match('/^energy-full:\s*(.*) (.*)$/', $roworig, $data)) {
                    $bat['full_capacity'] = str_replace(',', '.', $data[1])+0;
                    if (!isset($bat['capacity_unit'])) {
                        $bat['capacity_unit'] = trim($data[2]);

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

                if (preg_match('/^[dD]esign capacity:\s*(.*) (.*)$/', $roworig, $data)
                   || preg_match('/^energy-full-design:\s*(.*) (.*)$/', $roworig, $data)) {
                    $bat['design_capacity'] = str_replace(',', '.', $data[1])+0;
                    if (!isset($bat['capacity_unit'])) {
                        $bat['capacity_unit'] = trim($data[2]);

• Read up
  Read up

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

Open

class BAT extends PSI_Plugin

• Exclude checks
  • Disable engine
  • Disable check

The property $_filecontent is not named in camelCase.

Open

class BAT 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 BAT 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;
}

Each PHP statement must be on a line by itself

Open

                                case  7: $batstat = 'Charging and High'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 5: $techn = 'NiMH'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 6: $techn = 'Li-ion'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 2: $techn = 'Unknown'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 5: $techn = 'NiMH'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 6: $techn = 'Li-ion'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case  2: $batstat = 'AC connected'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 6: $techn = 'Li-ion'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case  4: $batstat = 'Low'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case  8: $batstat = 'Charging and Low'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case  8: $batstat = 'Charging and Low'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 11: $batstat = 'Partially Charged'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 4: $techn = 'NiCd'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 5: $techn = 'NiMH'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 5: $techn = 'NiMH'; break;

• Exclude checks
  • Disable engine
  • Disable check

CASE keyword must be followed by a single space

Open

                                case  1: $batstat = 'Discharging'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case  1: $batstat = 'Discharging'; break;

• Exclude checks
  • Disable engine
  • Disable check

CASE keyword must be followed by a single space

Open

                                case  7: $batstat = 'Charging and High'; break;

• Exclude checks
  • Disable engine
  • Disable check

CASE keyword must be followed by a single space

Open

                                case  8: $batstat = 'Charging and Low'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case  9: $batstat = 'Charging and Critical'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 11: $batstat = 'Partially Charged'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 2: $techn = 'Unknown'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 3: $techn = 'PbAc'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 6: $techn = 'Li-ion'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 3: $techn = 'PbAc'; break;

• 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

Each PHP statement must be on a line by itself

Open

                                case  3: $batstat = 'Fully Charged'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case  7: $batstat = 'Charging and High'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 8: $techn = 'Li-poly'; break;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    private $_result = array();

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case  5: $batstat = 'Critical'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 4: $techn = 'NiCd'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 6: $techn = 'Li-ion'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 2: $techn = 'Unknown'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 2: $techn = 'Unknown'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 4: $techn = 'NiCd'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 2: $techn = 'Unknown'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 4: $techn = 'NiCd'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 5: $techn = 'NiMH'; break;

• Exclude checks
  • Disable engine
  • Disable check

CASE keyword must be followed by a single space

Open

                                case  4: $batstat = 'Low'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case  4: $batstat = 'Low'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case  5: $batstat = 'Critical'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case  5: $batstat = 'Critical'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case  6: $batstat = 'Charging'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 11: $batstat = 'Partially Charged'; break;

• Exclude checks
  • Disable engine
  • Disable check

The DEFAULT body must start on the line following the statement

Open

                                default: $batstat = '';

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 1: $techn = 'Other'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 3: $techn = 'PbAc'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 3: $techn = 'PbAc'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 7: $techn = 'Zinc-air'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 8: $techn = 'Li-poly'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 6: $techn = 'Li-ion'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 4: $techn = 'NiCd'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case  4: $batstat = 'Low'; break;

• Exclude checks
  • Disable engine
  • Disable check

CASE keyword must be followed by a single space

Open

                                case  5: $batstat = 'Critical'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 3: $techn = 'PbAc'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 4: $techn = 'NiCd'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case  1: $batstat = 'Discharging'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 1: $techn = 'Other'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 3: $techn = 'PbAc'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 8: $techn = 'Li-poly'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case  2: $batstat = 'AC connected'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 2: $techn = 'Unknown'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 1: $techn = 'Other'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 7: $techn = 'Zinc-air'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case  6: $batstat = 'Charging'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case  8: $batstat = 'Charging and Low'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case  9: $batstat = 'Charging and Critical'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 10: $batstat = 'Undefined'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 1: $techn = 'Other'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 7: $techn = 'Zinc-air'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case  3: $batstat = 'Fully Charged'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case  3: $batstat = 'Fully Charged'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 10: $batstat = 'Undefined'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 1: $techn = 'Other'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 5: $techn = 'NiMH'; break;

• Exclude checks
  • Disable engine
  • Disable check

CASE keyword must be followed by a single space

Open

                                case  3: $batstat = 'Fully Charged'; break;

• Exclude checks
  • Disable engine
  • Disable check

CASE keyword must be followed by a single space

Open

                                case  9: $batstat = 'Charging and Critical'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 7: $techn = 'Zinc-air'; break;

• Exclude checks
  • Disable engine
  • Disable check

CASE keyword must be followed by a single space

Open

                                case  2: $batstat = 'AC connected'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case  2: $batstat = 'AC connected'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case  9: $batstat = 'Charging and Critical'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 10: $batstat = 'Undefined'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case 8: $techn = 'Li-poly'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case  1: $batstat = 'Discharging'; break;

• Exclude checks
  • Disable engine
  • Disable check

CASE keyword must be followed by a single space

Open

                                case  6: $batstat = 'Charging'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case  6: $batstat = 'Charging'; break;

• Exclude checks
  • Disable engine
  • Disable check

Terminating statement must be on a line by itself

Open

                                case  7: $batstat = 'Charging and High'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 1: $techn = 'Other'; break;

• Exclude checks
  • Disable engine
  • Disable check

Each PHP statement must be on a line by itself

Open

                                case 7: $techn = 'Zinc-air'; break;

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at end of control structure

Open

• 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

Terminating statement must be on a line by itself

Open

                                case 8: $techn = 'Li-poly'; break;

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 8: $techn = 'Li-poly'; break;

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at end of control structure

Open

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                                case 7: $techn = 'Zinc-air'; break;

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at end of control structure

Open

• 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

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

Open

            break;

• Exclude checks
  • Disable engine
  • Disable check

Blank line found at end of control structure

Open

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 125 characters

Open

                    $bufferBFCC = CommonFunctions::getWMI($_wmi, 'BatteryFullChargedCapacity', array('FullChargedCapacity'));

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 132 characters

Open

                            $buffer[$bi]['info'] .= 'POWER_SUPPLY_VOLTAGE_MIN_DESIGN='.($bufferWPB[$bi]['DesignVoltage']*1000)."\n";

• 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

Inline control structures are not allowed

Open

                            if ($batstat != '') $buffer[$bi]['state'] .= 'POWER_SUPPLY_STATUS='.$batstat."\n";

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 228 characters

Open

                    $this->global_error->addError("WMI connect error", "PhpSysInfo can not connect to the WMI interface for security reasons.\nCheck an authentication mechanism for the directory where phpSysInfo is installed.");

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 182 characters

Open

                    $bufferWPB = CommonFunctions::getWMI($_cim, 'Win32_PortableBattery', array('DesignVoltage', 'Chemistry', 'DesignCapacity', 'FullChargeCapacity', 'Manufacturer'));

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 136 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/voltage_max_design'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 177 characters

Open

                $bufferWB = CommonFunctions::getWMI($_cim, 'Win32_Battery', array('Caption', 'Name', 'EstimatedChargeRemaining', 'DesignVoltage', 'BatteryStatus', 'Chemistry'));

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 132 characters

Open

                                $buffer[$bi]['state'] .= 'POWER_SUPPLY_ENERGY_NOW='.($bufferBS[$bi]['RemainingCapacity']*1000)."\n";

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

                        if ($techn != '') $buffer[$bi]['info'] .= 'POWER_SUPPLY_TECHNOLOGY='.$techn."\n";

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

                            if ($capacity != '') $buffer[$bi]['state'] .= 'POWER_SUPPLY_ENERGY_NOW='.(round($capacity*$bufferWPB[$bi]['DesignCapacity']*10)."\n");

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 123 characters

Open

                                $buffer[$bi]['state'] .= 'POWER_SUPPLY_VOLTAGE_NOW='.($bufferBS[$bi]['Voltage']*1000)."\n";

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 162 characters

Open

                            if ($capacity != '') $buffer[$bi]['state'] .= 'POWER_SUPPLY_ENERGY_NOW='.(round($capacity*$bufferWPB[$bi]['DesignCapacity']*10)."\n");

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 126 characters

Open

                            if (CommonFunctions::rfts($batdevices[$i], $pbuffer, 1, 4096, false) && trim($pbuffer[0]==="1")) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 133 characters

Open

                            $buffer[$bi]['info'] .= 'POWER_SUPPLY_ENERGY_FULL_DESIGN='.($bufferWPB[$bi]['DesignCapacity']*1000)."\n";

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 126 characters

Open

                        if (!isset($bufferWPB[$bi]['FullChargeCapacity']) && isset($bufferBFCC[$bi]['FullChargedCapacity'])) {

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

                            if (isset($bufferWPB[$bi]['DesignCapacity']) && ($bufferWPB[$bi]['DesignCapacity']>0))

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 126 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/batt_vol'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 128 characters

Open

                } elseif (preg_match('/^native-path:\s*(.*)$/', $roworig, $data) && isset($data[1][0]) && ($data[1][0]!=='/')) {

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

            if (sizeof($bat)>0) $this->_result[$bi] = $bat;

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 136 characters

Open

            if ((!isset($bat_item['remaining_capacity']) || (isset($bat_item['full_capacity']) && ($bat_item['full_capacity'] == 0))) &&

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 128 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/energy_now'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 124 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/health'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 135 characters

Open

                                   && (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/serial_number'))!==null)) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 139 characters

Open

                                CommonFunctions::rfts(preg_replace('/\/present$/', '/uevent', $batdevices[$i]), $infoitem, 0, 4096, false);

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 133 characters

Open

                                   || (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/voltage_max'))!==null)) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 129 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/charge_full'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 124 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/status'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 124 characters

Open

                    $xmlbat->addAttribute("RemainingCapacity", round($bat_item['capacity']*$bat_item['full_capacity']/100));

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

            if (isset($this->_filecontent[$bi]['info'])) foreach ($this->_filecontent[$bi]['info'] as $roworig) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 123 characters

Open

                } elseif (preg_match('/^hw.sensors.acpibat0.watthour3=(.*) Wh \(remaining capacity\)/', $roworig, $data)) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 129 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/energy_full'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 127 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/batt_temp'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 128 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/charge_now'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 130 characters

Open

                            $buffer[$bi]['info'] .= 'POWER_SUPPLY_ENERGY_FULL='.($bufferWPB[$bi]['FullChargeCapacity']*1000)."\n";

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 126 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/capacity'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 133 characters

Open

                               (($bufferBS[$bi]['RemainingCapacity']>0) || ($hasvolt && ($bufferBS[$bi]['RemainingCapacity']==0)))) {

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

                            if ($capacity != '') $buffer[$bi]['state'] .= 'POWER_SUPPLY_ENERGY_NOW='.(round($capacity*$bufferWPB[$bi]['FullChargeCapacity']*10)."\n");

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 124 characters

Open

                               && !preg_match('/^Device: \/org\/freedesktop\/UPower\/devices\/DisplayDevice/', $infoitem)) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 143 characters

Open

                            $rfts_bs = CommonFunctions::rfts(preg_replace('/\/info$/', '/state', $batdevices[$i]), $stateitem, 0, 4096, false);

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 135 characters

Open

                                if ((($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/voltage_min_design'))!==null)

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 129 characters

Open

                                $buffer[$bi]['state'] .= 'POWER_SUPPLY_VOLTAGE_NOW='.($bufferWB[$bi]['DesignVoltage']*1000)."\n";

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 166 characters

Open

                            if ($capacity != '') $buffer[$bi]['state'] .= 'POWER_SUPPLY_ENERGY_NOW='.(round($capacity*$bufferWPB[$bi]['FullChargeCapacity']*10)."\n");

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

                    if ($data[1]>0) $bat['battery_temperature'] = $data[1]/100;

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 137 characters

Open

                                $buffer[$bi]['info'] .= 'POWER_SUPPLY_ENERGY_FULL_DESIGN='.($bufferWPB[$bi]['DesignCapacity']*1000)."\n";

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 130 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/manufacturer'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 129 characters

Open

                                    if (preg_match('/^\/sys\/class\/power_supply\/(.+)\/present$/', $batdevices[$i], $batname)) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 124 characters

Open

                } elseif (preg_match('/^hw.sensors.acpibat0.watthour0=(.*) Wh \(last full capacity\)$/', $roworig, $data)) {

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

                            if ($capacity != '') $buffer[$bi]['state'] .= 'POWER_SUPPLY_CAPACITY='.$capacity."\n";

• Exclude checks
  • Disable engine
  • Disable check

Inline control structures are not allowed

Open

            if (isset($this->_filecontent[$bi]['state'])) foreach ($this->_filecontent[$bi]['state'] as $roworig) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 121 characters

Open

                } elseif (preg_match('/^hw.sensors.acpibat0.watthour4=(.*) Wh \(design capacity\)$/', $roworig, $data)) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 129 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/voltage_now'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 122 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/temp'))!==null) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 168 characters

Open

                if (isset($bat_item['design_voltage_max']) && ($bat_item['design_voltage_max']>0) && ($bat_item['design_voltage_max'] != $bat_item['design_voltage'])) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 128 characters

Open

                                if (($buffer1 = CommonFunctions::rolv($batdevices[$i], '/\/present$/', '/technology'))!==null) {

• 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 at least 28 spaces, found 24

Open

                        $buffer[$bi]['info'] = '';

• Exclude checks
  • Disable engine
  • Disable check

Closing brace must be on a line by itself

Open

                                case  1: $batstat = 'Discharging'; break;

• Exclude checks
  • Disable engine
  • Disable check