app/core/CalendarOutboxTasks.cpp
| Line | Branch | Exec | Source |
|---|---|---|---|
| 1 | #include "CalendarOutbox.h" | ||
| 2 | #include "data/CalendarStore.h" | ||
| 3 | #include "data/ICalTodoCodec.h" | ||
| 4 | |||
| 5 | 115 | bool CalendarOutbox::saveTask(const CalendarTask &task) { | |
| 6 | 115 | auto *store = m_deps.store; | |
| 7 | 115 | auto stored = task; | |
| 8 |
1/1✓ Branch 3 → 4 taken 115 times.
|
115 | const auto previous = store->taskFor(task.uid, task.accountId, task.calendarPath); |
| 9 |
2/2✓ Branch 5 → 6 taken 86 times.
✓ Branch 5 → 18 taken 29 times.
|
115 | if (previous) { |
| 10 | 86 | stored.etag = previous->etag; stored.resourceHref = previous->resourceHref; | |
| 11 |
2/2✓ Branch 11 → 12 taken 57 times.
✓ Branch 11 → 14 taken 29 times.
|
86 | if (stored.icalRaw.isEmpty()) stored.icalRaw = previous->icalRaw; |
| 12 |
2/2✓ Branch 15 → 16 taken 79 times.
✓ Branch 15 → 18 taken 7 times.
|
86 | if (stored.resourceIcal.isEmpty()) stored.resourceIcal = previous->resourceIcal; |
| 13 | } | ||
| 14 |
1/1✓ Branch 18 → 19 taken 115 times.
|
115 | const QByteArray built = ICalTodoCodec::buildResource(stored); |
| 15 |
1/2✗ Branch 20 → 21 not taken.
✓ Branch 20 → 22 taken 115 times.
|
115 | if (built.isEmpty()) return false; |
| 16 | // S106-133: the stored row carries the VTODO just built, so its reminder | ||
| 17 | // times and its raw component agree. A row with the new due date, the | ||
| 18 | // old component and the old reminder times read as a manual reminder | ||
| 19 | // edit on the next anchor move (relative trigger frozen as an absolute | ||
| 20 | // stamp), and the reminder fired on the old day until a push. The | ||
| 21 | // resource text stays the server's: it is the merge base of a conflict. | ||
| 22 | 115 | CalendarTask rebuilt = stored; | |
| 23 |
1/1✓ Branch 23 → 24 taken 115 times.
|
115 | ICalTodoCodec::adoptPushedResource(&rebuilt, built); |
| 24 | 115 | stored.icalRaw = rebuilt.icalRaw; | |
| 25 | 115 | stored.alarmTimes = rebuilt.alarmTimes; | |
| 26 |
2/3✓ Branch 26 → 27 taken 115 times.
✗ Branch 27 → 28 not taken.
✓ Branch 27 → 29 taken 115 times.
|
115 | if (!store->beginTransaction()) return false; |
| 27 |
1/1✓ Branch 29 → 30 taken 115 times.
|
115 | bool ok = store->upsertTask(stored); |
| 28 |
6/13✓ Branch 33 → 34 taken 115 times.
✓ Branch 34 → 35 taken 115 times.
✗ Branch 34 → 37 not taken.
✓ Branch 35 → 36 taken 114 times.
✓ Branch 35 → 37 taken 1 time.
✓ Branch 38 → 39 taken 115 times.
✗ Branch 38 → 40 not taken.
✓ Branch 40 → 41 taken 115 times.
✗ Branch 40 → 42 not taken.
✗ Branch 77 → 78 not taken.
✗ Branch 77 → 79 not taken.
✗ Branch 81 → 82 not taken.
✗ Branch 81 → 83 not taken.
|
230 | ok = store->enqueueOutbox(task.accountId, task.calendarPath, task.uid, QStringLiteral("put"), stored.resourceHref, stored.etag) > 0 && ok; |
| 29 |
1/1✓ Branch 42 → 43 taken 115 times.
|
115 | auto entry = store->outboxEntry(task.accountId, task.calendarPath, task.uid); |
| 30 |
5/7✓ Branch 44 → 45 taken 115 times.
✗ Branch 44 → 61 not taken.
✓ Branch 55 → 56 taken 115 times.
✓ Branch 56 → 57 taken 115 times.
✗ Branch 56 → 59 not taken.
✓ Branch 57 → 58 taken 114 times.
✓ Branch 57 → 59 taken 1 time.
|
230 | if (entry) { entry->componentType = QStringLiteral("VTODO"); entry->dependencyKey.clear(); ok = store->updateOutbox(*entry) && ok; } |
| 31 |
3/3✓ Branch 61 → 62 taken 1 time.
✓ Branch 61 → 64 taken 114 times.
✓ Branch 62 → 63 taken 1 time.
|
115 | if (!ok) { store->rollbackTransaction(); return false; } |
| 32 |
2/3✓ Branch 64 → 65 taken 114 times.
✗ Branch 65 → 66 not taken.
✓ Branch 65 → 67 taken 114 times.
|
114 | if (!store->commitTransaction()) return false; |
| 33 |
2/2✓ Branch 67 → 68 taken 114 times.
✓ Branch 68 → 69 taken 114 times.
|
114 | publishChange(); requestFlush(); return true; |
| 34 | 115 | } | |
| 35 | |||
| 36 | 14 | bool CalendarOutbox::removeTask(const QString &uid, const QString &accountId, const QString &calendarPath) { | |
| 37 | 14 | auto *store = m_deps.store; | |
| 38 |
1/1✓ Branch 2 → 3 taken 14 times.
|
14 | const auto previous = store->taskFor(uid, accountId, calendarPath); |
| 39 |
1/2✗ Branch 4 → 5 not taken.
✓ Branch 4 → 6 taken 14 times.
|
14 | if (!previous) return false; |
| 40 |
1/1✓ Branch 6 → 7 taken 14 times.
|
14 | const auto existing = store->outboxEntry(accountId, calendarPath, uid); |
| 41 |
2/3✓ Branch 7 → 8 taken 14 times.
✗ Branch 8 → 9 not taken.
✓ Branch 8 → 10 taken 14 times.
|
14 | if (!store->beginTransaction()) return false; |
| 42 |
2/4✓ Branch 10 → 11 taken 14 times.
✗ Branch 11 → 12 not taken.
✓ Branch 11 → 14 taken 14 times.
✗ Branch 12 → 13 not taken.
|
14 | if (!store->deleteTask(uid, accountId, calendarPath)) { store->commitTransaction(); return false; } |
| 43 |
2/2✓ Branch 15 → 16 taken 8 times.
✓ Branch 15 → 18 taken 6 times.
|
14 | const QString etag = existing ? existing->baseEtag : previous->etag; |
| 44 |
9/11✓ Branch 22 → 23 taken 9 times.
✓ Branch 22 → 27 taken 5 times.
✓ Branch 23 → 24 taken 9 times.
✓ Branch 25 → 26 taken 8 times.
✓ Branch 25 → 27 taken 1 time.
✓ Branch 28 → 29 taken 9 times.
✓ Branch 28 → 30 taken 5 times.
✓ Branch 30 → 31 taken 8 times.
✓ Branch 30 → 33 taken 6 times.
✗ Branch 63 → 64 not taken.
✗ Branch 63 → 65 not taken.
|
14 | if (etag.isEmpty() && !m_inFlight.contains(key(accountId, calendarPath, uid))) { |
| 45 |
2/2✓ Branch 31 → 32 taken 8 times.
✓ Branch 32 → 54 taken 8 times.
|
8 | store->removeOutbox(accountId, calendarPath, uid); store->removeResource(accountId, calendarPath, uid); |
| 46 | } else { | ||
| 47 |
1/1✓ Branch 37 → 38 taken 6 times.
|
12 | store->enqueueOutbox(accountId, calendarPath, uid, QStringLiteral("delete"), previous->resourceHref, etag); |
| 48 |
2/3✓ Branch 40 → 41 taken 6 times.
✓ Branch 42 → 43 taken 6 times.
✗ Branch 42 → 52 not taken.
|
6 | if (auto entry = store->outboxEntry(accountId, calendarPath, uid)) { |
| 49 |
1/1✓ Branch 51 → 52 taken 6 times.
|
6 | entry->componentType = QStringLiteral("VTODO"); store->updateOutbox(*entry); |
| 50 | 6 | } | |
| 51 | } | ||
| 52 |
3/3✓ Branch 54 → 55 taken 14 times.
✓ Branch 55 → 56 taken 14 times.
✓ Branch 56 → 57 taken 14 times.
|
14 | store->commitTransaction(); publishChange(); requestFlush(); return true; |
| 53 | 14 | } | |
| 54 | |||
| 55 | 9 | bool CalendarOutbox::moveTask(const CalendarTask &source, const CalendarTask &target) { | |
| 56 | 9 | auto *store = m_deps.store; | |
| 57 |
5/11✓ Branch 3 → 4 taken 9 times.
✗ Branch 3 → 7 not taken.
✓ Branch 4 → 5 taken 9 times.
✗ Branch 6 → 7 not taken.
✓ Branch 6 → 8 taken 9 times.
✓ Branch 9 → 10 taken 9 times.
✗ Branch 9 → 11 not taken.
✗ Branch 11 → 12 not taken.
✓ Branch 11 → 13 taken 9 times.
✗ Branch 82 → 83 not taken.
✗ Branch 82 → 84 not taken.
|
9 | if (source.uid != target.uid || store->taskFor(target.uid, target.accountId, target.calendarPath)) return false; |
| 58 | 9 | auto moved = target; moved.etag.clear(); moved.resourceHref.clear(); | |
| 59 | // Persist both operations before dispatch. Source deletion waits until | ||
| 60 | // destination creation is confirmed, including across process restart. | ||
| 61 |
2/3✓ Branch 16 → 17 taken 9 times.
✗ Branch 17 → 18 not taken.
✓ Branch 17 → 19 taken 9 times.
|
9 | if (!store->beginTransaction()) return false; |
| 62 |
1/1✓ Branch 19 → 20 taken 9 times.
|
9 | bool ok = store->moveTask(source, moved); |
| 63 |
1/2✓ Branch 20 → 21 taken 9 times.
✗ Branch 20 → 70 not taken.
|
9 | if (ok) { |
| 64 |
1/1✓ Branch 26 → 27 taken 9 times.
|
18 | store->enqueueOutbox(target.accountId, target.calendarPath, target.uid, QStringLiteral("put"), {}, {}); |
| 65 |
2/3✓ Branch 31 → 32 taken 9 times.
✓ Branch 33 → 34 taken 9 times.
✗ Branch 33 → 43 not taken.
|
9 | if (auto entry = store->outboxEntry(target.accountId, target.calendarPath, target.uid)) { |
| 66 |
1/1✓ Branch 42 → 43 taken 9 times.
|
9 | entry->componentType = QStringLiteral("VTODO"); store->updateOutbox(*entry); |
| 67 | 9 | } | |
| 68 |
2/2✓ Branch 45 → 46 taken 5 times.
✓ Branch 45 → 70 taken 4 times.
|
9 | if (!source.etag.isEmpty()) { |
| 69 |
1/1✓ Branch 49 → 50 taken 5 times.
|
10 | store->enqueueOutbox(source.accountId, source.calendarPath, source.uid, QStringLiteral("delete"), source.resourceHref, source.etag); |
| 70 |
2/3✓ Branch 52 → 53 taken 5 times.
✓ Branch 54 → 55 taken 5 times.
✗ Branch 54 → 68 not taken.
|
5 | if (auto entry = store->outboxEntry(source.accountId, source.calendarPath, source.uid)) { |
| 71 | 5 | entry->componentType = QStringLiteral("VTODO"); | |
| 72 |
2/2✓ Branch 62 → 63 taken 5 times.
✓ Branch 67 → 68 taken 5 times.
|
5 | entry->dependencyKey = key(target.accountId, target.calendarPath, target.uid); store->updateOutbox(*entry); |
| 73 | 5 | } | |
| 74 | } | ||
| 75 | } | ||
| 76 |
1/3✗ Branch 70 → 71 not taken.
✓ Branch 70 → 73 taken 9 times.
✗ Branch 71 → 72 not taken.
|
9 | if (!ok) { store->rollbackTransaction(); return false; } |
| 77 |
2/3✓ Branch 73 → 74 taken 9 times.
✗ Branch 74 → 75 not taken.
✓ Branch 74 → 76 taken 9 times.
|
9 | if (!store->commitTransaction()) return false; |
| 78 |
2/2✓ Branch 76 → 77 taken 9 times.
✓ Branch 77 → 78 taken 9 times.
|
9 | publishChange(); requestFlush(); return true; |
| 79 | 9 | } | |
| 80 | |||
| 81 | |||
| 82 | 514 | void CalendarOutbox::publishChange() { | |
| 83 |
2/2✓ Branch 2 → 3 taken 66 times.
✓ Branch 2 → 4 taken 448 times.
|
514 | if (m_batchDepth > 0) m_batchChanged = true; |
| 84 | 448 | else emit storeChanged(); | |
| 85 | 514 | } | |
| 86 | 42 | void CalendarOutbox::endBatch() { | |
| 87 |
3/6✓ Branch 2 → 3 taken 42 times.
✗ Branch 2 → 4 not taken.
✗ Branch 3 → 4 not taken.
✓ Branch 3 → 5 taken 42 times.
✗ Branch 6 → 7 not taken.
✓ Branch 6 → 8 taken 42 times.
|
42 | if (m_batchDepth <= 0 || --m_batchDepth > 0) return; |
| 88 |
1/2✓ Branch 8 → 9 taken 42 times.
✗ Branch 8 → 10 not taken.
|
42 | if (m_batchChanged) { m_batchChanged = false; emit storeChanged(); } |
| 89 | 42 | requestFlush(); | |
| 90 | } | ||
| 91 |