Skip to content
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

Fixed the deletion of a file #59

Merged
merged 15 commits into from
Nov 15, 2020
Merged

Fixed the deletion of a file #59

merged 15 commits into from
Nov 15, 2020

Conversation

chinleung
Copy link
Contributor

@chinleung chinleung commented Oct 28, 2019

This PR fixes laravel/nova-issues#1936 by updating the updateFields method in Eminiarts\Tabs\TabsOnEdit to return a Eminiarts\Tabs\FieldCollection, which extends Laravel\Nova\Fields\FieldCollection instead of a regular Illuminate\Support\Collection to have access to findFieldByAttribute method.

Also overwritten the method of whereInstanceOf to search through the fields in the tabs instead of the first level element.

@ibrunotome
Copy link

Hello @bajramemini, can you merge this to solve the bug on TabsOnEdit?

@bolechen
Copy link

Hello @bajramemini, can you merge this to solve the bug on TabsOnEdit?

@RVxLab
Copy link
Collaborator

RVxLab commented Oct 1, 2020

Hey! Sorry it took this long for a response.

This has become quite a big PR, I'd need to have a look locally first.

@RVxLab RVxLab self-assigned this Oct 19, 2020
@RVxLab
Copy link
Collaborator

RVxLab commented Oct 20, 2020

This seems to have been fixed in #62 and #63, can you check with version 1.2.3 of this package?

@golonix
Copy link

golonix commented Nov 10, 2020

I'm using Nova 3.14.0 and the issue still exists. With FieldCollection class from this MR, it works as expected.

@RVxLab
Copy link
Collaborator

RVxLab commented Nov 15, 2020

@golonix Thanks for confirming this. I'll look into merging this soon.

@RVxLab RVxLab merged commit ac57e9a into eminiarts:master Nov 15, 2020
@RVxLab
Copy link
Collaborator

RVxLab commented Nov 15, 2020

Thanks for your PR :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deleting file results in 500 server error
6 participants