OrderAdmin::getEditForm() - Code Metrics - Inspection of "Refactor/ss4 (#309)" - dynamic/foxystripe - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( f8f77e...d2a4ab )

by Jason

created 2018-03-06 03:01 UTC

OrderAdmin::getEditForm() A

↳ Parent: OrderAdmin

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

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

<?php

namespace Dynamic\FoxyStripe\Admin;

use SilverStripe\Admin\ModelAdmin;

class OrderAdmin extends ModelAdmin
{
    /**
     * @var array
     */
    private static $managed_models = array(

        'Order',
    );

    /**
     * @var string
     */
    private static $url_segment = 'orders';


    /**
     * @var string
     */
    private static $menu_title = 'Orders';


    /**
     * @param null $id

     * @param null $fields
     *
     * @return \SilverStripe\Forms\Form
     */
    public function getEditForm($id = null, $fields = null)
    {
        $form = parent::getEditForm($id, $fields);

        $gridFieldName = $this->sanitiseClassName($this->modelClass);
        $gridField = $form->Fields()->fieldByName($gridFieldName);

        // GridField configuration
        $config = $gridField->getConfig();

        // remove edit icon
        $config->removeComponentsByType('GridFieldEditButton');

        return $form;
    }
}


1			<?php
2
3			namespace Dynamic\FoxyStripe\Admin;
4
5			use SilverStripe\Admin\ModelAdmin;
6
7			class OrderAdmin extends ModelAdmin
8			{
9			/**
10			* @var array
11			*/
12			private static $managed_models = array(
			0 ignored issues – show introduced 2018-03-03 21:50 UTC by Report Bug Copy Issue Report The private property `$managed_models` is not used, and could be removed. Loading history...
13			'Order',
14			);
15
16			/**
17			* @var string
18			*/
19			private static $url_segment = 'orders';
			0 ignored issues – show introduced 2018-03-03 21:50 UTC by Report Bug Copy Issue Report The private property `$url_segment` is not used, and could be removed. Loading history...
20
21			/**
22			* @var string
23			*/
24			private static $menu_title = 'Orders';
			0 ignored issues – show introduced 2018-03-03 21:50 UTC by Report Bug Copy Issue Report The private property `$menu_title` is not used, and could be removed. Loading history...
25
26			/**
27			* @param null $id
			0 ignored issues – show Documentation Bug introduced 2018-03-03 21:50 UTC by Report Bug Copy Issue Report Are you sure the doc-type for parameter `$fields` is correct as it would always require `null` to be passed? Loading history... Documentation Bug introduced 2018-03-03 21:50 UTC by Report Bug Copy Issue Report Are you sure the doc-type for parameter `$id` is correct as it would always require `null` to be passed? Loading history...
28			* @param null $fields
29			*
30			* @return \SilverStripe\Forms\Form
31			*/
32			public function getEditForm($id = null, $fields = null)
33			{
34			$form = parent::getEditForm($id, $fields);
35
36			$gridFieldName = $this->sanitiseClassName($this->modelClass);
37			$gridField = $form->Fields()->fieldByName($gridFieldName);
38
39			// GridField configuration
40			$config = $gridField->getConfig();
41
42			// remove edit icon
43			$config->removeComponentsByType('GridFieldEditButton');
44
45			return $form;
46			}
47			}
48

dynamic / foxystripe

Push — master ( f8f77e...d2a4ab )

OrderAdmin::getEditForm() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like