Taskforcedev\LaravelForum\Models\ForumReply

Complexity

Total Complexity

4

Size/Duplication

Total Lines	49
Duplicated Lines	28.57 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	author()	0	6	1
A	valid()	14	14	2
A	getReplyCount()	0	5	1

<?php

namespace Taskforcedev\LaravelForum\Models;

use \DB;
use \Validator;
use Taskforcedev\LaravelForum\Helpers\UserHelper;

/**
 * Class ForumReply
 *
 * @property string  $body      Post body.
 * @property integer $post_id   Post ID.
 * @property integer $author_id Author ID.
 *
 * @package Taskforcedev\LaravelForum\Models
 */
class ForumReply extends AbstractModel
{
    protected $table = 'forum_post_replies';

    protected $hidden = [];
    protected $fillable = ['body', 'post_id', 'author_id'];

    /**
     * Eloquent Relation.
     * @return mixed
     */
    public function author()
    {
        $userHelper = new UserHelper();
        $model = $userHelper->getUserModel();
        return $this->belongsTo($model);

It seems like $model defined by $userHelper->getUserModel() on line 32 can also be of type boolean; however, Illuminate\Database\Eloquent\Model::belongsTo() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * Validates model data.
     * @param array $data The data to validate.
     *
     * @return boolean
     */
    public static function valid($data)
    {
        $rules = [
            'body' => 'required|min:3',
            'post_id' => 'required|min:1|integer|exists:forum_posts,id',
            'author_id' => 'required|min:1|integer|exists:users,id'
        ];
        $validator = Validator::make($data, $rules);

        if ($validator->passes()) {
            return true;
        }
        return false;
    }

    /**
     * Gets the number of replies from the database.
     * @return mixed
     */
    public static function getReplyCount()
    {
        $results = DB::select('select COUNT(*) as count from forum_post_replies');
        return $results[0]->count;
    }
}