dcasia / nova-filepond

A Nova field for uploading File, Image and Video using Filepond.
MIT License
49 stars 28 forks source link

Uploading file 404 process #18

Closed dacastro4 closed 4 years ago

dacastro4 commented 4 years ago

Hey. I recently installed this package but every time I try to upload any type of file I get a 404. It says that /nova-vendor/nova-filepond/process doesn't exist. I check the code for it and I see it, I also see it in the route:list. You have any idea why this could be happening? It doesn't happen with anything else but this call and the /load works fine.

milewski commented 4 years ago

Perhaps its because it failed to create a temp file on your system? https://github.com/dcasia/nova-filepond/issues/16

dacastro4 commented 4 years ago

Which temp file? I don't think I'm aware of it

On Wed, Feb 26, 2020, 9:54 PM Rafael Milewski notifications@github.com wrote:

Perhaps its because it failed to create a temp file on your system?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/dcasia/nova-filepond/issues/18?email_source=notifications&email_token=AB3SPIVFCFVAYB7DR37YWUTRE4TOXA5CNFSM4K4RQJCKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENCW5AY#issuecomment-591752835, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB3SPIRKWGIZCA74R2RDLTDRE4TOXANCNFSM4K4RQJCA .

milewski commented 4 years ago

Try changing:

https://github.com/dcasia/nova-filepond/blob/master/src/Http/Controllers/FilepondController.php#L61

/tmp to sys_get_temp_dir() if this works for you I can submit a PR with this change

dacastro4 commented 4 years ago

I just check what you mention but I don't think it has anything to do with that because it's not even reaching the controller. It's returning a 404 meaning, it's not finding the route for some reason

milewski commented 4 years ago

It's possible that something inside that method is throwing a not found error and that's why you see a 404, I suggest you add a dd() on the very first line of the process method and see if this is the case.

dacastro4 commented 4 years ago

I found the issue. I fixed it on my forked repo but I'm going to create a PR so you can see the changes and take something out of it if you want.