fixxx

1 files changed, 34 insertions, 18 deletions

diff --git a/korganizer/calendarview.cpp b/korganizer/calendarview.cpp
index 65750af..426e8f9 100644
--- a/korganizer/calendarview.cpp
+++ b/korganizer/calendarview.cpp
@@ -663,2 +663,3 @@ void CalendarView::nextConflict( bool all, bool allday )
         test->setTagged( false );
+        test = testlist.next();
     }
@@ -669,18 +670,32 @@ void CalendarView::nextConflict( bool all, bool allday )
     Event * cE = 0;
+    Event * cE2 = 0;
     QPtrList<Event> testlist2 = testlist;
+    test = testlist.first();
+    bool skip = false;
     while ( test ) {
-        Event * test2 = testlist2.first();
-        while ( test2 ) {
-            if ( !test2->isTagged() ) {
-                if ( test->isOverlapping ( test2, &retVal, true ) ) {
-                    if ( ! found ) {
-                        if ( retVal >= startDT ) {
-                            conflict = retVal; 
-                            cE = test;
-                            found = true;
-                        }
-                    } else {
-                        if ( retVal >= startDT && retVal < conflict ) {
-                            conflict = retVal; 
-                            cE = test;
+        skip = false;
+        if ( !all ) skip = ( allday != test->doesFloat() );
+        if ( !skip ) {
+            Event * test2 = testlist2.first();
+            while ( test2 ) { 
+                skip = false;
+                if ( !all ) skip = ( allday != test2->doesFloat() );
+                if ( !skip ) {
+                    if ( !test2->isTagged() ) {
+                        if ( test->isOverlapping ( test2, &retVal, &startDT ) ) {
+                            //qDebug("overlap ");
+                            if ( ! found ) {
+                                if ( retVal >= startDT ) {
+                                    conflict = retVal; 
+                                    cE = test;
+                                    cE2 = test2;
+                                    found = true;
+                                }
+                            } else {
+                                if ( retVal >= startDT && retVal < conflict ) {
+                                    conflict = retVal; 
+                                    cE = test;
+                                    cE2 = test2;
+                                }
+                            }
                         }
@@ -688,4 +703,4 @@ void CalendarView::nextConflict( bool all, bool allday )
                 }
+                test2 = testlist2.next();
             }
-            test2 = testlist2.next();
         }
@@ -700,3 +715,3 @@ void CalendarView::nextConflict( bool all, bool allday )
         mViewManager->agendaView()->setStartHour( hour  );
-        topLevelWidget()->setCaption( i18n("Conflict %1 <-> %2"). arg( mConflictingEvent->summary().left( 20 ) ).arg( cE->summary().left( 20 ) ) );
+        topLevelWidget()->setCaption( i18n("Conflict %1 <-> %2"). arg( cE->summary().left( 20 ) ).arg( cE2->summary().left( 20 ) ) );
         return;
@@ -704,3 +719,3 @@ void CalendarView::nextConflict( bool all, bool allday )
 
-    topLevelWidget()->setCaption( i18n("No conflict found within the next two years") );
+    topLevelWidget()->setCaption( i18n("No conflict found") );
     qDebug("No conflict found ");
@@ -2796,5 +2811,6 @@ void CalendarView::checkConflictForEvent()
     Event * cE = 0;
+    QDateTime current = QDateTime::currentDateTime();
     while ( test ) {
         if ( !test->doesFloat() ) {
-            if ( mConflictingEvent->isOverlapping ( test, &retVal, true ) ) {
+            if ( mConflictingEvent->isOverlapping ( test, &retVal, &current ) ) {
                 if ( ! found ) {