dynamic / silverstripe-locator

src/Page/LocationPage.php

Indentation +256 added lines, -256 removed lines

@@ -31,260 +31,260 @@
  */
 class LocationPage extends \Page implements PermissionProvider
 {
-    /**
-     * @var string
-     */
-    private static $singular_name = 'Location';
-
-    /**
-     * @var string
-     */
-    private static $plural_name = 'Locations';
-
-    /**
-     * @var array
-     */
-    private static $db = [
-        'Title' => 'Varchar(255)',
-        'Featured' => 'Boolean',
-        'Website' => 'Varchar(255)',
-        'Phone' => 'Varchar(40)',
-        'Email' => 'Varchar(255)',
-        'Fax' => 'Varchar(45)',
-        'Import_ID' => 'Int',
-        'LegacyObjectID' => 'Int',
-    ];
-
-    private static $many_many = [
-        'Categories' => LocationCategory::class,
-    ];
-
-    /**
-     * @var string
-     */
-    private static $table_name = 'LocationPage';
-
-    /**
-     * @var array
-     */
-    private static $casting = [
-        'distance' => 'Decimal(9,3)',
-    ];
-
-    /**
-     * @var string
-     */
-    private static $default_sort = 'Title';
-
-    /**
-     * api access via Restful Server module
-     *
-     * @var bool
-     */
-    private static $api_access = true;
-
-    /**
-     * search fields for Model Admin
-     *
-     * @var array
-     */
-    private static $searchable_fields = [
-        'Title',
-        'Address',
-        'City',
-        'State',
-        'PostalCode',
-        'Country',
-        'Website',
-        'Phone',
-        'Email',
-        'Featured',
-    ];
-
-    /**
-     * columns for grid field
-     *
-     * @var array
-     */
-    private static $summary_fields = [
-        'Title',
-        'Address',
-        'Address2',
-        'City',
-        'State',
-        'PostalCode',
-        'CountryCode',
-        'Phone' => 'Phone',
-        'Fax' => 'Fax',
-        'Email' => 'Email',
-        'Website' => 'Website',
-        'Featured',
-        'CategoryList',
-        'Lat',
-        'Lng',
-        'Import_ID',
-    ];
-
-    /**
-     * Coords status for $summary_fields
-     *
-     * @return string
-     */
-    public function getCoords()
-    {
-        return ($this->Lat != 0 && $this->Lng != 0) ? 'true' : 'false';
-    }
-
-    /**
-     * @return string
-     */
-    public function getCategoryList()
-    {
-        if ($this->Categories()->count()) {
-            return implode(', ', $this->Categories()->column('Name'));
-        }
-
-        return '';
-    }
-
-    /**
-     * @return bool|string
-     */
-    public function getCountryCode()
-    {
-        if ($this->Country) {
-            return strtoupper($this->Country);
-        }
-
-        return false;
-    }
-
-    /**
-     * custom labels for fields
-     *
-     * @param bool $includerelations
-     * @return array|string
-     */
-    public function fieldLabels($includerelations = true)
-    {
-        $labels = parent::fieldLabels($includerelations);
-
-        $labels['Title'] = 'Name';
-        $labels['Address2'] = 'Address 2';
-        $labels['PostalCode'] = 'Postal Code';
-        $labels['Categories.Name'] = 'Categories';
-        $labels['Featured.NiceAsBoolean'] = 'Featured';
-
-        return $labels;
-    }
-
-    /**
-     * @return FieldList
-     */
-    public function getCMSFields()
-    {
-        $this->beforeUpdateCMSFields(function (FieldList $fields) {
-            $fields->addFieldsToTab(
-                'Root.Main',
-                [
-                    CheckboxField::create('Featured')
-                        ->setTitle('Featured'),
-                    TextField::create('Website')
-                        ->setTitle('Website')
-                        ->setDescription('Include the http/https (example: https://google.com)'),
-                    TextField::create('Phone')
-                        ->setTitle('Phone'),
-                    EmailField::create('Email')
-                        ->setTitle('Email'),
-                    TextField::create('Fax')
-                        ->setTitle('Fax'),
-                ],
-                'Content'
-            );
-
-            if ($this->exists()) {
-                $fields->addFieldToTab(
-                    'Root.Categories',
-                    GridField::create(
-                        'Categories',
-                        'Categories',
-                        $this->Categories(),
-                        $catConfig = GridFieldConfig_RelationEditor::create()
-                    )
-                );
-
-                $catConfig->removeComponentsByType([
-                    GridFieldAddExistingAutocompleter::class,
-                ])->addComponents([
-                    new GridFieldAddExistingSearchButton()
-                ]);
-            }
-        });
-
-        return parent::getCMSFields();
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool
-     */
-    public function canView($member = null, $context = [])
-    {
-        return true;
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool|int
-     */
-    public function canEdit($member = null, $context = [])
-    {
-        return Permission::check('Location_EDIT', 'any', $member);
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool|int
-     */
-    public function canDelete($member = null, $context = [])
-    {
-        return Permission::check('Location_DELETE', 'any', $member);
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool|int
-     */
-    public function canCreate($member = null, $context = [])
-    {
-        return Permission::check('Location_CREATE', 'any', $member);
-    }
-
-    /**
-     * @return array
-     */
-    public function providePermissions()
-    {
-        return [
-            'Location_EDIT' => 'Edit a Location',
-            'Location_DELETE' => 'Delete a Location',
-            'Location_CREATE' => 'Create a Location',
-        ];
-    }
-
-    /**
-     * @return string
-     */
-    public function getWebsiteURL()
-    {
-        $url = $this->Website;
-
-        $this->extend('updateWebsiteURL', $url);
-
-        return $url;
-    }
+	/**
+	 * @var string
+	 */
+	private static $singular_name = 'Location';
+
+	/**
+	 * @var string
+	 */
+	private static $plural_name = 'Locations';
+
+	/**
+	 * @var array
+	 */
+	private static $db = [
+		'Title' => 'Varchar(255)',
+		'Featured' => 'Boolean',
+		'Website' => 'Varchar(255)',
+		'Phone' => 'Varchar(40)',
+		'Email' => 'Varchar(255)',
+		'Fax' => 'Varchar(45)',
+		'Import_ID' => 'Int',
+		'LegacyObjectID' => 'Int',
+	];
+
+	private static $many_many = [
+		'Categories' => LocationCategory::class,
+	];
+
+	/**
+	 * @var string
+	 */
+	private static $table_name = 'LocationPage';
+
+	/**
+	 * @var array
+	 */
+	private static $casting = [
+		'distance' => 'Decimal(9,3)',
+	];
+
+	/**
+	 * @var string
+	 */
+	private static $default_sort = 'Title';
+
+	/**
+	 * api access via Restful Server module
+	 *
+	 * @var bool
+	 */
+	private static $api_access = true;
+
+	/**
+	 * search fields for Model Admin
+	 *
+	 * @var array
+	 */
+	private static $searchable_fields = [
+		'Title',
+		'Address',
+		'City',
+		'State',
+		'PostalCode',
+		'Country',
+		'Website',
+		'Phone',
+		'Email',
+		'Featured',
+	];
+
+	/**
+	 * columns for grid field
+	 *
+	 * @var array
+	 */
+	private static $summary_fields = [
+		'Title',
+		'Address',
+		'Address2',
+		'City',
+		'State',
+		'PostalCode',
+		'CountryCode',
+		'Phone' => 'Phone',
+		'Fax' => 'Fax',
+		'Email' => 'Email',
+		'Website' => 'Website',
+		'Featured',
+		'CategoryList',
+		'Lat',
+		'Lng',
+		'Import_ID',
+	];
+
+	/**
+	 * Coords status for $summary_fields
+	 *
+	 * @return string
+	 */
+	public function getCoords()
+	{
+		return ($this->Lat != 0 && $this->Lng != 0) ? 'true' : 'false';
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getCategoryList()
+	{
+		if ($this->Categories()->count()) {
+			return implode(', ', $this->Categories()->column('Name'));
+		}
+
+		return '';
+	}
+
+	/**
+	 * @return bool|string
+	 */
+	public function getCountryCode()
+	{
+		if ($this->Country) {
+			return strtoupper($this->Country);
+		}
+
+		return false;
+	}
+
+	/**
+	 * custom labels for fields
+	 *
+	 * @param bool $includerelations
+	 * @return array|string
+	 */
+	public function fieldLabels($includerelations = true)
+	{
+		$labels = parent::fieldLabels($includerelations);
+
+		$labels['Title'] = 'Name';
+		$labels['Address2'] = 'Address 2';
+		$labels['PostalCode'] = 'Postal Code';
+		$labels['Categories.Name'] = 'Categories';
+		$labels['Featured.NiceAsBoolean'] = 'Featured';
+
+		return $labels;
+	}
+
+	/**
+	 * @return FieldList
+	 */
+	public function getCMSFields()
+	{
+		$this->beforeUpdateCMSFields(function (FieldList $fields) {
+			$fields->addFieldsToTab(
+				'Root.Main',
+				[
+					CheckboxField::create('Featured')
+						->setTitle('Featured'),
+					TextField::create('Website')
+						->setTitle('Website')
+						->setDescription('Include the http/https (example: https://google.com)'),
+					TextField::create('Phone')
+						->setTitle('Phone'),
+					EmailField::create('Email')
+						->setTitle('Email'),
+					TextField::create('Fax')
+						->setTitle('Fax'),
+				],
+				'Content'
+			);
+
+			if ($this->exists()) {
+				$fields->addFieldToTab(
+					'Root.Categories',
+					GridField::create(
+						'Categories',
+						'Categories',
+						$this->Categories(),
+						$catConfig = GridFieldConfig_RelationEditor::create()
+					)
+				);
+
+				$catConfig->removeComponentsByType([
+					GridFieldAddExistingAutocompleter::class,
+				])->addComponents([
+					new GridFieldAddExistingSearchButton()
+				]);
+			}
+		});
+
+		return parent::getCMSFields();
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool
+	 */
+	public function canView($member = null, $context = [])
+	{
+		return true;
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool|int
+	 */
+	public function canEdit($member = null, $context = [])
+	{
+		return Permission::check('Location_EDIT', 'any', $member);
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool|int
+	 */
+	public function canDelete($member = null, $context = [])
+	{
+		return Permission::check('Location_DELETE', 'any', $member);
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool|int
+	 */
+	public function canCreate($member = null, $context = [])
+	{
+		return Permission::check('Location_CREATE', 'any', $member);
+	}
+
+	/**
+	 * @return array
+	 */
+	public function providePermissions()
+	{
+		return [
+			'Location_EDIT' => 'Edit a Location',
+			'Location_DELETE' => 'Delete a Location',
+			'Location_CREATE' => 'Create a Location',
+		];
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getWebsiteURL()
+	{
+		$url = $this->Website;
+
+		$this->extend('updateWebsiteURL', $url);
+
+		return $url;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -183,7 +183,7 @@
      */
     public function getCMSFields()
     {
-        $this->beforeUpdateCMSFields(function (FieldList $fields) {
+        $this->beforeUpdateCMSFields(function(FieldList $fields) {
             $fields->addFieldsToTab(
                 'Root.Main',
                 [

src/admin/LocationAdmin.php

Indentation +70 added lines, -70 removed lines

@@ -17,83 +17,83 @@
 class LocationAdmin extends ModelAdmin
 {
 
-    /**
-     * @var array
-     */
-    private static $managed_models = [
-        Location::class,
-        LocationCategory::class,
-    ];
+	/**
+	 * @var array
+	 */
+	private static $managed_models = [
+		Location::class,
+		LocationCategory::class,
+	];
 
-    /**
-     * @var array
-     */
-    private static $model_importers = [
-        Location::class => LocationCsvBulkLoader::class,
-        LocationCategory::class => CsvBulkLoader::class,
-    ];
+	/**
+	 * @var array
+	 */
+	private static $model_importers = [
+		Location::class => LocationCsvBulkLoader::class,
+		LocationCategory::class => CsvBulkLoader::class,
+	];
 
-    /**
-     * @var string
-     */
-    private static $menu_title = 'Locator';
-    /**
-     * @var string
-     */
-    private static $url_segment = 'locator';
+	/**
+	 * @var string
+	 */
+	private static $menu_title = 'Locator';
+	/**
+	 * @var string
+	 */
+	private static $url_segment = 'locator';
 
-    /**
-     * @return array
-     */
-    public function getExportFields()
-    {
-        if ($this->modelClass == Location::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',
-            ];
-        }
+	/**
+	 * @return array
+	 */
+	public function getExportFields()
+	{
+		if ($this->modelClass == Location::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();
-        }
+		if (!isset($fields)) {
+			$fields = parent::getExportFields();
+		}
 
-        $this->extend('updateGetExportFields', $fields);
+		$this->extend('updateGetExportFields', $fields);
 
-        return $fields;
-    }
+		return $fields;
+	}
 
-    /**
-     * @param null $id
-     * @param null $fields
-     * @return $this|Form
-     */
-    public function getEditForm($id = null, $fields = null)
-    {
-        $form = parent::getEditForm($id, $fields);
+	/**
+	 * @param null $id
+	 * @param null $fields
+	 * @return $this|Form
+	 */
+	public function getEditForm($id = null, $fields = null)
+	{
+		$form = parent::getEditForm($id, $fields);
 
-        if ($this->modelClass == Location::class) {
-            /** @var GridField $gridField */
-            if ($gridField = $form->Fields()->fieldByName($this->sanitiseClassName($this->modelClass))) {
-                $gridField->getConfig()
-                    ->removeComponentsByType('GridFieldDeleteAction');
-            }
-        }
+		if ($this->modelClass == Location::class) {
+			/** @var GridField $gridField */
+			if ($gridField = $form->Fields()->fieldByName($this->sanitiseClassName($this->modelClass))) {
+				$gridField->getConfig()
+					->removeComponentsByType('GridFieldDeleteAction');
+			}
+		}
 
-        return $form;
-    }
+		return $form;
+	}
 }

src/Tasks/LocationToPageTask.php

Indentation +58 added lines, -58 removed lines

@@ -15,72 +15,72 @@
  */
 class LocationToPageTask extends BuildTask
 {
-    /**
-     * @var string
-     */
-    protected $title = 'Locator - Location to Page Task';
+	/**
+	 * @var string
+	 */
+	protected $title = 'Locator - Location to Page Task';
 
-    /**
-     * @var string
-     */
-    private static $segment = 'locator-location-to-page-task';
+	/**
+	 * @var string
+	 */
+	private static $segment = 'locator-location-to-page-task';
 
-    /**
-     * @var bool
-     */
-    private static $auto_publish_location = false;
+	/**
+	 * @var bool
+	 */
+	private static $auto_publish_location = false;
 
-    /**
-     * @param \SilverStripe\Control\HTTPRequest $request
-     */
-    public function run($request)
-    {
-        $this->migrateLocations();
-    }
+	/**
+	 * @param \SilverStripe\Control\HTTPRequest $request
+	 */
+	public function run($request)
+	{
+		$this->migrateLocations();
+	}
 
-    /**
-     *
-     */
-    protected function migrateLocations()
-    {
-        if (!$parent = Locator::get()->first()) {
-            $parent = Locator::create();
-            $parent->Title = "Locator";
-            $parent->writeToStage(Versioned::DRAFT);
-        }
+	/**
+	 *
+	 */
+	protected function migrateLocations()
+	{
+		if (!$parent = Locator::get()->first()) {
+			$parent = Locator::create();
+			$parent->Title = "Locator";
+			$parent->writeToStage(Versioned::DRAFT);
+		}
 
-        /** @var Location $location */
-        foreach ($this->getLocations() as $location) {
-            if (!LocationPage::get()->filter('LegacyObjectID', $location->ID)->first()) {
-                if ($location->hasMethod('isPublished')) {
-                    $published = $location->isPublished();
-                }
+		/** @var Location $location */
+		foreach ($this->getLocations() as $location) {
+			if (!LocationPage::get()->filter('LegacyObjectID', $location->ID)->first()) {
+				if ($location->hasMethod('isPublished')) {
+					$published = $location->isPublished();
+				}
 
-                /** @var LocationPage $newLocation */
-                $newLocation = Injector::inst()->create(LocationPage::class, $location->toMap(), false);
-                $newLocation->setClassName(LocationPage::class);
-                $newLocation->ID = null;
-                $newLocation->ParentID = $parent->ID;
-                $newLocation->LegacyObjectID = $location->ID;
+				/** @var LocationPage $newLocation */
+				$newLocation = Injector::inst()->create(LocationPage::class, $location->toMap(), false);
+				$newLocation->setClassName(LocationPage::class);
+				$newLocation->ID = null;
+				$newLocation->ParentID = $parent->ID;
+				$newLocation->LegacyObjectID = $location->ID;
 
-                $this->extend('preLocationMigrationUpdate', $location, $newLocation);
+				$this->extend('preLocationMigrationUpdate', $location, $newLocation);
 
-                $newLocation->writeToStage(Versioned::DRAFT);
+				$newLocation->writeToStage(Versioned::DRAFT);
 
-                if ((isset($published) && $published) || $this->config()->get('auto_publish_location')) {
-                    $newLocation->publishSingle();
-                }
-            }
-        }
-    }
+				if ((isset($published) && $published) || $this->config()->get('auto_publish_location')) {
+					$newLocation->publishSingle();
+				}
+			}
+		}
+	}
 
-    /**
-     * @return \Generator
-     */
-    protected function getLocations()
-    {
-        foreach (Location::get() as $location) {
-            yield $location;
-        }
-    }
+	/**
+	 * @return \Generator
+	 */
+	protected function getLocations()
+	{
+		foreach (Location::get() as $location) {
+			yield $location;
+		}
+	}
 }

src/Tasks/LocationPublishTask.php

Indentation +66 added lines, -66 removed lines

@@ -11,76 +11,76 @@
  */
 class LocationPublishTask extends BuildTask
 {
-    /**
-     * @var string
-     */
-    protected $title = 'Publish all Locations';
-    /**
-     * @var string
-     */
-    protected $description = 'Migration task - pre versioning on Location';
-    /**
-     * @var bool
-     */
-    protected $enabled = true;
+	/**
+	 * @var string
+	 */
+	protected $title = 'Publish all Locations';
+	/**
+	 * @var string
+	 */
+	protected $description = 'Migration task - pre versioning on Location';
+	/**
+	 * @var bool
+	 */
+	protected $enabled = true;
 
-    /**
-     * @param $request
-     */
-    public function run($request)
-    {
-        $class = ($request->getVar('locationclass')) ? $request->getVar('locationclass') : 'Location';
-        $this->publishLocations($class);
-    }
+	/**
+	 * @param $request
+	 */
+	public function run($request)
+	{
+		$class = ($request->getVar('locationclass')) ? $request->getVar('locationclass') : 'Location';
+		$this->publishLocations($class);
+	}
 
-    /**
-     * @param string $class
-     * @return \Generator
-     */
-    protected function iterateLocations($class)
-    {
-        foreach ($class::get()->filter('ShowInLocator', true) as $location) {
-            yield $location;
-        }
-    }
+	/**
+	 * @param string $class
+	 * @return \Generator
+	 */
+	protected function iterateLocations($class)
+	{
+		foreach ($class::get()->filter('ShowInLocator', true) as $location) {
+			yield $location;
+		}
+	}
 
-    /**
-     * mark all ProductDetail records as ShowInMenus = 0.
-     *
-     * @param string $class
-     */
-    public function publishLocations($class)
-    {
-        if (!isset($class) || !class_exists($class) || !$class instanceof Location) {
-            $class = 'Location';
-        }
-        $ct = 0;
-        $publish = function ($location) use (&$ct) {
-            if (!$location->isPublished()) {
-                $title = $location->Title;
-                $location->writeToStage('Stage');
-                $location->publish('Stage', 'Live');
-                static::write_message($title);
-                ++$ct;
-            }
-        };
+	/**
+	 * mark all ProductDetail records as ShowInMenus = 0.
+	 *
+	 * @param string $class
+	 */
+	public function publishLocations($class)
+	{
+		if (!isset($class) || !class_exists($class) || !$class instanceof Location) {
+			$class = 'Location';
+		}
+		$ct = 0;
+		$publish = function ($location) use (&$ct) {
+			if (!$location->isPublished()) {
+				$title = $location->Title;
+				$location->writeToStage('Stage');
+				$location->publish('Stage', 'Live');
+				static::write_message($title);
+				++$ct;
+			}
+		};
 
-        foreach ($this->iterateLocations($class) as $location) {
-            $publish($location);
-        }
+		foreach ($this->iterateLocations($class) as $location) {
+			$publish($location);
+		}
 
-        static::write_message("{$ct} locations updated.");
-    }
+		static::write_message("{$ct} locations updated.");
+	}
 
-    /**
-     * @param $message
-     */
-    protected static function write_message($message)
-    {
-        if (Director::is_cli()) {
-            echo "{$message}\n";
-        } else {
-            echo "{$message}<br><br>";
-        }
-    }
+	/**
+	 * @param $message
+	 */
+	protected static function write_message($message)
+	{
+		if (Director::is_cli()) {
+			echo "{$message}\n";
+		} else {
+			echo "{$message}<br><br>";
+		}
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
             $class = 'Location';
         }
         $ct = 0;
-        $publish = function ($location) use (&$ct) {
+        $publish = function($location) use (&$ct) {
             if (!$location->isPublished()) {
                 $title = $location->Title;
                 $location->writeToStage('Stage');

src/Tasks/LocationCategoriesTask.php

Indentation +68 added lines, -68 removed lines

@@ -14,72 +14,72 @@
 class LocationCategoriesTask extends BuildTask
 {
 
-    /**
-     * @var string
-     */
-    protected $title = 'Location categories to many_many';
-
-    /**
-     * @var string
-     */
-    protected $description = 'Migration task - Converts locations to have multiple categories';
-
-    /**
-     * @var bool
-     */
-    protected $enabled = true;
-
-    /**
-     * @param $request
-     */
-    public function run($request)
-    {
-        /** @var DataObject $class */
-        $class = ($request->getVar('locationclass')) ? $request->getVar('locationclass') : Location::class;
-        $class::add_extension(LocationCategoryExtension::class);
-
-
-        $categories = [];
-
-        $convert = function (DataObject $location) use (&$categories) {
-            /** @var Location $location */
-            // skip if no category
-            if ($location->CategoryID > 0) {
-                $categories[$location->CategoryID][] = $location->ID;
-            }
-        };
-
-        foreach ($this->iterateLocations($class) as $location) {
-            $convert($location);
-        }
-
-        $catCt = 0;
-        $locCT = 0;
-
-        foreach ($categories as $categoryID => $locations) {
-            /** @var LocationCategory $category */
-            $category = LocationCategory::get()->byID($categoryID);
-            $category->Locations()->addMany($locations);
-
-            $catCt++;
-            $locCT += count($locations);
-        }
-
-        echo "{$catCt} categories converted<br />";
-        echo "{$locCT} location relations converted<br />";
-
-        $time = microtime(true) - $_SERVER["REQUEST_TIME_FLOAT"];
-        echo "Process Time: {$time} seconds";
-    }
-
-    /**
-     * @param string $class
-     * @return \Generator
-     */
-    protected function iterateLocations($class)
-    {
-        foreach ($class::get() as $location) {
-            yield $location;
-        }
-    }
+	/**
+	 * @var string
+	 */
+	protected $title = 'Location categories to many_many';
+
+	/**
+	 * @var string
+	 */
+	protected $description = 'Migration task - Converts locations to have multiple categories';
+
+	/**
+	 * @var bool
+	 */
+	protected $enabled = true;
+
+	/**
+	 * @param $request
+	 */
+	public function run($request)
+	{
+		/** @var DataObject $class */
+		$class = ($request->getVar('locationclass')) ? $request->getVar('locationclass') : Location::class;
+		$class::add_extension(LocationCategoryExtension::class);
+
+
+		$categories = [];
+
+		$convert = function (DataObject $location) use (&$categories) {
+			/** @var Location $location */
+			// skip if no category
+			if ($location->CategoryID > 0) {
+				$categories[$location->CategoryID][] = $location->ID;
+			}
+		};
+
+		foreach ($this->iterateLocations($class) as $location) {
+			$convert($location);
+		}
+
+		$catCt = 0;
+		$locCT = 0;
+
+		foreach ($categories as $categoryID => $locations) {
+			/** @var LocationCategory $category */
+			$category = LocationCategory::get()->byID($categoryID);
+			$category->Locations()->addMany($locations);
+
+			$catCt++;
+			$locCT += count($locations);
+		}
+
+		echo "{$catCt} categories converted<br />";
+		echo "{$locCT} location relations converted<br />";
+
+		$time = microtime(true) - $_SERVER["REQUEST_TIME_FLOAT"];
+		echo "Process Time: {$time} seconds";
+	}
+
+	/**
+	 * @param string $class
+	 * @return \Generator
+	 */
+	protected function iterateLocations($class)
+	{
+		foreach ($class::get() as $location) {
+			yield $location;
+		}
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
 
         $categories = [];
 
-        $convert = function (DataObject $location) use (&$categories) {
+        $convert = function(DataObject $location) use (&$categories) {
             /** @var Location $location */
             // skip if no category
             if ($location->CategoryID > 0) {

src/Tasks/LocationCategoryExtension.php

Indentation +3 added lines, -3 removed lines

@@ -13,7 +13,7 @@
  */
 class LocationCategoryExtension extends DataExtension
 {
-    private static $has_one = [
-        'Category' => LocationCategory::class,
-    ];
+	private static $has_one = [
+		'Category' => LocationCategory::class,
+	];
 }

src/Tasks/EmailAddressTask.php

Indentation +17 added lines, -17 removed lines

@@ -8,22 +8,22 @@
 
 class EmailAddressTask extends BuildTask
 {
-    protected $title = 'Email Address Task'; // title of the script
-    protected $description = "Convert depreciated 'Email Address' field to new 'Email' field.";
+	protected $title = 'Email Address Task'; // title of the script
+	protected $description = "Convert depreciated 'Email Address' field to new 'Email' field.";
 
-    public function run($request)
-    {
-        Config::inst()->update('DataObject', 'validation_enabled', false);
-        $ct = 0;
-        $updateEmail = function ($location) use (&$ct) {
-            if (!$location->Email && $location->EmailAddress) {
-                $location->Email = $location->EmailAddress;
-                $location->write();
-                ++$ct;
-            }
-        };
-        Location::get()->each($updateEmail);
-        Config::inst()->update('DataObject', 'validation_enabled', true);
-        echo '<p>'.$ct.' Locations updated</p>';
-    }
+	public function run($request)
+	{
+		Config::inst()->update('DataObject', 'validation_enabled', false);
+		$ct = 0;
+		$updateEmail = function ($location) use (&$ct) {
+			if (!$location->Email && $location->EmailAddress) {
+				$location->Email = $location->EmailAddress;
+				$location->write();
+				++$ct;
+			}
+		};
+		Location::get()->each($updateEmail);
+		Config::inst()->update('DataObject', 'validation_enabled', true);
+		echo '<p>'.$ct.' Locations updated</p>';
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
     {
         Config::inst()->update('DataObject', 'validation_enabled', false);
         $ct = 0;
-        $updateEmail = function ($location) use (&$ct) {
+        $updateEmail = function($location) use (&$ct) {
             if (!$location->Email && $location->EmailAddress) {
                 $location->Email = $location->EmailAddress;
                 $location->write();

src/Model/LocationCategory.php

Indentation +121 added lines, -121 removed lines

@@ -23,125 +23,125 @@
  */
 class LocationCategory extends DataObject
 {
-    /**
-     * @var string
-     */
-    private static $singular_name = 'Category';
-
-    /**
-     * @var string
-     */
-    private static $plural_name = 'Categories';
-
-    /**
-     * @var array
-     */
-    private static $db = [
-        'Name' => 'Varchar(100)',
-    ];
-
-    /**
-     * @var array
-     */
-    private static $belongs_many_many = [
-        'Locators' => Locator::class,
-        'LocationSet' => Location::class,
-    ];
-
-    /**
-     * @var string
-     */
-    private static $table_name = 'LocationCategory';
-
-    /**
-     * @var string
-     */
-    private static $default_sort = 'Name';
-
-    /**
-     * @return \SilverStripe\Forms\FieldList
-     */
-    public function getCMSFields()
-    {
-        $this->beforeUpdateCMSFields(function (FieldList $fields) {
-            $fields->removeByName([
-                'Locations',
-                'LocationSet',
-                'Locators',
-                'LinkTracking',
-                'FileTracking',
-            ]);
-
-            if ($this->ID) {
-                // Locations
-                $config = GridFieldConfig_RelationEditor::create();
-                $config->removeComponentsByType([
-                    GridFieldAddExistingAutocompleter::class,
-                    GridFieldAddNewButton::class,
-                ])
-                    ->addComponents([
-                        new GridFieldAddExistingSearchButton(),
-                    ]);
-                $locations = $this->Locations();
-                $locationField = GridField::create('Locations', 'Locations', $locations, $config);
-
-                $fields->addFieldsToTab('Root.Main', [
-                    $locationField,
-                ]);
-            }
-        });
-
-        $fields = parent::getCMSFields();
-
-        return $fields;
-    }
-
-    /**
-     * For backwards compatability
-     * @return ManyManyList
-     */
-    public function Locations()
-    {
-        return $this->LocationSet();
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool
-     */
-    public function canView($member = null, $context = [])
-    {
-        return true;
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool|int
-     */
-    public function canEdit($member = null, $context = [])
-    {
-        return Permission::check('Location_EDIT', 'any', $member);
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool|int
-     */
-    public function canDelete($member = null, $context = [])
-    {
-        return Permission::check('Location_DELETE', 'any', $member);
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool|int
-     */
-    public function canCreate($member = null, $context = [])
-    {
-        return Permission::check('Location_CREATE', 'any', $member);
-    }
+	/**
+	 * @var string
+	 */
+	private static $singular_name = 'Category';
+
+	/**
+	 * @var string
+	 */
+	private static $plural_name = 'Categories';
+
+	/**
+	 * @var array
+	 */
+	private static $db = [
+		'Name' => 'Varchar(100)',
+	];
+
+	/**
+	 * @var array
+	 */
+	private static $belongs_many_many = [
+		'Locators' => Locator::class,
+		'LocationSet' => Location::class,
+	];
+
+	/**
+	 * @var string
+	 */
+	private static $table_name = 'LocationCategory';
+
+	/**
+	 * @var string
+	 */
+	private static $default_sort = 'Name';
+
+	/**
+	 * @return \SilverStripe\Forms\FieldList
+	 */
+	public function getCMSFields()
+	{
+		$this->beforeUpdateCMSFields(function (FieldList $fields) {
+			$fields->removeByName([
+				'Locations',
+				'LocationSet',
+				'Locators',
+				'LinkTracking',
+				'FileTracking',
+			]);
+
+			if ($this->ID) {
+				// Locations
+				$config = GridFieldConfig_RelationEditor::create();
+				$config->removeComponentsByType([
+					GridFieldAddExistingAutocompleter::class,
+					GridFieldAddNewButton::class,
+				])
+					->addComponents([
+						new GridFieldAddExistingSearchButton(),
+					]);
+				$locations = $this->Locations();
+				$locationField = GridField::create('Locations', 'Locations', $locations, $config);
+
+				$fields->addFieldsToTab('Root.Main', [
+					$locationField,
+				]);
+			}
+		});
+
+		$fields = parent::getCMSFields();
+
+		return $fields;
+	}
+
+	/**
+	 * For backwards compatability
+	 * @return ManyManyList
+	 */
+	public function Locations()
+	{
+		return $this->LocationSet();
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool
+	 */
+	public function canView($member = null, $context = [])
+	{
+		return true;
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool|int
+	 */
+	public function canEdit($member = null, $context = [])
+	{
+		return Permission::check('Location_EDIT', 'any', $member);
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool|int
+	 */
+	public function canDelete($member = null, $context = [])
+	{
+		return Permission::check('Location_DELETE', 'any', $member);
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool|int
+	 */
+	public function canCreate($member = null, $context = [])
+	{
+		return Permission::check('Location_CREATE', 'any', $member);
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
      */
     public function getCMSFields()
     {
-        $this->beforeUpdateCMSFields(function (FieldList $fields) {
+        $this->beforeUpdateCMSFields(function(FieldList $fields) {
             $fields->removeByName([
                 'Locations',
                 'LocationSet',

src/Model/Location.php

Indentation +252 added lines, -252 removed lines

@@ -28,256 +28,256 @@
  */
 class Location extends DataObject implements PermissionProvider
 {
-    /**
-     * @var string
-     */
-    private static $singular_name = 'Location';
-
-    /**
-     * @var string
-     */
-    private static $plural_name = 'Locations';
-
-    /**
-     * @var array
-     */
-    private static $db = [
-        'Title' => 'Varchar(255)',
-        'Featured' => 'Boolean',
-        'Website' => 'Varchar(255)',
-        'Phone' => 'Varchar(40)',
-        'Email' => 'Varchar(255)',
-        'Fax' => 'Varchar(45)',
-        'Import_ID' => 'Int',
-    ];
-
-    private static $many_many = [
-        'Categories' => LocationCategory::class,
-    ];
-
-    /**
-     * @var string
-     */
-    private static $table_name = 'Location';
-
-    /**
-     * @var array
-     */
-    private static $casting = [
-        'distance' => 'Decimal(9,3)',
-    ];
-
-    /**
-     * @var string
-     */
-    private static $default_sort = 'Title';
-
-    /**
-     * api access via Restful Server module
-     *
-     * @var bool
-     */
-    private static $api_access = true;
-
-    /**
-     * search fields for Model Admin
-     *
-     * @var array
-     */
-    private static $searchable_fields = [
-        'Title',
-        'Address',
-        'City',
-        'State',
-        'PostalCode',
-        'Country',
-        'Website',
-        'Phone',
-        'Email',
-        'Featured',
-    ];
-
-    /**
-     * columns for grid field
-     *
-     * @var array
-     */
-    private static $summary_fields = [
-        'Title',
-        'Address',
-        'Address2',
-        'City',
-        'State',
-        'PostalCode',
-        'CountryCode',
-        'Phone' => 'Phone',
-        'Fax' => 'Fax',
-        'Email' => 'Email',
-        'Website' => 'Website',
-        'Featured',
-        'CategoryList',
-        'Lat',
-        'Lng',
-        'Import_ID',
-    ];
-
-    /**
-     * Coords status for $summary_fields
-     *
-     * @return string
-     */
-    public function getCoords()
-    {
-        return ($this->Lat != 0 && $this->Lng != 0) ? 'true' : 'false';
-    }
-
-    /**
-     * @return string
-     */
-    public function getCategoryList()
-    {
-        if ($this->Categories()->count()) {
-            return implode(', ', $this->Categories()->column('Name'));
-        }
-
-        return '';
-    }
-
-    /**
-     * @return bool|string
-     */
-    public function getCountryCode()
-    {
-        if ($this->Country) {
-            return strtoupper($this->Country);
-        }
-
-        return false;
-    }
-
-    /**
-     * custom labels for fields
-     *
-     * @param bool $includerelations
-     * @return array|string
-     */
-    public function fieldLabels($includerelations = true)
-    {
-        $labels = parent::fieldLabels($includerelations);
-        $labels['Title'] = 'Name';
-        $labels['Address2'] = 'Address 2';
-        $labels['PostalCode'] = 'Postal Code';
-        $labels['Categories.Name'] = 'Categories';
-        $labels['Featured.NiceAsBoolean'] = 'Featured';
-
-        return $labels;
-    }
-
-    /**
-     * @return FieldList
-     */
-    public function getCMSFields()
-    {
-        $this->beforeUpdateCMSFields(function ($fields) {
-            $fields->removeByName([
-                'Import_ID',
-                'LinkTracking',
-                'FileTracking',
-            ]);
-
-            $fields->dataFieldByName('Website')
-                ->setAttribute('placeholder', 'http://');
-
-            $fields->replaceField('Email', EmailField::create('Email'));
-
-            $featured = $fields->dataFieldByName('Featured')
-                ->setDescription('Location will display near the top of the results list');
-            $fields->insertAfter(
-                'Fax',
-                $featured
-            );
-
-            if ($this->ID) {
-                $categories = $fields->dataFieldByName('Categories');
-                $config = $categories->getConfig();
-                $config->removeComponentsByType([
-                    GridFieldAddExistingAutocompleter::class,
-                ])
-                    ->addComponents([
-                        new GridFieldAddExistingSearchButton(),
-                    ]);
-            }
-        });
-
-        $fields = parent::getCMSFields();
-
-        // allow to be extended via DataExtension
-        $this->extend('updateLocationFields', $fields);
-
-        return $fields;
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool
-     */
-    public function canView($member = null, $context = [])
-    {
-        return true;
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool|int
-     */
-    public function canEdit($member = null, $context = [])
-    {
-        return Permission::check('Location_EDIT', 'any', $member);
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool|int
-     */
-    public function canDelete($member = null, $context = [])
-    {
-        return Permission::check('Location_DELETE', 'any', $member);
-    }
-
-    /**
-     * @param null $member
-     * @param array $context
-     * @return bool|int
-     */
-    public function canCreate($member = null, $context = [])
-    {
-        return Permission::check('Location_CREATE', 'any', $member);
-    }
-
-    /**
-     * @return array
-     */
-    public function providePermissions()
-    {
-        return [
-            'Location_EDIT' => 'Edit a Location',
-            'Location_DELETE' => 'Delete a Location',
-            'Location_CREATE' => 'Create a Location',
-        ];
-    }
-
-    /**
-     * @return string
-     */
-    public function getWebsiteURL()
-    {
-        $url = $this->Website;
-
-        $this->extend('updateWebsiteURL', $url);
-
-        return $url;
-    }
+	/**
+	 * @var string
+	 */
+	private static $singular_name = 'Location';
+
+	/**
+	 * @var string
+	 */
+	private static $plural_name = 'Locations';
+
+	/**
+	 * @var array
+	 */
+	private static $db = [
+		'Title' => 'Varchar(255)',
+		'Featured' => 'Boolean',
+		'Website' => 'Varchar(255)',
+		'Phone' => 'Varchar(40)',
+		'Email' => 'Varchar(255)',
+		'Fax' => 'Varchar(45)',
+		'Import_ID' => 'Int',
+	];
+
+	private static $many_many = [
+		'Categories' => LocationCategory::class,
+	];
+
+	/**
+	 * @var string
+	 */
+	private static $table_name = 'Location';
+
+	/**
+	 * @var array
+	 */
+	private static $casting = [
+		'distance' => 'Decimal(9,3)',
+	];
+
+	/**
+	 * @var string
+	 */
+	private static $default_sort = 'Title';
+
+	/**
+	 * api access via Restful Server module
+	 *
+	 * @var bool
+	 */
+	private static $api_access = true;
+
+	/**
+	 * search fields for Model Admin
+	 *
+	 * @var array
+	 */
+	private static $searchable_fields = [
+		'Title',
+		'Address',
+		'City',
+		'State',
+		'PostalCode',
+		'Country',
+		'Website',
+		'Phone',
+		'Email',
+		'Featured',
+	];
+
+	/**
+	 * columns for grid field
+	 *
+	 * @var array
+	 */
+	private static $summary_fields = [
+		'Title',
+		'Address',
+		'Address2',
+		'City',
+		'State',
+		'PostalCode',
+		'CountryCode',
+		'Phone' => 'Phone',
+		'Fax' => 'Fax',
+		'Email' => 'Email',
+		'Website' => 'Website',
+		'Featured',
+		'CategoryList',
+		'Lat',
+		'Lng',
+		'Import_ID',
+	];
+
+	/**
+	 * Coords status for $summary_fields
+	 *
+	 * @return string
+	 */
+	public function getCoords()
+	{
+		return ($this->Lat != 0 && $this->Lng != 0) ? 'true' : 'false';
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getCategoryList()
+	{
+		if ($this->Categories()->count()) {
+			return implode(', ', $this->Categories()->column('Name'));
+		}
+
+		return '';
+	}
+
+	/**
+	 * @return bool|string
+	 */
+	public function getCountryCode()
+	{
+		if ($this->Country) {
+			return strtoupper($this->Country);
+		}
+
+		return false;
+	}
+
+	/**
+	 * custom labels for fields
+	 *
+	 * @param bool $includerelations
+	 * @return array|string
+	 */
+	public function fieldLabels($includerelations = true)
+	{
+		$labels = parent::fieldLabels($includerelations);
+		$labels['Title'] = 'Name';
+		$labels['Address2'] = 'Address 2';
+		$labels['PostalCode'] = 'Postal Code';
+		$labels['Categories.Name'] = 'Categories';
+		$labels['Featured.NiceAsBoolean'] = 'Featured';
+
+		return $labels;
+	}
+
+	/**
+	 * @return FieldList
+	 */
+	public function getCMSFields()
+	{
+		$this->beforeUpdateCMSFields(function ($fields) {
+			$fields->removeByName([
+				'Import_ID',
+				'LinkTracking',
+				'FileTracking',
+			]);
+
+			$fields->dataFieldByName('Website')
+				->setAttribute('placeholder', 'http://');
+
+			$fields->replaceField('Email', EmailField::create('Email'));
+
+			$featured = $fields->dataFieldByName('Featured')
+				->setDescription('Location will display near the top of the results list');
+			$fields->insertAfter(
+				'Fax',
+				$featured
+			);
+
+			if ($this->ID) {
+				$categories = $fields->dataFieldByName('Categories');
+				$config = $categories->getConfig();
+				$config->removeComponentsByType([
+					GridFieldAddExistingAutocompleter::class,
+				])
+					->addComponents([
+						new GridFieldAddExistingSearchButton(),
+					]);
+			}
+		});
+
+		$fields = parent::getCMSFields();
+
+		// allow to be extended via DataExtension
+		$this->extend('updateLocationFields', $fields);
+
+		return $fields;
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool
+	 */
+	public function canView($member = null, $context = [])
+	{
+		return true;
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool|int
+	 */
+	public function canEdit($member = null, $context = [])
+	{
+		return Permission::check('Location_EDIT', 'any', $member);
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool|int
+	 */
+	public function canDelete($member = null, $context = [])
+	{
+		return Permission::check('Location_DELETE', 'any', $member);
+	}
+
+	/**
+	 * @param null $member
+	 * @param array $context
+	 * @return bool|int
+	 */
+	public function canCreate($member = null, $context = [])
+	{
+		return Permission::check('Location_CREATE', 'any', $member);
+	}
+
+	/**
+	 * @return array
+	 */
+	public function providePermissions()
+	{
+		return [
+			'Location_EDIT' => 'Edit a Location',
+			'Location_DELETE' => 'Delete a Location',
+			'Location_CREATE' => 'Create a Location',
+		];
+	}
+
+	/**
+	 * @return string
+	 */
+	public function getWebsiteURL()
+	{
+		$url = $this->Website;
+
+		$this->extend('updateWebsiteURL', $url);
+
+		return $url;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -178,7 +178,7 @@
      */
     public function getCMSFields()
     {
-        $this->beforeUpdateCMSFields(function ($fields) {
+        $this->beforeUpdateCMSFields(function($fields) {
             $fields->removeByName([
                 'Import_ID',
                 'LinkTracking',