ShippingSettings::beforeSave() - Code Metrics - Inspection of "Allow drivers to be enabled / disabled" - scottbedard/oc-shop-plugin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( b7e188...4523b4 )

by Scott

created 2017-02-05 14:31 UTC

ShippingSettings::beforeSave() A

↳ Parent: ShippingSettings

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	4
rs	10
cc	1
eloc	2
nc	1
nop	0

<?php namespace Bedard\Shop\Models;

use Flash;
use Lang;
use Model;
use October\Rain\Exception\ValidationException;

/**
 * Cart Settings Model.
 */
class ShippingSettings extends Model
{
    /**
     * @var array   Behaviors
     */
    public $implement = ['System.Behaviors.SettingsModel'];

    /**
     * @var string  Settings code
     */
    public $settingsCode = 'bedard_shop_settings_shipping';

    /**
     * @var string  Settings fields
     */
    public $settingsFields = 'fields.yaml';

    /**
     * Before save.
     *
     * @return void
     */
    public function beforeSave()
    {
        $this->validateDrivers(self::get('enabled_drivers'));
    }

    /**
     * Ensure that only one driver is enabled.
     *
     * @param   array|null                              $drivers
     * @throws  \October\Rain\Database\ModelException
     * @return  void
     */
    protected function validateDrivers(array $drivers = null)
    {
        if (! $drivers || ! is_array($drivers)) {
            Flash::error(Lang::get('bedard.shop::lang.shipping.form.validation_none'));
            throw new ValidationException(null);
        }

        // @todo: allow more than one shipping driver to be enabled
        if (count($drivers) !== 1) {
            Flash::error(Lang::get('bedard.shop::lang.shipping.form.validation_multiple'));
            throw new ValidationException(null);
        }
    }
}


1			<?php namespace Bedard\Shop\Models;
2
3			use Flash;
4			use Lang;
5			use Model;
6			use October\Rain\Exception\ValidationException;
7
8			/**
9			* Cart Settings Model.
10			*/
11			class ShippingSettings extends Model
12			{
13			/**
14			* @var array Behaviors
15			*/
16			public $implement = ['System.Behaviors.SettingsModel'];
17
18			/**
19			* @var string Settings code
20			*/
21			public $settingsCode = 'bedard_shop_settings_shipping';
22
23			/**
24			* @var string Settings fields
25			*/
26			public $settingsFields = 'fields.yaml';
27
28			/**
29			* Before save.
30			*
31			* @return void
32			*/
33			public function beforeSave()
34			{
35			$this->validateDrivers(self::get('enabled_drivers'));
36			}
37
38			/**
39			* Ensure that only one driver is enabled.
40			*
41			* @param array\|null $drivers
42			* @throws \October\Rain\Database\ModelException
43			* @return void
44			*/
45			protected function validateDrivers(array $drivers = null)
46			{
47			if (! $drivers \|\| ! is_array($drivers)) {
48			Flash::error(Lang::get('bedard.shop::lang.shipping.form.validation_none'));
49			throw new ValidationException(null);
50			}
51
52			// @todo: allow more than one shipping driver to be enabled
53			if (count($drivers) !== 1) {
54			Flash::error(Lang::get('bedard.shop::lang.shipping.form.validation_multiple'));
55			throw new ValidationException(null);
56			}
57			}
58			}
59

scottbedard / oc-shop-plugin

Push — master ( b7e188...4523b4 )

ShippingSettings::beforeSave() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like