glassfish
  1. glassfish
  2. GLASSFISH-15993

Properties: runtime exception on cancel for DAS instance properties

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1_b42
    • Fix Version/s: 3.1.1_b01
    • Component/s: admin_gui
    • Labels:
      None
    • Environment:

      promoted build b43

      Description

      Steps to reproduce:

      1. Click on server (Admin Server), Properties tab.
      2. Click on Instance Properties.
      3. Click Cancel - runtime exeption is printed on screen.

      This only happens for DAS Instance Properties tab. Cancel works fine on System Properties screen for DAS and on all other properties tabs for clusters, clustered instances and standalone instances.

        Activity

        Hide
        lidiam added a comment -

        Removed the 3_1-exclude tag, so that this issue can be evaluated.

        Show
        lidiam added a comment - Removed the 3_1-exclude tag, so that this issue can be evaluated.
        Hide
        Anissa Lam added a comment -

        Not show stopper. exclude from 3.1

        Show
        Anissa Lam added a comment - Not show stopper. exclude from 3.1
        Hide
        Anissa Lam added a comment -

        Fixed in trunk.

        Project: glassfish
        Repository: svn
        Revision: 45186
        Author: anilam
        Date: 2011-02-20 00:32:27 UTC
        Link:
        Log Message:
        ------------
        GLASSFISH-15993. Specify parent page for DAS instance properties page's cancel button redirect to.

        Revisions:
        ----------
        45186

        Modified Paths:
        ---------------
        trunk/v3/admingui/common/src/main/resources/appServer/instanceProperties.jsf

        Diffs:
        ------
        Index: trunk/v3/admingui/common/src/main/resources/appServer/instanceProperties.jsf
        ===================================================================
        — trunk/v3/admingui/common/src/main/resources/appServer/instanceProperties.jsf (revision 45185)
        +++ trunk/v3/admingui/common/src/main/resources/appServer/instanceProperties.jsf (revision 45186)
        @@ -2,7 +2,7 @@

        DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.

        • Copyright (c) 2010 Oracle and/or its affiliates. All rights reserved.
          + Copyright (c) 2010-2011 Oracle and/or its affiliates. All rights reserved.

        The contents of this file are subject to the terms of either the GNU
        General Public License Version 2 only ("GPL") or the Common Development
        @@ -54,8 +54,7 @@
        urlencode(value="#

        {pageSession.instanceName}

        " encoding="UTF-8" result="#

        {pageSession.encodedInstanceName}");
        setSessionAttribute(key="serverInstTabs" value="instanceProps");
        setPageSessionAttribute(key="listLink" value="#{request.contextPath}/common/appServer/instanceProperties.jsf?instanceName=${instanceName}");
        -
        -
        + setPageSessionAttribute(key="parentPage", value="#{request.contextPath}/common/appServer/serverInstGeneralPe.jsf?instanceName=#{pageSession.encodedInstanceName}

        ");
        setPageSessionAttribute(key="selfPage", value="#

        {request.contextPath}

        /common/appServer/instanceProperties.jsf?instanceName=#

        {pageSession.encodedInstanceName}");
        setPageSessionAttribute(key="parentUrl", value="#{sessionScope.REST_URL}/servers/server/#{pageSession.encodedInstanceName}

        ");
        setPageSessionAttribute(key="selfUrl", value="#

        {pageSession.parentUrl}

        ");

        Show
        Anissa Lam added a comment - Fixed in trunk. Project: glassfish Repository: svn Revision: 45186 Author: anilam Date: 2011-02-20 00:32:27 UTC Link: Log Message: ------------ GLASSFISH-15993 . Specify parent page for DAS instance properties page's cancel button redirect to. Revisions: ---------- 45186 Modified Paths: --------------- trunk/v3/admingui/common/src/main/resources/appServer/instanceProperties.jsf Diffs: ------ Index: trunk/v3/admingui/common/src/main/resources/appServer/instanceProperties.jsf =================================================================== — trunk/v3/admingui/common/src/main/resources/appServer/instanceProperties.jsf (revision 45185) +++ trunk/v3/admingui/common/src/main/resources/appServer/instanceProperties.jsf (revision 45186) @@ -2,7 +2,7 @@ DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER. Copyright (c) 2010 Oracle and/or its affiliates. All rights reserved. + Copyright (c) 2010-2011 Oracle and/or its affiliates. All rights reserved. The contents of this file are subject to the terms of either the GNU General Public License Version 2 only ("GPL") or the Common Development @@ -54,8 +54,7 @@ urlencode(value="# {pageSession.instanceName} " encoding="UTF-8" result="# {pageSession.encodedInstanceName}"); setSessionAttribute(key="serverInstTabs" value="instanceProps"); setPageSessionAttribute(key="listLink" value="#{request.contextPath}/common/appServer/instanceProperties.jsf?instanceName=${instanceName}"); - - + setPageSessionAttribute(key="parentPage", value="#{request.contextPath}/common/appServer/serverInstGeneralPe.jsf?instanceName=#{pageSession.encodedInstanceName} "); setPageSessionAttribute(key="selfPage", value="# {request.contextPath} /common/appServer/instanceProperties.jsf?instanceName=# {pageSession.encodedInstanceName}"); setPageSessionAttribute(key="parentUrl", value="#{sessionScope.REST_URL}/servers/server/#{pageSession.encodedInstanceName} "); setPageSessionAttribute(key="selfUrl", value="# {pageSession.parentUrl} ");
        Hide
        lidiam added a comment -

        Verified in build b08.

        Show
        lidiam added a comment - Verified in build b08.

          People

          • Assignee:
            Anissa Lam
            Reporter:
            lidiam
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: