Venue::setDefaultLocation() - Code Metrics - Inspection of "Bump node-sass from 4.10.0 to 4.14.1" - xoco70/kendozone - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#45)

unknown

created 2020-09-03 23:56 UTC

Venue::setDefaultLocation() B

↳ Parent: Venue

Complexity

Conditions	7
Paths	12

Size

Total Lines	17
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	7

Importance

Changes

Metric	Value
cc	7
eloc	12
nc	12
nop	3
dl	0
loc	17
ccs	12
cts	12
cp	1
crap	7
rs	8.8333
c	0
b	0
f	0

<?php

namespace App;

use Illuminate\Database\Eloquent\Model;

class Venue extends Model
{
    public $timestamps = true;
    protected $table = 'venue';
    protected $fillable = [
        'venue_name',
        'address',
        'details',
        'city',
        'CP',
        'state',
        'country_id',
        'latitude',
        'longitude',
    ];

    /**
     * A Venue Belongs to a Country
     * @return \Illuminate\Database\Eloquent\Relations\BelongsTo
     */
    public function country()
    {
        return $this->belongsTo(Country::class);
    }


    /**
     * Get the Current User location
     * @param $tournament
     * @param $latitude
     * @param $longitude
     * @return $this
     */
    public function setDefaultLocation($tournament, $latitude, $longitude)
    {
        $userLat = $tournament != null ? $tournament->owner->latitude : null;
        $userLng = $tournament != null ? $tournament->owner->longitude : null;

        if ($latitude != null && $longitude != null) {
            $this->latitude = $latitude;
            $this->longitude = $longitude;
        } else if (!isNullOrEmptyString($userLat) && !isNullOrEmptyString($userLng)) {
            $this->latitude = $userLat;
            $this->longitude = $userLng;
        } else {
            //TODO Should popup for user localization
            $this->latitude = 0;
            $this->longitude = 0;
        }
        return $this;
    }
}


1		<?php
2
3		namespace App;
4
5		use Illuminate\Database\Eloquent\Model;
6
7		class Venue extends Model
8		{
9		public $timestamps = true;
10		protected $table = 'venue';
11		protected $fillable = [
12		'venue_name',
13		'address',
14		'details',
15		'city',
16		'CP',
17		'state',
18		'country_id',
19		'latitude',
20		'longitude',
21		];
22
23		/**
24		* A Venue Belongs to a Country
25		* @return \Illuminate\Database\Eloquent\Relations\BelongsTo
26		*/
27		public function country()
28		{
29		return $this->belongsTo(Country::class);
30		}
31
32
33		/**
34		* Get the Current User location
35		* @param $tournament
36		* @param $latitude
37		* @param $longitude
38		* @return $this
39		*/
40	8	public function setDefaultLocation($tournament, $latitude, $longitude)
41		{
42	8	$userLat = $tournament != null ? $tournament->owner->latitude : null;
43	8	$userLng = $tournament != null ? $tournament->owner->longitude : null;
44
45	8	if ($latitude != null && $longitude != null) {
46	6	$this->latitude = $latitude;
47	6	$this->longitude = $longitude;
48	7	} else if (!isNullOrEmptyString($userLat) && !isNullOrEmptyString($userLng)) {
49	6	$this->latitude = $userLat;
50	6	$this->longitude = $userLng;
51		} else {
52		//TODO Should popup for user localization
53	1	$this->latitude = 0;
54	1	$this->longitude = 0;
55		}
56	8	return $this;
57		}
58		}
59

xoco70 / kendozone

Pull Request — master (#45)

Venue::setDefaultLocation() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like