ICEfaces
  1. ICEfaces
  2. ICE-8825

ace:dataTable - Summary row is not updated with the correct data

          Details

          • Assignee Priority:
            P1
          • Salesforce Case Reference:
          • Workaround Description:
            Hide
            Change:
                                    <ace:column colspan="1">
                                        <h:outputText id="stockTotal" value="#{testBean.groupTotal('item.itemType', 'item.itemStock', index)}"/>
                                    </ace:column>
            to:
                                    <ace:column colspan="1">
                                        #{testBean.groupTotal('item.itemType', 'item.itemStock', index)}
                                    </ace:column>
            Show
            Change:                         <ace:column colspan="1">                             <h:outputText id="stockTotal" value="#{testBean.groupTotal('item.itemType', 'item.itemStock', index)}"/>                         </ace:column> to:                         <ace:column colspan="1">                             #{testBean.groupTotal('item.itemType', 'item.itemStock', index)}                         </ace:column>

            Description

            An ace:dataTable is set to show a summary row for totals of a column. On the first load of the table contents the summary row calculation is displayed correctly. If the table us reloaded with different data, but had the same number of rows, the summary row calculations are in correct.

            The data coming from the server is correct as well as the update that is sent. The client side simply does not show the correct data.

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Arran Mccullough made changes -
              Field Original Value New Value
              Salesforce Case Reference 5007000000OC8bfAAD
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case that shows the issue.

              Steps:

              • Load welcomeICEfaces.jsf
              • Click on the Load List One button.
              • The Totals are correct 9, 4, and 3.
              • Click on the Load List Two button.
              • The Totals are not correct, they should be 18, 10, 12.
              Show
              Arran Mccullough added a comment - Attached test case that shows the issue. Steps: Load welcomeICEfaces.jsf Click on the Load List One button. The Totals are correct 9, 4, and 3. Click on the Load List Two button. The Totals are not correct, they should be 18, 10, 12.
              Arran Mccullough made changes -
              Attachment Case11683ExampleWAR.zip [ 15112 ]
              Attachment Case11683Example.zip [ 15113 ]
              Attachment first-list-load.jpg [ 15114 ]
              Attachment second-list-load.jpg [ 15115 ]
              Ken Fyten made changes -
              Assignee Nils Lundquist [ nils.lundquist ]
              Fix Version/s EE-3.2.0.GA [ 10332 ]
              Fix Version/s 3.3 [ 10370 ]
              Assignee Priority P1 [ 10010 ]
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #32801 Thu Dec 13 12:51:09 MST 2012 nils.lundquist ICE-8825 - Conditional row subcomponents now get unique container ID prefixes.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTable.java
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTableRowRenderer.java
              Nils Lundquist made changes -
              Status Open [ 1 ] In Progress [ 3 ]
              Nils Lundquist made changes -
              Workaround Description Change:
                                      <ace:column colspan="1">
                                          <h:outputText id="stockTotal" value="#{testBean.groupTotal('item.itemType', 'item.itemStock', index)}"/>
                                      </ace:column>
              to:
                                      <ace:column colspan="1">
                                          #{testBean.groupTotal('item.itemType', 'item.itemStock', index)}
                                      </ace:column>
              Hide
              Permalink
              Nils Lundquist added a comment -

              As I've described in the workaround this can be fixed by removing the outputText that renders the EL and instead rendering the EL directly into the markup.

              The updates coming from the server contain the correct conditional row values, so I suspect some ID inconsistency is preventing the DOM from updating. I'm investigating this further.

              Show
              Nils Lundquist added a comment - As I've described in the workaround this can be fixed by removing the outputText that renders the EL and instead rendering the EL directly into the markup. The updates coming from the server contain the correct conditional row values, so I suspect some ID inconsistency is preventing the DOM from updating. I'm investigating this further.
              Hide
              Permalink
              Nils Lundquist added a comment -

              Revision #32801
              Committed by nils.lundquist
              Moments ago
              ICE-8825 - Conditional row subcomponents now get unique container ID prefixes.

              Show
              Nils Lundquist added a comment - Revision #32801 Committed by nils.lundquist Moments ago ICE-8825 - Conditional row subcomponents now get unique container ID prefixes.
              Nils Lundquist made changes -
              Status In Progress [ 3 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Fix Version/s EE-3.2.0.BETA [ 10573 ]
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Nils Lundquist
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: