railpage/railpagecore

Method getUnreviewedImages has 33 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function getUnreviewedImages($page = 1, $itemsPerPage = 25) {
        
        if (!$this->User instanceof User) {
            throw new Exception("Cannot find unreviewed images - no user has been set");
        }

Method reviewImage has 29 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function reviewImage(Image $imageObject, User $userObject, $publish = 0, $pick = 0) {
        
        $publish = (bool) intval($publish);
        $pick = (bool) intval($pick); 
        

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

Open

    public function getSkippedImages($page = 1, $itemsPerPage = 25) {
        
        $query = "SELECT SQL_CALC_FOUND_ROWS i.*
            FROM image_flags_skip AS f
            LEFT JOIN image AS i ON f.image_id = i.id

• Read up
  Read up

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

Open

    public function getRejectedImages($page = 1, $itemsPerPage = 25) {
        
        $query = "SELECT SQL_CALC_FOUND_ROWS u.user_id, u.username, CONCAT('/user/', u.user_id) AS url, i.*
            FROM image_flags AS f
            LEFT JOIN nuke_users AS u ON f.screened_by = u.user_id

• Read up
  Read up

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

Open

        $row['user_avatar'] = [
            "tiny" => AvatarUtility::Format($av, 25, 25),
            "thumb" => AvatarUtility::Format($av, 50, 50),
            "small" => AvatarUtility::Format($av, 75, 75),
            "medium" => AvatarUtility::Format($av, 100, 100)

• Read up
  Read up

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

Open

    public function getSkippedBy(Image $imageObject) {
        
        $query = "SELECT u.username, u.user_id FROM nuke_users AS u LEFT JOIN image_flags_skip AS p ON p.user_id = u.user_id WHERE p.image_id = ?";
        
        $rs = $this->db->fetchAll($query, $imageObject->id); 

• Read up
  Read up

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

Open

        if ($provider == null) {
            $where = [ "id = ?" => $id ];
            $this->db->delete("image", $where); 
            
            return $this;

• Read up
  Read up

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

Open

        $params = [ 
            $this->User->id,
            ($page - 1) * $itemsPerPage, 
            $itemsPerPage 
        ];

• Read up
  Read up

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

Open

        if (!filter_var($userObject->id, FILTER_VALIDATE_INT)) {
            return false;
        }

• Read up
  Read up

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

Open

    public function getScreeners() {
        
        $query = "SELECT u.user_id, u.username, CONCAT('/user/', u.user_id) AS url, COUNT(*) AS num
            FROM image_flags AS f
            LEFT JOIN nuke_users AS u ON f.screened_by = u.user_id 

• Read up
  Read up

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

Open

        if ($pick == true) {
            $publish = true;
        }

• Read up
  Read up

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

Open

            $this->db->quote(intval($imageObject->id)), 

• Read up
  Read up

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

Open

            $this->db->quote(intval($imageObject->id)), 

• Read up
  Read up

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

Open

            $this->db->quote(intval($userObject->id)), 

• Read up
  Read up

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

Open

            $this->db->quote(intval($userObject->id))

• Read up
  Read up

Blank line found at start of control structure

Open

        if (!$date) {

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check

Function closing brace must go on the next line following the body; found 1 blank lines before brace

Open

    }

• Exclude checks
  • Disable engine
  • Disable check