Merge lp:~boiko/texts-scope/fix_event_parsing into lp:texts-scope

[Gustavo Pichorim Boiko]

In the second step, process only the new events, not the whole list.

In the second step, process only the new events, not the whole list.

[Kyle Nitzsche (knitzsche)]

like the calls-scope MR, this fix is needed but is not quite enough since we still get duplicated texts.

See https://code.launchpad.net/~knitzsche/texts-scope/fix_event_parsing_fixMR

I will merge that into trunk (due to time constraints) and release as 1.5 for validation.

review: Approve

[Kyle Nitzsche (knitzsche)]

all merged to trunk with additional noted fix and etc.

[Gustavo Pichorim Boiko (boiko)]

In fact looking at the code, that while (going) thing is not needed anymore, maybe the right fix would be to remove it, but as you fixed and merged that already, let's fix that in a later version.

Unmerged revisions

=== modified file 'src/query.cpp'
--- src/query.cpp	2015-10-16 21:51:16 +0000
+++ src/query.cpp	2015-10-28 18:51:06 +0000
@@ -155,7 +155,7 @@
                 ev_l.append(l);
 
                 // and process them too
-                Q_FOREACH(const Event &event, ev_l) {
+                Q_FOREACH(const Event &event, l) {
                     QDate ev_date = event.timestamp().date();
 
                     // if the date is newer than the end date, keep searching