Nicklas\Comment\Modules\Post

Complexity

Total Complexity

2

Size/Duplication

Total Lines	88
Duplicated Lines	53.41 %

Coupling/Cohesion

Components	1
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
B	getAllPosts()	25	25	1
A	getPost()	22	22	1

<?php

namespace Nicklas\Comment\Modules;

/**
 * A database driven model.
 */
class Post extends ActiveRecordModelExtender
{

        /**
     * @var string $tableName name of the database table.
     */
    protected $tableName = "ramverk1_posts";

    /**
     * Columns in the table.
     *
     * @var integer $id primary key auto incremented.
     */
    public $id;
    public $user;
    public $questionId; # Connection to new Question()
    public $accepted; # default "no"

    public $type; # question/answer
    public $text;

    public $created;

    public $di;



    /**
     * Returns post with markdown and gravatar
     * @param string $sql
     * @param array $param

There is no parameter named $param. Did you maybe mean $params?

     *
     * @return array
     */
    public function getAllPosts($sql, $params)

This method seems to be duplicated in your project.

    {
        $posts = $this->findAllWhere("$sql", $params);

        return array_map(function ($post) {

            // Get e-mail for Post
            $user = new User($this->db);
            $post->img = $user->getGravatar($post->user);

            // Get comments for Post
            $comment = new Comment($this->db);
            $post->comments = $comment->getComments("parentId = ?", [$post->id]);


            // Get votes for Post
            $vote = new Vote($this->db);
            $post->vote = $vote->getVote("parentId = ? AND parentType = ?", [$post->id, "post"]);

            // Get text
            $post->markdown = $this->getMD($post->text);

            return $post;
        }, $posts);
    }

    /**
     * return question/answer, three attributes are set, comments connected to them is an array.
     *
     * @return object
    */
    public function getPost($sql, $params)

This method seems to be duplicated in your project.

    {
        $post = $this->findWhere("$sql", $params);

        // Get e-mail for question
        $user = new User($this->db);
        $post->img = $user->getGravatar($post->user);

        // Get comments for Post
        $comment = new Comment($this->db);
        $post->comments = $comment->getComments("parentId = ?", [$post->id]);

        // Get votes for Post
        $vote = new Vote($this->db);
        $post->vote = $vote->getVote("parentId = ? AND parentType = ?", [$post->id, "post"]);
        
        // Start setting attributes
        $post->markdown = $this->getMD($post->text);


        return $post;
    }
}