AborstIfEnvVariableIsnotInstalled::abortsIfEnvVarIsNotInstalled() - Code Metrics - Inspection of "Sync" - acacha/forge-publish - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( f0c745...274b6a )

by Sergi Tur

created 2017-12-04 12:59 UTC

abortsIfEnvVarIsNotInstalled() A

↳ Parent: AborstIfEnvVariableIsnotInstalled

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	5
nc	2
nop	1
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0

<?php

namespace Acacha\ForgePublish\Commands\Traits;

use Acacha\ForgePublish\Exceptions\EnvironmentVariableNotFoundException;

/**
 * Trait AborstIfEnvVariableIsnotInstalled
 *
 * @package Acacha\ForgePublish\Commands
 */
trait AborstIfEnvVariableIsnotInstalled
{
    /**
     * Skip if env var is not installed.
     */
    protected function abortsIfEnvVarIsNotInstalled($env_var)
    {
        if (! $this->fp_env($env_var)) {
            $this->info("No $env_var key found in .env file.");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->info('Please configure this .env variable manually or run php artisan publish:init. Skipping...');
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            throw new EnvironmentVariableNotFoundException($env_var);
        }
    }

    /**
     * Get Forge publish env.
     *
     * @param $env_var
     * @return null
     */
    protected function fp_env($env_var) {
        return fp_env($env_var);
    }
}


1			<?php
2
3			namespace Acacha\ForgePublish\Commands\Traits;
4
5			use Acacha\ForgePublish\Exceptions\EnvironmentVariableNotFoundException;
6
7			/**
8			* Trait AborstIfEnvVariableIsnotInstalled
9			*
10			* @package Acacha\ForgePublish\Commands
11			*/
12			trait AborstIfEnvVariableIsnotInstalled
13			{
14			/**
15			* Skip if env var is not installed.
16			*/
17			protected function abortsIfEnvVarIsNotInstalled($env_var)
18			{
19			if (! $this->fp_env($env_var)) {
20			$this->info("No $env_var key found in .env file.");
			0 ignored issues – show Bug introduced 2017-12-04 13:01 UTC by Report Bug Copy Issue Report It seems like `info()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
21			$this->info('Please configure this .env variable manually or run php artisan publish:init. Skipping...');
			0 ignored issues – show Bug introduced 2017-12-04 13:01 UTC by Report Bug Copy Issue Report It seems like `info()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
22			throw new EnvironmentVariableNotFoundException($env_var);
23			}
24			}
25
26			/**
27			* Get Forge publish env.
28			*
29			* @param $env_var
30			* @return null
31			*/
32			protected function fp_env($env_var) {
33			return fp_env($env_var);
34			}
35			}
36

acacha / forge-publish

Push — master ( f0c745...274b6a )

abortsIfEnvVarIsNotInstalled() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like