[nemo-qml-plugin-calendar] Always return stored CalendarEvent object …

…even if not part of any notebook anymore.

The CalendarEvent objects created by the manager are stored in
the mEventObjects private array. This array is always growing.
When an event is not listed anymore in the mEvents array
because the notebook it belongs to is not visible anymore,
fetching its CalendarEvent object should still be valid.
So move the array search before checking that the event is
currently part of any notebook.

src/calendarmanager.cpp

@@ -135,16 +135,16 @@ void CalendarManager::setDefaultNotebook(const QString &notebookUid)
 
 CalendarEvent* CalendarManager::eventObject(const QString &eventUid, const KDateTime &recurrenceId)
 {
-    CalendarData::Event event = getEvent(eventUid, recurrenceId);
-    if (event.isValid()) {
-        QMultiHash<QString, CalendarEvent *>::iterator it = mEventObjects.find(eventUid);
-        while (it != mEventObjects.end() && it.key() == eventUid) {
-            if ((*it)->recurrenceId() == recurrenceId) {
-                return *it;
-            }
-            ++it;
+    QMultiHash<QString, CalendarEvent *>::iterator it = mEventObjects.find(eventUid);
+    while (it != mEventObjects.end() && it.key() == eventUid) {
+        if ((*it)->recurrenceId() == recurrenceId) {
+            return *it;
         }
+        ++it;
+    }
 
+    CalendarData::Event event = getEvent(eventUid, recurrenceId);
+    if (event.isValid()) {
         CalendarEvent *calendarEvent = new CalendarEvent(this, eventUid, recurrenceId);
         mEventObjects.insert(eventUid, calendarEvent);
         return calendarEvent;