NewMessage UI - NPE when no Group Recipient is selected FMSG-36 #resolve

danielcorreia96 · sergiofbsilva · commit 99bc7a5b6633 · 2018-05-22T15:23:21.000+01:00
diff --git a/messaging-core/src/main/java/org/fenixedu/messaging/core/ui/MessageBean.java b/messaging-core/src/main/java/org/fenixedu/messaging/core/ui/MessageBean.java
@@ -38,6 +38,7 @@
 import java.util.Date;
 import java.util.HashSet;
 import java.util.Locale;
+import java.util.Optional;
 import java.util.Set;
 import java.util.stream.Collectors;
 
@@ -112,7 +113,7 @@ public void setSingleRecipients(final String singleRecipients) {
     }
 
     public Set<String> getSelectedRecipients() {
-        return selectedRecipients;
+        return Optional.ofNullable(selectedRecipients).orElseGet(HashSet::new);
     }
 
     public void setSelectedRecipients(final Set<String> recipients) {

Original file line number	Diff line number	Diff line change
`@@ -38,6 +38,7 @@`
`38`	`38`	`import java.util.Date;`
`39`	`39`	`import java.util.HashSet;`
`40`	`40`	`import java.util.Locale;`
	`41`	`+import java.util.Optional;`
`41`	`42`	`import java.util.Set;`
`42`	`43`	`import java.util.stream.Collectors;`
`43`	`44`
`@@ -112,7 +113,7 @@ public void setSingleRecipients(final String singleRecipients) {`
`112`	`113`	`}`
`113`	`114`
`114`	`115`	`public Set<String> getSelectedRecipients() {`
`115`		`- return selectedRecipients;`
	`116`	`+ return Optional.ofNullable(selectedRecipients).orElseGet(HashSet::new);`
`116`	`117`	`}`
`117`	`118`
`118`	`119`	`public void setSelectedRecipients(final Set<String> recipients) {`