ICEfaces
  1. ICEfaces
  2. ICE-9585

ace:dataTable Row/Cell Editing with filtering not working

    Details

      Description

      I have problem when I use row editing in combination with filtering.

      How to reproduce it:
      I downloaded code from:
      [url]http://icefaces-showcase.icesoft.org/showcase.jsf?grp=aceMenu&exp=dataTableBean[/url]
      and did small modification as you can see on attached picture.

      After that when I make filtering, i get wrong data to change as you can see on picture also.

        Activity

        Zoran Djuric created issue -
        Zoran Djuric made changes -
        Field Original Value New Value
        Attachment dataTable bug.jpg [ 16424 ]
        Hide
        Zoran Djuric added a comment -

        One more comment.

        To make workaround for this issue, we use datatable without paging. But because we have a lot of element in list, when we use scrollbar and position at the middle of the screen, then click on EDIT button, scroll bar position is reset to the top

        Scrollbar position should remain unchanged, so instead <a href="#" ... or <a .. without href
        should be used:

        <a href ="" onclick="something();return false"

        Show
        Zoran Djuric added a comment - One more comment. To make workaround for this issue, we use datatable without paging. But because we have a lot of element in list, when we use scrollbar and position at the middle of the screen, then click on EDIT button, scroll bar position is reset to the top Scrollbar position should remain unchanged, so instead <a href="#" ... or <a .. without href should be used: <a href ="" onclick="something();return false"
        Hide
        Nils Lundquist added a comment -

        This issue is due to the filtered data set not being used to setup context when using visitTree to render ace:dataTable cell subregions.

        Show
        Nils Lundquist added a comment - This issue is due to the filtered data set not being used to setup context when using visitTree to render ace:dataTable cell subregions.
        Judy Guglielmin made changes -
        Salesforce Case Reference 5007000000YZKHTAA5
        Ken Fyten made changes -
        Assignee Arturo Zambrano [ artzambrano ]
        Fix Version/s 4.0 [ 10770 ]
        Fix Version/s EE-3.3.0.GA_P02 [ 11371 ]
        Assignee Priority P2 [ 10011 ]
        Hide
        Arturo Zambrano added a comment - - edited

        Committed fix to 4.0 trunk at revision 39883 and to the 3.3 EE maintenance branch at revision 39885. The fix consists in regenerating the data model before executing visitTree(), so that the filtered data set is used, if available.

        Show
        Arturo Zambrano added a comment - - edited Committed fix to 4.0 trunk at revision 39883 and to the 3.3 EE maintenance branch at revision 39885. The fix consists in regenerating the data model before executing visitTree(), so that the filtered data set is used, if available.
        Arturo Zambrano made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #39883 Tue Feb 04 14:18:07 MST 2014 art.zambrano ICE-9585 modification to regenerate data model before executing visitTree(), so that the filtered data set is used if applicable
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datatable/DataTable.java
        Hide
        Ken Fyten added a comment -

        Re-open to be applied to the maintenance branch also.

        Show
        Ken Fyten added a comment - Re-open to be applied to the maintenance branch also.
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Arturo Zambrano made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 4.0 [ 11382 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Arturo Zambrano
            Reporter:
            Zoran Djuric
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: