Project

General

Profile

Actions
Copy link

bug #6780

closed

ConversationHolder isTransactionActive() returns true even if the transaction is completed

Added by Andreas Kohlbecker almost 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
New
Assignee:
Andreas Kohlbecker
Category:
cdmlib
Target version:
Release 4.12
Start date:
Due date:
% Done:

50%

Estimated time:
Severity:
normal
Found in Version:

Description

The ConversationHolder method isTransactionActive() (always?) returns true in cases when the transaction has been rolled back or was committed.

this mehtod should be changed like this:


public boolean isTransactionActive(){
        return transactionStatus != null && !transactionStatus.isCompleted();
    }

#4335 is possibly a duplicate of this

Actions
Copy link
 #2

Updated by Andreas Kohlbecker over 6 years ago

  • Status changed from New to In Progress
  • Assignee changed from Andreas Müller to Andreas Kohlbecker
  • Target version changed from Unassigned CDM tickets to Release 4.11
Actions
Copy link
 #3

Updated by Andreas Kohlbecker over 6 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 50
Actions
Copy link
 #4

Updated by Andreas Kohlbecker over 6 years ago

I needed to revert (cdm-vaadin|b84bb2fd ) a prevous commit which was making use of the fixed isTransactionActive() method due to problems with the transaction synchronization being not active:

2017-10-25 18:03:20.859:WARN:oejs.HttpChannel:qtp748658608-47: /app/distribution
java.lang.IllegalStateException: Cannot deactivate transaction synchronization - not active
    at org.springframework.transaction.support.TransactionSynchronizationManager.clearSynchronization(TransactionSynchronizationManager.java:329)
    at org.springframework.transaction.support.TransactionSynchronizationManager.clear(TransactionSynchronizationManager.java:464)
    at org.springframework.transaction.support.AbstractPlatformTransactionManager.cleanupAfterCompletion(AbstractPlatformTransactionManager.java:1013)
    at org.springframework.transaction.support.AbstractPlatformTransactionManager.processCommit(AbstractPlatformTransactionManager.java:811)
    at org.springframework.transaction.support.AbstractPlatformTransactionManager.commit(AbstractPlatformTransactionManager.java:730)
    at eu.etaxonomy.cdm.api.conversation.ConversationHolder.commit(ConversationHolder.java:341)
    at eu.etaxonomy.vaadin.mvp.AbstractPresenter.unbindConversation(AbstractPresenter.java:194)
    at eu.etaxonomy.vaadin.mvp.AbstractPresenter.onRequestEnd(AbstractPresenter.java:253)
    at eu.etaxonomy.cdm.vaadin.server.CdmSpringVaadinServletService.lambda$1(CdmSpringVaadinServletService.java:56)
    at java.util.ArrayList.forEach(ArrayList.java:1249)
    at eu.etaxonomy.cdm.vaadin.server.CdmSpringVaadinServletService.requestEnd(CdmSpringVaadinServletService.java:56)
    at com.vaadin.server.VaadinService.handleRequest(VaadinService.java:1450)
    at com.vaadin.server.VaadinServlet.service(VaadinServlet.java:380)
Actions
Copy link
 #5

Updated by Andreas Kohlbecker over 6 years ago

  • Target version changed from Release 4.11 to Release 4.12
Actions
Copy link
 #6

Updated by Andreas Müller over 6 years ago

  • Status changed from Resolved to Feedback
Actions
Copy link
 #7

Updated by Andreas Kohlbecker over 6 years ago

  • Status changed from Feedback to Closed

reviewed by a.mueller: "Deine vorgeschlagenen Änderungen im ConversationHolder sind dort ja weiterhin drin. Lediglich in vaadin hast du wohl etwas reverted.
Kann man das Ticket evtl. doch schließen jetzt? "

Ok, dann ist also alles gut und du hast hiermit auch gleich das Review gemacht, also schließe ich das Ticket.

Actions
Copy link

Also available in: Atom PDF