ICEfaces
  1. ICEfaces
  2. ICE-6879

Alternate delta submit algorithm

          Details

          • Type: Improvement Improvement
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 2.0.2
          • Fix Version/s: 2.1-Beta, 3.0
          • Component/s: Bridge, Framework
          • Labels:
            None
          • Environment:
            client + server
          • Assignee Priority:
            P2

            Description

            An alternate algorithm can be imagined for the delta submit where the parameter diffing is always calculated against the parameters calculated off the current document state. This avoids chained parameter diffing that can sometimes can be frail. Also the current algorithm when it breaks, the diffing will remain broken with no chance of recovering (except a page reload).
            This alternate algorithm is also simpler to implement.

              Activity

              Ascending order - Click to sort in descending order
              Mircea Toma created issue -
              Mircea Toma made changes -
              Field Original Value New Value
              Assignee Mircea Toma [ mircea.toma ]
              Ken Fyten made changes -
              Salesforce Case []
              Assignee Priority P3
              Ken Fyten made changes -
              Salesforce Case []
              Fix Version/s 2.1 [ 10241 ]
              Ken Fyten made changes -
              Salesforce Case []
              Assignee Priority P3 P1
              Ken Fyten made changes -
              Salesforce Case []
              Assignee Priority P1 P2
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #25023 Wed Jul 13 14:17:34 MDT 2011 mircea.toma ICE-6879 Simplified delta submit algorithm to use the parameters calculated off the DOM form as the comparing base for the delta calculations.
              Files Changed
              Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/javascript/application.js
              Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/DeltaSubmitPhaseListener.java
              Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/javascript/submit.js
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #25026 Wed Jul 13 15:11:43 MDT 2011 mircea.toma ICE-6879 Removed DeltaSubmitParametersPropagation navigation handler since the new delta submit algorithm builds the previous parameters from the "old" DOM which is already propagated by the PropagatingNavigationHandler.
              Files Changed
              Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/resources/META-INF/faces-config.xml
              Commit graph DEL /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/application/DeltaSubmitParametersPropagation.java
              Hide
              Permalink
              Mircea Toma added a comment - - edited

              Changed the delta submit algorithm to use the parameters calculated off the DOM form as the comparing base for the delta calculations.

              Show
              Mircea Toma added a comment - - edited Changed the delta submit algorithm to use the parameters calculated off the DOM form as the comparing base for the delta calculations.
              Hide
              Permalink
              Mircea Toma added a comment -

              Removed DeltaSubmitParametersPropagation navigation handler since the new delta submit algorithm builds the previous parameters from the "old" DOM which is already propagated by the PropagatingNavigationHandler.

              Show
              Mircea Toma added a comment - Removed DeltaSubmitParametersPropagation navigation handler since the new delta submit algorithm builds the previous parameters from the "old" DOM which is already propagated by the PropagatingNavigationHandler.
              Mircea Toma made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s 2.1-Beta [ 10291 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Mircea Toma
                  Reporter:
                  Mircea Toma
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  0 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: