array_diff(): Argument #2 must be of type array, int given about filament

[danharrin]

Could you please try and reproduce this in a new repository? I see theres quite a lot of code in the existing one that is not related and I think there might be something weird in the setup that might be causing this. While you're reproducing from scratch, its quite likely that you'll find what might be wrong to provide more detailed information.

[danharrin]

For example, you're currently using a BelongsToMany relationship, but your Select doesn't have multiple()? I guess that's where the issue lies and I didn't account for that when I made the change. Not seen anyone do that before to be honest

[scottjo]

HIya Dan - that seems to be the issue - by adding multiple() to the select field it works a treat. Thats my issue - at present the users want to have a single 'category' for a product but in the short to medium term they are going to move to the products belonging to more than one category. This worked in 3.2.78 but no longer in 3.2.83. I don't really want to go through my production code base removing those many-to-many relationships

(thanks for your help and for a great great product)

[danharrin]

It should be a simple fix, I will work on it probably tomorrow and get a fix out

[danharrin]

If you want to speed up the process, look on line 999 of Select.php and wrap that line in an Arr::wrap() function call, and PR it in so I can quickly merge. If you do that I can squeeze it into todays release

[scottjo]

Sorry Dan - away from my machine for the rest of the day. Thanks again for your help.JonSent from my iPhoneOn 29 May 2024, at 15:30, Dan Harrin ***@***.***> wrote: If you want to speed up the process, look on line 999 of Select.php and wrap that line in an Arr::wrap() function call, and PR it in so I can quickly merge. If you do that I can squeeze it into todays release —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: ***@***.***>

[danharrin]

Fixed by #13010