dynamic / foxystripe

src/Admin/OrderAdmin.php

<?php

namespace Dynamic\FoxyStripe\Admin;

use Dynamic\FoxyStripe\Model\Order;
use SilverStripe\Admin\ModelAdmin;
use SilverStripe\Forms\GridField\GridField;
use SilverStripe\Forms\GridField\GridFieldConfig;
use SilverStripe\Forms\GridField\GridFieldEditButton;

class OrderAdmin extends ModelAdmin
{
    /**
     * @var array
     */
    private static $managed_models = array(

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);

Are you sure the assignment to $gridField is correct as $form->Fields()->fieldByName($gridFieldName) targeting SilverStripe\Forms\FieldList::fieldByName() seems to always return null.

        // GridField configuration
        /** @var GridFieldConfig $config */
        $config = $gridField->getConfig();

        // remove edit icon
        $config->removeComponentsByType(GridFieldEditButton::class);

        return $form;
    }
}