[Bug] Upload subfield contains wrong url in repeteable relationship field

[CamusMX7]

Bug report

What I did

I have had this problem pending for a few months, I had reported it with the title "Upload field contains wrong url" previously and some solutions were applied. I followed what the updated documentation says at: https://backpackforlaravel.com/docs/6.x/crud-uploaders#handling-uploads-in-relationship-fields

What I expected to happen

That the files will be loaded and all the urls will correctly point to the corresponding file for each repeatable element.

What happened

Everything seemed to work fine when I tried it with 1 element, but when I tried to add more the URL points to something temporary.

What I've already tried to fix it

nothing

Backpack, Laravel, PHP, DB version

LARAVEL VERSION:

10.24.0.0

BACKPACK PACKAGE VERSIONS:

backpack/basset: 1.2.0
backpack/crud: 6.2.1
backpack/generators: v4.0.2
backpack/permissionmanager: 7.0.1
backpack/pro: 2.0.13
backpack/theme-coreuiv2: 1.2.1
backpack/theme-coreuiv4: 1.1.1
backpack/theme-tabler: 1.1.1

[CamusMX7]

Good afternoon, @tabacitu is there anything new on this topic? or any temporary alternative solution?

Thank you

[karandatwani92]

Hey @CamusMX7,

I'm so sorry🙏, I missed replying to you. Answering this was my responsibility.

I tried a similar thing and it worked for me:

Code

CRUD::field('items')->subfields([
            [
                'name'    => 'description',
                'label'    => 'file_upload_demo',
                'type'    => 'upload',
                'withFiles' => true,
                'wrapper' => [
                    'class' => 'form-group col-md-8',
                ],
            ],
            [
                'name'       => 'quantity',
                'type'       => 'number',
                'attributes' => ['step' => 'any'],
                'wrapper'    => [
                    'class' => 'form-group col-md-2',
                ],
            ],
            [
                'name'       => 'unit_price',
                'type'       => 'number',
                'attributes' => ['step' => 'any'],
                'wrapper'    => [
                    'class' => 'form-group col-md-2',
                ],
            ],
        ])->reorder('order')->hint('<small class="float-right">Create/update/delete InvoiceItem entries over a <code>hasMany</code> relationship (1-n).</small>');
    }

Relationships

Invoice model

class Invoice extends Model
{
public function items()
    {
        return $this->hasMany(\App\Models\PetShop\InvoiceItem::class);
    }
}

InvoiceItem model

class InvoiceItem extends Model
{
    public function invoice()
    {
        return $this->belongsTo(\App\Models\PetShop\Invoice::class);
    }
}

Preview

I created two items here & accessed their URLs too. It worked!

let me know if I missing something.

[CamusMX7]

@karandatwani92 Thank you for your response.
I'm a bit desperate, because it's a problem I've been exposing since July.
It should be noted that I am working with a BelongsToMany relationship.
I was first using the "repeatable" component as I specified when I reported a bug:

#5232

When I was finally understood my DB is like this:

- castings: id, others
- locutores: id, others
- casting_locutor: casting_id, locutor_id, audioc

Pedro stayed to correct it, but I had no more news. When I asked why I saw the topic closed. I saw that there was a new article, which I mentioned here:

https://backpackforlaravel.com/docs/6.x/crud-uploaders#handling-uploads-in-relationship-fields

I changed the component from repeatable type to relationship type but I had the problem that I presented here. Now, I did an update today and I no longer have that problem BUT, now I have the same initial problem (from my July post). The folder of the repeated URL.

My field and subield definition is:

CRUD::field([
    'name'          => 'locutors',
    'type'          => "relationship",
    'pivotSelect'=> [
        // 'attribute' => "title", // attribute on model that is shown to user
        'placeholder' => 'Selecciona un locutor',
        'wrapper' => [
            'class' => 'form-group col-md-6',
        ],
        // by default we call a $model->all(). you can configure it like in any other select
        'options' => function($model) {
            return $model->where('type', 'primary');
        },
        // note that just like any other select, enabling ajax will require you to provide an url for the field
        // to fetch available options. You can use the FetchOperation or manually create the enpoint.
        'ajax' => true,
        'data_source' => backpack_url('casting/fetch/locutor'),
        'attributes' => [
            'required'  => 'required',
            'class'     => 'selector_locutor'
        ],
    ],
    'subfields'   => [
                     [   // Upload
                        'name'      => 'audioc',
                        'label'     => 'Audio mp3 menor a 4MB',
                        'type'      => 'upload',
                        'withFiles' => [
                            'disk' => 'public', // the disk where file will be stored
                            'path' => 'castings', // the path inside the disk where file will be stored
                            //'fileNamer' => function($file, $uploader) { return 'the_file_name.png'; },
                        ],
                        'wrapper' => [
                            'class' => 'form-group col-md-4',
                        ]
                        ],
                        [   // select_from_array
                            'name'        => 'email_visto',
                            'label'       => "Email visto",
                            'type'        => 'select_from_array',
                            'options'     => (new \App\Models\Casting)->cat_si_no(),
                            'allows_null' => false,
                            'wrapper' => [
                                'class' => 'form-group col-md-2',
                            ],
                            'default'     => 0,
                            // 'allows_multiple' => true, // OPTIONAL; needs you to cast this to array in your model;
                            'attributes' => array('readonly' => 'readonly', 'disabled' => 'disabled')
                        ]
    ],
]);

If I remove the path in withFiles everything works fine, but it is essential to move the files to the corresponding folder.

I think the two behaviors (the bug I initially reported and this one) are related and the difference has been when doing updates, but I have followed all the steps in the documentation, the recommendations and there is always a problem with this component. It has been quite a headache these months. Please, I really need to solve this because although it seems something simple it is fundamental in what I am developing and it is the only thing I need to deliver before the end of the month.

[CamusMX7]

Hello @pxpm ,
Is there any news on this topic?

Last weekend I upgraded to the latest version of the system in production, but they are reporting the incorrect URL problem again:

It is not uploading correctly (On upgrades it seems to jump between the two problems, one is the incorrect upload throwing a temporary link and the other is it uploads them correctly but replicates the folder containing the files).

Sorry for the insistence, but it is a problem that comes from the middle of the year and affects a lot.

cheers