[rhq] modules/enterprise

lkrejci lkrejci at fedoraproject.org
Tue Aug 14 13:52:20 UTC 2012


 modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/groups/detail/configuration/GroupResourceConfigurationEditView.java |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

New commits:
commit 19cfeb90dcbf853a5aacf76656b4218af16d642f
Author: Lukas Krejci <lkrejci at redhat.com>
Date:   Tue Aug 14 15:51:48 2012 +0200

    [BZ 848066] - The group resource configuration page shows the config editor again
    Also made the layout consistent with the single resource config page by
    moving the button bar to the bottom of the page.

diff --git a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/groups/detail/configuration/GroupResourceConfigurationEditView.java b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/groups/detail/configuration/GroupResourceConfigurationEditView.java
index f6d6657..c5467b8 100644
--- a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/groups/detail/configuration/GroupResourceConfigurationEditView.java
+++ b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/inventory/groups/detail/configuration/GroupResourceConfigurationEditView.java
@@ -111,9 +111,8 @@ public class GroupResourceConfigurationEditView extends LocatableVLayout impleme
                 save();
             }
         });
+        this.saveButton.disable();
         buttonbar.addMember(saveButton);
-        // The button bar will remain hidden until the configuration has been successfully loaded.
-        buttonbar.setVisible(false);
         return buttonbar;
     }
 
@@ -125,7 +124,7 @@ public class GroupResourceConfigurationEditView extends LocatableVLayout impleme
         }
 
         this.refreshing = true;
-        this.saveButton.disable();
+        
         if (editor != null) {
             editor.destroy();
             removeMember(editor);
@@ -146,6 +145,7 @@ public class GroupResourceConfigurationEditView extends LocatableVLayout impleme
             this.editor.addPropertyValueChangeListener(this);
             this.editor.setReadOnly(!this.resourcePermission.isConfigureWrite());
             addMember(this.editor);
+            addMember(createButtonBar());            
             this.refreshing = false; // when we get here, we know we are done the refresh
         }
     }
@@ -197,7 +197,6 @@ public class GroupResourceConfigurationEditView extends LocatableVLayout impleme
                                             "The server did not return the configuration for one or more member resources.");
                                     }
                                     memberConfigurations.add(memberConfiguration);
-                                    addMember(createButtonBar());
                                 }
                                 initEditor();
                             }




More information about the rhq-commits mailing list