[rhq] modules/enterprise

Jiri Kremser jkremser at fedoraproject.org
Tue Aug 28 12:36:12 UTC 2012


 modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/alert/definitions/CliNotificationSenderForm.java |    5 +++++
 1 file changed, 5 insertions(+)

New commits:
commit 9fda60f21e354f3b33a1ecf85b81cae19e0af1af
Author: Jirka Kremser <jkremser at redhat.com>
Date:   Tue Aug 28 14:34:23 2012 +0200

    [BZ 850818 - Globally uncaught exception on clicking OK button while adding CLI Script alert notification to the alert definition] Added the check whether the repository has been picked

diff --git a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/alert/definitions/CliNotificationSenderForm.java b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/alert/definitions/CliNotificationSenderForm.java
index f63eb55..b212043 100644
--- a/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/alert/definitions/CliNotificationSenderForm.java
+++ b/modules/enterprise/gui/coregui/src/main/java/org/rhq/enterprise/gui/coregui/client/alert/definitions/CliNotificationSenderForm.java
@@ -403,6 +403,11 @@ public class CliNotificationSenderForm extends AbstractNotificationSenderForm {
     }
 
     private void validatePackage(final AsyncCallback<Void> callback) {
+        if (config.selectedRepo == null) {
+            repoSelector.setIcons(failureIcon);
+            callback.onFailure(null);
+            return;
+        }
         getConfiguration().put(new PropertySimple(PROP_REPO_ID, config.selectedRepo.getId()));
 
         if (packageSelector.getSelectedIndex() == 0) {




More information about the rhq-commits mailing list