From bd1d9356b5904db6c2023c4d3eb9c6cecdbeb2aa Mon Sep 17 00:00:00 2001
From: Tobias Doerffel <tobias.doerffel@gmail.com>
Date: Sun, 30 Mar 2014 12:31:47 +0200
Subject: [PATCH] TrackContentObjectView: add journal checkpoint in remove()
 rather than close()

The remove() slot is only called upon user interaction (i.e. right click)
and thus the correct place where to make checkpoints.
---
 src/core/track.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/core/track.cpp b/src/core/track.cpp
index 5bdf09f7f..82d040eca 100644
--- a/src/core/track.cpp
+++ b/src/core/track.cpp
@@ -324,8 +324,6 @@ bool trackContentObjectView::fixedTCOs()
  */
 bool trackContentObjectView::close()
 {
-	m_trackView->getTrack()->addJournalCheckPoint();
-
 	m_trackView->getTrackContentWidget()->removeTCOView( this );
 	return QWidget::close();
 }
@@ -342,6 +340,8 @@ bool trackContentObjectView::close()
  */
 void trackContentObjectView::remove()
 {
+	m_trackView->getTrack()->addJournalCheckPoint();
+
 	// delete ourself
 	close();
 	m_tco->deleteLater();
@@ -2246,7 +2246,6 @@ void trackView::mouseMoveEvent( QMouseEvent * _me )
 		// a track-widget not equal to ourself?
 		if( track_at_y != NULL && track_at_y != this )
 		{
-			addJournalCheckPoint();
 			// then move us up/down there!
 			if( _me->y() < 0 )
 			{