user_insert::send_message() - Code Metrics - Inspection of "Merge branch 'main' of https://github.com/the-real..." - the-real-sumsome/subrocks-2012 - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — main ( 24361c...cb6120 )

by chief

created 2021-10-11 05:42 UTC

user_insert::send_message() A

↳ Parent: user_insert

Complexity

Conditions	1
Paths	1

Size

Total Lines	9
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	8
nc	1
nop	6
dl	0
loc	9
rs	10
c	0
b	0
f	0

<?php

/**
* @Auther: bhief
* @Version: 1.0
*
* Inserts user information
*
**/
class user_insert {
    public $__db;

	public function __construct($conn){
        $this->__db = $conn;
	}

    function check_view_channel($vidid, $user) {

        $stmt = $this->__db->prepare("SELECT * FROM channel_views WHERE viewer = :user AND channel = :vidid");
        $stmt->bindParam(":user", $user);
        $stmt->bindParam(":vidid", $vidid);
        $stmt->execute();
        if($stmt->rowCount() === 0) {
            $this->add_view_channel($vidid, $user);
        }
    }
    
    function add_view_channel($vidid, $user) {

        $stmt = $this->__db->prepare("INSERT INTO channel_views (viewer, channel) VALUES (:user, :vidid)");
        $stmt->bindParam(":user", $user);
        $stmt->bindParam(":vidid", $vidid);
        $stmt->execute();
    }   

    function send_message($author, $subject, $to, $message, $video = "", $type = "nm") {

        $stmt = $this->__db->prepare("INSERT INTO pms (owner, subject, touser, message, video_attribute, type) VALUES (:owner, :subject, :touser, :message, :video, :type)");
        $stmt->bindParam(":owner", $author);
        $stmt->bindParam(":subject", $subject);
        $stmt->bindParam(":touser", $to);
        $stmt->bindParam(":message", $message);
        $stmt->bindParam(":video", $video);
        $stmt->bindParam(":type", $type);
        $stmt->execute();
    }   
}

?>


1			<?php
2
3			/**
4			* @Auther: bhief
5			* @Version: 1.0
6			*
7			* Inserts user information
8			*
9			**/
10			class user_insert {
11			public $__db;
12
13			public function __construct($conn){
14			$this->__db = $conn;
15			}
16
17			function check_view_channel($vidid, $user) {
			0 ignored issues – show Best Practice introduced 2021-10-01 05:40 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
18			$stmt = $this->__db->prepare("SELECT * FROM channel_views WHERE viewer = :user AND channel = :vidid");
19			$stmt->bindParam(":user", $user);
20			$stmt->bindParam(":vidid", $vidid);
21			$stmt->execute();
22			if($stmt->rowCount() === 0) {
23			$this->add_view_channel($vidid, $user);
24			}
25			}
26
27			function add_view_channel($vidid, $user) {
			0 ignored issues – show Best Practice introduced 2021-10-01 05:40 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
28			$stmt = $this->__db->prepare("INSERT INTO channel_views (viewer, channel) VALUES (:user, :vidid)");
29			$stmt->bindParam(":user", $user);
30			$stmt->bindParam(":vidid", $vidid);
31			$stmt->execute();
32			}
33
34			function send_message($author, $subject, $to, $message, $video = "", $type = "nm") {
			0 ignored issues – show Best Practice introduced 2021-10-11 05:46 UTC by Report Bug Copy Issue Report It is generally recommended to explicitly declare the visibility for methods. Adding explicit visibility (`private`, `protected`, or `public`) is generally recommend to communicate to other developers how, and from where this method is intended to be used. Loading history...
35			$stmt = $this->__db->prepare("INSERT INTO pms (owner, subject, touser, message, video_attribute, type) VALUES (:owner, :subject, :touser, :message, :video, :type)");
36			$stmt->bindParam(":owner", $author);
37			$stmt->bindParam(":subject", $subject);
38			$stmt->bindParam(":touser", $to);
39			$stmt->bindParam(":message", $message);
40			$stmt->bindParam(":video", $video);
41			$stmt->bindParam(":type", $type);
42			$stmt->execute();
43			}
44			}
45
46			?>
			0 ignored issues – show Best Practice introduced 2021-10-01 05:40 UTC by Report Bug Copy Issue Report It is not recommended to use PHP's closing tag `?>` in files other than templates. Using a closing tag in PHP files that only contain PHP code is not recommended as you might accidentally add whitespace after the closing tag which would then be output by PHP. This can cause severe problems, for example headers cannot be sent anymore. A simple precaution is to leave off the closing tag as it is not required, and it also has no negative effects whatsoever. Loading history...

the-real-sumsome / subrocks-2012

Push — main ( 24361c...cb6120 )

user_insert::send_message() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like