Need a warning dialog for non-matching charset in form posting

ASSIGNED
Assigned to

Status

()

Core
HTML: Form Submission
P3
normal
ASSIGNED
19 years ago
9 years ago

People

(Reporter: Frank Tang, Assigned: nhottanscp)

Tracking

({intl})

Trunk
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 years ago
If we type some text in form but the text cannot convert to the document 
charset, it will be nice we warn the user and provide the following information
1. It try to post as charset XXXX
2. The following field name YYY have text ZZZZ which cannot be represent as 
charset  XXXX and will be change to LLLL
3. Provide two choice- [Do not submit] [Submit it anyway]
(Reporter)

Comment 1

19 years ago
The concept is similar to 28908
Status: NEW → ASSIGNED
(Reporter)

Updated

19 years ago
Target Milestone: M17
(Reporter)

Comment 2

18 years ago
jbetak- you probably can help since you are now familar with the form posting 
code. Please look at how wallet and security code file prompt dialogbox at the 
form posting code. 
Assignee: ftang → jbetak
Status: ASSIGNED → NEW
Status: NEW → ASSIGNED
reassigning to ftang for resource reallocation
Assignee: jbetak → ftang
Status: ASSIGNED → NEW
(Reporter)

Comment 4

18 years ago
mark assign.
Status: NEW → ASSIGNED
(Reporter)

Comment 5

18 years ago
nhotta- can you teach me how to throw a warning dialog ?
(Assignee)

Comment 6

18 years ago
Mail code uses nsIPrompt::Confirm() for alert.
http://lxr.mozilla.org/seamonkey/source/mailnews/compose/src/nsMsgPrompts.cpp

Comment 7

18 years ago
Updating QA contact.
QA Contact: ckritzer → vladimire
(Reporter)

Comment 8

18 years ago
nhotta, can you add this ?
reassign browser related bug to nhotta.
Assignee: ftang → nhotta
Status: ASSIGNED → NEW
Target Milestone: M17 → ---

Comment 9

18 years ago
what will the data in question (ZZZZ) be converted to?

would it make sense to warn the user when they typed it in instead of when they
submitted it?
(Reporter)

Comment 10

18 years ago
mark it as Future
Target Milestone: --- → Future
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED

Updated

17 years ago
Blocks: 70838

Comment 11

17 years ago
NAOKI:
  could you take a look at this one please?

I have bug 70838 assigned to me and that one has a miletone set.

If you cannot take care about this bug, go ahead and reassign it to me.

Thx!
(Assignee)

Comment 12

17 years ago
We need to check the error code NS_ERROR_UENC_NOMAPPING of
nsIUnicodeEncoder::Conver().
http://lxr.mozilla.org/seamonkey/source/content/html/content/src/nsFormSubmitter.cpp#1312

But I have no idea how to post alert in that code.
Alexandru, do you have any idea?
Keywords: intl
(Assignee)

Comment 13

17 years ago
cc to alexsavulov@netscape.com, please see my last comment.

Comment 14

17 years ago
Naoki:
I'm looking at it. As soon as I have a wayto do that i'll let you know.

Comment 15

17 years ago
Naoki:

Unfortuantely the source code of nsFormSubmitter.cpp changed after j.keisers
checkin. However since then there is a piece of code that opens a notification
dialog. 

If that's what you need then loock in nsFormSubmitter::OnSubmit after

398 nsresult notifyStatus = formSubmitObserver->Notify(formElement,
399                                                    window,
400                                                    actionURL,
401                                                    &cancelSubmit);

Hope this helps!

(Assignee)

Comment 16

17 years ago
Okay, I will look intot this 0.9.8.
Target Milestone: Future → mozilla0.9.8
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.8 → ---
(Assignee)

Updated

17 years ago
Target Milestone: --- → mozilla1.2
(Assignee)

Updated

16 years ago
Target Milestone: mozilla1.2alpha → ---

Comment 17

16 years ago
*** Bug 189275 has been marked as a duplicate of this bug. ***

Comment 18

16 years ago
are there any news on this? You said you'll look at it more than 2 years
ago ;-)

Updated

13 years ago
Blocks: 228779
QA Contact: vladimire → form-submission
You need to log in before you can comment on or make changes to this bug.