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

        Ted Goddard created issue -
        Ted Goddard made changes -
        Field Original Value New Value
        Assignee Ted Goddard [ ted.goddard ]
        Ken Fyten made changes -
        Salesforce Case []
        Assignee Priority P1
        Ted Goddard made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 2.1-Beta [ 10291 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: