16683: Tweak other calls to user listing to work with federated users
authorPeter Amstutz <peter.amstutz@curii.com>
Fri, 14 Aug 2020 18:49:03 +0000 (14:49 -0400)
committerPeter Amstutz <peter.amstutz@curii.com>
Fri, 14 Aug 2020 18:50:40 +0000 (14:50 -0400)
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz@curii.com>

src/store/collections-content-address-panel/collections-content-address-middleware-service.ts
src/store/data-explorer/data-explorer-middleware-service.ts
src/store/group-details-panel/group-details-panel-middleware-service.ts
src/store/users/users-actions.ts
src/views-components/sharing-dialog/participant-select.tsx

index 72da1d2e0144927395175ad4739f227b9cb02071..a68d13bdec52aa3c6c7ddaf0d5d84593202ab47d 100644 (file)
@@ -77,25 +77,24 @@ export class CollectionsWithSameContentAddressMiddlewareService extends DataExpl
                     }
                 });
                 const responseUsers = await this.services.userService.list({
-                    limit: dataExplorer.rowsPerPage,
-                    offset: dataExplorer.page * dataExplorer.rowsPerPage,
                     filters: new FilterBuilder()
                         .addIn('uuid', userUuids)
-                        .getFilters()
+                        .getFilters(),
+                    count: "none"
                 });
                 const responseGroups = await this.services.groupsService.list({
-                    limit: dataExplorer.rowsPerPage,
-                    offset: dataExplorer.page * dataExplorer.rowsPerPage,
                     filters: new FilterBuilder()
                         .addIn('uuid', groupUuids)
-                        .getFilters()
+                        .getFilters(),
+                    count: "none"
                 });
                 responseUsers.items.map(it => {
                     api.dispatch<any>(ownerNameActions.SET_OWNER_NAME({
                         name: it.uuid === userUuid
                             ? 'User: Me'
                             : `User: ${getUserDisplayName(it)}`,
-                        uuid: it.uuid }));
+                        uuid: it.uuid
+                    }));
                 });
                 responseGroups.items.map(it => {
                     api.dispatch<any>(ownerNameActions.SET_OWNER_NAME({ name: `Project: ${it.name}`, uuid: it.uuid }));
index 219e76030a14e12470bafa55442094b9c6239f74..a7c32f42d66ae54507811ca56289f6ccc783dc62 100644 (file)
@@ -36,6 +36,7 @@ export const getDataExplorerColumnFilters = <T>(columns: DataColumns<T>, columnN
 export const dataExplorerToListParams = (dataExplorer: DataExplorer) => ({
     limit: dataExplorer.rowsPerPage,
     offset: dataExplorer.page * dataExplorer.rowsPerPage,
+    count: "none"
 });
 
 export const listResultsToDataExplorerItemsMeta = <R>({ itemsAvailable, offset, limit }: ListResults<R>) => ({
index 94f78a58e75116e32febd2164a6a346508a40fa4..5aff4e7b303fe34e8fcbdfd90647408c19da4478 100644 (file)
@@ -36,7 +36,8 @@ export class GroupDetailsPanelMiddlewareService extends DataExplorerMiddlewareSe
                 const users = await this.services.userService.list({
                     filters: new FilterBuilder()
                         .addIn('uuid', permissions.items.map(item => item.headUuid))
-                        .getFilters()
+                        .getFilters(),
+                    count: "none"
                 });
                 api.dispatch(GroupDetailsPanelActions.SET_ITEMS({
                     ...listResultsToDataExplorerItemsMeta(permissions),
index 7b12fe75633ec403a8978bb15c9ece5bc9012cce..8f696fa29ad6e0d9e7122b5cbdc82d04a1d93a3f 100644 (file)
@@ -157,7 +157,7 @@ export const userBindedActions = bindDataExplorerActions(USERS_PANEL_ID);
 
 export const loadUsersData = () =>
     async (dispatch: Dispatch, getState: () => RootState, services: ServiceRepository) => {
-        await services.userService.list();
+        await services.userService.list({ count: "none" });
     };
 
 export const loadUsersPanel = () =>
index 5d062da0ef7ae612179c2e9edfc14e5622eec739..ea3775e93ad652a8463b95b3e28656e74a8f03de 100644 (file)
@@ -41,7 +41,7 @@ interface ParticipantSelectState {
 }
 
 const getDisplayName = (item: GroupResource & UserResource) => {
-    switch(item.kind) {
+    switch (item.kind) {
         case ResourceKind.USER:
             return getUserDisplayName(item, true);
         case ResourceKind.GROUP:
@@ -131,14 +131,14 @@ export const ParticipantSelect = connect()(
             const filterUsers = new FilterBuilder()
                 .addILike('any', value)
                 .getFilters();
-            const userItems: ListResults<any> = await userService.list({ filters: filterUsers, limit });
+            const userItems: ListResults<any> = await userService.list({ filters: filterUsers, limit, count: "none" });
 
             const filterGroups = new FilterBuilder()
                 .addNotIn('group_class', [GroupClass.PROJECT])
                 .addILike('name', value)
                 .getFilters();
 
-            const groupItems: ListResults<any> = await groupsService.list({ filters: filterGroups, limit });
+            const groupItems: ListResults<any> = await groupsService.list({ filters: filterGroups, limit, count: "none" });
             this.setState({
                 suggestions: this.props.onlyPeople
                     ? userItems.items