DMSGridFieldEditButton::getColumnContent() - Code Metrics - Inspection of "Merge pull request #177 from creative-commoners/pu..." - silverstripe/silverstripe-dms - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d57fc8...4bdeb9 )

by Franco

created 2017-06-14 04:14 UTC

DMSGridFieldEditButton::getColumnContent() A

↳ Parent: DMSGridFieldEditButton

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

class DMSGridFieldEditButton extends GridFieldEditButton implements GridField_ColumnProvider
namespace YourVendor;

class YourClass { }
{

    /**
     * Overriding the parent method to change the template that the DMS edit button will be rendered with based on
     * whether or not the user has edit permissions.
     *
     * @param GridField $gridField
     * @param DataObject $record
     * @param string $columnName
     *
     * @return string - the HTML for the column

     */
    public function getColumnContent($gridField, $record, $columnName)
    {
        $data = new ArrayData(array(
            'Link' => Controller::join_links($gridField->Link('item'), $record->ID, 'edit')
        ));

        $template = $record->canEdit() ? 'GridFieldEditButton' : 'GridFieldViewButton';

        return $data->renderWith($template);
    }
}


1			<?php
2
3			class DMSGridFieldEditButton extends GridFieldEditButton implements GridField_ColumnProvider
			0 ignored issues – show Coding Style Compatibility introduced 2017-06-13 04:27 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
6			/**
7			* Overriding the parent method to change the template that the DMS edit button will be rendered with based on
8			* whether or not the user has edit permissions.
9			*
10			* @param GridField $gridField
11			* @param DataObject $record
12			* @param string $columnName
13			*
14			* @return string - the HTML for the column
			0 ignored issues – show Documentation introduced 2017-06-13 04:27 UTC by Report Bug Copy Issue Report Should the return type not be `HTMLText`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
15			*/
16			public function getColumnContent($gridField, $record, $columnName)
17			{
18			$data = new ArrayData(array(
19			'Link' => Controller::join_links($gridField->Link('item'), $record->ID, 'edit')
20			));
21
22			$template = $record->canEdit() ? 'GridFieldEditButton' : 'GridFieldViewButton';
23
24			return $data->renderWith($template);
25			}
26			}
27

silverstripe / silverstripe-dms

Push — master ( d57fc8...4bdeb9 )

DMSGridFieldEditButton::getColumnContent() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like