glassfish
  1. glassfish
  2. GLASSFISH-20963

when clicked the icon of "Connector Resources", it forward to the "Admin Object Resources" page

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0_b89_RC5
    • Fix Version/s: 4.1
    • Component/s: admin_gui
    • Labels:
      None
    • Environment:

      linux

      Description

      when clicked the icon of "Connector Resources", it forward to the "Admin Object Resources" page

        Activity

        Hide
        lzg5039 added a comment -

        Hi Anissa
        My fix as follows,modify the line of 61,can you give me some advices

         
        https://svn.java.net/svn/glassfish~svn/trunk/main/appserver/admingui/jca/src/main/resources/jcaTreeNode.jsf
        
        60 <!facet image>
        -61(modify before)        <sun:iconHyperlink id="restIcon" icon="TREE_FOLDER" url="/jca/adminObjectResources.jsf"  border="0" immediate="true" alt="$resource{i18n.tree.connectorResources}" />
        +61(modify after)         <sun:iconHyperlink id="restIcon" icon="TREE_FOLDER" url="/jca/connectorResources.jsf"  border="0" immediate="true" alt="$resource{i18n.tree.connectorResources}" />
        62 </facet>
        
        Show
        lzg5039 added a comment - Hi Anissa My fix as follows,modify the line of 61,can you give me some advices https://svn.java.net/svn/glassfish~svn/trunk/main/appserver/admingui/jca/src/main/resources/jcaTreeNode.jsf 60 <!facet image> -61(modify before) <sun:iconHyperlink id="restIcon" icon="TREE_FOLDER" url="/jca/adminObjectResources.jsf" border="0" immediate="true" alt="$resource{i18n.tree.connectorResources}" /> +61(modify after) <sun:iconHyperlink id="restIcon" icon="TREE_FOLDER" url="/jca/connectorResources.jsf" border="0" immediate="true" alt="$resource{i18n.tree.connectorResources}" /> 62 </facet>
        Hide
        Anissa Lam added a comment -

        Thanks for catching this. Change looks fine, you can go ahead and commit that.
        thanks.

        Show
        Anissa Lam added a comment - Thanks for catching this. Change looks fine, you can go ahead and commit that. thanks.
        Hide
        lzg5039 added a comment -

        Thanks for reviewing it,now I do not have right to commit source,this time could you commit it for me?
        thanks.

        Show
        lzg5039 added a comment - Thanks for reviewing it,now I do not have right to commit source,this time could you commit it for me? thanks.
        Hide
        Jeremy_Lv added a comment -

        Checked in the changes as r63084

        Show
        Jeremy_Lv added a comment - Checked in the changes as r63084

          People

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

            Dates

            • Created:
              Updated:
              Resolved: