CBSNexora/cbsnexora-backend
4
0
Fork 0
Code Issues Pull Requests Actions 37 Packages Projects Releases Wiki Activity

Fix bug in generating notification.

Browse Source 
When querying the users required to get a notification, add new topic subscribers
to the current set of subscribers, instead of replacing the currents set of subscribers
with the new topic subscribers.
This commit is contained in:
Anh3h
2019-01-21 12:21:24 -05:00
parent 4fd584c87f
commit f63a8dea81
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
fineract-provider/src/main/java/org/apache/fineract/notification/service/NotificationDomainServiceImpl.java
+4 -2
View File
@@ -51,7 +51,9 @@ import javax.jms.Queue;
import java.util.ArrayList;
import java.util.Collection;
import java.util.HashSet;
import java.util.List;
import java.util.Set;
import java.util.Map;
@Service
@@ -583,7 +585,7 @@ public class NotificationDomainServiceImpl implements NotificationDomainService
private List<Long> retrieveSubscribers(Long officeId, String permission) {
Collection<TopicSubscriberData> topicSubscribers = new ArrayList<>();
Set<TopicSubscriberData> topicSubscribers = new HashSet<>();
List<Long> subscriberIds = new ArrayList<>();
Long entityId = officeId;
String entityType= "";
@@ -596,7 +598,7 @@ public class NotificationDomainServiceImpl implements NotificationDomainService
for (Role curRole : allRoles) {
if (curRole.hasPermissionTo(permission) || curRole.hasPermissionTo("ALL_FUNCTIONS")) {
String memberType = curRole.getName();
topicSubscribers = topicSubscriberReadPlatformService.getSubscribers(entityId, entityType, memberType);
topicSubscribers.addAll(topicSubscriberReadPlatformService.getSubscribers(entityId, entityType, memberType));
}
}