yii2-attachments icon indicating copy to clipboard operation
yii2-attachments copied to clipboard

This is a new pull request check it

Open hexl opened this issue 9 years ago • 13 comments

if the file name is chinese, for example: 中文文件名测试.jpg add '' to the file name when saving it to the move it to the temp folder. remove the first '' when insert the file name into database.

hexl avatar May 10 '15 03:05 hexl

Bro, I have just tested that and I have not caught that error on my Mac. What OS do you use?

CTOlet avatar May 23 '15 04:05 CTOlet

Windows Seven SP1 simple Chinese

hexl avatar May 23 '15 05:05 hexl

if the file name is chinese file name, the pathinfo can not get the right file name.

hexl avatar May 23 '15 05:05 hexl

I have used chinese file name but have not seen any error

CTOlet avatar May 23 '15 15:05 CTOlet

Just now I have tested uploading and downloading on mac os and windows and have not seen that problem. When do you have that error? I can not understand you. My file name is 中文文件名测试.xlsx

CTOlet avatar Jun 19 '15 15:06 CTOlet

How about this? http://php.net/manual/vote-note.php?id=79980&page=function.pathinfo&vote=up

hexl avatar Jun 19 '15 16:06 hexl

7 years ago?? Are you serious? Which version of the PHP do you use?

CTOlet avatar Jun 19 '15 17:06 CTOlet

i use 5.4

hexl avatar Jun 20 '15 00:06 hexl

the recent comments about file name http://php.net/manual/vote-note.php?id=117298&page=function.pathinfo&vote=up

hexl avatar Jun 20 '15 01:06 hexl

Are you sure about 5.4? On http://php.net/manual/vote-note.php?id=117298&page=function.pathinfo&vote=up the bug is risen if you use 5.3

CTOlet avatar Jun 20 '15 07:06 CTOlet

I'm sure. But i wonder if it's my php.ini or http.conf config error. I found the bug in this framework. https://github.com/liu21st/thinkphp/blob/master/ThinkPHP/Library/Think/Upload.class.php#L370

hexl avatar Jun 20 '15 08:06 hexl

So, do you solved the problem?

CTOlet avatar Jun 22 '15 06:06 CTOlet

This PR has solved the problem.

hexl avatar Jun 22 '15 06:06 hexl