0009534: whole day events display problem
authorAlexander Stintzing <a.stintzing@metaways.de>
Fri, 9 May 2014 08:52:02 +0000 (10:52 +0200)
committerPhilipp Schüle <p.schuele@metaways.de>
Wed, 14 May 2014 13:57:50 +0000 (15:57 +0200)
Seit einiger Zeit fällt mir auf, dass ganztägige Termine nicht mehr eindeutig den
Tagen zuzuordnen sind. Das heißt (s. Screenshot) das die Termine über die Tage
hinaus gezogen sind.
Außerdem tauchten bspw. am 16.1. der ganztägige Termin dort doppelt auf. Leider
ist das auf dem momentanen Screenshot nicht (mehr) zu sehen, weil der Termin
urplötzlich nicht mehr doppelt angezeigt wird.

Fakt ist: Es wurden bis eben noch zwei ganztägige Termine angezeigt, obwohl sich
dort nur einer befinden sollte. Es machte hierbei keinen Unterschied welchen Termin
ich anklickte, ich kam offensichtlich immer zur gleichen Detailansicht.

https://forge.tine20.org/mantisbt/view.php?id=9534

Change-Id: I6a2dcb50b8e71cc9384a499d437d2932c895a8c5
Reviewed-on: http://gerrit.tine20.com/customers/614
Tested-by: Jenkins CI (http://ci.tine20.com/)
Reviewed-by: Philipp Schüle <p.schuele@metaways.de>
Tested-by: Philipp Schüle <p.schuele@metaways.de>
tine20/Calendar/js/DaysView.js

index fb8f284..d3b72f4 100644 (file)
@@ -304,7 +304,7 @@ Ext.extend(Tine.Calendar.DaysView, Ext.Container, {
                     
                     if (! event.get('is_all_day_event') && targetDate.is_all_day_event && event.duration < Date.msDAY) {
                         // draged from scroller -> dropped to allDay and duration less than a day
-                        event.set('dtend', targetDate.add(Date.DAY, 1));
+                        event.set('dtend', targetDate.add(Date.DAY, 1).add(Date.SECOND, -1));
                     } else if (event.get('is_all_day_event') && !targetDate.is_all_day_event) {
                         // draged from allDay -> droped to scroller will be resetted to hone hour
                         event.set('dtend', targetDate.add(Date.HOUR, 1));
@@ -1156,7 +1156,8 @@ Ext.extend(Tine.Calendar.DaysView, Ext.Container, {
                 var timePart = this.timeScale.getAt(parts[2]);
                 date = date.add(Date.MINUTE, timePart.get('minutes'));
                 date.is_all_day_event = false;
-            }   
+            }
+            
             return date;
         }
     },
@@ -1532,4 +1533,4 @@ Ext.extend(Tine.Calendar.DaysView, Ext.Container, {
     }
 });
 
-Ext.reg('Tine.Calendar.DaysView', Tine.Calendar.DaysView);
\ No newline at end of file
+Ext.reg('Tine.Calendar.DaysView', Tine.Calendar.DaysView);