Transaction Were DeadLocked when releasing invoice

Hi @abdallaahmed61 ,

Could you perform the update using the graph object instead of DBupdate?

This is due to running internal processes. To figure out which processes might be blocking each other, you should show the entire process you are trying to run (calling actions, methods, etc.).

A Transaction deadlocked error refers to database updates and means that two different processes are trying to change the same record in the same SQL table at the same time.

Unfortunately from this code, I can’t say what the exact problem is, but I can suggest some changes which will help to avoid errors like this.

From this code, I saw you have a graph MixPaymentsEntry created for the MixPayments record. I hope this graph can be used for data manipulation in the MixPayments table.

• It is not necessary to use PXTransactionScope if the code uses only DB direct operation. In this code, you can just remove the PXTransactionScope.
• If the MixPayments records update depends on the Invoice Release process the best practice is to override the ReleaseInvoiceProc method and write your code before or after calling base method.

 public delegate void ReleaseInvoiceProcDelegate(List<ARRegister> list, bool isMassProcess);
 [PXOverride]
 public void ReleaseInvoiceProc(List<ARRegister> list, bool isMassProcess, ReleaseInvoiceProcDelegate releaseInvoiceProc)
 {
     using (PXTransactionScope scope = new PXTransactionScope())
     {
         //TODO add your code here
         releaseInvoiceProc(list, isMassProcess);
         //TODO add your code here
         scope.Complete();
     }
 }

You can write code without direct database updates, as @jinin pointed out above.

For example something like this: 

 public virtual void UpdateMixPayments(List<ARRegister> list)
 {
     foreach (ARRegister arregister in list)
     {
         ARInvoice invoice = ARInvoice.PK.Find(this.Base, arregister.DocType, arregister.RefNbr);

         ARTran invLine = ARTran.FK.Invoice.SelectChildren(Base, invoice).FirstOrDefault();
         if (invLine != null)
         {
             var mixPayments = Pay.Select(invLine.SOOrderNbr, invLine.SOOrderType);

             MixPaymentsEntry mixPaymentsGraph = PXGraph.CreateInstance<MixPaymentsEntry>();

             foreach (MixPayments mix in mixPayments)
             {
                 mixPaymentsGraph.Clear();
                 if (mix.RefNbr is null || mix.DocType is null || mix.RefNbr == " <NEW>")
                 {
                     mixPaymentsGraph.MixPayments.Current = mix;//Where MixPayments is primary view of MixPaymentsEntry graph
                     mixPaymentsGraph.MixPayments.Current.DocType = invoice.DocType;
                     mixPaymentsGraph.MixPayments.Current.RefNbr = invoice.RefNbr;
                     mixPaymentsGraph.MixPayments.Update(mixPaymentsGraph.MixPayments.Current);
                     mixPaymentsGraph.Save.Press();
                 }
             }
         }
     }
 }

Of course, the code written inside of second foreach loop depends on which type graph is MixPaymentsEntry. If it is the ListView the calling Clear() and Save action can be out of the foreach.

Then call this method during the process where you want to update records