dynamic / silverstripe-locator

src/admin/LocationsAdmin.php

<?php

namespace Dynamic\Locator\Admin;

use Dynamic\Locator\Bulkloader\LocationCsvBulkLoader;
use Dynamic\Locator\Model\LocationCategory;
use Dynamic\Locator\Page\LocationPage;
use LittleGiant\CatalogManager\ModelAdmin\CatalogPageAdmin;
use SilverStripe\Dev\CsvBulkLoader;
use SilverStripe\Forms\Form;
use SilverStripe\Forms\GridField\GridField;

/**
 * Class LocationAdmin
 * @package Dynamic\Locator\Admin
 */
class LocationsAdmin extends CatalogPageAdmin
{

    /**
     * @var array
     */
    private static $managed_models = [

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

        LocationPage::class,
        LocationCategory::class,
    ];

    /**
     * @var array
     */
    private static $model_importers = [

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

        LocationCategory::class => CsvBulkLoader::class,
    ];

    /**
     * @var string
     */
    private static $menu_title = 'Locations';

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

    /**
     * @var string
     */
    private static $url_segment = 'locations';

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

    /**
     * @return array
     */
    public function getExportFields()
    {
        if ($this->modelClass == LocationPage::class) {
            $fields = [
                'Title' => 'Name',
                'Address' => 'Address',
                'Address2' => 'Address2',
                'City' => 'City',
                'State' => 'State',
                'PostalCode' => 'PostalCode',
                'CountryCode' => 'Country',
                'Phone' => 'Phone',
                'Fax' => 'Fax',
                'Email' => 'Email',
                'Website' => 'Website',
                'Featured' => 'Featured',
                'CategoryList' => 'Categories',
                'Lat' => 'Lat',
                'Lng' => 'Lng',
                'Import_ID' => 'Import_ID',
            ];
        }

        if (!isset($fields)) {
            $fields = parent::getExportFields();
        }

        $this->extend('updateGetExportFields', $fields);

        return $fields;
    }

    /**
     * @param null $id

Are you sure the doc-type for parameter $id is correct as it would always require null to be passed?

     * @param null $fields

Are you sure the doc-type for parameter $fields is correct as it would always require null to be passed?

     * @return $this|Form
     */
    public function getEditForm($id = null, $fields = null)
    {
        $form = parent::getEditForm($id, $fields);

        if ($this->modelClass == LocationPage::class) {
            /** @var GridField $gridField */
            if ($gridField = $form->Fields()->fieldByName($this->sanitiseClassName($this->modelClass))) {
                $gridField->getConfig()
                    ->removeComponentsByType('GridFieldDeleteAction');
            }
        }

        return $form;
    }
}