Using allFields() method breaks export of relationship and computed fields about laravel-nova-excel HOT 8 CLOSED

allFields is a bit of a wrong naming. What it actually does is getting all "attributes" from the Eloquent model and doesn't consult the Nova resource to see what kind of field it is.

I'll probably deprecate and replace this method in the next release by allAttributes to align with the actual functionality.

If you want to have an export that represents how the nova resource index page looks, you should just not using allFields at all and it should work out of the box.

from laravel-nova-excel.

allFields is a bit of a wrong naming. What it actually does is getting all "attributes" from the Eloquent model and doesn't consult the Nova resource to see what kind of field it is.

I'll probably deprecate and replace this method in the next release by allAttributes to align with the actual functionality.

If you want to have an export that represents how the nova resource index page looks, you should just not using allFields at all and it should work out of the box.

What should I do if I want to include all fields including relationships and computed values?

from laravel-nova-excel.

@pelmered with "all fields" do you mean all fields that are defined as "index" fields on the nova resource? Than you don't have to do anything, those are used by default.

from laravel-nova-excel.

@patrickbrouwers No, all fields that are defined on the resource no matter where they are displayed. In my case I would like to include all those fields except one.

I think this is the most common use case, because you probably want to avoid side scrolling to use the buttons on the row.

from laravel-nova-excel.

I think currently you would have to specify all fields with ->only() that you want in the export.
We might need to change allFields to do that instead. Feel free to PR that functionality. Happy to incorporate it as a solution in the package.

from laravel-nova-excel.

+1 on the ability to have a true allFields functionality!

from laravel-nova-excel.

Same here. If using only(), when having a relationship with user_id, I can use user that will display the whole json object, but not user.email

from laravel-nova-excel.

PR was merged

from laravel-nova-excel.