BelongsToMany pivot fields are not showing on index table

[milewski]

• Laravel Version: 8.68.1
• Nova Version: 3.30.0
• PHP Version: 8.0

Description:

The latest version of nova v3.30.0 breaks the pivot fields on the index page..

// Model 
public function exports(): BelongsToMany
{
    return $this
        ->belongsToMany(Export::class)
        ->using(ExportSegment::class)
        ->withPivot([ 'status' ]);
}

// Resource
BelongsToMany::make('Exports', 'exports', Export::class)->fields(function () {
    return [
        StatusField::make('Status', 'status'),
    ];
}),

Reverting to 3.29.0 the fields works as expected..

[crynobone]

Our nova-dusk-suite along with our Dusk test suite would have reported this if that was the case. Please provide full reproducing code in order for us to debug your use case

[milewski]

This is my setup:

<?php

declare(strict_types = 1);

namespace App\Nova\Resources\Export;

use App\Models\Export\Export as ExportModel;
use App\Nova\Resources\Export\Segments\Segment;
use App\Nova\Resources\Resource;
use Illuminate\Http\Request;
use Laravel\Nova\Fields\BelongsToMany;
use Laravel\Nova\Fields\ID;

class Export extends Resource
{
    public static string $model = ExportModel::class;

    public static $with = [ 'segments' ];

    public function fields(Request $request): array
    {
        return [
            ID::make(),
            BelongsToMany::make('Segments', 'segments', Segment::class),
        ];
    }
}

<?php

declare(strict_types = 1);

namespace App\Nova\Resources\Export\Segments;

use App\Models\Export\Segment as SegmentModel;
use App\Nova\Resources\Export\Export;
use App\Nova\Resources\Resource;
use Illuminate\Http\Request;
use Laravel\Nova\Fields\BelongsToMany;
use Laravel\Nova\Fields\ID;
use Laravel\Nova\Fields\Text;
use Laravel\Nova\Http\Requests\NovaRequest;

class Segment extends Resource
{
    public static string $model = SegmentModel::class;

    public function fields(Request $request): array
    {
        return [];
    }

    public function fieldsForIndex(NovaRequest $request): array
    {
        return [
            ID::make(),
            BelongsToMany::make('Exports', 'exports', Export::class)->fields(function () {
                return [
                    Text::make('Status', 'status'),
                ];
            }),
        ];
    }
}

Nova: 3.29.0

Accessing: https://nova.docker.localhost/resources/exports/1

Nova: 3.30.0

Accessing: https://nova.docker.localhost/resources/exports/1

[crynobone]

class Export extends Resource
{
    public static string $model = ExportModel::class;

    public static $with = [ 'segments' ];

    public function fields(Request $request): array
    {
        return [
            ID::make(),
            BelongsToMany::make('Segments', 'segments', Segment::class)->fields(function () {
                return [
                    Text::make('Status', 'status'),
                ];
            }),
        ];
    }
}

This will solve the issue. See #3533

[crynobone]

[akatche]

Hi, I have noticed the same behaviour after updating to 3.3.0. In my case I have 2 resources, Digitalizacion Resource and Estado Resource.
If I go back to v3.29.0 everything works again.

• Laravel Version: 8.68.1
• Nova Version: 3.30.0
• PHP Version: 8.0

<?php

class Digitalizacion extends Resource
{
    public function fields(Request $request): array
    {
        return [
               /// Removed other fields on top
               BelongsToMany::make('Estados','estados',Estado::class)->fields(function () {
                return [
                        Text::make('Usuario', 'user_id')->displayUsing(function ($field, $resource) {
                            return $resource->user->name;
                        })->exceptOnForms(),

                        Hidden::make('Usuario', 'user_id')->default(function ($request) {
                            return $request->user()->id;
                        })->onlyOnForms(),

                        DateTime::make('Fecha','created_at')
                            ->format('DD/MM/YYYY HH:MM')
                            ->exceptOnForms()
                            ->sortable(),
                ];
            })
       ];
    }
}

class Digitalizacion extends Model
{
    public function estados()
    {
        return $this->belongsToMany(
            Estado::class
        )
            ->using(DigitalizacionEstado::class)
            ->withTimestamps()
            ->withPivot('user_id')
            ->orderByDesc('created_at');
    }
}

<?php

class Estado extends Resource
{
    public function fields(Request $request): array
    {
        return [
               /// Removed other fields on top
                     BelongsToMany::make('Digitalizacion','digitalizacion',Digitalizacion::class)->fields(function () {
                return [
                    Text::make('Usuario', 'user_id')->displayUsing(function ($field, $resource) {
                        return $resource->user->name;
                    })->exceptOnForms(),

                    Hidden::make('Usuario', 'user_id')->default(function ($request) {
                        return $request->user()->id;
                    })->onlyOnForms(),

                    DateTime::make('Fecha','created_at')
                        ->format('DD/MM/YYYY hh:mm')
                        ->exceptOnForms()
                        ->sortable(),
                ];
            }),
       ];
    }
}

class Estado extends Model
{
    public function digitalizacion()
    {
        return $this->belongsToMany(Digitalizacion::class)
            ->using(DigitalizacionEstado::class)
            ->withTimestamps()
            ->withPivot('user_id');
    }
}

[crynobone]

@akatche update withPivot() to include all fields you want to display.

[akatche]

Thanks for your prompt reply @crynobone, just tried with those changes and still not working on 3.30. on 3.29 is working as expected.

[crynobone]

Please create new issue with full reproducing code for us to debug your use case.

[milewski]

I think it is still the same case @crynobone ... I didn't understand the link you referred me to, and moving the code from fieldsForIndex to fields did not really work, I still have the same issue...

[crynobone]

@milewski read my first response to you #3615 (comment)

and moving the code from fieldsForIndex to fields did not really work, I still have the same issue...

No idea how this is related to anything I said.

[milewski]

I think I got it now, your suggestion was to move the fields from Segment to Export .. haven't tried that.. will see if I can make it work that way...