← Back to team overview

touch-packages team mailing list archive

[Bug 1462989] [NEW] containsItems returned wrong value when checking all day event

 

Public bug reported:

Related issue to 
https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1437305

For all day event(startTime == endTime), if all day event created at 2015-06-08
so src/imports/organizer/qdeclarativeorganizermodel.cpp[line:966]
    QList<bool> QDeclarativeOrganizerModel::containsItems(const QDateTime &start, const QDateTime &end, int interval)
   {
    ....
    ....
    startTime = item->itemStartTime();  //(2015-06-08  00:00:00.000)
    endTime = item->itemEndTime();      //(2015-06-08  00:00:00.000)

    // if dateTime.at(i) == (2015-06-07  00:00:00.000), it will return true, actually which would return false since all day event should occur at 2015-06-08 after this line 
    if (!endTime.isNull() && endTime > dateTime.at(i) && endTime <= dateTime.at(i + 1)) {
          occupiedTimeSlots[i] = true;
           break;
    }

** Affects: qtpim-opensource-src (Ubuntu)
     Importance: Undecided
         Status: New

** Patch added: "1437305.patch"
   https://bugs.launchpad.net/bugs/1462989/+attachment/4411510/+files/1437305.patch

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to qtpim-opensource-src in
Ubuntu.
https://bugs.launchpad.net/bugs/1462989

Title:
  containsItems returned wrong value when checking all day event

Status in qtpim-opensource-src package in Ubuntu:
  New

Bug description:
  Related issue to 
  https://bugs.launchpad.net/ubuntu-calendar-app/+bug/1437305

  For all day event(startTime == endTime), if all day event created at 2015-06-08
  so src/imports/organizer/qdeclarativeorganizermodel.cpp[line:966]
      QList<bool> QDeclarativeOrganizerModel::containsItems(const QDateTime &start, const QDateTime &end, int interval)
     {
      ....
      ....
      startTime = item->itemStartTime();  //(2015-06-08  00:00:00.000)
      endTime = item->itemEndTime();      //(2015-06-08  00:00:00.000)

      // if dateTime.at(i) == (2015-06-07  00:00:00.000), it will return true, actually which would return false since all day event should occur at 2015-06-08 after this line 
      if (!endTime.isNull() && endTime > dateTime.at(i) && endTime <= dateTime.at(i + 1)) {
            occupiedTimeSlots[i] = true;
             break;
      }

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/qtpim-opensource-src/+bug/1462989/+subscriptions


Follow ups

References