ICEfaces
  1. ICEfaces
  2. ICE-9068

ace:dataTable - ace:ajax event="editSubmit" tag prevents saving changed inputs

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: EE-3.2.0.GA
          • Fix Version/s: 3.3
          • Component/s: ACE-Components
          • Labels:
            None
          • Environment:
            All
          • Assignee Priority:
            P1

            Description

            When defining the rowEditListener and rowEditCancelListener methods for the ace:rowEditor, there methods are not called on save of an edit and on cancel of an edit.

            I also tried using an <ace:ajax event="editSubmit"/> tag for the ace:dataTable. This didn't help and it also prevented the changed inputs from being displayed once saved except for the last row in the table.
            1. Zip Archive
              Case12041Example.zip
              20 kB
              Arran Mccullough
            2. Zip Archive
              Case12041ExampleWAR.zip
              9.34 MB
              Arran Mccullough

              Activity

              Ascending order - Click to sort in descending order
              Arran Mccullough created issue -
              Hide
              Permalink
              Arran Mccullough added a comment -

              Attached test case that shows issue.

              Steps:

              • Load welcomeICEfaces.jsf
              • Select a row to edit
              • Change some inputs and save the row.
              • Server side logs should show some output but this is not seen.
              Show
              Arran Mccullough added a comment - Attached test case that shows issue. Steps: Load welcomeICEfaces.jsf Select a row to edit Change some inputs and save the row. Server side logs should show some output but this is not seen.
              Arran Mccullough made changes -
              Field Original Value New Value
              Attachment Case12041ExampleWAR.zip [ 15513 ]
              Attachment Case12041Example.zip [ 15514 ]
              Hide
              Permalink
              Arran Mccullough added a comment -

              Support case #12041

              Show
              Arran Mccullough added a comment - Support case #12041
              Ken Fyten made changes -
              Assignee Nils Lundquist [ nils.lundquist ]
              Fix Version/s 3.3 [ 10370 ]
              Assignee Priority P1 [ 10010 ]
              Hide
              Permalink
              Nils Lundquist added a comment -

              Just attempted simple case in showcase, where the listeners worked as expected.

              Looking closer now to determine the difference between the test case and this example.

              Show
              Nils Lundquist added a comment - Just attempted simple case in showcase, where the listeners worked as expected. Looking closer now to determine the difference between the test case and this example.
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #33979 Tue Mar 19 14:08:59 MDT 2013 nils.lundquist ICE-9068 - CellEditor now changes facet visibility in updateModel phase. Prevents incomplete update due to 'rendered' being false when executing table children via 'iterate' rather with 'visitTree'.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/celleditor/CellEditor.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #33980 Tue Mar 19 14:37:39 MDT 2013 nils.lundquist ICE-9068 - CellEditor no longer contains unneeded rendered adjustments. Tiggered in RowEditor phases since addition of toggleOnInvalidEdit.
              Files Changed
              Commit graph MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/celleditor/CellEditor.java
              Hide
              Permalink
              Nils Lundquist added a comment -

              Revision #33980
              Committed by nils.lundquist
              Moments ago
              ICE-9068 - CellEditor no longer contains unneeded rendered adjustments. Tiggered in RowEditor phases since addition of toggleOnInvalidEdit.

              I've resolved the issue with the ace:ajax 'submitEvent'. The other issue doesn't appear to be reproducible.

              Show
              Nils Lundquist added a comment - Revision #33980 Committed by nils.lundquist Moments ago ICE-9068 - CellEditor no longer contains unneeded rendered adjustments. Tiggered in RowEditor phases since addition of toggleOnInvalidEdit. I've resolved the issue with the ace:ajax 'submitEvent'. The other issue doesn't appear to be reproducible.
              Hide
              Permalink
              Nils Lundquist added a comment -

              EditSubmit behaviours have been fixed, remaining issue when combined with ToggleOnInvalidEdit is handled in : ICE-9087.

              Show
              Nils Lundquist added a comment - EditSubmit behaviours have been fixed, remaining issue when combined with ToggleOnInvalidEdit is handled in : ICE-9087 .
              Nils Lundquist made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Ken Fyten made changes -
              Summary ace:rowEditor - rowEditListener and rowEditCancelListener not called ace:dataTable - ace:ajax event="editSubmit" tag prevents saving changed inputs
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: