dynamic / silverstripe-foxy-orders

src/Admin/OrderAdmin.php

<?php

namespace Dynamic\Foxy\Orders\Admin;

use Dynamic\Foxy\Orders\Model\Order;
use SilverStripe\Admin\ModelAdmin;
use SilverStripe\Forms\GridField\GridField;
use SilverStripe\Forms\GridField\GridFieldConfig;
use SilverStripe\Forms\GridField\GridFieldConfig_RecordViewer;
use SilverStripe\Forms\GridField\GridFieldEditButton;

/**
 * Class OrderAdmin
 * @package Dynamic\Foxy\Orders\Admin
 */
class OrderAdmin extends ModelAdmin
{
    /**
     * @var array
     */
    private static $managed_models = [

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

        Order::class,
    ];

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

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

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

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

    /**
     * @var int
     */
    private static $menu_priority = 4;

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

    /**
     * @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 \SilverStripe\Forms\Form
     */
    public function getEditForm($id = null, $fields = null)
    {
        $form = parent::getEditForm($id, $fields);

        $gridFieldName = $this->sanitiseClassName($this->modelClass);
        /** @var GridField $gridField */
        $gridField = $form->Fields()->fieldByName($gridFieldName);

        /** @var $config GridFieldConfig_RecordViewer */
        $gridField->setConfig($config = GridFieldConfig_RecordViewer::create());

        return $form;
    }
}