ICEfaces
  1. ICEfaces
  2. ICE-9000

ace:dateTimeEntry - convert to use ice.ace.lazy

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Major 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.

        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 ICE-9000: ace:dateTimeEntry - convert to use ice.ace.lazy.
        Files Changed
        Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datetimeentry/datetimeentry.js
        Hide
        yip.ng added a comment - - edited

        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

        Show
        yip.ng added a comment - - edited 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
        yip.ng made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        yip.ng added a comment - - edited

        ICE-9271, ICE-9291.

        Show
        yip.ng added a comment - - edited ICE-9271 , ICE-9291.
        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 ]

          People

          • Assignee:
            yip.ng
            Reporter:
            Nils Lundquist
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: