Details
-
Type: Task
-
Status: Closed
-
Priority: Major
-
Resolution: Fixed
-
Affects Version/s: 3.2
-
Fix Version/s: EE-3.3.0.GA, 4.0.BETA, 4.0
-
Component/s: ACE-Components
-
Labels:None
-
Environment:*
-
Assignee Priority:P2
Description
Convert component from current lazy initialization style to use new ice.ace.lazy.
Tests had shown that current implementation registers ice.onElementUpdate for DateTimeEntry even when not yet initialized.
Tests had shown that current implementation registers ice.onElementUpdate for DateTimeEntry even when not yet initialized.
Activity
Nils Lundquist
created issue -
Nils Lundquist
made changes -
Field | Original Value | New Value |
---|---|---|
Assignee | Nils Lundquist [ nils.lundquist ] |
Ken Fyten
made changes -
Assignee | Nils Lundquist [ nils.lundquist ] | yip.ng [ yip.ng ] |
Fix Version/s | EE-3.3.0.GA [ 10572 ] | |
Fix Version/s | 3.4 [ 10770 ] | |
Assignee Priority | P2 [ 10011 ] |
Repository | Revision | Date | User | Message |
ICEsoft Public SVN Repository | #34899 | Wed May 08 13:37:10 MDT 2013 | yip.ng | |
Files Changed | ||||
MODIFY
/icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datetimeentry/datetimeentry.js
|
yip.ng
made changes -
Status | Open [ 1 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Ken Fyten
made changes -
Issue Type | Bug [ 1 ] | Task [ 3 ] |
Ken Fyten
made changes -
Fix Version/s | 4.0 [ 11382 ] |
Ken Fyten
made changes -
Status | Resolved [ 5 ] | Closed [ 6 ] |
Done exactly same as checkboxbutton (and linkButton and pushbutton.) But this was actually not the cause of "registers ice.onElementUpdate ... even when not yet initialized." There was already a form of lazy init. since "rev. 31899 01/11/12 10:43 AM."
The cause was actually a side effect of the lazy init. setup for popup calendar. The onElementUpdate in question was used to clean up lazy init. setup objects. It was registered right after lazy init. setup mostly likely to utilize closure access to the objects. A separate onElementUpdate was registered right after lazy init. construction.
Changed to merge the two into one and done after lazy init. construction. This also solved the issue raised by Mark about "double registration for same id."
M: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\resources\icefaces.ace\datetimeentry\datetimeentry.js#34899