ICEfaces
  1. ICEfaces
  2. ICE-8475

Implement client instance look up mechanism for the ace:dataTable

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1
    • Fix Version/s: 4.0.BETA, 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      ICEfaces
    • Assignee Priority:
      P1

      Description

      ace:dataTable require an implementation of client instance look up mechanism. In other words we should provide a way for developers to access global list of ace:dataTable widgetVars. This implementation should be similar to our ace:list component functionality where particular instance of ice.ace.List could be located in the global ice.ace.Lists collection.

        Issue Links

          Activity

          Evgheni Sadovoi created issue -
          Evgheni Sadovoi made changes -
          Field Original Value New Value
          Salesforce Case [5007000000NI5ln]
          Description ace:dataTable require an implementation of client instance look up mechanism. In other words we should provide a way for developers to access global list of ace:dataTable widgetVars somehow. This implementation should be similar to our ace:list component functionality where particular instance of ice.ace.List could be located in the global ice.ace.Lists collection. ace:dataTable require an implementation of client instance look up mechanism. In other words we should provide a way for developers to access global list of ace:dataTable widgetVars. This implementation should be similar to our ace:list component functionality where particular instance of ice.ace.List could be located in the global ice.ace.Lists collection.
          Hide
          Mark Collette added a comment -

          Once the concerns in ICE-8477 are addressed, we can find a way for enumerating the dataTable objects. Likely that will not involve a central list, but rather from providing an API for finding the dataTable javascript objects within the page.

          A central list would create memory leak problems, and would likely have problems of old defunct objects remaining in it.

          Show
          Mark Collette added a comment - Once the concerns in ICE-8477 are addressed, we can find a way for enumerating the dataTable objects. Likely that will not involve a central list, but rather from providing an API for finding the dataTable javascript objects within the page. A central list would create memory leak problems, and would likely have problems of old defunct objects remaining in it.
          Mark Collette made changes -
          Link This issue depends on ICE-8477 [ ICE-8477 ]
          Hide
          Mark Collette added a comment -

          It should now be possible to add ACE components to lists, as long as ICE-8477 / ICE-8500 cleans up the components from those lists when they're updated or removed from the view.

          Show
          Mark Collette added a comment - It should now be possible to add ACE components to lists, as long as ICE-8477 / ICE-8500 cleans up the components from those lists when they're updated or removed from the view.
          Ken Fyten made changes -
          Fix Version/s 3.4 [ 10770 ]
          Ken Fyten made changes -
          Assignee Nils Lundquist [ nils.lundquist ]
          Assignee Priority P1 [ 10010 ]
          Hide
          Nils Lundquist added a comment -

          Revision #37281
          Committed by nils.lundquist
          A minute ago
          ICE-8475 - added ice.ace.instance, ice.ace.destroy and ice.ace.deadStates and demonstrating their use on ace:DataTable. Also modified ice.ace.create to add widget states to the DOM elements of all components, enabling the use of ice.ace.instance for all components using ice.ace.create. This new state saving is confirmed with the ace:dateTimeEntry & ace:dataTable memoryLeak tests not to introduce any new leaks.

          Show
          Nils Lundquist added a comment - Revision #37281 Committed by nils.lundquist A minute ago ICE-8475 - added ice.ace.instance, ice.ace.destroy and ice.ace.deadStates and demonstrating their use on ace:DataTable. Also modified ice.ace.create to add widget states to the DOM elements of all components, enabling the use of ice.ace.instance for all components using ice.ace.create. This new state saving is confirmed with the ace:dateTimeEntry & ace:dataTable memoryLeak tests not to introduce any new leaks.
          Nils Lundquist made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #37281 Tue Jul 30 17:33:24 MDT 2013 nils.lundquist ICE-8475 - added ice.ace.instance, ice.ace.destroy and ice.ace.deadStates and demonstrating their use on ace:DataTable. Also modified ice.ace.create to add widget states to the DOM elements of all components, enabling the use of ice.ace.instance for all components using ice.ace.create. This new state saving is confirmed with the ace:dateTimeEntry & ace:dataTable memoryLeak tests not to introduce any new leaks.
          Files Changed
          Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/core/core.js
          Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/datatable/datatable.js
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #37282 Wed Jul 31 07:52:23 MDT 2013 nils.lundquist ICE-8475 - remove deadState when component is rebuilt.
          Files Changed
          Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/resources/icefaces.ace/core/core.js
          Mark Collette made changes -
          Link This issue blocks ICE-9500 [ ICE-9500 ]
          Ken Fyten made changes -
          Fix Version/s 4.0 [ 11382 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Nils Lundquist
              Reporter:
              Evgheni Sadovoi
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: