Merge pull request #3059 from gradido/admin-order-usersearch-desc

feat(admin): order user search desc, fix pagination

admin/src/graphql/searchUsers.js

@@ -1,12 +1,19 @@
 import gql from 'graphql-tag'
 
 export const searchUsers = gql`
-  query ($searchText: String!, $currentPage: Int, $pageSize: Int, $filters: SearchUsersFilters) {
+  query (
+    $query: String!
+    $filters: SearchUsersFilters
+    $currentPage: Int = 0
+    $pageSize: Int = 25
+    $order: Order = ASC
+  ) {
     searchUsers(
-      searchText: $searchText
+      query: $query
+      filters: $filters
       currentPage: $currentPage
       pageSize: $pageSize
-      filters: $filters
+      order: $order
     ) {
       userCount
       userList {

admin/src/pages/UserSearch.spec.js

@@ -10,11 +10,20 @@ const apolloQueryMock = jest.fn().mockResolvedValue({
       userCount: 4,
       userList: [
         {
-          userId: 1,
-          firstName: 'Bibi',
-          lastName: 'Bloxberg',
-          email: 'bibi@bloxberg.de',
-          creation: [200, 400, 600],
+          userId: 4,
+          firstName: 'New',
+          lastName: 'User',
+          email: 'new@user.ch',
+          creation: [1000, 1000, 1000],
+          emailChecked: false,
+          deletedAt: null,
+        },
+        {
+          userId: 3,
+          firstName: 'Peter',
+          lastName: 'Lustig',
+          email: 'peter@lustig.de',
+          creation: [0, 0, 0],
           emailChecked: true,
           deletedAt: null,
         },
@@ -28,23 +37,14 @@ const apolloQueryMock = jest.fn().mockResolvedValue({
           deletedAt: new Date(),
         },
         {
-          userId: 3,
-          firstName: 'Peter',
-          lastName: 'Lustig',
-          email: 'peter@lustig.de',
-          creation: [0, 0, 0],
+          userId: 1,
+          firstName: 'Bibi',
+          lastName: 'Bloxberg',
+          email: 'bibi@bloxberg.de',
+          creation: [200, 400, 600],
           emailChecked: true,
           deletedAt: null,
         },
-        {
-          userId: 4,
-          firstName: 'New',
-          lastName: 'User',
-          email: 'new@user.ch',
-          creation: [1000, 1000, 1000],
-          emailChecked: false,
-          deletedAt: null,
-        },
       ],
     },
   },
@@ -79,9 +79,10 @@ describe('UserSearch', () => {
       expect(apolloQueryMock).toBeCalledWith(
         expect.objectContaining({
           variables: {
-            searchText: '',
+            query: '',
             currentPage: 1,
             pageSize: 25,
+            order: 'DESC',
             filters: {
               byActivated: null,
               byDeleted: null,
@@ -100,9 +101,10 @@ describe('UserSearch', () => {
         expect(apolloQueryMock).toBeCalledWith(
           expect.objectContaining({
             variables: {
-              searchText: '',
+              query: '',
               currentPage: 1,
               pageSize: 25,
+              order: 'DESC',
               filters: {
                 byActivated: false,
                 byDeleted: null,
@@ -122,9 +124,10 @@ describe('UserSearch', () => {
         expect(apolloQueryMock).toBeCalledWith(
           expect.objectContaining({
             variables: {
-              searchText: '',
+              query: '',
               currentPage: 1,
               pageSize: 25,
+              order: 'DESC',
               filters: {
                 byActivated: null,
                 byDeleted: true,
@@ -144,9 +147,10 @@ describe('UserSearch', () => {
         expect(apolloQueryMock).toBeCalledWith(
           expect.objectContaining({
             variables: {
-              searchText: '',
+              query: '',
               currentPage: 2,
               pageSize: 25,
+              order: 'DESC',
               filters: {
                 byActivated: null,
                 byDeleted: null,
@@ -166,9 +170,10 @@ describe('UserSearch', () => {
         expect(apolloQueryMock).toBeCalledWith(
           expect.objectContaining({
             variables: {
-              searchText: 'search string',
+              query: 'search string',
               currentPage: 1,
               pageSize: 25,
+              order: 'DESC',
               filters: {
                 byActivated: null,
                 byDeleted: null,
@@ -185,9 +190,10 @@ describe('UserSearch', () => {
           expect(apolloQueryMock).toBeCalledWith(
             expect.objectContaining({
               variables: {
-                searchText: '',
+                query: '',
                 currentPage: 1,
                 pageSize: 25,
+                order: 'DESC',
                 filters: {
                   byActivated: null,
                   byDeleted: null,

admin/src/pages/UserSearch.vue

@@ -49,7 +49,7 @@
       pills
       size="lg"
       v-model="currentPage"
-      per-page="perPage"
+      :per-page="perPage"
       :total-rows="rows"
       align="center"
       :hide-ellipsis="true"
@@ -97,10 +97,11 @@ export default {
         .query({
           query: searchUsers,
           variables: {
-            searchText: this.criteria,
+            query: this.criteria,
+            filters: this.filters,
             currentPage: this.currentPage,
             pageSize: this.perPage,
-            filters: this.filters,
+            order: 'DESC',
           },
           fetchPolicy: 'no-cache',
         })

backend/src/graphql/arg/SearchUsersArgs.ts

@@ -1,21 +0,0 @@
-import { ArgsType, Field, Int } from 'type-graphql'
-
-import { SearchUsersFilters } from '@arg/SearchUsersFilters'
-
-@ArgsType()
-export class SearchUsersArgs {
-  @Field(() => String)
-  searchText: string
-
-  @Field(() => Int, { nullable: true })
-  // eslint-disable-next-line type-graphql/invalid-nullable-input-type
-  currentPage?: number
-
-  @Field(() => Int, { nullable: true })
-  // eslint-disable-next-line type-graphql/invalid-nullable-input-type
-  pageSize?: number
-
-  // eslint-disable-next-line type-graphql/wrong-decorator-signature
-  @Field(() => SearchUsersFilters, { nullable: true, defaultValue: null })
-  filters?: SearchUsersFilters | null
-}

backend/src/graphql/resolver/UserResolver.test.ts

@@ -2110,7 +2110,7 @@ describe('UserResolver', () => {
 
   describe('search users', () => {
     const variablesWithoutTextAndFilters = {
-      searchText: '',
+      query: '',
       currentPage: 1,
       pageSize: 25,
       filters: null,

backend/src/graphql/resolver/UserResolver.ts

@@ -23,7 +23,7 @@ import { v4 as uuidv4 } from 'uuid'
 
 import { CreateUserArgs } from '@arg/CreateUserArgs'
 import { Paginated } from '@arg/Paginated'
-import { SearchUsersArgs } from '@arg/SearchUsersArgs'
+import { SearchUsersFilters } from '@arg/SearchUsersFilters'
 import { UnsecureLoginArgs } from '@arg/UnsecureLoginArgs'
 import { UpdateUserInfosArgs } from '@arg/UpdateUserInfosArgs'
 import { OptInType } from '@enum/OptInType'
@@ -638,8 +638,11 @@ export class UserResolver {
   @Authorized([RIGHTS.SEARCH_USERS])
   @Query(() => SearchUsersResult)
   async searchUsers(
+    @Arg('query', () => String) query: string,
+    @Arg('filters', () => SearchUsersFilters, { nullable: true })
+    filters: SearchUsersFilters | null | undefined,
     @Args()
-    { searchText, currentPage = 1, pageSize = 25, filters }: SearchUsersArgs,
+    { currentPage = 1, pageSize = 25, order = Order.ASC }: Paginated,
     @Ctx() context: Context,
   ): Promise<SearchUsersResult> {
     const clientTimezoneOffset = getClientTimezoneOffset(context)
@@ -657,15 +660,16 @@ export class UserResolver {
       userFields.map((fieldName) => {
         return 'user.' + fieldName
       }),
-      searchText,
+      query,
       filters ?? null,
       currentPage,
       pageSize,
+      order,
     )
 
     if (users.length === 0) {
       return {
-        userCount: 0,
+        userCount: count,
         userList: [],
       }
     }

backend/src/seeds/graphql/queries.ts

@@ -69,12 +69,19 @@ export const sendResetPasswordEmail = gql`
 `
 
 export const searchUsers = gql`
-  query ($searchText: String!, $currentPage: Int, $pageSize: Int, $filters: SearchUsersFilters) {
+  query (
+    $query: String!
+    $filters: SearchUsersFilters
+    $currentPage: Int = 1
+    $pageSize: Int = 25
+    $order: Order = ASC
+  ) {
     searchUsers(
-      searchText: $searchText
+      query: $query
+      filters: $filters
       currentPage: $currentPage
       pageSize: $pageSize
-      filters: $filters
+      order: $order
     ) {
       userCount
       userList {

backend/src/typeorm/repository/User.ts

@@ -1,7 +1,8 @@
 import { Brackets, EntityRepository, IsNull, Not, Repository } from '@dbTools/typeorm'
 import { User as DbUser } from '@entity/User'
 
-import { SearchUsersFilters } from '@/graphql/arg/SearchUsersFilters'
+import { SearchUsersFilters } from '@arg/SearchUsersFilters'
+import { Order } from '@enum/Order'
 
 @EntityRepository(DbUser)
 export class UserRepository extends Repository<DbUser> {
@@ -11,6 +12,7 @@ export class UserRepository extends Repository<DbUser> {
     filters: SearchUsersFilters | null,
     currentPage: number,
     pageSize: number,
+    order = Order.ASC,
   ): Promise<[DbUser[], number]> {
     const query = this.createQueryBuilder('user')
       .select(select)
@@ -46,6 +48,7 @@ export class UserRepository extends Repository<DbUser> {
     }
 
     return query
+      .orderBy({ 'user.id': order })
       .take(pageSize)
       .skip((currentPage - 1) * pageSize)
       .getManyAndCount()