AutoValidate::validate() - Code Metrics - Inspection of "Fixed the boot method to actually call each trait'..." - tylercd100/laravel-model-improved - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (62cc10)

by Tyler

created 2017-09-13 18:07 UTC

AutoValidate::validate() A

↳ Parent: AutoValidate

Complexity

Conditions	2
Paths	2

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	14
ccs	0
cts	8
cp	0
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	9
nc	2
nop	0
crap	6

<?php

namespace Tylercd100\Database\Eloquent\Traits;

use Tylercd100\Database\Eloquent\Exceptions\ValidationException;
use Illuminate\Support\Facades\Validator;

trait AutoValidate
{
    /**
     * An array of validation rules
     * @var array
     */
    protected $rules = [];

    /**
     * An array of validation messages
     * @var array
     */
    protected $messages = [];

    /**
     * Validates the Model
     * @return boolean Whether or not the input is valid
     */
    public function validate()
    {
        $data = $this->toArray();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $rules = $this->getValidationRules();
        $messages = $this->getValidationMessages();

        $validator = Validator::make($data, $rules, $messages);
        
        if ($validator->fails()) {
            throw new ValidationException($validator->errors()->first());
        } else {
            return true;
        }
    }

    /**
     * Returns an array of validation rules
     * @return array An Array of validation rules
     */
    protected function getValidationRules()
    {
        return $this->rules;
    }

    /**
     * Returns an array of validation messages
     * @return array An Array of validation messages
     */
    protected function getValidationMessages()
    {
        return $this->messages;
    }
}


1			<?php
2
3			namespace Tylercd100\Database\Eloquent\Traits;
4
5			use Tylercd100\Database\Eloquent\Exceptions\ValidationException;
6			use Illuminate\Support\Facades\Validator;
7
8			trait AutoValidate
9			{
10			/**
11			* An array of validation rules
12			* @var array
13			*/
14			protected $rules = [];
15
16			/**
17			* An array of validation messages
18			* @var array
19			*/
20			protected $messages = [];
21
22			/**
23			* Validates the Model
24			* @return boolean Whether or not the input is valid
25			*/
26			public function validate()
27			{
28			$data = $this->toArray();
			0 ignored issues – show Bug introduced 2017-05-03 00:18 UTC by Report Bug Copy Issue Report It seems like `toArray()` 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...
29			$rules = $this->getValidationRules();
30			$messages = $this->getValidationMessages();
31
32			$validator = Validator::make($data, $rules, $messages);
33
34			if ($validator->fails()) {
35			throw new ValidationException($validator->errors()->first());
36			} else {
37			return true;
38			}
39			}
40
41			/**
42			* Returns an array of validation rules
43			* @return array An Array of validation rules
44			*/
45			protected function getValidationRules()
46			{
47			return $this->rules;
48			}
49
50			/**
51			* Returns an array of validation messages
52			* @return array An Array of validation messages
53			*/
54			protected function getValidationMessages()
55			{
56			return $this->messages;
57			}
58			}
59

tylercd100 / laravel-model-improved

Branch — master (62cc10)

AutoValidate::validate() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like