Function locate has a Cognitive Complexity of 38 (exceeds 5 allowed). Consider refactoring.

Open

    public function locate($segments)
    {
        // Clear var $this->directory before search controller in function locate() of the Router class.
        // Solve the problem of trying to load a "root" controller using Modules::run('controller/method') after loading a module controller
        // with Modules::run('module/controller/method')

• Read up
  Read up

Method locate has 60 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function locate($segments)
    {
        // Clear var $this->directory before search controller in function locate() of the Router class.
        // Solve the problem of trying to load a "root" controller using Modules::run('controller/method') after loading a module controller
        // with Modules::run('module/controller/method')

Function _set_module_path has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    protected function _set_module_path(&$_route)
    {
        if (! empty($_route)) {
            // Are module/directory/controller/method segments being specified?
            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

• Read up
  Read up

Function _set_request has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    protected function _set_request($segments = [])
    {
        if ($this->translate_uri_dashes === true) {
            foreach (range(0, 2) as $v) {
                isset($segments[$v]) && $segments[$v] = str_replace('-', '_', $segments[$v]);

• Read up
  Read up

Avoid too many return statements within this method.

Open

            return array_slice($segments, 1);

Avoid too many return statements within this method.

Open

                return array_slice($segments, 2);

Avoid too many return statements within this method.

Open

            return $segments;

Avoid too many return statements within this method.

Open

                return array_slice($segments, 1);

The method locate() has an NPath complexity of 4704. The configured NPath complexity threshold is 200.

Open

    public function locate($segments)
    {
        // Clear var $this->directory before search controller in function locate() of the Router class.
        // Solve the problem of trying to load a "root" controller using Modules::run('controller/method') after loading a module controller
        // with Modules::run('module/controller/method')

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

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

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

Open

    protected function _set_module_path(&$_route)
    {
        if (! empty($_route)) {
            // Are module/directory/controller/method segments being specified?
            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

Open

    public function locate($segments)
    {
        // Clear var $this->directory before search controller in function locate() of the Router class.
        // Solve the problem of trying to load a "root" controller using Modules::run('controller/method') after loading a module controller
        // with Modules::run('module/controller/method')

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

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

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

Open

                    case 2: $_route = ($this->located < 2) ? $module.'/'.$directory : $directory.'/index';

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

Open

            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

Open

            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

Open

                    case 4: $_route = ($this->located == 3) ? $class.'/'.$method : $method.'/index';

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

Open

                    case 2: $_route = ($this->located < 2) ? $module.'/'.$directory : $directory.'/index';

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

Open

            if ($this->locate([$module, $directory, $class])) {

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

Open

                    case 3: $_route = ($this->located == 2) ? $directory.'/'.$class : $class.'/index';

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

Open

                    case 3: $_route = ($this->located == 2) ? $directory.'/'.$class : $class.'/index';

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

Open

            if ($this->locate([$module, $directory, $class])) {

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

Open

                    case 4: $_route = ($this->located == 3) ? $class.'/'.$method : $method.'/index';

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

Open

            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

Open

                    case 3: $_route = ($this->located == 2) ? $directory.'/'.$class : $class.'/index';

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

Open

                    case 4: $_route = ($this->located == 3) ? $class.'/'.$method : $method.'/index';

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

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

Avoid using static access to class 'Modules' in method 'locate'.

Open

        if (isset($segments[0]) && $routes = Modules::parse_routes($segments[0], implode('/', $segments))) {

• 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 '143', column '36').

Open

    public function locate($segments)
    {
        // Clear var $this->directory before search controller in function locate() of the Router class.
        // Solve the problem of trying to load a "root" controller using Modules::run('controller/method') after loading a module controller
        // with Modules::run('module/controller/method')

• 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 '160', column '24').

Open

    public function locate($segments)
    {
        // Clear var $this->directory before search controller in function locate() of the Router class.
        // Solve the problem of trying to load a "root" controller using Modules::run('controller/method') after loading a module controller
        // with Modules::run('module/controller/method')

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

Open

                    } else {
                        $this->located = -1;
                    }

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

Open

        } else {
            $segments[1] = 'index';
        }

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

Open

        } else {
            [$module, $directory, $controller] = array_pad($segments, 3, null);
        }

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

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

The phpdoc comment for @method cannot occur on a method

Open

     * @method fetch_module

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->config

Open

        $ext = $this->config->item('controller_suffix').EXT;

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @method fetch_module'

Open

     * @method fetch_module

• Exclude checks
  • Disable engine
  • Disable check

The phpdoc comment for @method cannot occur on a method

Open

     * @method _set_404override_controller

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->directory (Did you mean $directory)

Open

                $this->directory = $module.'/'.$directory.'/';

• Exclude checks
  • Disable engine
  • Disable check

The phpdoc comment for @method cannot occur on a method

Open

     * @method locate

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \APPPATH

Open

        if (is_dir(APPPATH.'controllers/'.$module.'/')) {

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @method _set_module_path'

Open

     * @method _set_module_path

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->translate_uri_dashes

Open

        if ($this->translate_uri_dashes === true) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->uri

Open

        $this->uri->rsegments = $segments;

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @method _set_request'

Open

     * @method _set_request

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->routes

Open

        $this->_set_module_path($this->routes['404_override']);

• Exclude checks
  • Disable engine
  • Disable check

The phpdoc comment for @method cannot occur on a method

Open

     * @method _set_default_controller

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->class

Open

        if (empty($this->class)) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->directory (Did you mean $directory)

Open

                        $this->directory .= $directory.'/';

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \APPPATH

Open

        if (is_file(APPPATH.'controllers/'.ucfirst($module).$ext)) {

• Exclude checks
  • Disable engine
  • Disable check

The phpdoc comment for @method cannot occur on a method

Open

     * @method set_class

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @method set_class'

Open

     * @method set_class

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @method locate'

Open

     * @method locate

• Exclude checks
  • Disable engine
  • Disable check

The phpdoc comment for @method cannot occur on a method

Open

     * @method _set_module_path

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->config

Open

        $suffix = $this->config->item('controller_suffix');

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @method _set_404override_controller'

Open

     * @method _set_404override_controller

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @method _set_default_controller'

Open

     * @method _set_default_controller

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @param [type] $segments [description]'

Open

     * @param  [type] $segments [description]

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->directory (Did you mean $directory)

Open

        if (! empty($this->directory)) {

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @return [type] [description]'

Open

     * @return [type]           [description]

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->directory (Did you mean $directory)

Open

            $this->directory = $module.'/';

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @param [type] &$_route [description]'

Open

     * @param  [type]  &$_route [description]

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared class \CI_Router (Did you mean class \MX_Router or class \MY_Router)

Open

        parent::set_class($class);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared class \CI_Router (Did you mean class \MX_Router or class \MY_Router)

Open

        parent::_set_default_controller();

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->directory (Did you mean $directory)

Open

                $this->directory = $offset.$module.'/controllers/';

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \APPPATH

Open

            if (is_file(APPPATH.'controllers/'.$module.'/'.ucfirst($directory).$ext)) {

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->directory

Open

        if (empty($this->directory)) {

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @param [type] $class [description]'

Open

     * @param  [type]    $class [description]

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->directory

Open

        $this->directory = null;

• Exclude checks
  • Disable engine
  • Disable check

Saw unextractable annotation for comment '* @return [type] [description]'

Open

     * @return [type]       [description]

• Exclude checks
  • Disable engine
  • Disable check

Call to undeclared method \MX_Router::set_method

Open

            $this->set_method($segments[1]);

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->default_controller

Open

            $this->_set_module_path($this->default_controller);

• Exclude checks
  • Disable engine
  • Disable check

Class extends undeclared class \CI_Router (Did you mean class \MX_Router or class \MY_Router)

Open

class MX_Router extends CI_Router

• Exclude checks
  • Disable engine
  • Disable check

The phpdoc comment for @method cannot occur on a method

Open

     * @method _set_request

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared property \MX_Router->directory (Did you mean $directory)

Open

                $this->directory = $module.'/';

• Exclude checks
  • Disable engine
  • Disable check

Reference to undeclared constant \APPPATH

Open

            if ($controller && is_file(APPPATH . 'controllers/' . $module . '/' . $directory . '/' . ucfirst($controller) . $ext)) {

• Exclude checks
  • Disable engine
  • Disable check

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

Open

class MX_Router extends CI_Router

• Exclude checks
  • Disable engine
  • Disable check

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 38 and the first side effect is on line 1.

Open

<?php defined('BASEPATH') or exit('No direct script access allowed');

• Exclude checks
  • Disable engine
  • Disable check

The parameter $_route is not named in camelCase.

Open

    protected function _set_module_path(&$_route)
    {
        if (! empty($_route)) {
            // Are module/directory/controller/method segments being specified?
            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

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

The class MX_Router is not named in CamelCase.

Open

class MX_Router extends CI_Router
{
    public $module;
    private $located = 0;

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

class class_name {
}

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

Open

    protected function _set_request($segments = [])

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                    case 1: $_route = $module.'/index';

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    protected function _set_module_path(&$_route)

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                    case 2: $_route = ($this->located < 2) ? $module.'/'.$directory : $directory.'/index';

• Exclude checks
  • Disable engine
  • Disable check

Method name "MX_Router::_set_request" is not in camel caps format

Open

    protected function _set_request($segments = [])

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    protected function _set_default_controller()

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                    case 3: $_route = ($this->located == 2) ? $directory.'/'.$class : $class.'/index';

• Exclude checks
  • Disable engine
  • Disable check

Method name "MX_Router::set_class" is not in camel caps format

Open

    public function set_class($class)

• Exclude checks
  • Disable engine
  • Disable check

Method name "MX_Router::_set_default_controller" is not in camel caps format

Open

    protected function _set_default_controller()

• Exclude checks
  • Disable engine
  • Disable check

The CASE body must start on the line following the statement

Open

                    case 4: $_route = ($this->located == 3) ? $class.'/'.$method : $method.'/index';

• Exclude checks
  • Disable engine
  • Disable check

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

Open

    protected function _set_404override_controller()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MX_Router::fetch_module" is not in camel caps format

Open

    public function fetch_module()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MX_Router::_set_404override_controller" is not in camel caps format

Open

    protected function _set_404override_controller()

• Exclude checks
  • Disable engine
  • Disable check

Method name "MX_Router::_set_module_path" is not in camel caps format

Open

    protected function _set_module_path(&$_route)

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 132 characters

Open

            if ($controller && is_file(APPPATH . 'controllers/' . $module . '/' . $directory . '/' . ucfirst($controller) . $ext)) {

• Exclude checks
  • Disable engine
  • Disable check

Line exceeds 120 characters; contains 140 characters

Open

        // Solve the problem of trying to load a "root" controller using Modules::run('controller/method') after loading a module controller

• Exclude checks
  • Disable engine
  • Disable check

Class name "MX_Router" is not in camel caps format

Open

class MX_Router extends CI_Router

• Exclude checks
  • Disable engine
  • Disable check

The variable $_route is not named in camelCase.

Open

    protected function _set_module_path(&$_route)
    {
        if (! empty($_route)) {
            // Are module/directory/controller/method segments being specified?
            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

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

The variable $_route is not named in camelCase.

Open

    protected function _set_module_path(&$_route)
    {
        if (! empty($_route)) {
            // Are module/directory/controller/method segments being specified?
            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

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

The variable $_route is not named in camelCase.

Open

    protected function _set_module_path(&$_route)
    {
        if (! empty($_route)) {
            // Are module/directory/controller/method segments being specified?
            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

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

The variable $_route is not named in camelCase.

Open

    protected function _set_module_path(&$_route)
    {
        if (! empty($_route)) {
            // Are module/directory/controller/method segments being specified?
            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

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

The variable $_route is not named in camelCase.

Open

    protected function _set_module_path(&$_route)
    {
        if (! empty($_route)) {
            // Are module/directory/controller/method segments being specified?
            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

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

The variable $_route is not named in camelCase.

Open

    protected function _set_module_path(&$_route)
    {
        if (! empty($_route)) {
            // Are module/directory/controller/method segments being specified?
            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

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

The method _set_404override_controller is not named in camelCase.

Open

    protected function _set_404override_controller()
    {
        $this->_set_module_path($this->routes['404_override']);
    }

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

class ClassName {
    public function get_name() {
    }
}

The method set_class is not named in camelCase.

Open

    public function set_class($class)
    {
        $suffix = $this->config->item('controller_suffix');
        // Fixing Error Message: strpos(): Non-string needles will be interpreted as strings in the future.
        // Use an explicit chr() call to preserve the current behavior.

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

class ClassName {
    public function get_name() {
    }
}

The method _set_request is not named in camelCase.

Open

    protected function _set_request($segments = [])
    {
        if ($this->translate_uri_dashes === true) {
            foreach (range(0, 2) as $v) {
                isset($segments[$v]) && $segments[$v] = str_replace('-', '_', $segments[$v]);

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

class ClassName {
    public function get_name() {
    }
}

The method _set_default_controller is not named in camelCase.

Open

    protected function _set_default_controller()
    {
        if (empty($this->directory)) {
            // set the default controller module path
            $this->_set_module_path($this->default_controller);

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

class ClassName {
    public function get_name() {
    }
}

The method fetch_module is not named in camelCase.

Open

    public function fetch_module()
    {
        return $this->module;
    }

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

class ClassName {
    public function get_name() {
    }
}

The method _set_module_path is not named in camelCase.

Open

    protected function _set_module_path(&$_route)
    {
        if (! empty($_route)) {
            // Are module/directory/controller/method segments being specified?
            $sgs = sscanf($_route, '%[^/]/%[^/]/%[^/]/%s', $module, $directory, $class, $method);

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

class ClassName {
    public function get_name() {
    }
}