Dynamic\Locator\LocationCsvBulkLoader

Complexity

Total Complexity

2

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	getCategoryByName()	0	11	2

<?php

namespace Dynamic\Locator;

use SilverStripe\Dev\CsvBulkLoader;
use SilverStripe\Core\Convert;

/**
 * Class LocationCsvBulkLoader
 */
class LocationCsvBulkLoader extends CsvBulkLoader
{

    /**
     * @var array
     */
    public $columnMap = array(
        'Name' => 'Title',
        'City' => 'Suburb',
        'EmailAddress' => 'Email',
        'Import_ID' => 'Import_ID',
    );

    /**
     * @var array
     */
    public $duplicateChecks = array(
        'Import_ID' => 'Import_ID'
    );

    /**
     * @var array
     */
    /*
    public $relationCallbacks = array(
       'Category.Name' => array(
           'relationname' => 'Category',
           'callback' => 'getCategoryByName',
        ),
    );*/

    /**
     * @param $obj
     * @param $val
     * @param $record
     * @return \SilverStripe\ORM\DataObject|static
     */
    public static function getCategoryByName(&$obj, $val, $record)

The parameter $record is not used and could be removed.

The parameter $obj is not used and could be removed.

    {
        $val = Convert::raw2sql($val);
        $category = LocationCategory::get()->filter(array('Name' => $val))->First();
        if (!$category) {

$category is of type SilverStripe\ORM\DataObject, thus it always evaluated to true.

            $category = LocationCategory::create();
            $category->Name = $val;
            $category->write();
        }

        return $category;
    }
}