-
Notifications
You must be signed in to change notification settings - Fork 879
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
Image field and HasTraslation trait issue #5484
Comments
@miquelangeld thanks for the report. 🙏 There is a PR for uploaders that we are testing rn, that fixes a lot of stuff and it may fix this too. I would rather evaluate this after we finish working on that PR. It's #5478 in CRUD and https://github.com/Laravel-Backpack/PRO/pull/232 . I will get back here before merging those PR's to make sure this is covered. Cheers |
Hi @pxpm I updated CRUD from 6.6.4 => 6.7.5 and now seems that all works fine. Tomorrow I will do more tests to confirm. I think the PR that you mentioned it's not merged yet isn't? |
Confirmed. It's working |
Bug report
What I did
When I enable HasTranslation trait, the image fields delete the image path when I save the form even without any change, the translatable field is not the image field. If I remove the trait, then all works fine.
Backpack, Laravel, PHP, DB version
When I run
php artisan backpack:version
the output is:PHP VERSION:
PHP 8.3.4 (cli) (built: Mar 16 2024 08:40:08) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.3.4, Copyright (c) Zend Technologies
with Zend OPcache v8.3.4, Copyright (c), by Zend Technologies
with Xdebug v3.3.1, Copyright (c) 2002-2023, by Derick Rethans
LARAVEL VERSION:
10.45.1.0
BACKPACK PACKAGE VERSIONS:
backpack/basset: 1.2.4
backpack/crud: 6.6.4
backpack/devtools: 3.0.4
backpack/editable-columns: 3.0.7
backpack/generators: v4.0.3
backpack/pagemanager: 3.3.0
backpack/permissionmanager: 7.2.0
backpack/pro: 2.1.7
backpack/theme-tabler: 1.2.3
The text was updated successfully, but these errors were encountered: