IT4Change/gradido
3
0
Fork 0
mirror of https://github.com/IT4Change/gradido.git synced 2025-12-13 07:45:54 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1128 from gradido/bugfix_transactions_alterAutoIncrAfterRollback

Browse Source 
Changed the Auto increment so that it is done after the rollback and …
This commit is contained in:
Hannes Heine 2021-11-22 23:27:46 +01:00 committed by GitHub
commit e922a4895c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
backend/src/graphql/resolver/TransactionResolver.ts
View File

@ -613,9 +613,6 @@ export class TransactionResolver {
await queryRunner.commitTransaction() await queryRunner.commitTransaction()
} catch (e) { } catch (e) {
await queryRunner.rollbackTransaction() await queryRunner.rollbackTransaction()
throw e
} finally {
await queryRunner.release()
// TODO: This is broken code - we should never correct an autoincrement index in production // TODO: This is broken code - we should never correct an autoincrement index in production
// according to dario it is required tho to properly work. The index of the table is used as // according to dario it is required tho to properly work. The index of the table is used as
// index for the transaction which requires a chain without gaps // index for the transaction which requires a chain without gaps
@ -627,6 +624,9 @@ export class TransactionResolver {
// eslint-disable-next-line no-console // eslint-disable-next-line no-console
console.log('problems with reset auto increment: %o', error) console.log('problems with reset auto increment: %o', error)
}) })
throw e
} finally {
await queryRunner.release()
} }
// send notification email // send notification email
// TODO: translate // TODO: translate