FormArtisan::resolve_strategies() - Code Metrics - Inspection of "Feature/validator - new abstract factory for creat..." - macder/wfv-validation - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Pull Request — master (#119)

by Maciej

created 2017-05-19 17:37 UTC

FormArtisan::resolve_strategies() C

↳ Parent: Project

Complexity

Conditions	7
Paths	11

Size

Total Lines	31
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	7
eloc	17
nc	11
nop	0
dl	0
loc	31
rs	6.7272
c	0
b	0
f	0

<?php

namespace WFV\Artisan;
defined( 'ABSPATH' ) or die();
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);

use WFV\Contract\ArtisanInterface;
use WFV\Collection\ErrorCollection;
use WFV\Collection\InputCollection;
use WFV\Collection\MessageCollection;
use WFV\Collection\RuleCollection;

use WFV\FormComposite;
use WFV\Validator;

/**
 *
 *
 * @since 0.10.0
 */
class FormArtisan implements ArtisanInterface {

	/**
	 *
	 *
	 * @since 0.10.0
	 * @var array
	 */
	public $collection = array();

	/**
	 *
	 *
	 * @since 0.11.0
	 * @var WFV\Validator
	 */
	public $validator;

	/**
	 *
	 *
	 * @since 0.10.0
	 * @access protected
	 * @var array
	 */
	protected $config = array();

	/**
	 *
	 *
	 * @since 0.10.0
	 * @access protected
	 * @var WFV\FormComposite
	 */
	protected $form;

	/**
	 *
	 *
	 * @since 0.11.0
	 *
	 * @param array $config
	 */
	function __construct( array $config ) {

		$this->config = $config;
	}

	/**
	 * Return the final Form instance
	 *
	 * @since 0.10.0
	 *
	 * @return WFV\FormComposite
	 */
	public function actualize() {
		return $this->form;
	}

	/**
	 * Creates the instance of FormComposite
	 *
	 * @since 0.10.0
	 *
	 * @param string $action
	 * @return WFV\Artisan\FormArtisan
	 */
	public function create( $action ) {
		$this->form = new FormComposite( $this, $action );

		return $this;
	}

	/**
	 * Create instance of ErrorCollection
	 * Save it in $collection array property
	 *
	 * @since 0.10.0
	 *
	 * @return WFV\Artisan\FormArtisan
	 */
	public function errors() {
		$this->collection['errors'] = new ErrorCollection( $this->labels() );
		return $this;
	}

	/**
	 * Create instance of InputCollection
	 * Save it in $collection array property
	 *
	 * @since 0.10.0
	 *
	 * @param array $data
	 * @return WFV\Artisan\FormArtisan
	 */
	public function input( array $data = [] ) {
		$input = $data[0];
		$trim = $data[1];
		$this->collection['input'] = new InputCollection( $input, $trim );
		return $this;
	}

	/**
	 *
	 *
	 * @since 0.10.0
	 *
	 * @return WFV\Artisan\FormArtisan
	 */
	public function messages() {
		$this->collection['messages'] = new MessageCollection( $this->config );
		return $this;
	}

	/**
	 * Create instance of RuleCollection
	 * Save it in $collection array property
	 *
	 * @since 0.10.0
	 *
	 * @return WFV\Artisan\FormArtisan
	 */
	public function rules() {
		foreach( $this->config as $field => $options ) {
			$rules[ $field ] = $options['rules'];
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
		}
		$this->collection['rules'] = new RuleCollection( $rules );
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
		return $this;
	}

	/**
	 * Create instance of WFV\Validator
	 * Save it in $validator property
	 *
	 * @since 0.11.0
	 *
	 * @return WFV\Artisan\FormArtisan
	 */
	public function validator() {
		$this->validator = new Validator();

		return $this;
	}

	/**
	 * Returns an array of human friendly field labels
	 *  as defined in the config array
	 *
	 * @since 0.11.0
	 * @access protected
	 *
	 * @return array
	 */
	protected function labels() {
		return array_map( function( $item ) {
			return $item['label'];
		}, $this->config);
	}
}


Pull Request — master (#119)

FormArtisan::resolve_strategies() C

Complexity

Size

Duplication

Importance

Logical Operators are used for Control-Flow

Available Fixes

macder / wfv-validation

Pull Request — master (#119)

FormArtisan::resolve_strategies() C

Complexity

Size

Duplication

Importance

Logical Operators are used for Control-Flow

Available Fixes

Duplication Side-by-Side

Filter issues like