ICEfaces
  1. ICEfaces
  2. ICE-7244

MyFaces 2: component ids with datatables include row index twice

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.2
    • Fix Version/s: 3.0.RC2, 3.0
    • Component/s: Framework, ICE-Components
    • Labels:
      None
    • Environment:
      MyFaces 2 ICEfaces 2 datatables
    • Assignee Priority:
      P1
    • Affects:
      Compatibility/Configuration

      Description

      When running under MyFaces, components within datatables are being assigned ids where the row index appears to be appended twice. So for example, in Mojarra where an element within a datatable might be identified as;

      <span id="iceform:datTbl:2:number" class="iceOutTxt">1</span>

      in MyFaces, the same element is identified as:

      <span id="iceform:datTbl:2:2:number" class="iceOutTxt">1</span>

      While it doesn't appear to strictly break any application functionality that I can identify, it does cause distress to many of our Selenium tests where we attempt to find elements within a datatable.
      1. ICE-7244.patch
        10 kB
        Deryk Sinotte
      1. datatable-ids.png
        501 kB

        Issue Links

          Activity

          Repository Revision Date User Message
          ICEsoft Public SVN Repository #28147 Sat Mar 03 01:23:08 MST 2012 mark.collette ICE-7244 cleanup debug code
          Files Changed
          Commit graph MODIFY /icefaces3/branches/icefaces-3.0.x-maintenance/icefaces/compat/components/src/main/java/com/icesoft/faces/component/ext/HtmlDataTable.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #28146 Sat Mar 03 01:21:09 MST 2012 mark.collette ICE-7244 cleanup debug code
          Files Changed
          Commit graph MODIFY /icefaces3/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/ext/HtmlDataTable.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #26929 Fri Dec 16 14:05:10 MST 2011 mark.collette ICE-7244 : MyFaces 2: component ids with datatables include row index twice
          Files Changed
          Commit graph MODIFY /icefaces3/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/ext/renderkit/TableRenderer.java
          Commit graph MODIFY /icefaces3/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/ext/RowSelector.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #26886 Wed Dec 14 18:30:07 MST 2011 mark.collette ICE-7244 : MyFaces 2: component ids with datatables include row index twice
          Files Changed
          Commit graph MODIFY /icefaces3/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/ext/renderkit/TableRenderer.java
          Commit graph MODIFY /icefaces3/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/ext/UIColumns.java
          Commit graph MODIFY /icefaces3/trunk/icefaces/core/src/main/java/org/icefaces/impl/component/UISeriesBase.java
          Commit graph MODIFY /icefaces3/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/ext/HtmlDataTable.java
          Commit graph MODIFY /icefaces3/trunk/icefaces/compat/core/src/main/java/com/icesoft/faces/renderkit/dom_html_basic/TableRenderer.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #25788 Fri Sep 30 16:00:25 MDT 2011 ken.fyten ICE-7244 / ICE-7291 - Revert change to ExtendedExceptionHandler that causes View cannot be restored error to appear when session expires.
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/application/ExtendedExceptionHandler.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #25751 Thu Sep 29 16:44:18 MDT 2011 deryk.sinotte ICE-7244: remove the MyFaces fix as the row index id issue is worse with it in there
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/panelseries/UISeries.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #25695 Wed Sep 28 11:45:37 MDT 2011 deryk.sinotte ICE-7244: small adjustment to our CharacterEncodingHandler to handle Content-Type headers with multiple charset= entries. Prevents unnecesary warning messages.
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/util/CharacterEncodingHandler.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #25689 Wed Sep 28 11:00:03 MDT 2011 deryk.sinotte ICE-7244: removed a snippet of a code to fix MyFaces which was causing porlets to fail when ViewStates were applied to all forms on a page
          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/util/CharacterEncodingHandler.java
          Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/application/ExtendedExceptionHandler.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #25644 Mon Sep 26 14:57:41 MDT 2011 deryk.sinotte ICE-7244: put back original row index logic but ignore it if running under MyFaces
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/panelseries/UISeries.java
          Repository Revision Date User Message
          ICEsoft Public SVN Repository #25609 Fri Sep 23 10:31:54 MDT 2011 deryk.sinotte ICE-7244: remove a workaround for adding row indexes to client ids that is now fixed in MyFaces
          Files Changed
          Commit graph MODIFY /icefaces2/trunk/icefaces/compat/components/src/main/java/com/icesoft/faces/component/panelseries/UISeries.java

            People

            • Assignee:
              Mark Collette
              Reporter:
              Deryk Sinotte
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: