ICEfaces
  1. ICEfaces
  2. ICE-6765

NullPointerException being thrown from DeltaSubmitParametersPropagation.handleNavigation(FacesContext, String, String) during ace:fileEntry file upload

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-2.0.0.GA, 3.0
    • Fix Version/s: EE-2.0.0.GA, 2.0.2
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      Liferay 6.0.6 + Tomcat 6.0.29 + PortletFaces Bridge 2.0.0-SNAPSHOT (April 7th, 2011)

      Description

      Here's the patch:


      Index: core/src/main/java/org/icefaces/impl/application/DeltaSubmitParametersPropagation.java
      ===================================================================
      --- core/src/main/java/org/icefaces/impl/application/DeltaSubmitParametersPropagation.java (revision 24358)
      +++ core/src/main/java/org/icefaces/impl/application/DeltaSubmitParametersPropagation.java (working copy)
      @@ -51,7 +51,7 @@
       
           public void handleNavigation(FacesContext context, String fromAction, String outcome) {
               NavigationCase navigationCase = getNavigationCase(context, fromAction, outcome);
      - if (navigationCase.isRedirect()) {
      + if ((navigationCase != null) && (navigationCase.isRedirect())) {
                   handler.handleNavigation(context, fromAction, outcome);
               } else {
                   UIViewRoot viewRoot = context.getViewRoot();
      @@ -63,7 +63,13 @@
                       viewAttributes.put(DeltaSubmitPhaseListener.PreviousParameters, idToPreviousParametersMapping);
                   }
                   for (UIForm form: forms) {
      - idToPreviousParametersMapping.put(form.getId(), new HashMap((Map) form.getAttributes().get(DeltaSubmitPhaseListener.PreviousParameters)));
      + Map<String, Object> formAttributes = form.getAttributes();
      + if (formAttributes != null) {
      + Map previousParametersMap = (Map) formAttributes.get(DeltaSubmitPhaseListener.PreviousParameters);
      + if (previousParametersMap != null) {
      + idToPreviousParametersMapping.put(form.getId(), new HashMap(previousParametersMap));
      + }
      + }
                   }
                   handler.handleNavigation(context, fromAction, outcome);
                   //propagate previously calculated submit parameters

        Activity

        Neil Griffin created issue -
        Deryk Sinotte made changes -
        Field Original Value New Value
        Salesforce Case []
        Component/s Framework [ 10013 ]
        Fix Version/s EE-2.0.0.GA [ 10263 ]
        Fix Version/s 2.1 [ 10241 ]
        Assignee Priority P1
        Assignee Mircea Toma [ mircea.toma ]
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Fix Version/s 2.0.2 [ 10273 ]
        Fix Version/s 2.1 [ 10241 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Assignee Priority P1

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Neil Griffin
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: