NewsFeed::create() - Code Metrics - OmarElGabry/miniPHP - Measure and Improve Code Quality continuously with Scrutinizer

NewsFeed::create() B
last analyzed 2018-01-04 10:22 UTC

↳ Parent: NewsFeed

Complexity

Conditions	3
Paths	3

Size

Total Lines	24
Code Lines	16

Duplication

Lines	24
Ratio	100 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	24
loc	24
rs	8.9713
cc	3
eloc	16
nc	3
nop	2

<?php

 /**
  * NewsFeed Class
  *
  * @license    http://opensource.org/licenses/MIT The MIT License (MIT)
  * @author     Omar El Gabry <[email protected]>
  */
class NewsFeed extends Model{
namespace YourVendor;

class YourClass { }

    /**
     * get all news feed.
     *
     * @access public
     * @param  integer  $pageNum
     * @return array
     *
     */
    public function getAll($pageNum = 1){


        $pagination = Pagination::pagination("newsfeed", "", [], $pageNum);
        $offset     = $pagination->getOffset();
        $limit      = $pagination->perPage;

        $database   = Database::openConnection();
        $query  = "SELECT newsfeed.id AS id, users.profile_picture, users.id AS user_id, users.name AS user_name, newsfeed.content, newsfeed.date ";
        $query .= "FROM users, newsfeed ";
        $query .= "WHERE users.id = newsfeed.user_id ";
        $query .= "ORDER BY newsfeed.date DESC ";
        $query .= "LIMIT $limit OFFSET $offset";

        $database->prepare($query);
        $database->execute();
        $newsfeed = $database->fetchAllAssociative();

        return array("newsfeed" => $newsfeed, "pagination" => $pagination);
     }

    /**
     * get news feed by Id.
     *
     * @param  string  $newsfeedId
     * @return array
      */
    public function getById($newsfeedId){


        $database = Database::openConnection();
        $query  = "SELECT newsfeed.id AS id, users.profile_picture, users.id AS user_id, users.name AS user_name, newsfeed.content, newsfeed.date ";
        $query .= "FROM users, newsfeed ";
        $query .= "WHERE newsfeed.id = :id ";
        $query .= "AND users.id = newsfeed.user_id  LIMIT 1 ";

        $database->prepare($query);
        $database->bindValue(':id', (int)$newsfeedId);
        $database->execute();

        $feed = $database->fetchAllAssociative();
        return $feed;
     }

    /**
     * create news feed.
     *
     * @param  integer $userId
     * @param  string  $content
     * @return array feed created
     * @throws Exception if feed couldn't be created
     */
    public function create($userId, $content){


        $validation = new Validation();
        if(!$validation->validate(['Content'   => [$content, "required|minLen(4)|maxLen(300)"]])) {
            $this->errors = $validation->errors();
            return false;
        }

        $database = Database::openConnection();
        $query    = "INSERT INTO newsfeed (user_id, content) VALUES (:user_id, :content)";

        $database->prepare($query);
        $database->bindValue(':user_id', $userId);
        $database->bindValue(':content', $content);
        $database->execute();

        if($database->countRows() !== 1){
            throw new Exception("Couldn't add news feed");
        }

        $newsfeedId = $database->lastInsertedId();
        $feed = $this->getById($newsfeedId);
        return $feed;
     }

    /**
     * update news feed.
     *
     * @param  string  $newsfeedId
     * @param  string  $content
     * @return array   feed created
     * @throws Exception if feed couldn't be updated
     */
    public function update($newsfeedId, $content){


        $validation = new Validation();
        if(!$validation->validate(['Content'   => [$content, "required|minLen(4)|maxLen(300)"]])) {
            $this->errors = $validation->errors();
            return false;
        }

        $database = Database::openConnection();
        $query = "UPDATE newsfeed SET content = :content WHERE id = :id LIMIT 1";

        $database->prepare($query);
        $database->bindValue(':content', $content);
        $database->bindValue(':id', $newsfeedId);
        $result = $database->execute();

        if(!$result){
            throw new Exception("Couldn't update newsfeed of ID: " . $newsfeedId);
        }

        $feed = $this->getById($newsfeedId);
        return $feed;
     }

 }

1		<?php
2
3		/**
4		* NewsFeed Class
5		*
6		* @license http://opensource.org/licenses/MIT The MIT License (MIT)
7		* @author Omar El Gabry <[email protected]>
8		*/
9		class NewsFeed extends Model{
		0 ignored issues – show Coding Style Compatibility introduced 2015-11-27 07:13 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
10
11		/**
12		* get all news feed.
13		*
14		* @access public
15		* @param integer $pageNum
16		* @return array
17		*
18		*/
19	View Code Duplication	public function getAll($pageNum = 1){
		0 ignored issues – show Duplication introduced 2018-01-04 10:22 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
20
21		$pagination = Pagination::pagination("newsfeed", "", [], $pageNum);
22		$offset = $pagination->getOffset();
23		$limit = $pagination->perPage;
24
25		$database = Database::openConnection();
26		$query = "SELECT newsfeed.id AS id, users.profile_picture, users.id AS user_id, users.name AS user_name, newsfeed.content, newsfeed.date ";
27		$query .= "FROM users, newsfeed ";
28		$query .= "WHERE users.id = newsfeed.user_id ";
29		$query .= "ORDER BY newsfeed.date DESC ";
30		$query .= "LIMIT $limit OFFSET $offset";
31
32		$database->prepare($query);
33		$database->execute();
34		$newsfeed = $database->fetchAllAssociative();
35
36		return array("newsfeed" => $newsfeed, "pagination" => $pagination);
37		}
38
39		/**
40		* get news feed by Id.
41		*
42		* @param string $newsfeedId
43		* @return array
44		*/
45	View Code Duplication	public function getById($newsfeedId){
		0 ignored issues – show Duplication introduced 2018-01-04 10:22 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
46
47		$database = Database::openConnection();
48		$query = "SELECT newsfeed.id AS id, users.profile_picture, users.id AS user_id, users.name AS user_name, newsfeed.content, newsfeed.date ";
49		$query .= "FROM users, newsfeed ";
50		$query .= "WHERE newsfeed.id = :id ";
51		$query .= "AND users.id = newsfeed.user_id LIMIT 1 ";
52
53		$database->prepare($query);
54		$database->bindValue(':id', (int)$newsfeedId);
55		$database->execute();
56
57		$feed = $database->fetchAllAssociative();
58		return $feed;
59		}
60
61		/**
62		* create news feed.
63		*
64		* @param integer $userId
65		* @param string $content
66		* @return array feed created
67		* @throws Exception if feed couldn't be created
68		*/
69	View Code Duplication	public function create($userId, $content){
		0 ignored issues – show Duplication introduced 2018-01-04 10:22 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
70
71		$validation = new Validation();
72		if(!$validation->validate(['Content' => [$content, "required\|minLen(4)\|maxLen(300)"]])) {
73		$this->errors = $validation->errors();
74		return false;
75		}
76
77		$database = Database::openConnection();
78		$query = "INSERT INTO newsfeed (user_id, content) VALUES (:user_id, :content)";
79
80		$database->prepare($query);
81		$database->bindValue(':user_id', $userId);
82		$database->bindValue(':content', $content);
83		$database->execute();
84
85		if($database->countRows() !== 1){
86		throw new Exception("Couldn't add news feed");
87		}
88
89		$newsfeedId = $database->lastInsertedId();
90		$feed = $this->getById($newsfeedId);
91		return $feed;
92		}
93
94		/**
95		* update news feed.
96		*
97		* @param string $newsfeedId
98		* @param string $content
99		* @return array feed created
100		* @throws Exception if feed couldn't be updated
101		*/
102	View Code Duplication	public function update($newsfeedId, $content){
		0 ignored issues – show Duplication introduced 2018-01-04 10:22 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
103
104		$validation = new Validation();
105		if(!$validation->validate(['Content' => [$content, "required\|minLen(4)\|maxLen(300)"]])) {
106		$this->errors = $validation->errors();
107		return false;
108		}
109
110		$database = Database::openConnection();
111		$query = "UPDATE newsfeed SET content = :content WHERE id = :id LIMIT 1";
112
113		$database->prepare($query);
114		$database->bindValue(':content', $content);
115		$database->bindValue(':id', $newsfeedId);
116		$result = $database->execute();
117
118		if(!$result){
119		throw new Exception("Couldn't update newsfeed of ID: " . $newsfeedId);
120		}
121
122		$feed = $this->getById($newsfeedId);
123		return $feed;
124		}
125
126		}

OmarElGabry / miniPHP

GitHub Access Token became invalid

NewsFeed::create() B last analyzed 2018-01-04 10:22 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

NewsFeed::create() B
last analyzed 2018-01-04 10:22 UTC