user_insert

Complexity

Total Complexity

5

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	check_view_channel()	0	7	2
A	add_view_channel()	0	5	1
A	__construct()	0	2	1
A	send_message()	0	9	1

<?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) {

It is generally recommended to explicitly declare the visibility for methods.

        $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) {

It is generally recommended to explicitly declare the visibility for methods.

        $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") {

It is generally recommended to explicitly declare the visibility for methods.

        $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();
    }   
}

?>

It is not recommended to use PHP's closing tag ?> in files other than templates.