Добавить ID для иерархии BelongsToMany

[YouMixx]

У элемента BelongsToMany есть возможность построения древа, добавив функцию ->tree и указав первым параметром поле, содержащее ID родительского элемента. Но нет возможность заменить дефолтное поле ID, на какое-то свое.

Лично у меня с этим возникла проблема. Я парсил категории с стороннего сервиса по API. С него мне возвращались поле id и parent_id.

Но я не хотел сохранять в БД дефолтное id (primary_key) id-шником стороннего сервиса, поэтому для него я сделал поле uuID.
В итоге у меня вышло это:

Т.е. мое предложение заключается в том, чтобы была возможность указать поле, содержащее ID родительского элемента.

---

Я это у себя уже реализовал. Но код требует анализа.

<?php

declare(strict_types=1);

namespace App\MoonShine\Fields;

use Closure;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Collection;
use MoonShine\Contracts\Fields\HasFields;
use MoonShine\Contracts\Fields\HasPivot;
use MoonShine\Contracts\Fields\Relationships\HasAsyncSearch;
use MoonShine\Contracts\Fields\Relationships\HasRelatedValues;
use MoonShine\Contracts\Fields\Relationships\HasRelationship;
use MoonShine\Fields\Select;
use MoonShine\Traits\Fields\CheckboxTrait;
use MoonShine\Traits\Fields\SelectTransform;
use MoonShine\Traits\Fields\WithAsyncSearch;
use MoonShine\Traits\Fields\WithPivot;
use MoonShine\Traits\Fields\WithRelatedValues;
use MoonShine\Traits\WithFields;
use Throwable;

class BelongsToMany extends Select implements
    HasRelationship,
    HasRelatedValues,
    HasPivot,
    HasFields,
    HasAsyncSearch
{
    use WithFields;
    use WithPivot;
    use WithRelatedValues;
    use CheckboxTrait;
    use SelectTransform;
    use WithAsyncSearch;

    protected static string $view = 'moonshine::fields.belongs-to-many';

    protected bool $group = true;

    protected bool $tree = false;

    protected string $treeHtml = '';

    protected string $treeParentColumn = '';

    protected string $treeIDColumn = '';

    public function tree(string $treeParentColumn, string $treeIDColumn = 'id'): static
    {
        $this->treeParentColumn = $treeParentColumn;
        $this->treeIDColumn = $treeIDColumn;
        $this->tree = true;

        return $this;
    }

    public function isTree(): bool
    {
        return $this->tree;
    }

    public function buildTreeHtml(Model $item): string
    {
        $related = $this->getRelated($item);
        $query = $related->newModelQuery();

        if (is_callable($this->valuesQuery)) {
            $query = call_user_func($this->valuesQuery, $query);
        }

        $data = $query->get();

        $this->treePerformHtml($data);

        return $this->treeHtml();
    }

    private function treePerformHtml(Collection $data): void
    {
        $this->makeTree($this->treePerformData($data));

        $this->treeHtml = (string) str($this->treeHtml())->wrap(
            "<ul class='tree-list'>",
            "</ul>"
        );
    }

    private function makeTree(
        array $performedData,
        int|string $parent_id = 0,
        int $offset = 0
    ): void {
        if (isset($performedData[$parent_id])) {
            foreach ($performedData[$parent_id] as $item) {
                $element = view(
                    'moonshine::components.form.input-composition',
                    [
                        'attributes' => $this->attributes()->merge([
                            'type' => 'checkbox',
                            'id' => $this->id((string) $item->getKey()),
                            'name' => $this->name(),
                            'value' => $item->getKey(),
                            'class' => 'form-group-inline',
                        ]),
                        'beforeLabel' => true,
                        'label' => $item->{$this->resourceTitleField()},
                    ]
                );

                $this->treeHtml .= str($element)->wrap(
                    "<li style='margin-left: " . ($offset * 30) . "px'>",
                    "</li>"
                );

                $this->makeTree($performedData, $item->{$this->treeIDColumn}, $offset + 1);
            }
        }
    }

    private function treePerformData(Collection $data): array
    {
        $performData = [];

        foreach ($data as $item) {
            $parent = is_null($item->{$this->treeParentColumn()})
                ? 0
                : $item->{$this->treeParentColumn()};

            $performData[$parent][$item->getKey()] = $item;
        }

        return $performData;
    }

    public function treeParentColumn(): string
    {
        return $this->treeParentColumn;
    }

    public function treeHtml(): string
    {
        return $this->treeHtml;
    }

    /**
     * @throws Throwable
     */
    public function save(Model $item): Model
    {
        $values = $this->requestValue() ?: [];
        $sync = [];

        if ($this->hasFields()) {
            foreach ($values as $index => $value) {
                foreach ($this->getFields() as $field) {
                    $sync[$value][$field->field()] = $field->requestValue(
                        $index
                    );
                }
            }
        } else {
            $sync = $values;
        }

        $item->{$this->relation()}()->sync($sync);

        return $item;
    }

    public function exportViewValue(Model $item): string
    {
        return collect($item->{$this->relation()})
            ->map(fn ($item) => $item->{$this->resourceTitleField()})
            ->implode(';');
    }

    /**
     * @deprecated Will be deleted, use asyncSearch
     */
    public function onlySelected(
        string $relation,
        string $searchColumn = null,
        ?Closure $searchQuery = null,
        ?Closure $searchValueCallback = null
    ): static {
        return $this->asyncSearch(
            $searchColumn,
            asyncSearchQuery: $searchQuery,
            asyncSearchValueCallback: $searchValueCallback
        );
    }
}

Я добавил переменную $treeIDColumn, а в функции tree() принимаю вторым необязательным аргументом это самое поле.
Далее, все-что я делаю, это на 117 строке использую это поле, вместо ->getKey.

[lee-to]

Вам нужно делать pull requests а не создавать обсуждения

[YouMixx]

Я не уверен, что мой код хороший, поэтому создаю обсуждение.
А код - просто показываю как для примера.

[lee-to]

так берется getKey
установите у модели primary ваш uuid и все будет работать без фиксов

[YouMixx]

Это может сказаться на других моментах.
Это мелкое предложение, которое явно не будет лишним.
Если кому нужно - воспользуется, если нет - нет.