Uploaded image for project: 'mod-finance-storage'
  1. mod-finance-storage
  2. MODFISTO-293

Error All expected transactions already processed

    XMLWordPrintable

Details

    • ACQ Sprint 137
    • 1
    • Thunderjet
    • Lotus (R1 2022) Bug Fix
    • Hide
      See https://wiki.folio.org/display/DD/How+to+fix+issues+with+finance+data
      (this is not a complete workaround, budget might have to be fixed)
      Show
      See https://wiki.folio.org/display/DD/How+to+fix+issues+with+finance+data (this is not a complete workaround, budget might have to be fixed)
    • Implementation coding issue

    Description

      Overview:
      When an error occurs in createTransaction or updateTransaction, the temporary transactions are currently not cleaned up. Because the same id is currently used to process transactions for orders or invoices again, and because processAllOrNothing compares the number of temporary transactions to the number of expected transactions, this causes the processing of failed temporary transactions with some new ones, and the subsequent error "All expected transactions already processed".

      Since the calling mods will give up on the whole set of transactions if a single one fails, we should remove all the related temporary transactions in case of failure, for any case of failure (including validation).

      MODFISTO-268 will resolve this issue, but the resolution is too complex to be included in a hot fix. On the other hand, cleaning up the temp transactions is easy and can be included in a hot fix.

      Steps to Reproduce:
      See related stories.

      Expected Results:
      User can try again if an operation (such as invoice approval) fails.

      Actual Results:
      The operation keeps failing with the message "All expected transactions already processed".

      Additional Information:
      In AllOrNothingTransactionService, deleteTempTransactions should be called if there is an error within createTransaction, updateTransaction or processAllOrNothing.

      Interested parties:

      TestRail: Results

        Attachments

          Issue Links

            Activity

              People

                damieng Damien Guillaume
                damieng Damien Guillaume
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved:

                  TestRail: Runs

                    TestRail: Cases