SilverStripe\Taxonomy\Extensions\TaxonomyTermUrlExtension

Complexity

Total Complexity

6

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	onBeforeWrite()	0	11	4
A	updateCMSFields()	0	6	2

<?php

namespace SilverStripe\Taxonomy\Extensions;

use SilverStripe\Taxonomy\TaxonomyTerm;
use SilverStripe\ORM\DataExtension;
use SilverStripe\CMS\Model\SiteTree;

The type SilverStripe\CMS\Model\SiteTree was not found. Maybe you did not declare it correctly or list all dependencies?

use SilverStripe\Forms\FieldList;

/**
* Adds a URLSegment to TaxonomyTerm that gets
* generated from the name if not specified.
*
* @package taxonomy
*/
class TaxonomyTermUrlExtension extends DataExtension
{
    private static $db = array(

The private property $db is not used, and could be removed.

        'URLSegment' => 'Varchar(255)',
    );

    public function updateCMSFields(FieldList $fields)
    {
        $field = $fields->dataFieldByName('URLSegment');
        if ($field) {

$field is of type SilverStripe\Forms\FormField, thus it always evaluated to true.

            $field->setDescription('Leave blank to generate from name');
            $fields->insertAfter('Name', $field);
        }
    }

    /**
     *  Set the URL segment allowing filtering by url slug
     *
     *  {@inheritDoc}
     */
    public function onBeforeWrite() {
         parent::onBeforeWrite();
         // Write the URLSegment to allow for Taxonomy navigation only if one is not supplied
        if ($this->owner->Name && $this->owner->URLSegment == null) {
            $this->owner->setField('URLSegment', SiteTree::singleton()->generateURLSegment($this->owner->Name));
             // Ensure that this object has a non-conflicting URLSegment value.
            $count = 2;
            while (TaxonomyTerm::get()->filter(['URLSegment' => $this->owner->URLSegment])->exclude(array('ID' => $this->owner->ID))->exists())
            {
                $this->owner->setField('URLSegment', preg_replace('/-[0-9]+$/', null, $this->owner->URLSegment) . '-' . $count);
                $count++;
            }
        }
    }
}