MyFaces CODI
  1. MyFaces CODI
  2. EXTCDI-266

WindowContextManagerObserver causes an Exception when redirecting if the requested view does not exists

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 1.0.3
    • Fix Version/s: 1.0.4
    • Component/s: None
    • Labels:
      None

      Description

      if the current request does not contain a windowId the WindowContextManagerObserver tries to redirect to currend view and appends the windowId. But if the current view does not exists it calls getActionUrl with null. This causes an IllegalArgumentException instead of 404.

      I will add a patch which prevent the redirect if the current viewId is null.

      1. EXTCDI-266.patch
        0.8 kB
        Marcus Büttner

        Activity

        Marcus Büttner created issue -
        Marcus Büttner made changes -
        Field Original Value New Value
        Attachment EXTCDI-266.patch [ 12514117 ]
        Gerhard Petracek made changes -
        Assignee Gerhard Petracek [ gpetracek ]
        Gerhard Petracek made changes -
        Status Open [ 1 ] In Progress [ 3 ]
        Gerhard Petracek made changes -
        Attachment EXTCDI-266.patch [ 12514121 ]
        Gerhard Petracek made changes -
        Status In Progress [ 3 ] Resolved [ 5 ]
        Fix Version/s 1.0.4 [ 12319465 ]
        Resolution Fixed [ 1 ]
        Gerhard Petracek made changes -
        Attachment EXTCDI-266.patch [ 12514121 ]
        Hide
        Gerhard Petracek added a comment -

        minor because there is a workaround (cdi alternative for WindowContextManagerObserver)

        Show
        Gerhard Petracek added a comment - minor because there is a workaround (cdi alternative for WindowContextManagerObserver)
        Gerhard Petracek made changes -
        Priority Major [ 3 ] Minor [ 4 ]
        Gerhard Petracek made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        38m 59s 1 Gerhard Petracek 10/Feb/12 17:23
        In Progress In Progress Resolved Resolved
        48m 52s 1 Gerhard Petracek 10/Feb/12 18:11
        Resolved Resolved Closed Closed
        6d 26m 1 Gerhard Petracek 16/Feb/12 18:38

          People

          • Assignee:
            Gerhard Petracek
            Reporter:
            Marcus Büttner
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development