FileFormFactory::getUpdateAction() - Code Metrics - Inspection of "Pulls/4.0/309 insert update media button label" - silverstripe/silverstripe-asset-admin - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#499)

unknown

created 2017-06-11 22:22 UTC

FileFormFactory::getUpdateAction() A

↳ Parent: FileFormFactory

Complexity

Conditions	4
Paths	2

Size

Total Lines	10
Code Lines	6

Duplication

Lines	10
Ratio	100 %

Importance

Changes

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

<?php

namespace SilverStripe\AssetAdmin\Forms;

use SilverStripe\Assets\File;
use SilverStripe\Control\RequestHandler;
use SilverStripe\Forms\CheckboxField;
use SilverStripe\Forms\FieldGroup;
use SilverStripe\Forms\DatetimeField;
use SilverStripe\Forms\FieldList;
use SilverStripe\Forms\FormAction;
use SilverStripe\Forms\HiddenField;
use SilverStripe\Forms\LiteralField;
use SilverStripe\Forms\Tab;
use SilverStripe\Forms\TabSet;
use SilverStripe\Forms\TextField;

class FileFormFactory extends AssetFormFactory
{
    protected function getFormFieldTabs($record, $context = [])
    {
        // Add extra tab
        $tabs = TabSet::create(
            'Editor',
            $this->getFormFieldDetailsTab($record, $context),
            $this->getFormFieldSecurityTab($record, $context),
            $this->getFormFieldUsageTab($record, $context),
            $this->getFormFieldHistoryTab($record, $context)
        );

        // All non-admin forms are typically readonly
        switch ($this->getFormType($context)) {
            case static::TYPE_INSERT_MEDIA:
            case static::TYPE_UPDATE_MEDIA:
                $tabs->setReadonly(true);
                $tabs->unshift($this->getFormFieldAttributesTab($record, $context));
                break;
            case static::TYPE_INSERT_LINK:
            case static::TYPE_UPDATE_LINK:
                $tabs->setReadonly(true);
                $tabs->unshift($this->getFormFieldLinkOptionsTab($record, $context));
                break;
            case static::TYPE_SELECT:
                $tabs->setReadonly(true);
                break;
        }

        return $tabs;
    }

    /**
     * Build "Usage" tab
     *
     * @param File $record
     * @param array $context
     * @return Tab
     */
    protected function getFormFieldUsageTab($record, $context = [])

    {
        // Add new tab for usage
        return Tab::create(
            'Usage',
            DatetimeField::create(
                "Created",
                _t('SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.CREATED', 'First uploaded')
            )
                ->setReadonly(true),
            DatetimeField::create(
                "LastEdited",
                _t('SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.LASTEDIT', 'Last changed')
            )
                ->setReadonly(true)
        );
    }

    protected function getFormFieldDetailsTab($record, $context = [])
    {
        // Update details tab
        $tab = parent::getFormFieldDetailsTab($record, $context);

        $tab->insertBefore('Name', TextField::create("Title", File::singleton()->fieldLabel('Title')));

        if ($this->getFormType($context) !== static::TYPE_ADMIN) {
            $tab->push(LiteralField::create(
                'EditLink',
                sprintf(
                    '<a href="%s" class="%s" target="_blank"><i class="%s" />%s</a>',
                    $record->CMSEditLink(),
                    'btn btn-secondary-outline font-icon-edit editor__edit-link',
                    '',
                    _t('SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.EditLink', 'Edit original file')
                )
            ));
        }
        return $tab;
    }

    protected function getFormFieldLinkOptionsTab($record, $context = [])

    {
        return Tab::create(
            'LinkOptions',
            _t(__CLASS__ .'.LINKOPTIONS', 'Link options'),
            TextField::create(
                'Description',
                _t(__CLASS__.'.LINKDESCR', 'Link description')
            ),
            CheckboxField::create(
                'TargetBlank',
                _t(__CLASS__.'.LINKOPENNEWWIN', 'Open in new window/tab')
            )
        );
    }

    /**
     * Create tab for file attributes
     *
     * @param File $record
     * @param array $context
     * @return Tab
     */
    protected function getFormFieldAttributesTab($record, $context = [])
    {
        return Tab::create(
            'Placement',
            LiteralField::create(
                'AttributesDescription',
                '<p>'. _t(
                    'SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.AttributesDescription',
                    'These changes will only affect this particular placement of the file.'
                ) .'</p>'
            ),
            TextField::create('Caption', _t('SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.Caption', 'Caption'))
        );
    }

    protected function getFormFieldHistoryTab($record, $context = [])

    {
        return Tab::create(
            'History',
            HistoryListField::create('HistoryList')
                ->setRecord($record)
        );
    }

    protected function getFormFields(RequestHandler $controller = null, $formName, $context = [])
    {
        /** @var File $record */
        $record = $context['Record'];

        // Add status flag before extensions are triggered
        $this->beforeExtending('updateFormFields', function (FieldList $fields) use ($record) {
            // @todo move specs to a component/class, so it can update specs when a File is replaced
            $fields->insertAfter(
                'TitleHeader',
                LiteralField::create('FileSpecs', $this->getSpecsMarkup($record))
            );
            $fields->push(HiddenField::create('FileFilename'));
            $fields->push(HiddenField::create('FileHash'));
            $fields->push(HiddenField::create('FileVariant'));
        });

        return parent::getFormFields($controller, $formName, $context);
    }

    /**
     * Get publish action
     *
     * @param File $record
     * @return FormAction
     */
    protected function getPublishAction($record)
    {
        if (!$record || !$record->canPublish()) {
            return null;
        }

        // Build action
        $publishText = _t('SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.PUBLISH_BUTTON', 'Publish');
        /** @var FormAction $action */
        $action = FormAction::create('publish', $publishText)
            ->setIcon('rocket')
            ->setSchemaData(['data' => ['buttonStyle' => 'primary']]);

        return $action;
    }

    protected function getFormActions(RequestHandler $controller = null, $formName, $context = [])
    {
        $record = $context['Record'];
        $type = $this->getFormType($context);

        if ($type === static::TYPE_SELECT || $type === static::TYPE_INSERT_MEDIA) {
            $actionItems = array_filter([
                $this->getInsertAction($record),
            ]);
        } elseif ($type === static::TYPE_UPDATE_MEDIA) {
            $actionItems = array_filter([
                $this->getUpdateAction($record),
            ]);
        } elseif ($type === static::TYPE_INSERT_LINK) {
            $actionItems = array_filter([
                $this->getInsertLinkAction($record),
            ]);
        } elseif ($type === static::TYPE_UPDATE_LINK) {
            $actionItems = array_filter([
                $this->getUpdateLinkAction($record),
            ]);
        } else {
            $actionItems = array_filter([
                $this->getSaveAction($record),
                $this->getPublishAction($record)
            ]);
        }

        // Group all actions
        if (count($actionItems) > 1) {
            $actionItems = [
                FieldGroup::create($actionItems)
                    ->setName('Actions')
                    ->addExtraClass('btn-group')
            ];
        }

        if ($type === static::TYPE_ADMIN) {
            // Add popover
            $popover = $this->getPopoverMenu($record);
            if ($popover) {
                $actionItems[] = $popover;
            }
        }

        // Build
        $actions = new FieldList($actionItems);

        // Update
        $this->invokeWithExtensions('updateFormActions', $actions, $controller, $formName, $context);
        return $actions;
    }

    /**
     * get HTML for status icon
     *
     * @param File $record
     * @return null|string
     */
    protected function getSpecsMarkup($record)
    {
        if (!$record || !$record->exists()) {
            return null;
        }
        return sprintf(
            '<div class="editor__specs">%s %s</div>',
            $record->getSize(),
            $this->getStatusFlagMarkup($record)
        );
    }

    /**
     * Get published status flag
     *
     * @param File $record
     * @return null|string
     */
    protected function getStatusFlagMarkup($record)
    {
        if ($record && ($statusTitle = $record->getStatusTitle())) {
            return "<span class=\"editor__status-flag\">{$statusTitle}</span>";
        }
        return null;
    }

    /**
     * Get action for publishing
     *
     * @param File $record
     * @return FormAction
     */
    protected function getUnpublishAction($record)
    {
        // Check if record is unpublishable
        if (!$record || !$record->isPublished() || !$record->canUnpublish()) {
            return null;
        }

        // Build action
        $unpublishText = _t(
            'SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.UNPUBLISH_BUTTON',
            'Unpublish'
        );
        return FormAction::create('unpublish', $unpublishText)
            ->setIcon('cancel-circled');
    }

    /**
     * Get actions that go into the Popover menu
     *
     * @param $record
     * @return array
     */
    protected function getPopoverActions($record)
    {
        $this->beforeExtending('updatePopoverActions', function (&$actions, $record) {
            // add the unpublish action to the start of the array
            array_unshift($actions, $this->getUnpublishAction($record));
        });

        return parent::getPopoverActions($record);
    }

    /**
     * @param File $record
     * @return FormAction
     */
    protected function getInsertAction($record)

    {
        $action = null;
        if ($record && $record->isInDB() && $record->canEdit()) {
            /** @var FormAction $action */
            $action = FormAction::create('insert', _t(__CLASS__.'.INSERT_FILE', 'Insert file'))
                ->setSchemaData(['data' => ['buttonStyle' => 'primary']]);
        }
        return $action;
    }

    /**
     * @param File $record
     * @return FormAction
     */
    protected function getUpdateAction($record)

    {
        $action = null;
        if ($record && $record->isInDB() && $record->canEdit()) {
            /** @var FormAction $action */
            $action = FormAction::create('update', _t(__CLASS__.'.UPDATE_FILE', 'Update file'))
                ->setSchemaData(['data' => ['buttonStyle' => 'primary']]);
        }
        return $action;
    }

    /**
     * @param File $record
     * @return FormAction
     */
    protected function getInsertLinkAction($record)

    {
        $action = null;
        if ($record && $record->isInDB() && $record->canEdit()) {
            /** @var FormAction $action */
            $action = FormAction::create('insert', _t(__CLASS__.'.INSERT_LINK', 'Link to file'))
                ->setSchemaData(['data' => ['buttonStyle' => 'primary']]);
        }
        return $action;
    }

    /**
     * @param File $record
     * @return FormAction
     */
    protected function getUpdateLinkAction($record)

    {
        $action = null;
        if ($record && $record->isInDB() && $record->canEdit()) {
            /** @var FormAction $action */
            $action = FormAction::create('insert', _t(__CLASS__.'.UPDATE_LINK', 'Update link'))
                ->setSchemaData(['data' => ['buttonStyle' => 'primary']]);
        }
        return $action;
    }
}


1		<?php
2
3		namespace SilverStripe\AssetAdmin\Forms;
4
5		use SilverStripe\Assets\File;
6		use SilverStripe\Control\RequestHandler;
7		use SilverStripe\Forms\CheckboxField;
8		use SilverStripe\Forms\FieldGroup;
9		use SilverStripe\Forms\DatetimeField;
10		use SilverStripe\Forms\FieldList;
11		use SilverStripe\Forms\FormAction;
12		use SilverStripe\Forms\HiddenField;
13		use SilverStripe\Forms\LiteralField;
14		use SilverStripe\Forms\Tab;
15		use SilverStripe\Forms\TabSet;
16		use SilverStripe\Forms\TextField;
17
18		class FileFormFactory extends AssetFormFactory
19		{
20		protected function getFormFieldTabs($record, $context = [])
21		{
22		// Add extra tab
23		$tabs = TabSet::create(
24		'Editor',
25		$this->getFormFieldDetailsTab($record, $context),
26		$this->getFormFieldSecurityTab($record, $context),
27		$this->getFormFieldUsageTab($record, $context),
28		$this->getFormFieldHistoryTab($record, $context)
29		);
30
31		// All non-admin forms are typically readonly
32		switch ($this->getFormType($context)) {
33		case static::TYPE_INSERT_MEDIA:
34		case static::TYPE_UPDATE_MEDIA:
35		$tabs->setReadonly(true);
36		$tabs->unshift($this->getFormFieldAttributesTab($record, $context));
37		break;
38		case static::TYPE_INSERT_LINK:
39		case static::TYPE_UPDATE_LINK:
40		$tabs->setReadonly(true);
41		$tabs->unshift($this->getFormFieldLinkOptionsTab($record, $context));
42		break;
43		case static::TYPE_SELECT:
44		$tabs->setReadonly(true);
45		break;
46		}
47
48		return $tabs;
49		}
50
51		/**
52		* Build "Usage" tab
53		*
54		* @param File $record
55		* @param array $context
56		* @return Tab
57		*/
58		protected function getFormFieldUsageTab($record, $context = [])
		0 ignored issues – show Unused Code introduced 2017-05-05 02:59 UTC by Report Bug Copy Issue Report The parameter `$record` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2016-11-17 21:46 UTC by Report Bug Copy Issue Report The parameter `$context` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
59		{
60		// Add new tab for usage
61		return Tab::create(
62		'Usage',
63		DatetimeField::create(
64		"Created",
65		_t('SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.CREATED', 'First uploaded')
66		)
67		->setReadonly(true),
68		DatetimeField::create(
69		"LastEdited",
70		_t('SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.LASTEDIT', 'Last changed')
71		)
72		->setReadonly(true)
73		);
74		}
75
76		protected function getFormFieldDetailsTab($record, $context = [])
77		{
78		// Update details tab
79		$tab = parent::getFormFieldDetailsTab($record, $context);
80
81		$tab->insertBefore('Name', TextField::create("Title", File::singleton()->fieldLabel('Title')));
82
83		if ($this->getFormType($context) !== static::TYPE_ADMIN) {
84		$tab->push(LiteralField::create(
85		'EditLink',
86		sprintf(
87		'<a href="%s" class="%s" target="_blank"><i class="%s" />%s</a>',
88		$record->CMSEditLink(),
89		'btn btn-secondary-outline font-icon-edit editor__edit-link',
90		'',
91		_t('SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.EditLink', 'Edit original file')
92		)
93		));
94		}
95		return $tab;
96		}
97
98		protected function getFormFieldLinkOptionsTab($record, $context = [])
		0 ignored issues – show Unused Code introduced 2017-05-31 03:51 UTC by Report Bug Copy Issue Report The parameter `$record` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history... Unused Code introduced 2016-11-17 21:46 UTC by Report Bug Copy Issue Report The parameter `$context` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
99		{
100		return Tab::create(
101		'LinkOptions',
102		_t(__CLASS__ .'.LINKOPTIONS', 'Link options'),
103		TextField::create(
104		'Description',
105		_t(__CLASS__.'.LINKDESCR', 'Link description')
106		),
107		CheckboxField::create(
108		'TargetBlank',
109		_t(__CLASS__.'.LINKOPENNEWWIN', 'Open in new window/tab')
110		)
111		);
112		}
113
114		/**
115		* Create tab for file attributes
116		*
117		* @param File $record
118		* @param array $context
119		* @return Tab
120		*/
121		protected function getFormFieldAttributesTab($record, $context = [])
122		{
123		return Tab::create(
124		'Placement',
125		LiteralField::create(
126		'AttributesDescription',
127		'<p>'. _t(
128		'SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.AttributesDescription',
129		'These changes will only affect this particular placement of the file.'
130		) .'</p>'
131		),
132		TextField::create('Caption', _t('SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.Caption', 'Caption'))
133		);
134		}
135
136		protected function getFormFieldHistoryTab($record, $context = [])
		0 ignored issues – show Unused Code introduced 2017-05-31 03:51 UTC by Report Bug Copy Issue Report The parameter `$context` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
137		{
138		return Tab::create(
139		'History',
140		HistoryListField::create('HistoryList')
141		->setRecord($record)
142		);
143		}
144
145		protected function getFormFields(RequestHandler $controller = null, $formName, $context = [])
146		{
147		/** @var File $record */
148		$record = $context['Record'];
149
150		// Add status flag before extensions are triggered
151		$this->beforeExtending('updateFormFields', function (FieldList $fields) use ($record) {
152		// @todo move specs to a component/class, so it can update specs when a File is replaced
153		$fields->insertAfter(
154		'TitleHeader',
155		LiteralField::create('FileSpecs', $this->getSpecsMarkup($record))
156		);
157		$fields->push(HiddenField::create('FileFilename'));
158		$fields->push(HiddenField::create('FileHash'));
159		$fields->push(HiddenField::create('FileVariant'));
160		});
161
162		return parent::getFormFields($controller, $formName, $context);
163		}
164
165		/**
166		* Get publish action
167		*
168		* @param File $record
169		* @return FormAction
170		*/
171		protected function getPublishAction($record)
172		{
173		if (!$record \|\| !$record->canPublish()) {
174		return null;
175		}
176
177		// Build action
178		$publishText = _t('SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.PUBLISH_BUTTON', 'Publish');
179		/** @var FormAction $action */
180		$action = FormAction::create('publish', $publishText)
181		->setIcon('rocket')
182		->setSchemaData(['data' => ['buttonStyle' => 'primary']]);
183
184		return $action;
185		}
186
187		protected function getFormActions(RequestHandler $controller = null, $formName, $context = [])
188		{
189		$record = $context['Record'];
190		$type = $this->getFormType($context);
191
192		if ($type === static::TYPE_SELECT \|\| $type === static::TYPE_INSERT_MEDIA) {
193		$actionItems = array_filter([
194		$this->getInsertAction($record),
195		]);
196		} elseif ($type === static::TYPE_UPDATE_MEDIA) {
197		$actionItems = array_filter([
198		$this->getUpdateAction($record),
199		]);
200		} elseif ($type === static::TYPE_INSERT_LINK) {
201		$actionItems = array_filter([
202		$this->getInsertLinkAction($record),
203		]);
204		} elseif ($type === static::TYPE_UPDATE_LINK) {
205		$actionItems = array_filter([
206		$this->getUpdateLinkAction($record),
207		]);
208		} else {
209		$actionItems = array_filter([
210		$this->getSaveAction($record),
211		$this->getPublishAction($record)
212		]);
213		}
214
215		// Group all actions
216		if (count($actionItems) > 1) {
217		$actionItems = [
218		FieldGroup::create($actionItems)
219		->setName('Actions')
220		->addExtraClass('btn-group')
221		];
222		}
223
224		if ($type === static::TYPE_ADMIN) {
225		// Add popover
226		$popover = $this->getPopoverMenu($record);
227		if ($popover) {
228		$actionItems[] = $popover;
229		}
230		}
231
232		// Build
233		$actions = new FieldList($actionItems);
234
235		// Update
236		$this->invokeWithExtensions('updateFormActions', $actions, $controller, $formName, $context);
237		return $actions;
238		}
239
240		/**
241		* get HTML for status icon
242		*
243		* @param File $record
244		* @return null\|string
245		*/
246		protected function getSpecsMarkup($record)
247		{
248		if (!$record \|\| !$record->exists()) {
249		return null;
250		}
251		return sprintf(
252		'<div class="editor__specs">%s %s</div>',
253		$record->getSize(),
254		$this->getStatusFlagMarkup($record)
255		);
256		}
257
258		/**
259		* Get published status flag
260		*
261		* @param File $record
262		* @return null\|string
263		*/
264		protected function getStatusFlagMarkup($record)
265		{
266		if ($record && ($statusTitle = $record->getStatusTitle())) {
267		return "<span class=\"editor__status-flag\">{$statusTitle}</span>";
268		}
269		return null;
270		}
271
272		/**
273		* Get action for publishing
274		*
275		* @param File $record
276		* @return FormAction
277		*/
278		protected function getUnpublishAction($record)
279		{
280		// Check if record is unpublishable
281		if (!$record \|\| !$record->isPublished() \|\| !$record->canUnpublish()) {
282		return null;
283		}
284
285		// Build action
286		$unpublishText = _t(
287		'SilverStripe\\AssetAdmin\\Controller\\AssetAdmin.UNPUBLISH_BUTTON',
288		'Unpublish'
289		);
290		return FormAction::create('unpublish', $unpublishText)
291		->setIcon('cancel-circled');
292		}
293
294		/**
295		* Get actions that go into the Popover menu
296		*
297		* @param $record
298		* @return array
299		*/
300		protected function getPopoverActions($record)
301		{
302		$this->beforeExtending('updatePopoverActions', function (&$actions, $record) {
303		// add the unpublish action to the start of the array
304		array_unshift($actions, $this->getUnpublishAction($record));
305		});
306
307		return parent::getPopoverActions($record);
308		}
309
310		/**
311		* @param File $record
312		* @return FormAction
313		*/
314	View Code Duplication	protected function getInsertAction($record)
		0 ignored issues – show Duplication introduced 2016-10-17 01:27 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
315		{
316		$action = null;
317		if ($record && $record->isInDB() && $record->canEdit()) {
318		/** @var FormAction $action */
319		$action = FormAction::create('insert', _t(__CLASS__.'.INSERT_FILE', 'Insert file'))
320		->setSchemaData(['data' => ['buttonStyle' => 'primary']]);
321		}
322		return $action;
323		}
324
325		/**
326		* @param File $record
327		* @return FormAction
328		*/
329	View Code Duplication	protected function getUpdateAction($record)
		0 ignored issues – show Duplication introduced 2017-05-31 03:51 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
330		{
331		$action = null;
332		if ($record && $record->isInDB() && $record->canEdit()) {
333		/** @var FormAction $action */
334		$action = FormAction::create('update', _t(__CLASS__.'.UPDATE_FILE', 'Update file'))
335		->setSchemaData(['data' => ['buttonStyle' => 'primary']]);
336		}
337		return $action;
338		}
339
340		/**
341		* @param File $record
342		* @return FormAction
343		*/
344	View Code Duplication	protected function getInsertLinkAction($record)
		0 ignored issues – show Duplication introduced 2017-05-31 03:51 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
345		{
346		$action = null;
347		if ($record && $record->isInDB() && $record->canEdit()) {
348		/** @var FormAction $action */
349		$action = FormAction::create('insert', _t(__CLASS__.'.INSERT_LINK', 'Link to file'))
350		->setSchemaData(['data' => ['buttonStyle' => 'primary']]);
351		}
352		return $action;
353		}
354
355		/**
356		* @param File $record
357		* @return FormAction
358		*/
359	View Code Duplication	protected function getUpdateLinkAction($record)
		0 ignored issues – show Duplication introduced 2017-06-09 04:20 UTC by Report Bug Copy Issue Report This method seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
360		{
361		$action = null;
362		if ($record && $record->isInDB() && $record->canEdit()) {
363		/** @var FormAction $action */
364		$action = FormAction::create('insert', _t(__CLASS__.'.UPDATE_LINK', 'Update link'))
365		->setSchemaData(['data' => ['buttonStyle' => 'primary']]);
366		}
367		return $action;
368		}
369		}
370

silverstripe / silverstripe-asset-admin

Pull Request — master (#499)

FileFormFactory::getUpdateAction() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like