TrackingRelationships::model() - Code Metrics - Inspection of "FIX phpunit requirement" - sfneal/tracking - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 6b0e6b...43789e )

by Stephen

created 2020-11-12 15:02 UTC

TrackingRelationships::model() C

↳ Parent: TrackingRelationships

Complexity

Conditions	12
Paths	7

Size

Total Lines	30
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	12
eloc	14
c	1
b	0
f	0
nc	7
nop	0
dl	0
loc	30
rs	6.9666

How to fix Complexity

<?php

namespace Sfneal\Tracking\Models\Traits;

use Domain\Plans\Models\Plan;
filter:
    dependency_paths: ["lib/*"]
use Domain\Plans\Models\PlanManagement;
filter:
    dependency_paths: ["lib/*"]
use Domain\Projects\Models\Project;
filter:
    dependency_paths: ["lib/*"]
use Domain\Tasks\Models\Task;
filter:
    dependency_paths: ["lib/*"]
use Domain\Tasks\Models\TaskRecord;
filter:
    dependency_paths: ["lib/*"]
use Illuminate\Database\Eloquent\Relations\BelongsTo;

// todo: probably move back to project
trait TrackingRelationships
{
    /**
     * Retrieve the TrackActivity instances Plan model if one exists.
     *
     * @return mixed|null
     */
    public function model()
    {
        // Plan
        if ($this->model_table == 'plan') {
            return $this->plan;
        } elseif (inString($this->model_table, 'plan_management')) {
            // Return Plan model if it exists
            if ($this->planManagement->plan) {
                return $this->planManagement->plan;
            }

            // Otherwise return the Project model
            else {
                return $this->planManagement->project;
            }
        }

        // Project
        elseif ($this->model_table == 'project') {
            return $this->project;
        }

        // Task
        elseif ($this->model_table == 'task_record' && $this->taskRecord && $this->taskRecord->task && $this->taskRecord->task->project) {
            return $this->taskRecord->task->project;
        } elseif ($this->model_table == 'task' && $this->task && $this->task->project) {
            return $this->task->project;
        }

        return null;
    }

    /**
     * Retrieve a route for this model.
     *
     * @return string
     */
    public function modelRoute()
    {
        // Plan
        if ($this->model() instanceof Plan) {
            return route('plans.plan.show', ['plan'=>$this->model()->getKey(), 'modal'=>1]);
        }

        // Project
        elseif ($this->model() instanceof Project) {
            return route('projects.show', ['project'=>$this->model()->getKey()]);
        }
    }

    /**
     * Retrieve the link class.
     *
     * @return string
     */
    public function linkClass(): string
    {
        if ($this->model() instanceof Plan) {
            return 'modal-link';
        } else {
            return '';
        }
    }

    /**
     * Plan relationship.
     *
     * @return BelongsTo
     */
    public function plan()
    {
        return $this->belongsTo(Plan::class, 'model_key', 'plan_id');
        return $this->/** @scrutinizer ignore-call */ belongsTo(Plan::class, 'model_key', 'plan_id');
    }

    /**
     * Plan management relationship.
     *
     * @return BelongsTo
     */
    public function planManagement()
    {
        return $this->belongsTo(PlanManagement::class, 'model_key', 'project_id');
    }

    /**
     * Project relationship.
     *
     * @return BelongsTo
     */
    public function project()
    {
        return $this->belongsTo(Project::class, 'model_key', 'project_id');
    }

    /**
     * Task relationship.
     *
     * @return BelongsTo
     */
    public function task()
    {
        return $this->belongsTo(Task::class, 'model_key', 'task_id');
    }

    /**
     * Task relationship.
     *
     * @return BelongsTo
     */
    public function taskRecord()
    {
        return $this->belongsTo(TaskRecord::class, 'model_key', 'record_id');
    }
}


sfneal / tracking

Push — master ( 6b0e6b...43789e )

TrackingRelationships::model() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like