helpers.php ➔ backpack_avatar_url() - Code Metrics - Inspection of "[WIP] Backpack 0.8.0 upgrade" - Laravel-Backpack/Base - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#183)

by Cristian

created 2017-11-02 09:37 UTC

helpers.php ➔ backpack_avatar_url() A

↳ Parent: Project

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	3
eloc	11
nc	3
nop	1
dl	0
loc	16
rs	9.4285
c	0
b	0
f	0

<?php

if (!function_exists('backpack_url')) {
    /**
     * Appends the configured backpack prefix and returns
     * the URL using the standard Laravel helpers.
     *
     * @param $path
     *
     * @return string
     */
    function backpack_url($path = null)
    {
        $path = !$path || (substr($path, 1, 1) == '/') ? $path : '/'.$path;

        return url(config('backpack.base.route_prefix', 'admin').$path);
    }
}

if (!function_exists('backpack_avatar')) {
    /**
     * Returns the avatar URL of a user.
     *
     * @param $user
     *
     * @return string
     */
    function backpack_avatar_url($user)
    {
        switch (config('backpack.base.avatar_type')) {
            case 'gravatar':
                return Gravatar::fallback('https://placehold.it/160x160/00a65a/ffffff/&text='.Auth::user()->name[0])->get($user->email);
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

            case 'placehold':
                return 'https://placehold.it/160x160/00a65a/ffffff/&text='.$user->name[0];
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

            default:
                return $user->{config('backpack.base.avatar_type')};
                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
        }
    }
}


1			<?php
2
3			if (!function_exists('backpack_url')) {
4			/**
5			* Appends the configured backpack prefix and returns
6			* the URL using the standard Laravel helpers.
7			*
8			* @param $path
9			*
10			* @return string
11			*/
12			function backpack_url($path = null)
13			{
14			$path = !$path \|\| (substr($path, 1, 1) == '/') ? $path : '/'.$path;
15
16			return url(config('backpack.base.route_prefix', 'admin').$path);
17			}
18			}
19
20			if (!function_exists('backpack_avatar')) {
21			/**
22			* Returns the avatar URL of a user.
23			*
24			* @param $user
25			*
26			* @return string
27			*/
28			function backpack_avatar_url($user)
29			{
30			switch (config('backpack.base.avatar_type')) {
31			case 'gravatar':
32			return Gravatar::fallback('https://placehold.it/160x160/00a65a/ffffff/&text='.Auth::user()->name[0])->get($user->email);
33			break;
			0 ignored issues – show Unused Code introduced 2017-09-11 14:37 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
34
35			case 'placehold':
36			return 'https://placehold.it/160x160/00a65a/ffffff/&text='.$user->name[0];
37			break;
			0 ignored issues – show Unused Code introduced 2017-09-11 14:37 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
38
39			default:
40			return $user->{config('backpack.base.avatar_type')};
41			break;
			0 ignored issues – show Unused Code introduced 2017-09-11 14:37 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
42			}
43			}
44			}
45

Laravel-Backpack / Base

Pull Request — master (#183)

helpers.php ➔ backpack_avatar_url() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like