Doctor::city() - Code Metrics - Inspection of "Thread answer 1" - flying-coders/SIFIKS - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#65)

by Faiq

created 2019-04-21 18:27 UTC

Doctor::city() A

↳ Parent: Doctor

Complexity

Conditions	1
Paths	1

Size

Total Lines	2
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	1
dl	0
loc	2
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php

namespace App;

use Carbon\Carbon;
use App\City;
use App\DoctorSpecialization;
use Illuminate\Notifications\Notifiable;
use Illuminate\Contracts\Auth\MustVerifyEmail;
use Illuminate\Foundation\Auth\User as Authenticatable;

class Doctor extends Authenticatable
{
    use Notifiable;


    protected $guard = 'doctor';

    /**
     * The attributes that are mass assignable.
     *
     * @var array
     */
    protected $fillable = [
        'name', 'biography', 'profile_picture', 'email', 'password'
    ];

    /**
     * The attributes that should be hidden for arrays.
     *
     * @var array
     */
    protected $hidden = [
        'password', 'remember_token',
    ];

    /**
     * The attributes that should be cast to native types.
     *
     * @var array
     */
    protected $casts = [
        'email_verified_at' => 'datetime',
    ];

    /**
     * @var array
     */
    protected $dates = [
        'created_at',
        'updated_at'
    ];

    /**
     * @return \Illuminate\Database\Eloquent\Relations\HasMany
     */
    public function article()
    {
        return $this->hasMany('App\Articles');
    }

//    /**
//     * @return \Illuminate\Database\Eloquent\Relations\HasMany
//     */
//    public function detail()
//    {
//        return $this->hasMany('App\DoctorDetail');
//    }

    /**
     * @return \Illuminate\Database\Eloquent\Relations\BelongsTo
     */
    public function specialty()
    {
        return $this->belongsTo('App\DoctorSpecialization', 'specialization_id');
    }

    public function city() {
        return $this->belongsTo('App\City', 'city_id');
    }

    public function thread() {
        return $this->hasMany('App\Thread', 'doctor_id');
    }

    /**
     * @param $str
     * @return string
     */
    public function trimStr($str)
    {
        if(strlen($str) > 20) {
            return substr($str, 0, 20)."...";
        }
        return $str;
    }

    /**
     * @return string
     */
    public function getGreetings()
    {
        $h = Carbon::now()->format('H');

        if ($h >= 0 && $h < 6) {
            return "<i class='fa far fa-surprise fa-2x'></i>&nbsp;&nbsp;Wow, ini masih pagi loh ";
        } elseif ($h >= 6 && $h < 11) {
            return "<i class='fa far fa-laugh-beam fa-2x'></i>&nbsp;&nbsp;Selamat pagi ";
        } elseif ($h >= 11 && $h <= 14) {
            return "<i class='fa far fa-smile-wink fa-2x'></i>&nbsp;&nbsp;Selamat siang ";
        } elseif ($h > 14 && $h <= 16) {
            return "<i class='fa far fa-smile-beam fa-2x'></i>&nbsp;&nbsp;Selamat sore ";
        } elseif ($h >= 17 && $h < 20) {
            return "<i class='fa far fa-grin-stars fa-2x'></i>&nbsp;&nbsp;Senja yang indah ya ";
        } else {
            return "<i class='fa far fa-bed fa-2x'></i>&nbsp;&nbsp;Sudah malam, sebaiknya kamu istirahat ";
        }
    }

    /**
     * @return mixed
     */
    public function getSpecialty()
    {
        $specialtyId = $this->getAttributeValue('specialization_id');

        return $specialty = DoctorSpecialization::find($specialtyId);

    }

    /**
     * @return mixed
     */
    public function getLocation()
    {
        $locationId = $this->getAttributeValue('city_id');

        return $loc = City::find($locationId);

    }
}


1			<?php
2
3			namespace App;
4
5			use Carbon\Carbon;
6			use App\City;
7			use App\DoctorSpecialization;
8			use Illuminate\Notifications\Notifiable;
9			use Illuminate\Contracts\Auth\MustVerifyEmail;
10			use Illuminate\Foundation\Auth\User as Authenticatable;
11
12			class Doctor extends Authenticatable
13			{
14			use Notifiable;
			0 ignored issues – show introduced 2019-04-05 00:46 UTC by Report Bug Copy Issue Report The trait `Illuminate\Notifications\Notifiable` requires some properties which are not provided by `App\Doctor`: `$email`, `$phone_number` Loading history...
15
16			protected $guard = 'doctor';
17
18			/**
19			* The attributes that are mass assignable.
20			*
21			* @var array
22			*/
23			protected $fillable = [
24			'name', 'biography', 'profile_picture', 'email', 'password'
25			];
26
27			/**
28			* The attributes that should be hidden for arrays.
29			*
30			* @var array
31			*/
32			protected $hidden = [
33			'password', 'remember_token',
34			];
35
36			/**
37			* The attributes that should be cast to native types.
38			*
39			* @var array
40			*/
41			protected $casts = [
42			'email_verified_at' => 'datetime',
43			];
44
45			/**
46			* @var array
47			*/
48			protected $dates = [
49			'created_at',
50			'updated_at'
51			];
52
53			/**
54			* @return \Illuminate\Database\Eloquent\Relations\HasMany
55			*/
56			public function article()
57			{
58			return $this->hasMany('App\Articles');
59			}
60
61			// /**
62			// * @return \Illuminate\Database\Eloquent\Relations\HasMany
63			// */
64			// public function detail()
65			// {
66			// return $this->hasMany('App\DoctorDetail');
67			// }
68
69			/**
70			* @return \Illuminate\Database\Eloquent\Relations\BelongsTo
71			*/
72			public function specialty()
73			{
74			return $this->belongsTo('App\DoctorSpecialization', 'specialization_id');
75			}
76
77			public function city() {
78			return $this->belongsTo('App\City', 'city_id');
79			}
80
81			public function thread() {
82			return $this->hasMany('App\Thread', 'doctor_id');
83			}
84
85			/**
86			* @param $str
87			* @return string
88			*/
89			public function trimStr($str)
90			{
91			if(strlen($str) > 20) {
92			return substr($str, 0, 20)."...";
93			}
94			return $str;
95			}
96
97			/**
98			* @return string
99			*/
100			public function getGreetings()
101			{
102			$h = Carbon::now()->format('H');
103
104			if ($h >= 0 && $h < 6) {
105			return "<i class='fa far fa-surprise fa-2x'></i>  Wow, ini masih pagi loh ";
106			} elseif ($h >= 6 && $h < 11) {
107			return "<i class='fa far fa-laugh-beam fa-2x'></i>  Selamat pagi ";
108			} elseif ($h >= 11 && $h <= 14) {
109			return "<i class='fa far fa-smile-wink fa-2x'></i>  Selamat siang ";
110			} elseif ($h > 14 && $h <= 16) {
111			return "<i class='fa far fa-smile-beam fa-2x'></i>  Selamat sore ";
112			} elseif ($h >= 17 && $h < 20) {
113			return "<i class='fa far fa-grin-stars fa-2x'></i>  Senja yang indah ya ";
114			} else {
115			return "<i class='fa far fa-bed fa-2x'></i>  Sudah malam, sebaiknya kamu istirahat ";
116			}
117			}
118
119			/**
120			* @return mixed
121			*/
122			public function getSpecialty()
123			{
124			$specialtyId = $this->getAttributeValue('specialization_id');
125
126			return $specialty = DoctorSpecialization::find($specialtyId);
			0 ignored issues – show Unused Code introduced 2019-04-11 18:45 UTC by Report Bug Copy Issue Report The assignment to `$specialty` is dead and can be removed. Loading history...
127			}
128
129			/**
130			* @return mixed
131			*/
132			public function getLocation()
133			{
134			$locationId = $this->getAttributeValue('city_id');
135
136			return $loc = City::find($locationId);
			0 ignored issues – show Unused Code introduced 2019-04-11 15:46 UTC by Report Bug Copy Issue Report The assignment to `$loc` is dead and can be removed. Loading history...
137			}
138			}
139

flying-coders / SIFIKS

Pull Request — master (#65)

Doctor::city() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like