Merge pull request #3086 from gradido/fix-gdt-error

fix(backend): gdt server error & gdt refetch policy

backend/src/graphql/model/GdtEntry.ts

@@ -1,25 +1,39 @@
-/* eslint-disable @typescript-eslint/no-unsafe-member-access */
 /* eslint-disable @typescript-eslint/no-unsafe-assignment */
 /* eslint-disable @typescript-eslint/no-explicit-any */
-/* eslint-disable @typescript-eslint/explicit-module-boundary-types */
 import { ObjectType, Field, Float, Int } from 'type-graphql'
 
 import { GdtEntryType } from '@enum/GdtEntryType'
 
 @ObjectType()
 export class GdtEntry {
-  constructor(json: any) {
-    this.id = json.id
-    this.amount = json.amount
-    this.date = json.date
-    this.email = json.email
-    this.comment = json.comment
-    this.couponCode = json.coupon_code
-    this.gdtEntryType = json.gdt_entry_type_id
-    this.factor = json.factor
-    this.amount2 = json.amount2
-    this.factor2 = json.factor2
-    this.gdt = json.gdt
+  constructor({
+    id,
+    amount,
+    date,
+    email,
+    comment,
+    // eslint-disable-next-line camelcase
+    coupon_code,
+    // eslint-disable-next-line camelcase
+    gdt_entry_type_id,
+    factor,
+    amount2,
+    factor2,
+    gdt,
+  }: any) {
+    this.id = id
+    this.amount = amount
+    this.date = date
+    this.email = email
+    this.comment = comment
+    // eslint-disable-next-line camelcase
+    this.couponCode = coupon_code
+    // eslint-disable-next-line camelcase
+    this.gdtEntryType = gdt_entry_type_id
+    this.factor = factor
+    this.amount2 = amount2
+    this.factor2 = factor2
+    this.gdt = gdt
   }
 
   @Field(() => Int)

backend/src/graphql/model/GdtEntryList.ts

@@ -1,20 +1,15 @@
-/* eslint-disable @typescript-eslint/no-unsafe-call */
-/* eslint-disable @typescript-eslint/no-unsafe-member-access */
-/* eslint-disable @typescript-eslint/no-unsafe-assignment */
-/* eslint-disable @typescript-eslint/no-explicit-any */
-/* eslint-disable @typescript-eslint/explicit-module-boundary-types */
 import { ObjectType, Field, Int, Float } from 'type-graphql'
 
 import { GdtEntry } from './GdtEntry'
 
 @ObjectType()
 export class GdtEntryList {
-  constructor(json: any) {
-    this.status = json.state
-    this.count = json.count
-    this.gdtEntries = json.gdtEntries ? json.gdtEntries.map((json: any) => new GdtEntry(json)) : []
-    this.gdtSum = json.gdtSum
-    this.timeUsed = json.timeUsed
+  constructor(status = '', count = 0, gdtEntries = [], gdtSum = 0, timeUsed = 0) {
+    this.status = status
+    this.count = count
+    this.gdtEntries = gdtEntries
+    this.gdtSum = gdtSum
+    this.timeUsed = timeUsed
   }
 
   @Field(() => String)

backend/src/graphql/resolver/GdtResolver.ts

@@ -6,6 +6,7 @@ import { Resolver, Query, Args, Ctx, Authorized, Arg, Int, Float } from 'type-gr
 
 import { Paginated } from '@arg/Paginated'
 import { Order } from '@enum/Order'
+import { GdtEntry } from '@model/GdtEntry'
 import { GdtEntryList } from '@model/GdtEntryList'
 
 import { apiGet, apiPost } from '@/apis/HttpRequest'
@@ -31,9 +32,17 @@ export class GdtResolver {
         `${CONFIG.GDT_API_URL}/GdtEntries/listPerEmailApi/${userEntity.emailContact.email}/${currentPage}/${pageSize}/${order}`,
       )
       if (!resultGDT.success) {
-        throw new LogError(resultGDT.data)
+        return new GdtEntryList()
       }
-      return new GdtEntryList(resultGDT.data)
+      const { state, count, gdtEntries, gdtSum, timeUsed } = resultGDT.data
+      return new GdtEntryList(
+        state,
+        count,
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-call, @typescript-eslint/no-explicit-any
+        gdtEntries ? gdtEntries.map((data: any) => new GdtEntry(data)) : [],
+        gdtSum,
+        timeUsed,
+      )
     } catch (err) {
       throw new LogError('GDT Server is not reachable')
     }

frontend/src/layouts/DashboardLayout.vue

@@ -276,7 +276,7 @@ export default {
           } = result
           this.GdtBalance =
             transactionList.balance.balanceGDT === null
-              ? null
+              ? 0
               : Number(transactionList.balance.balanceGDT)
           this.transactions = transactionList.transactions
           this.balance = Number(transactionList.balance.balance)

frontend/src/pages/Transactions.spec.js

@@ -142,6 +142,7 @@ describe('Transactions', () => {
             currentPage: 1,
             pageSize: 25,
           },
+          fetchPolicy: 'network-only',
         })
       })
 
@@ -170,6 +171,7 @@ describe('Transactions', () => {
               currentPage: 2,
               pageSize: 25,
             },
+            fetchPolicy: 'network-only',
           })
         })
       })

frontend/src/pages/Transactions.vue

@@ -59,6 +59,7 @@ export default {
             currentPage: this.currentPage,
             pageSize: this.pageSize,
           },
+          fetchPolicy: 'network-only',
         })
         .then((result) => {
           const {