drionix commented 4 years ago

Bug report

What I did:

Use table field

What I expected to happen:

Show table field in update route

What happened:

Here's the stack trace [Flare App link redacted]

What I've already tried to fix it:

Nothing. Tried to debug it but can't find anything

Backpack, Laravel, PHP, DB version:

PHP VERSION:

LARAVEL VERSION:

v7.12.0@c2fff1e9879494a6f853593b3c517dc9922bbb51

BACKPACK VERSION:

4.1.6@cbd4143d3eb8302916012af205565cd3183f274f

welcome[bot] commented 4 years ago

Hello there! Thanks for opening your first issue on this repo!

Just a heads-up: Here at Backpack we use Github Issues only for tracking bugs. Talk about new features is also acceptable. This helps a lot in keeping our focus on improving Backpack. If you issue is not a bug/feature, please help us out by closing the issue yourself and posting in the appropriate medium (see below). If you're not sure where it fits, it's ok, a community member will probably reply to help you with that.

Backpack communication mediums:

Bug Reports, Feature Requests - Github Issues (here);
Quick help (How do I do X) - Gitter Chatroom;
Long questions (I have done X and Y and it won't do Z wtf) - Stackoverflow, using the backpack-for-laravel tag;

Please keep in mind Backpack offers no official / paid support. Whatever help you receive here, on Gitter, Slack or Stackoverflow is thanks to our awesome awesome community members, who give up some of their time to help their peers. If you want to join our community, just start pitching in. We take pride in being a welcoming bunch.

Thank you!

-- Justin Case The Backpack Robot

tabacitu commented 4 years ago

@drionix did you get to the bottom of this? I couldn't receive the same error myself - maybe you can share your addField() call so we test with the same thing?

Cheers!

tabacitu commented 4 years ago

Ps. Excellent thing - sharing the stack trace using Flare App. Excellent!

pxpm commented 4 years ago

Hello @drionix

I think you have an outdate version of wrapper_start.blade.php.

I can spot one of the differences here: $field['wrapper'][$attributeKey] = !is_string($value) && is_callable($value) ? $value($crud, $field, $entry ?? null) : $value ?? '';

Line 7 of your provided error.

drionix commented 4 years ago

Hello @drionix

I think you have an outdate version of wrapper_start.blade.php.

I can spot one of the differences here: $field['wrapper'][$attributeKey] = !is_string($value) && is_callable($value) ? $value($crud, $field, $entry ?? null) : $value ?? '';

Line 7 of your provided error.

@pxpm Oh wow! That's exactly the problem! Thanks a lot for noticing. Can't express my gratitude with words.

I browse around a little bit and found out that this bug was apparently fixed in Backpack version 4.1.7 with Laravel-Backpack/CRUD#2862 while my current version is 4.1.6. However, there's no documentation concerning this version incompatibility.

This means that it is more of a Laravel-Backpack/CRUD bug rather than Laravel-Backpack/Settings'.

Laravel-Backpack / Settings

Using table field cause ViewException in latest version (Probably have something to do with Backpack 4.1) #102