Skip to content

Commit 0da7c15

Browse files
committed
Limit getMembers() and getSubgroups() to only fetching one object. These lists are only used to find the size of each
1 parent 9a3d330 commit 0da7c15

1 file changed

Lines changed: 12 additions & 2 deletions

File tree

src/app/access-control/group-registry/groups-registry.component.ts

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -216,18 +216,28 @@ export class GroupsRegistryComponent implements OnInit, OnDestroy {
216216

217217
/**
218218
* Get the members (epersons embedded value of a group)
219+
* NOTE: At this time we only grab the *first* member in order to receive the `totalElements` value
220+
* needed for our HTML template.
219221
* @param group
220222
*/
221223
getMembers(group: Group): Observable<RemoteData<PaginatedList<EPerson>>> {
222-
return this.ePersonDataService.findListByHref(group._links.epersons.href).pipe(getFirstSucceededRemoteData());
224+
return this.ePersonDataService.findListByHref(group._links.epersons.href, {
225+
currentPage: 1,
226+
elementsPerPage: 1,
227+
}).pipe(getFirstSucceededRemoteData());
223228
}
224229

225230
/**
226231
* Get the subgroups (groups embedded value of a group)
232+
* NOTE: At this time we only grab the *first* subgroup in order to receive the `totalElements` value
233+
* needed for our HTML template.
227234
* @param group
228235
*/
229236
getSubgroups(group: Group): Observable<RemoteData<PaginatedList<Group>>> {
230-
return this.groupService.findListByHref(group._links.subgroups.href).pipe(getFirstSucceededRemoteData());
237+
return this.groupService.findListByHref(group._links.subgroups.href, {
238+
currentPage: 1,
239+
elementsPerPage: 1,
240+
}).pipe(getFirstSucceededRemoteData());
231241
}
232242

233243
/**

0 commit comments

Comments
 (0)