-
Notifications
You must be signed in to change notification settings - Fork 73
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
array_flip(): Can only flip STRING and INTEGER values! #24
Comments
We don't use an |
Hi, Yeah I knew it was a general question, I was just curious if anyone had seen something similar before and could give me a direction to look. I will see what bits of the code I can sanitise and release. But it is a standard Lens and Filter setup, with a couple of columns created not from the direct SQL query and shows fine on the VueJS Table... |
After some digging around the issue seems to be around Nova's callbacks in Computed Fields. i.e. using: Text::make('Colour', function() {
return $this->getCurrentOptionValue('Colour');
}), or even just... Text::make('Colour', function() { return ''; }), in the fields() method causes the issue. Don't see this as an issue with laraval-nova-excel persay just means anyone using callbacks cant use the module for that specific resource/Lens :-( Regards, Paul. |
Hm, perhaps we can figure something out to get the value that the callback creates, but I'm not sure on which level this goes wrong. Let's leave the ticket open, I will have a look at it soon. |
WIP #26. Feel free to give it a test. |
Great thanks, will take a look in the next few days and let you know! 👍 |
Computed fields are released as 1.1 |
@patrickbrouwers i am getting this error because i use this field on my nova resource https://github.com/dillingham/nova-button Is there any option to exclude this filed in case to avoid error? Thanks, great package btw |
Same here |
me too @patrickbrouwers |
Hi,
When trying to Export to Excel (via a Lens), I am seeing an error:
any thoughts what can cause this?
The text was updated successfully, but these errors were encountered: