railpage/railpagecore

View on GitHub

Showing 1,011 of 9,214 total issues

The method validate() has an NPath complexity of 12288. The configured NPath complexity threshold is 200.
Open

    public function validate() {
        
        if (is_null($this->bbcode_uid)) {
            $this->bbcode_uid = "sausages";
        }
Severity: Minor
Found in lib/Forums/Post.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method commit() has an NPath complexity of 640. The configured NPath complexity threshold is 200.
Open

    public function commit() {
        if (empty($this->bbcode_uid)) {
            $this->bbcode_uid = crc32($this->text);
        }
        
Severity: Minor
Found in lib/Forums/Post.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

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

    function __construct($threadid = false) {
        
        if (RP_DEBUG) {
            global $site_debug;
            $debug_timer_start = microtime(true);
Severity: Minor
Found in lib/Forums/Thread.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method validate() has an NPath complexity of 512. The configured NPath complexity threshold is 200.
Open

    function validate() {
        if (empty($this->title)) {
            throw new Exception("Thread title cannot be empty");
        }
        
Severity: Minor
Found in lib/Forums/Thread.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method LatLonWoELookup() has an NPath complexity of 1158. The configured NPath complexity threshold is 200.
Open

    public static function LatLonWoELookup($lat, $lon, $force = false) {
        
        if (is_null($lon) && strpos($lat, ",") !== false) {
            $tmp = explode(",", $lat); 
            $lat = $tmp[0];
Severity: Minor
Found in lib/PlaceUtility.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method render() has an NPath complexity of 28800. The configured NPath complexity threshold is 200.
Open

    public function render() {
        
        if (!$this->userObject instanceof User) {
            throw new InvalidArgumentException("No valid user object has been provided"); 
        }
Severity: Minor
Found in lib/Prerender/Home.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method send() has 114 lines of code. Current threshold is set to 100. Avoid really long methods.
Open

        public function send($markAsSent = false) {
            
            $this->validate(); 
            
            $Smarty = AppCore::getSmarty(); 
Severity: Minor
Found in lib/Reminders/Reminder.php by phpmd

The method hybi10Encode() has an NPath complexity of 864. The configured NPath complexity threshold is 200.
Open

    private function hybi10Encode($payload, $type = 'text', $masked = true)
    {
        $frameHead = array();
        $payloadLength = strlen($payload);
        switch ($type) {
Severity: Minor
Found in lib/SocketIO.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method getPhotoLegacy() has an NPath complexity of 742. The configured NPath complexity threshold is 200.
Open

    public function getPhotoLegacy($photo_id = false) {
        if (!$photo_id) {
            throw new Exception("Cannot fetch photo info and sizes - no photo ID given"); 
            return false;
        }
Severity: Minor
Found in lib/Railcams/Camera.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method load() has an NPath complexity of 16202. The configured NPath complexity threshold is 200.
Open

    public function load() {
        $photodata = $this->Provider->getPhoto($this->id);
                
        $this->title = $photodata['title'];
        $this->description = $photodata['description'];
Severity: Minor
Found in lib/Railcams/Photo.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method getPhotoLegacy() has 105 lines of code. Current threshold is set to 100. Avoid really long methods.
Open

    public function getPhotoLegacy($photo_id = false) {
        if (!$photo_id) {
            throw new Exception("Cannot fetch photo info and sizes - no photo ID given"); 
            return false;
        }
Severity: Minor
Found in lib/Railcams/Camera.php by phpmd

The method send() has an NPath complexity of 392. The configured NPath complexity threshold is 200.
Open

        public function send($markAsSent = false) {
            
            $this->validate(); 
            
            $Smarty = AppCore::getSmarty(); 
Severity: Minor
Found in lib/Reminders/Reminder.php by phpmd

NPathComplexity

Since: 0.1

The NPath complexity of a method is the number of acyclic execution paths through that method. A threshold of 200 is generally considered the point where measures should be taken to reduce complexity.

Example

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

Source https://phpmd.org/rules/codesize.html#npathcomplexity

The method render() has 209 lines of code. Current threshold is set to 100. Avoid really long methods.
Open

    public function render() {
        
        if (!$this->userObject instanceof User) {
            throw new InvalidArgumentException("No valid user object has been provided"); 
        }
Severity: Minor
Found in lib/Prerender/Home.php by phpmd

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

    public static function lookup($type, $id, $activeOnly) {
        
        $Database = (new AppCore)->getDatabaseConnection(); 
        
        if ($type == "ip") {
Severity: Minor
Found in lib/BanControl/LookupUtility.php by phpmd

CyclomaticComplexity

Since: 0.1

Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

Example

// 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;
            }
        }
    }
}

Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

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

        public function parse() {
            if (empty($this->feeds)) {
                throw new Exception("No scraped RSS data was found (hint: Consume::scrape())");
            }
            
Severity: Minor
Found in lib/RSS/Consume.php by phpmd

CyclomaticComplexity

Since: 0.1

Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

Example

// 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;
            }
        }
    }
}

Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

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

        public function addTiming($day = false, $time = false, $going = "arr", $commencing = false, $expiring = false) {
            if (!$this->Point instanceof Point) {
                throw new Exception("Cannot add a timing because no valid timetable point has been set");
            }
            
Severity: Minor
Found in lib/Timetables/Train.php by phpmd

CyclomaticComplexity

Since: 0.1

Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

Example

// 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;
            }
        }
    }
}

Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

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

        public function getContents($page = 1, $items_per_page = 25) {
            if (empty($this->folder)) {
                throw new Exception("Cannot get folder contents - no folder specified"); 
            } 
            
Severity: Minor
Found in lib/PrivateMessages/Folder.php by phpmd

CyclomaticComplexity

Since: 0.1

Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

Example

// 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;
            }
        }
    }
}

Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

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

        private function load() {
            
            $this->mckey = sprintf("railpage:locations.location=%d", $this->id); 
            
            if (!$row = $this->Memcached->fetch($this->mckey)) {
Severity: Minor
Found in lib/Locations/Location.php by phpmd

CyclomaticComplexity

Since: 0.1

Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

Example

// 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;
            }
        }
    }
}

Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

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

        public function setCommodity($commodity) {
            switch (strtolower($commodity)) {
                case "intermodal" :
                    $this->commodity = self::COMMODITY_INTERMODAL;
                    break;
Severity: Minor
Found in lib/Timetables/Train.php by phpmd

CyclomaticComplexity

Since: 0.1

Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

Example

// 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;
            }
        }
    }
}

Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

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

        public function getRegions($country = false) {
            
            $timer = Debug::GetTimer();
            
            $return = false;
Severity: Minor
Found in lib/Locations/Locations.php by phpmd

CyclomaticComplexity

Since: 0.1

Complexity is determined by the number of decision points in a method plus one for the method entry. The decision points are 'if', 'while', 'for', and 'case labels'. Generally, 1-4 is low complexity, 5-7 indicates moderate complexity, 8-10 is high complexity, and 11+ is very high complexity.

Example

// 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;
            }
        }
    }
}

Source https://phpmd.org/rules/codesize.html#cyclomaticcomplexity

Severity
Category
Status
Source
Language