select all members when sending fees to the ORD if memberIds is empty
This commit is contained in:
@@ -33,6 +33,8 @@ const deleteFeesFromORD = (dateRange, memberIds) => {
|
||||
memberIdsMap[memberId] = true;
|
||||
});
|
||||
|
||||
const filterByMemberIds = memberIds.length > 0;
|
||||
|
||||
const feeIdsToRemove = [];
|
||||
|
||||
fetchedFees.forEach((fee) => {
|
||||
@@ -42,7 +44,9 @@ const deleteFeesFromORD = (dateRange, memberIds) => {
|
||||
|
||||
const isDateInDateRange = startDate.isSameOrBefore(date) && endDate.isSameOrAfter(date);
|
||||
const manuallyAddedFee = manualFeeNames.indexOf(name) !== -1;
|
||||
if (memberIdsMap[member] && isDateInDateRange && (status === UNPAID_FEE_STATUS) && !manuallyAddedFee) {
|
||||
const memberFeesShouldBeDeleted = filterByMemberIds ? memberIdsMap[member] : true;
|
||||
|
||||
if (memberFeesShouldBeDeleted && isDateInDateRange && (status === UNPAID_FEE_STATUS) && !manuallyAddedFee) {
|
||||
feeIdsToRemove.push(feeId);
|
||||
}
|
||||
});
|
||||
|
||||
@@ -10,14 +10,18 @@ const fetchAllMembershipsForMemberIds = (memberIds) => {
|
||||
const memberships = result.data || [];
|
||||
|
||||
if (Array.isArray(memberIds)){
|
||||
const filteredMemberships = [];
|
||||
memberships.forEach((membership) => {
|
||||
const { member } = membership;
|
||||
if (memberIds.indexOf(member) !== -1){
|
||||
filteredMemberships.push(membership);
|
||||
}
|
||||
});
|
||||
resolve(filteredMemberships);
|
||||
if (memberIds.length > 0){
|
||||
const filteredMemberships = [];
|
||||
memberships.forEach((membership) => {
|
||||
const { member } = membership;
|
||||
if (memberIds.indexOf(member) !== -1){
|
||||
filteredMemberships.push(membership);
|
||||
}
|
||||
});
|
||||
resolve(filteredMemberships);
|
||||
}else{
|
||||
resolve(memberships);
|
||||
}
|
||||
}else{
|
||||
reject(integrationServiceErrors.EXPECTED_MEMBER_IDS_ARRAY);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user