Details
-
Type: Bug
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 2.0-Alpha1
-
Component/s: ICE-Components
-
Labels:None
-
Environment:FF & IE
-
Affects:Compatibility/Configuration
Description
After switching from ice.singleSubmit() to ice.ser in calendar component, the following problems were observed:
- JS null pointer exception, see attached screenshot. This happens in code trying to do a diff. of component properties on the client side between renderings. Just check singleSubmit checkbox and select a date or time. It should trigger single submit and the error.
- In popup with text input, the display date formatting in the input field was done with a single submit after clicking OK in the calendar. This doesn't work anymore. Nothing happens until a full submit (by clicking on the the checkboxes.)
- The renderAsPopup and renderInputField checkboxes are submitted with ice.submit(). They don't seem to work anymore in IE. Sometimes nothing happends to change the calendar display mode; sometimes the display mode just gets out of sync with the checkboxes. It still works OK in FF.
Test app. in C:\svn\ossrepo\icefaces\scratchpads\sparkle\showcase\test
- JS null pointer exception, see attached screenshot. This happens in code trying to do a diff. of component properties on the client side between renderings. Just check singleSubmit checkbox and select a date or time. It should trigger single submit and the error.
- In popup with text input, the display date formatting in the input field was done with a single submit after clicking OK in the calendar. This doesn't work anymore. Nothing happens until a full submit (by clicking on the the checkboxes.)
- The renderAsPopup and renderInputField checkboxes are submitted with ice.submit(). They don't seem to work anymore in IE. Sometimes nothing happends to change the calendar display mode; sometimes the display mode just gets out of sync with the checkboxes. It still works OK in FF.
Test app. in C:\svn\ossrepo\icefaces\scratchpads\sparkle\showcase\test
Activity
Ken Fyten
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
Assignee Priority | P1 |
Ken Fyten
made changes -
Fix Version/s | 2.0.0 [ 10230 ] |
yip.ng
made changes -
Attachment | screenshot-10.png [ 12419 ] |
yip.ng
made changes -
Attachment | screenshot-9.png [ 12418 ] |
yip.ng
made changes -
Attachment | screenshot-8.png [ 12417 ] |
Mircea Toma
made changes -
Status | Reopened [ 4 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #21752 | Thu Jun 17 06:39:05 MDT 2010 | mircea.toma | |
Files Changed | ||||
MODIFY
/icefaces/scratchpads/glimmer/core/src/main/javascript/submit.js
|
Ken Fyten
made changes -
Resolution | Fixed [ 1 ] | |
Status | Resolved [ 5 ] | Reopened [ 4 ] |
Mircea Toma
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Ken Fyten
made changes -
Salesforce Case | [] | |
Assignee Priority | P2 | P1 |
Assignee | Yip Ng [ yip.ng ] | Mircea Toma [ mircea.toma ] |
yip.ng
made changes -
Attachment | screenshot-7.png [ 12405 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #21707 | Wed Jun 09 09:47:16 MDT 2010 | yip.ng | |
Files Changed | ||||
MODIFY
/icefaces/scratchpads/sparkle/component/src/org/icefaces/component/selectinputdate/SelectInputDate.java
MODIFY /icefaces/scratchpads/sparkle/showcase/test/web/calendar.xhtml MODIFY /icefaces/scratchpads/sparkle/component/src/org/icefaces/component/selectinputdate/SelectInputDateRenderer.java |
Ken Fyten
made changes -
Salesforce Case | [] | |
Assignee Priority | P2 |
Ken Fyten
made changes -
Salesforce Case | [] | |
Fix Version/s | 2.0-Beta [ 10231 ] | |
Fix Version/s | 2.0-Alpha3 [ 10032 ] | |
Assignee Priority | P1 |
Mircea Toma
made changes -
Assignee | Mircea Toma [ mircea.toma ] | Yip Ng [ yip.ng ] |
Mircea Toma
made changes -
Attachment | ICE-5681.patch [ 12369 ] |
Ken Fyten
made changes -
Assignee | Yip Ng [ yip.ng ] | Mircea Toma [ mircea.toma ] |
yip.ng
made changes -
Attachment | screenshot-6.png [ 12334 ] |
yip.ng
made changes -
Attachment | screenshot-5.png [ 12333 ] |
yip.ng
made changes -
Attachment | screenshot-4.png [ 12330 ] |
yip.ng
made changes -
Attachment | screenshot-3.png [ 12329 ] |
yip.ng
made changes -
Attachment | screenshot-2.png [ 12328 ] |
Mircea Toma
made changes -
Assignee | Mircea Toma [ mircea.toma ] | Yip Ng [ yip.ng ] |
Ken Fyten
made changes -
Salesforce Case | [] | |
Fix Version/s | 2.0-Alpha3 [ 10032 ] | |
Affects | [Compatibility/Configuration] | |
Assignee Priority | P1 | |
Assignee | Mircea Toma [ mircea.toma ] |
yip.ng
made changes -
Field | Original Value | New Value |
---|---|---|
Attachment | screenshot-1.png [ 12325 ] |
yip.ng
created issue -
Single submit uses a IE specific strategy for cloning the element to be submitted. It uses innerHTML and outerHTML of the element to copy the element, using Element.cloneNode does not copy also the state of the input[checkbox/radio] and select elements. IE6 and IE7 would serialize the state of the chckebox/radio properly but IE8 won't.
The fix applies a unique strategy for all browsers which moves temporarily the submitted element from its original form into the single submit form, submit the from and then put the element back. Also the state of the input[checkbox/radio] is restored programatically after the move since IE6 in this case loses the state.