The method xmlapi::xmlapi_query() calls the typical debug function print_r() which is mostly only used during development.

Open

                error_log("SimpleXML var_dump:\n" . print_r($response, true));

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

class SuspectCode {

    public function doSomething(array $items)
    {
        foreach ($items as $i => $item) {
            // …

            if ('qafoo' == $item) var_dump($i);

            // …
        }
    }
}

The method xmlapi::xmlapi_query() calls the typical debug function print_r() which is mostly only used during development.

Open

                error_log("Associative Array var_dump:\n" . print_r($response, true));

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

class SuspectCode {

    public function doSomething(array $items)
    {
        foreach ($items as $i => $item) {
            // …

            if ('qafoo' == $item) var_dump($i);

            // …
        }
    }
}

File xmlapi.php has 1064 lines of code (exceeds 250 allowed). Consider refactoring.

Open

<?php
/**
* cPanel XMLAPI Client Class
*
* This class allows for easy interaction with cPanel's XML-API allow functions within the XML-API to be called

xmlapi has 104 functions (exceeds 20 allowed). Consider refactoring.

Open

class xmlapi
{
    // should debugging statements be printed?
    private $debug            = false;

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

Open

    public function __construct($host = null, $user = null, $password = null)
    {
        // Check if debugging must be enabled
        if ((defined('XMLAPI_DEBUG')) && (XMLAPI_DEBUG == '1')) {
            $this->debug = true;

• Read up
  Read up

Function xmlapi_query has a Cognitive Complexity of 26 (exceeds 5 allowed). Consider refactoring.

Open

    public function xmlapi_query($function, $vars = [])
    {
        // Check to make sure all the data needed to perform the query is in place
        if (!$function) {
            throw new Exception('xmlapi_query() requires a function to be passed to it');

• Read up
  Read up

Method xmlapi_query has 67 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function xmlapi_query($function, $vars = [])
    {
        // Check to make sure all the data needed to perform the query is in place
        if (!$function) {
            throw new Exception('xmlapi_query() requires a function to be passed to it');

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

Open

    public function __construct($host = null, $user = null, $password = null)
    {
        // Check if debugging must be enabled
        if ((defined('XMLAPI_DEBUG')) && (XMLAPI_DEBUG == '1')) {
            $this->debug = true;

Consider simplifying this complex logical expression.

Open

        if (!isset($args['xemail']) || !isset($args['host']) || !isset($args['country']) || !isset($args['state']) || !isset($args['city']) || !isset($args['co']) || !isset($args['cod']) || !isset($args['email']) || !isset($args['pass'])) {
            error_log("generatessl requires that xemail, host, country, state, city, co, cod, email and pass are defined in the array passed to it");

            return false;
        }

Function setresellerpackagelimits has a Cognitive Complexity of 11 (exceeds 5 allowed). Consider refactoring.

Open

    public function setresellerpackagelimits($user, $no_limit, $package = null, $allowed = null, $number = null)
    {
        if (!isset($user) || !isset($no_limit)) {
            error_log("setresellerpackagelimits requires that a username and no_limit are passed to it by default");

• Read up
  Read up

Method api1_query has 28 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

Open

    public function setresellerpackagelimits($user, $no_limit, $package = null, $allowed = null, $number = null)
    {
        if (!isset($user) || !isset($no_limit)) {
            error_log("setresellerpackagelimits requires that a username and no_limit are passed to it by default");

Consider simplifying this complex logical expression.

Open

        if (!isset($args['user']) || !isset($args['domain']) || !isset($args['cert']) || !isset($args['key']) || !isset($args['cab']) || !isset($args['ip'])) {
            error_log("installssl requires that user, domain, cert, key, cab and ip are defined in the array passed to it");

            return false;
        }

Method setresellerpackagelimits has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    public function setresellerpackagelimits($user, $no_limit, $package = null, $allowed = null, $number = null)

Avoid too many return statements within this method.

Open

        return $response;

Function stat has a Cognitive Complexity of 6 (exceeds 5 allowed). Consider refactoring.

Open

    public function stat($username, $args = null)
    {
        if ((!isset($username)) || (!isset($args))) {
            error_log("stat requires that a username and options are passed to it");

• Read up
  Read up

The class xmlapi has 2338 lines of code. Current threshold is 1000. Avoid really long classes.

Open

class xmlapi
{
    // should debugging statements be printed?
    private $debug            = false;

• Exclude checks
  • Disable engine
  • Disable check

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

Open

        curl_setopt($curl, CURLOPT_HTTPHEADER, $header);

• 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 '$values' which will lead to PHP notices.

Open

            $values['display'] = substr($args, 0, -1);

• 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 '$values' which will lead to PHP notices.

Open

        return $this->api2_query($username, 'StatsBar', 'stat', $values);

• 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 '$header' which will lead to PHP notices.

Open

        $header[0] =$authstr .

• 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 '$values' which will lead to PHP notices.

Open

            $values['display'] = substr($display, 0, -1);

• 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 '759', column '23').

Open

            throw new Exception("curl_exec threw error \"" . curl_error($curl) . "\" for " . $url . "?" . $postdata);

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

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

Missing class import via use statement (line '343', column '23').

Open

            throw new Exception('non integer or negative integer passed to set_port');

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

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

Missing class import via use statement (line '663', column '23').

Open

            throw new Exception('invalid auth_type set');

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

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

Missing class import via use statement (line '378', column '23').

Open

            throw new Exception('https and http are the only protocols that can be passed to set_protocol');

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

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

Missing class import via use statement (line '417', column '23').

Open

            throw new Exception('json, xml, array and simplexml are the only allowed values for set_output');

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

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

Missing class import via use statement (line '591', column '23').

Open

            throw new Exception('only curl and fopen and allowed http clients');

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

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

Missing class import via use statement (line '236', column '27').

Open

                throw new Exception("No host defined");

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

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

Missing class import via use statement (line '253', column '23').

Open

            throw new Exception('allow_url_fopen and curl are neither available in this PHP configuration');

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

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

Missing class import via use statement (line '636', column '23').

Open

            throw new Exception('no authentication information has been set');

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

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

Missing class import via use statement (line '451', column '23').

Open

            throw new Exception('the only two allowable auth types arehash and path');

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

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

Missing class import via use statement (line '628', column '23').

Open

            throw new Exception('xmlapi_query() requires a function to be passed to it');

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

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

Missing class import via use statement (line '769', column '23').

Open

            throw new Exception('fopen_query called on system without allow_url_fopen enabled in php.ini');

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

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

Missing class import via use statement (line '632', column '23').

Open

            throw new Exception('no user has been set');

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

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

Avoid unused local variables such as '$key'.

Open

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

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

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

Avoid using count() function in for loops.

Open

        for ($int = 0; $int < count($args);  $int++) {
            $call['arg-' . $int] = $args[$int];
        }

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

class Foo {

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

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

The parameter $no_limit is not named in camelCase.

Open

    public function setresellerpackagelimits($user, $no_limit, $package = null, $allowed = null, $number = null)
    {
        if (!isset($user) || !isset($no_limit)) {
            error_log("setresellerpackagelimits requires that a username and no_limit are passed to it by default");

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The class xmlapi is not named in CamelCase.

Open

class xmlapi
{
    // should debugging statements be printed?
    private $debug            = false;

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

class class_name {
}

The property $auth_type is not named in camelCase.

Open

class xmlapi
{
    // should debugging statements be printed?
    private $debug            = false;

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

class ClassName {
    protected $property_name;
}

The parameter $reseller_cfg is not named in camelCase.

Open

    public function setresellerlimits($reseller_cfg)
    {
        if (!isset($reseller_cfg['user'])) {
            error_log("setresellerlimits requires that a user is defined in the array passed to it");

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The parameter $auth_type is not named in camelCase.

Open

    public function set_auth_type($auth_type)
    {
        if ($auth_type != 'hash' && $auth_type != 'pass') {
            throw new Exception('the only two allowable auth types arehash and path');
        }

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

class ClassName {
    public function doSomething($user_name) {
    }
}

The property $http_client is not named in camelCase.

Open

class xmlapi
{
    // should debugging statements be printed?
    private $debug            = false;

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

class ClassName {
    protected $property_name;
}

The 'get_debug()' method which returns a boolean should be named 'is...()' or 'has...()'

Open

    public function get_debug()
    {
        return $this->debug;
    }

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

class Foo {
    /**
     * @return boolean
     */
    public function getFoo() {} // bad
    /**
     * @return bool
     */
    public function isFoo(); // ok
    /**
     * @return boolean
     */
    public function getFoo($bar); // ok, unless checkParameterizedMethods=true
}

The variable $auth_type is not named in camelCase.

Open

    public function set_auth_type($auth_type)
    {
        if ($auth_type != 'hash' && $auth_type != 'pass') {
            throw new Exception('the only two allowable auth types arehash and path');
        }

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

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

The variable $reseller_cfg is not named in camelCase.

Open

    public function setresellerlimits($reseller_cfg)
    {
        if (!isset($reseller_cfg['user'])) {
            error_log("setresellerlimits requires that a user is defined in the array passed to it");

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

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

The variable $query_type is not named in camelCase.

Open

    public function xmlapi_query($function, $vars = [])
    {
        // Check to make sure all the data needed to perform the query is in place
        if (!$function) {
            throw new Exception('xmlapi_query() requires a function to be passed to it');

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

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

The variable $module_type is not named in camelCase.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

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

The variable $query_type is not named in camelCase.

Open

    public function xmlapi_query($function, $vars = [])
    {
        // Check to make sure all the data needed to perform the query is in place
        if (!$function) {
            throw new Exception('xmlapi_query() requires a function to be passed to it');

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

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

The variable $func_type is not named in camelCase.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

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

The variable $module_type is not named in camelCase.

Open

    public function api2_query($user, $module, $function, $args = [])
    {
        if (!isset($user) || !isset($module) || !isset($function)) {
            error_log("api2_query requires that a username, module and function are passed to it");

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

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

The variable $func_type is not named in camelCase.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

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

The variable $api_type is not named in camelCase.

Open

    public function api2_query($user, $module, $function, $args = [])
    {
        if (!isset($user) || !isset($module) || !isset($function)) {
            error_log("api2_query requires that a username, module and function are passed to it");

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

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

The variable $auth_type is not named in camelCase.

Open

    public function set_auth_type($auth_type)
    {
        if ($auth_type != 'hash' && $auth_type != 'pass') {
            throw new Exception('the only two allowable auth types arehash and path');
        }

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

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

The variable $api_type is not named in camelCase.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

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

The variable $api_type is not named in camelCase.

Open

    public function api2_query($user, $module, $function, $args = [])
    {
        if (!isset($user) || !isset($module) || !isset($function)) {
            error_log("api2_query requires that a username, module and function are passed to it");

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

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

The variable $func_type is not named in camelCase.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

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

The variable $func_type is not named in camelCase.

Open

    public function api2_query($user, $module, $function, $args = [])
    {
        if (!isset($user) || !isset($module) || !isset($function)) {
            error_log("api2_query requires that a username, module and function are passed to it");

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

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

The variable $api_type is not named in camelCase.

Open

    public function api2_query($user, $module, $function, $args = [])
    {
        if (!isset($user) || !isset($module) || !isset($function)) {
            error_log("api2_query requires that a username, module and function are passed to it");

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

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

The variable $no_limit is not named in camelCase.

Open

    public function setresellerpackagelimits($user, $no_limit, $package = null, $allowed = null, $number = null)
    {
        if (!isset($user) || !isset($no_limit)) {
            error_log("setresellerpackagelimits requires that a username and no_limit are passed to it by default");

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

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

The variable $query_type is not named in camelCase.

Open

    public function xmlapi_query($function, $vars = [])
    {
        // Check to make sure all the data needed to perform the query is in place
        if (!$function) {
            throw new Exception('xmlapi_query() requires a function to be passed to it');

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

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

The variable $module_type is not named in camelCase.

Open

    public function api2_query($user, $module, $function, $args = [])
    {
        if (!isset($user) || !isset($module) || !isset($function)) {
            error_log("api2_query requires that a username, module and function are passed to it");

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

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

The variable $module_type is not named in camelCase.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

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

The variable $module_type is not named in camelCase.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

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

The variable $func_type is not named in camelCase.

Open

    public function api2_query($user, $module, $function, $args = [])
    {
        if (!isset($user) || !isset($module) || !isset($function)) {
            error_log("api2_query requires that a username, module and function are passed to it");

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

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

The variable $auth_type is not named in camelCase.

Open

    public function set_auth_type($auth_type)
    {
        if ($auth_type != 'hash' && $auth_type != 'pass') {
            throw new Exception('the only two allowable auth types arehash and path');
        }

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

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

The variable $no_limit is not named in camelCase.

Open

    public function setresellerpackagelimits($user, $no_limit, $package = null, $allowed = null, $number = null)
    {
        if (!isset($user) || !isset($no_limit)) {
            error_log("setresellerpackagelimits requires that a username and no_limit are passed to it by default");

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

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

The variable $api_type is not named in camelCase.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

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

The variable $api_type is not named in camelCase.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

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

The variable $func_type is not named in camelCase.

Open

    public function api2_query($user, $module, $function, $args = [])
    {
        if (!isset($user) || !isset($module) || !isset($function)) {
            error_log("api2_query requires that a username, module and function are passed to it");

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

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

The variable $reseller_cfg is not named in camelCase.

Open

    public function setresellerlimits($reseller_cfg)
    {
        if (!isset($reseller_cfg['user'])) {
            error_log("setresellerlimits requires that a user is defined in the array passed to it");

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

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

The variable $module_type is not named in camelCase.

Open

    public function api2_query($user, $module, $function, $args = [])
    {
        if (!isset($user) || !isset($module) || !isset($function)) {
            error_log("api2_query requires that a username, module and function are passed to it");

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

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

The method set_debug is not named in camelCase.

Open

    public function set_debug($debug = 1)
    {
        $this->debug = $debug;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method fopen_query is not named in camelCase.

Open

    private function fopen_query($url, $postdata, $authstr)
    {
        if (!(ini_get('allow_url_fopen'))) {
            throw new Exception('fopen_query called on system without allow_url_fopen enabled in php.ini');
        }

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

class ClassName {
    public function get_name() {
    }
}

The method get_debug is not named in camelCase.

Open

    public function get_debug()
    {
        return $this->debug;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method hash_auth is not named in camelCase.

Open

    public function hash_auth($user, $hash)
    {
        $this->set_hash($hash);
        $this->set_user($user);
    }

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

class ClassName {
    public function get_name() {
    }
}

The method set_http_client is not named in camelCase.

Open

    public function set_http_client($client)
    {
        if (($client != 'curl') && ($client != 'fopen')) {
            throw new Exception('only curl and fopen and allowed http clients');
        }

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

class ClassName {
    public function get_name() {
    }
}

The method get_port is not named in camelCase.

Open

    public function get_port()
    {
        return $this->port;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method unserialize_xml is not named in camelCase.

Open

    private function unserialize_xml($input, $callback = null, $recurse = false)
    {
        // Get input, loading an xml string with simplexml if its the top level of recursion
        $data = ((!$recurse) && is_string($input)) ? simplexml_load_string($input) : $input;
        // Convert SimpleXMLElements to array

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

class ClassName {
    public function get_name() {
    }
}

The method api2_query is not named in camelCase.

Open

    public function api2_query($user, $module, $function, $args = [])
    {
        if (!isset($user) || !isset($module) || !isset($function)) {
            error_log("api2_query requires that a username, module and function are passed to it");

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

class ClassName {
    public function get_name() {
    }
}

The method return_xml is not named in camelCase.

Open

    public function return_xml()
    {
        $this->set_output('xml');
    }

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

class ClassName {
    public function get_name() {
    }
}

The method get_protocol is not named in camelCase.

Open

    public function get_protocol()
    {
        return $this->protocol;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method get_auth_type is not named in camelCase.

Open

    public function get_auth_type()
    {
        return $this->auth_type;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method get_http_client is not named in camelCase.

Open

    public function get_http_client()
    {
        return $this->http_client;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method get_host is not named in camelCase.

Open

    public function get_host()
    {
        return $this->host;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method set_auth_type is not named in camelCase.

Open

    public function set_auth_type($auth_type)
    {
        if ($auth_type != 'hash' && $auth_type != 'pass') {
            throw new Exception('the only two allowable auth types arehash and path');
        }

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

class ClassName {
    public function get_name() {
    }
}

The method password_auth is not named in camelCase.

Open

    public function password_auth($user, $pass)
    {
        $this->set_password($pass);
        $this->set_user($user);
    }

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

class ClassName {
    public function get_name() {
    }
}

The method set_password is not named in camelCase.

Open

    public function set_password($pass)
    {
        $this->auth_type = 'pass';
        $this->auth = $pass;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method get_output is not named in camelCase.

Open

    public function get_output()
    {
        return $this->output;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method set_output is not named in camelCase.

Open

    public function set_output($output)
    {
        if ($output != 'json' && $output != 'xml' && $output != 'array' && $output != 'simplexml') {
            throw new Exception('json, xml, array and simplexml are the only allowed values for set_output');
        }

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

class ClassName {
    public function get_name() {
    }
}

The method curl_query is not named in camelCase.

Open

    private function curl_query($url, $postdata, $authstr)
    {
        $curl = curl_init();
        curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, 0);
        // Return contents of transfer on curl_exec

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

class ClassName {
    public function get_name() {
    }
}

The method get_user is not named in camelCase.

Open

    public function get_user()
    {
        return $this->user;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method return_object is not named in camelCase.

Open

    public function return_object()
    {
        $this->set_output('simplexml');
    }

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

class ClassName {
    public function get_name() {
    }
}

The method set_host is not named in camelCase.

Open

    public function set_host($host)
    {
        $this->host = $host;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method set_port is not named in camelCase.

Open

    public function set_port($port)
    {
        if (!is_int($port)) {
            $port = intval($port);
        }

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

class ClassName {
    public function get_name() {
    }
}

The method set_user is not named in camelCase.

Open

    public function set_user($user)
    {
        $this->user = $user;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method api1_query is not named in camelCase.

Open

    public function api1_query($user, $module, $function, $args = [])
    {
        if (!isset($module) || !isset($function) || !isset($user)) {
            error_log("api1_query requires that a module and function are passed to it");

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

class ClassName {
    public function get_name() {
    }
}

The method xmlapi_query is not named in camelCase.

Open

    public function xmlapi_query($function, $vars = [])
    {
        // Check to make sure all the data needed to perform the query is in place
        if (!$function) {
            throw new Exception('xmlapi_query() requires a function to be passed to it');

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

class ClassName {
    public function get_name() {
    }
}

The method set_protocol is not named in camelCase.

Open

    public function set_protocol($proto)
    {
        if ($proto != 'https' && $proto != 'http') {
            throw new Exception('https and http are the only protocols that can be passed to set_protocol');
        }

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

class ClassName {
    public function get_name() {
    }
}

The method set_hash is not named in camelCase.

Open

    public function set_hash($hash)
    {
        $this->auth_type = 'hash';
        $this->auth = preg_replace("/(\n|\r|\s)/", '', $hash);
    }

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

class ClassName {
    public function get_name() {
    }
}