Angelov\Storgman\Documents\Document

Complexity

Total Complexity

28

Size/Duplication

Total Lines	171
Duplicated Lines	0 %

Coupling/Cohesion

Components	3
Dependencies	4

Importance

Changes

0

24 Methods

Rating	Name	Duplication	Size	Complexity
A	getId()	0	4	1
A	getTitle()	0	4	1
A	setTitle()	0	4	1
A	getDescription()	0	4	1
A	setDescription()	0	4	1
A	getUrl()	0	4	1
A	setUrl()	0	4	1
A	submitter()	0	4	1
A	getSubmitter()	0	4	1
A	setSubmitter()	0	4	1
A	openedBy()	0	4	1
A	addOpener()	0	4	1
A	getOpeners()	0	4	1
A	countOpenings()	0	4	1
A	countOpeners()	0	13	3
A	tags()	0	4	1
A	addTag()	0	4	1
A	getTags()	0	4	1
A	getCreatedAt()	0	4	1
A	getUpdatedAt()	0	4	1
A	setVisibleToAllMembers()	0	4	1
A	isVisibleToAllMembers()	0	4	1
A	setVisibleToBoardOnly()	0	4	1
A	save()	0	12	3

<?php

/**
 * Storgman - Student Organizations Management
 * Copyright (C) 2014-2016, Dejan Angelov <angelovdejan92@gmail.com>
 *
 * This file is part of Storgman.
 *
 * Storgman is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * Storgman is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with Storgman.  If not, see <http://www.gnu.org/licenses/>.
 *
 * @package Storgman
 * @copyright Copyright (C) 2014-2016, Dejan Angelov <angelovdejan92@gmail.com>
 * @license https://github.com/angelov/storgman/blob/master/LICENSE
 * @author Dejan Angelov <angelovdejan92@gmail.com>
 */

namespace Angelov\Storgman\Documents;

use Angelov\Storgman\Members\Member;
use Angelov\Storgman\Documents\Tags\Tag;
use Illuminate\Database\Eloquent\Model;

class Document extends Model
{
    protected $table = "documents";

    protected $tagList = [];
    protected $openedByList = [];

    public function getId()
    {
        return $this->getAttribute('id');
    }

    public function getTitle()
    {
        return $this->getAttribute('title');
    }

    public function setTitle($title)
    {
        $this->setAttribute('title', $title);
    }

    public function getDescription()
    {
        return $this->getAttribute('description');
    }

    public function setDescription($description)
    {
        $this->setAttribute('description', $description);
    }

    public function getUrl()
    {
        return $this->getAttribute('url');
    }

    public function setUrl($url)
    {
        $this->setAttribute('url', $url);
    }

    public function submitter()
    {
        return $this->belongsTo(Member::class, 'submitted_by');
    }

    /**
     * @return Member
     */
    public function getSubmitter()
    {
        return $this->submitter;

The property submitter does not exist on object<Angelov\Storgman\Documents\Document>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * @param Member $member
     */
    public function setSubmitter(Member $member)
    {
        $this->submitter()->associate($member);
    }

    public function openedBy()
    {
        return $this->belongsToMany(Member::class, 'document_openings')->withTimestamps();
    }

    public function addOpener(Member $member)
    {
        $this->openedByList[] = $member;
    }

    /**
     * @return Member[]
     */
    public function getOpeners()
    {
        return $this->openedBy->all();

The property openedBy does not seem to exist. Did you mean openedByList?

    }

    public function countOpenings()
    {
        return $this->openedBy->count();

The property openedBy does not seem to exist. Did you mean openedByList?

    }

    public function countOpeners()
    {
        $openers = $this->getOpeners();
        $counted = [];

        foreach ($openers as $opener) {
            if (! in_array($opener->getId(), $counted)) {
                $counted[] = $opener->getId();
            }
        }

        return count($counted);
    }

    public function tags()
    {
        return $this->belongsToMany(Tag::class);
    }

    /**
     * @param Tag $tag
     */
    public function addTag(Tag $tag)
    {
        $this->tagList[] = $tag;
    }

    /**
     * @return Tag[]
     */
    public function getTags()
    {
        return $this->tags;

The property tags does not exist on object<Angelov\Storgman\Documents\Document>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    /**
     * @return \DateTime
     */
    public function getCreatedAt()
    {
        return $this->getAttribute('created_at');
    }

    /**
     * @return \DateTime
     */
    public function getUpdatedAt()
    {
        return $this->getAttribute('updated_at');
    }

    /**
     * By default, all members have access to the document
     */
    public function setVisibleToAllMembers()
    {
        $this->setAttribute('board_only', false);
    }

    /**
     * @return bool
     */
    public function isVisibleToAllMembers()
    {
        return (! $this->getAttribute('board_only'));
    }

    public function setVisibleToBoardOnly()
    {
        $this->setAttribute('board_only', true);
    }

    public function save(array $options = [])
    {
        parent::save($options);

        foreach ($this->tagList as $tag) {
            $this->tags()->attach($tag);
        }

        foreach ($this->openedByList as $opener) {
            $this->openedBy()->attach($opener);
        }
    }
}