modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/plugin/pc/alert/AlertSender.java
| 8 ++++++--
modules/enterprise/server/plugins/alert-email/src/main/java/org/rhq/enterprise/server/plugins/alertEmail/EmailSender.java
| 2 +-
2 files changed, 7 insertions(+), 3 deletions(-)
New commits:
commit db911f971bb286fe600c4257b85f917bb3227004
Author: Joseph Marques <joseph(a)redhat.com>
Date: Tue Jun 29 15:33:19 2010 -0400
BZ-607898: fix for sending alert notifications with multiple direct email addresses
diff --git
a/modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/plugin/pc/alert/AlertSender.java
b/modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/plugin/pc/alert/AlertSender.java
index 02183f6..5a958b1 100644
---
a/modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/plugin/pc/alert/AlertSender.java
+++
b/modules/enterprise/server/jar/src/main/java/org/rhq/enterprise/server/plugin/pc/alert/AlertSender.java
@@ -118,9 +118,13 @@ public abstract class AlertSender<T extends
ServerPluginComponent> {
* where '|' delimits all elements as well as wraps
* the entire expression.
*/
- @SuppressWarnings("unchecked")
public static <T> List<T> unfence(String fencedData, Class<T> type)
{
- String[] elements = fencedData.split("\\|");
+ return unfence(fencedData, type, "\\|");
+ }
+
+ @SuppressWarnings("unchecked")
+ public static <T> List<T> unfence(String fencedData, Class<T> type,
String delimiter) {
+ String[] elements = fencedData.split(delimiter);
List<T> results = new ArrayList<T>(elements.length);
if (Integer.class.equals(type)) {
diff --git
a/modules/enterprise/server/plugins/alert-email/src/main/java/org/rhq/enterprise/server/plugins/alertEmail/EmailSender.java
b/modules/enterprise/server/plugins/alert-email/src/main/java/org/rhq/enterprise/server/plugins/alertEmail/EmailSender.java
index 773d99a..81b1383 100644
---
a/modules/enterprise/server/plugins/alert-email/src/main/java/org/rhq/enterprise/server/plugins/alertEmail/EmailSender.java
+++
b/modules/enterprise/server/plugins/alert-email/src/main/java/org/rhq/enterprise/server/plugins/alertEmail/EmailSender.java
@@ -40,7 +40,7 @@ public class EmailSender extends AlertSender {
return new SenderResult(ResultState.FAILURE, "No email address
given");
}
- List<String> emails = AlertSender.unfence(emailAddressString,
String.class);
+ List<String> emails = AlertSender.unfence(emailAddressString, String.class,
",");
return new SenderResult(ResultState.DEFERRED_EMAIL, "Sending to addresses:
" + emails, emails);
}
Show replies by date