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

Import file with long name (or arabic name) #14823

Open
Saifallak opened this issue Nov 17, 2024 · 2 comments
Open

Import file with long name (or arabic name) #14823

Saifallak opened this issue Nov 17, 2024 · 2 comments
Milestone

Comments

@Saifallak
Copy link
Contributor

Saifallak commented Nov 17, 2024

Package

filament/filament

Package Version

v3.2.124

Laravel Version

v11.x

Livewire Version

v3.x

PHP Version

8.3.x

Problem description

getting an error when importing large file names or Arabic file name ex:

SQLSTATE[22001]: String data, right truncated: 1406 Data too long for column 'file_path' at row 1 (Connection: mysql, SQL: insert into `imports` (`user_id`, `file_name`, `file_path`, `importer`, `total_rows`, `updated_at`, `created_at`) values (209, عملاء شهر يناير 2024 البدر للمبيعات اون لاين واوف لاين .csv, /code/storage/app/livewire-tmp/97dKPqjXMEhVTL3x1swh3I6hfqyYKK-meta2LnZhdmE2KfYoSDYtNmH2LEg2YrZhtin2YrYsSAyMDI0INin2YTYqNiv2LEg2YTZhNmF2KjZiti52KfYqiDYp9mI2YYg2YTYp9mK2YYg2YjYp9mI2YEg2YTYp9mK2YYgLmNzdg==-.csv, App\Filament\Imports\ContactImporter, 53, 2024-11-17 08:09:15, 2024-11-17 08:09:15))

Expected behavior

imported succssfully.

Steps to reproduce

just import file named عملاء شهر يناير 2024 البدر للمبيعات اون لاين واوف لاين .csv

Reproduction repository (issue will be closed if this is not valid)

https://github.com/AQuadic/NO-REPO-NEEDED

Relevant log output

Sorry for not including repo, but it can be reproduced on any repo and on demo. just file name like I said.

Donate 💰 to fund this issue

  • You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
  • 100% of the funding will be distributed between the Filament core team to run all aspects of the project.
  • Thank you in advance for helping us make maintenance sustainable!
Fund with Polar
@github-project-automation github-project-automation bot moved this to Todo in Roadmap Nov 17, 2024
@zepfietje zepfietje added this to the v3 milestone Nov 20, 2024
@btxtiger
Copy link

btxtiger commented Nov 23, 2024

@Saifallak
1406 Data too long for column 'file_path'

Looks like your file_path column is too small, probably varchar(255). You can try it with text type

@Saifallak
Copy link
Contributor Author

@Saifallak 1406 Data too long for column 'file_path'

Looks like your file_path column is too small, probably varchar(255). You can try it with text type

Yes you're right, but this is the migration created by filament, that's why i opemed issue to be changed in the library itself

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Todo
Development

No branches or pull requests

3 participants