Todo

Complexity

Total Complexity

6

Size/Duplication

Total Lines	50
Duplicated Lines	44 %

Coupling/Cohesion

Components	1
Dependencies	3

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getAll()	13	13	1
A	create()	0	23	3
A	delete()	9	9	2

 <?php

It is not recommended to output anything before PHP's opening tag in non-template files.

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 10 and the first side effect is on line 1.

 
 /**
  * Todo Class
  *
  * @license    http://opensource.org/licenses/MIT The MIT License (MIT)
  * @author     Omar El Gabry <omar.elgabry.93@gmail.com>
  */

class Todo extends Model{

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

    public function getAll(){

This method seems to be duplicated in your project.

        $database = Database::openConnection();
        $query  = "SELECT todo.id AS id, users.id AS user_id, users.name AS user_name, todo.content ";
        $query .= "FROM users, todo ";
        $query .= "WHERE users.id = todo.user_id ";

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

        return $todo;
     }

    public function create($userId, $content){

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

        // using database class to insert new todo
        $database = Database::openConnection();
        $query    = "INSERT INTO todo (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 create todo");
        }

        return true;
     }

    public function delete($id){

This method seems to be duplicated in your project.

        $database = Database::openConnection();
        $database->deleteById("todo", $id);

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