Summary: | upload filename can not contain characters other than latin1 | ||
---|---|---|---|
Product: | [Applications] konqueror | Reporter: | LiuCougar <liucougar> |
Component: | khtml | Assignee: | Konqueror Developers <konq-bugs> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Gentoo Packages | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Attachments: | Makes the file upload use the page encoding instead |
Description
LiuCougar
2003-07-27 12:42:51 UTC
Created attachment 2081 [details]
Makes the file upload use the page encoding instead
It should probably use the page's encoding, like other form elements do.
Can you please test the attached patch?
It works fine, thanks. Hold on for a bit, I don't think Thiago commited this yet :-) Subject: KDE_3_1_BRANCH: kdelibs/khtml/html CVS commit by thiago: Fixing in branch. CCMAIL:61721@bugs.kde.org M +1 -1 html_formimpl.cpp 1.300.2.11 --- kdelibs/khtml/html/html_formimpl.cpp #1.300.2.10:1.300.2.11 @@ -268,5 +268,5 @@ QByteArray HTMLFormElementImpl::formData QString onlyfilename = path.mid(path.findRev('/')+1); - hstr += ("; filename=\"" + onlyfilename + "\"").ascii(); + hstr += fixUpfromUnicode(codec, "; filename=\"" + onlyfilename + "\""); if(!static_cast<HTMLInputElementImpl*>(current)->value().isEmpty()) { Subject: kdelibs/khtml/html CVS commit by thiago: And fixing in HEAD. Thanks for reporting. CCMAIL:61721-done@bugs.kde.org M +2 -2 html_formimpl.cpp 1.328 --- kdelibs/khtml/html/html_formimpl.cpp #1.327:1.328 @@ -269,5 +269,5 @@ QByteArray HTMLFormElementImpl::formData QString onlyfilename = path.mid(path.findRev('/')+1); - hstr += ("; filename=\"" + onlyfilename + "\"").ascii(); + hstr += fixUpfromUnicode(codec, "; filename=\"" + onlyfilename + "\""); if(!static_cast<HTMLInputElementImpl*>(current)->value().isEmpty()) { KMimeType::Ptr ptr = KMimeType::findByURL(KURL(path)); |