ElementalAdmin::getEditForm() - Code Metrics - Inspection of "FIX: Display correct translated classname" - dnadesign/silverstripe-elemental - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 2ee2df...59c799 )

by Will

created 2016-05-23 21:57 UTC

ElementalAdmin::getEditForm() A

↳ Parent: ElementalAdmin

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

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

<?php

class ElementalAdmin extends ModelAdmin {
namespace YourVendor;

class YourClass { }

    private static $managed_models = array(

        'BaseElement'
    );

    private static $menu_title = 'Content Blocks';


    private static $url_segment = 'elemental';


    private static $menu_icon = "elemental/images/blocks.svg";


    public function getEditForm($id = null, $fields = null) {

        $form = parent::getEditForm($id, $fields);

        $grid = $form->Fields()
            ->dataFieldByName($this->sanitiseClassName($this->modelClass));

        $grid->getConfig()->removeComponentsByType('GridFieldAddNewButton');

        return $form;
    }
}


1			<?php
2
3			class ElementalAdmin extends ModelAdmin {
			0 ignored issues – show Coding Style Compatibility introduced 2016-05-23 22:00 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
4
5			private static $managed_models = array(
			0 ignored issues – show Comprehensibility introduced 2016-05-23 22:00 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2016-05-23 22:00 UTC by Report Bug Copy Issue Report The property `$managed_models` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
6			'BaseElement'
7			);
8
9			private static $menu_title = 'Content Blocks';
			0 ignored issues – show Comprehensibility introduced 2016-05-23 22:00 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2016-05-23 22:00 UTC by Report Bug Copy Issue Report The property `$menu_title` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
10
11			private static $url_segment = 'elemental';
			0 ignored issues – show Comprehensibility introduced 2016-05-23 22:00 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2016-05-23 22:00 UTC by Report Bug Copy Issue Report The property `$url_segment` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
12
13			private static $menu_icon = "elemental/images/blocks.svg";
			0 ignored issues – show Comprehensibility introduced 2016-05-23 22:00 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2016-05-23 22:00 UTC by Report Bug Copy Issue Report The property `$menu_icon` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
14
15			public function getEditForm($id = null, $fields = null) {
			0 ignored issues – show Documentation introduced 2016-05-23 22:00 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
16			$form = parent::getEditForm($id, $fields);
17
18			$grid = $form->Fields()
19			->dataFieldByName($this->sanitiseClassName($this->modelClass));
20
21			$grid->getConfig()->removeComponentsByType('GridFieldAddNewButton');
22
23			return $form;
24			}
25			}
26

dnadesign / silverstripe-elemental

Push — master ( 2ee2df...59c799 )

ElementalAdmin::getEditForm() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like