ICEfaces
  1. ICEfaces
  2. ICE-6793

org.icefaces.autoid incompatible with dynamically added components

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.0.1
    • Fix Version/s: 2.1-Beta, 3.0
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      ICEfaces
    • Assignee Priority:
      P1

      Description


      When components are dynamically added, the id modification performed by autoid can get out of sync:

      In Mojarra StateManagementStrategyImpl:


              // handle dynamic adds/removes
              List<String> removeList = stateContext.getDynamicRemoves();
              if (null != removeList && !removeList.isEmpty()) {
                  stateMap.put(CLIENTIDS_TO_REMOVE_NAME, removeList);
              }
              Map<String, ComponentStruct> addList = stateContext.getDynamicAdds();
              if (null != addList && !addList.isEmpty()) {
                  List<Object> savedAddList = new ArrayList<Object>(addList.size());
                  for (ComponentStruct s : addList.values()) {
                      savedAddList.add(s.saveState(context));
                  }
                  stateMap.put(CLIENTIDS_TO_ADD_NAME, savedAddList.toArray());
              }

      "dynamic adds" are populated with the original IDs set prior to autoid modification. When autoid modifies these IDs, they do not match the value expected by state saving.

        Activity

        Repository Revision Date User Message
        ICEsoft Public SVN Repository #24784 Wed Jun 08 16:10:44 MDT 2011 ted.goddard autoid applied only before rendering (ICE-6793)
        Files Changed
        Commit graph MODIFY /icefaces2/trunk/icefaces/core/src/main/java/org/icefaces/impl/event/MainEventListener.java

          People

          • Assignee:
            Ted Goddard
            Reporter:
            Ted Goddard
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: