presentation/QuickTaskActions.cpp
| Line | Branch | Exec | Source |
|---|---|---|---|
| 1 | #include "QuickTaskActions.h" | ||
| 2 | #include "QuickTaskModel.h" | ||
| 3 | #include "WorkspaceState.h" | ||
| 4 | #include "TaskConsoleInput.h" | ||
| 5 | #include "TaskEditorFields.h" | ||
| 6 | #include "TaskListLayout.h" | ||
| 7 | #include "PimEditorTime.h" | ||
| 8 | #include "app/core/MailSessionCore.h" | ||
| 9 | #include "app/core/AccountSession.h" | ||
| 10 | #include "app/core/CalendarOutbox.h" | ||
| 11 | #include "app/core/CalendarSyncCoordinator.h" | ||
| 12 | #include "data/CalendarStore.h" | ||
| 13 | #include "data/MailCache.h" | ||
| 14 | #include "data/TaskRecurrence.h" | ||
| 15 | #include "util/MessageId.h" | ||
| 16 | #include <QTimer> | ||
| 17 | #include <QSettings> | ||
| 18 | #include <QUuid> | ||
| 19 | #include <QTimeZone> | ||
| 20 | #include <QScopeGuard> | ||
| 21 | #include <algorithm> | ||
| 22 | #include <QLoggingCategory> | ||
| 23 | #include <QRegularExpression> | ||
| 24 |
4/7✓ Branch 2 → 3 taken 3 times.
✓ Branch 2 → 8 taken 26 times.
✓ Branch 4 → 5 taken 3 times.
✗ Branch 4 → 8 not taken.
✓ Branch 5 → 6 taken 3 times.
✗ Branch 10 → 11 not taken.
✗ Branch 10 → 12 not taken.
|
29 | Q_LOGGING_CATEGORY(lcTaskActions, "mailjd.tasks.actions") |
| 25 | 795 | QuickTaskActions::QuickTaskActions(MailSessionCore &session, WorkspaceState &workspace, QuickTaskModel &model, Deps deps, QObject *parent) | |
| 26 | 795 | : QObject(parent), m_session(session), m_workspace(workspace), m_tasks(model), m_deps(std::move(deps)) {} | |
| 27 | 108 | QVariantMap QuickTaskActions::taskRecord(const QString &uid, const QString &accountId, const QString &calendarPath) const { | |
| 28 | 108 | auto *sync = m_session.calendarSync(); | |
| 29 |
2/3✓ Branch 3 → 4 taken 108 times.
✗ Branch 3 → 5 not taken.
✓ Branch 4 → 6 taken 108 times.
|
108 | const auto task = sync ? sync->taskFor(uid, accountId, calendarPath) : std::nullopt; |
| 30 |
3/3✓ Branch 7 → 8 taken 103 times.
✓ Branch 7 → 10 taken 5 times.
✓ Branch 9 → 11 taken 103 times.
|
216 | return task ? QuickTaskModel::record(*task) : QVariantMap{}; |
| 31 | 108 | } | |
| 32 | |||
| 33 | 88 | QVariantMap QuickTaskActions::taskAt(int row) const { | |
| 34 | 88 | return m_tasks.taskAt(row); | |
| 35 | } | ||
| 36 | |||
| 37 | 16 | QVariantMap QuickTaskActions::createTaskFromInput(const QString &input) { | |
| 38 |
2/4✓ Branch 8 → 9 taken 16 times.
✓ Branch 8 → 10 taken 16 times.
✗ Branch 283 → 284 not taken.
✗ Branch 283 → 285 not taken.
|
64 | QVariantMap rejected{{QStringLiteral("ok"), false}}; |
| 39 |
1/2✗ Branch 13 → 14 not taken.
✓ Branch 13 → 26 taken 16 times.
|
16 | if (input.size() > 4096) { |
| 40 | ✗ | rejected.insert(QStringLiteral("error"), tr("The task title is too long.")); | |
| 41 | ✗ | return rejected; | |
| 42 | } | ||
| 43 | 16 | auto *sync = m_session.calendarSync(); | |
| 44 |
1/2✗ Branch 27 → 28 not taken.
✓ Branch 27 → 40 taken 16 times.
|
16 | if (!sync) { |
| 45 | ✗ | rejected.insert(QStringLiteral("error"), tr("No calendar storage available.")); | |
| 46 | ✗ | return rejected; | |
| 47 | } | ||
| 48 |
1/1✓ Branch 40 → 41 taken 16 times.
|
16 | const auto parts = TaskConsoleInput::split(input); |
| 49 |
1/1✓ Branch 41 → 42 taken 16 times.
|
16 | const QString summary = parts.summary.trimmed(); |
| 50 |
2/2✓ Branch 43 → 44 taken 1 time.
✓ Branch 43 → 56 taken 15 times.
|
16 | if (summary.isEmpty()) { |
| 51 |
2/2✓ Branch 44 → 45 taken 1 time.
✓ Branch 49 → 50 taken 1 time.
|
2 | rejected.insert(QStringLiteral("error"), tr("Enter a task title.")); |
| 52 | 1 | return rejected; | |
| 53 | } | ||
| 54 |
1/1✓ Branch 56 → 57 taken 15 times.
|
15 | const int priority = TaskConsoleInput::parsePriority(parts.priorityHint); |
| 55 |
2/2✓ Branch 57 → 58 taken 1 time.
✓ Branch 57 → 70 taken 14 times.
|
15 | if (priority < 0) { |
| 56 |
2/2✓ Branch 58 → 59 taken 1 time.
✓ Branch 63 → 64 taken 1 time.
|
2 | rejected.insert(QStringLiteral("error"), tr("Choose High, Normal, Low or None.")); return rejected; |
| 57 | } | ||
| 58 | 14 | QDateTime due; | |
| 59 |
2/2✓ Branch 72 → 73 taken 6 times.
✓ Branch 72 → 91 taken 8 times.
|
14 | if (!parts.dueHint.isEmpty()) { |
| 60 |
2/2✓ Branch 73 → 74 taken 6 times.
✓ Branch 74 → 75 taken 6 times.
|
6 | due = TaskConsoleInput::parseDue(parts.dueHint); |
| 61 |
3/3✓ Branch 77 → 78 taken 6 times.
✓ Branch 78 → 79 taken 1 time.
✓ Branch 78 → 91 taken 5 times.
|
6 | if (!due.isValid()) { |
| 62 |
1/1✓ Branch 84 → 85 taken 1 time.
|
2 | rejected.insert(QStringLiteral("error"), |
| 63 |
1/1✓ Branch 79 → 80 taken 1 time.
|
2 | tr("Enter the date as 2026-09-15 or 15.9.2026.")); |
| 64 | 1 | return rejected; | |
| 65 | } | ||
| 66 | } | ||
| 67 |
1/1✓ Branch 91 → 92 taken 13 times.
|
13 | const auto calendars = sync->calendars(); |
| 68 |
1/2✗ Branch 93 → 94 not taken.
✓ Branch 93 → 106 taken 13 times.
|
13 | if (calendars.isEmpty()) { |
| 69 | ✗ | rejected.insert(QStringLiteral("error"), | |
| 70 | ✗ | tr("No calendar exists yet; create one in the DAV settings first.")); | |
| 71 | ✗ | return rejected; | |
| 72 | } | ||
| 73 | 13 | const CalendarInfo *target = nullptr; | |
| 74 |
2/2✓ Branch 107 → 108 taken 9 times.
✓ Branch 107 → 125 taken 4 times.
|
13 | if (!parts.calendarHint.isEmpty()) { |
| 75 |
1/1✓ Branch 108 → 109 taken 9 times.
|
9 | target = matchTaskList(calendars, parts.calendarHint); |
| 76 |
2/2✓ Branch 109 → 110 taken 2 times.
✓ Branch 109 → 127 taken 7 times.
|
9 | if (!target) { |
| 77 |
3/3✓ Branch 110 → 111 taken 2 times.
✓ Branch 112 → 113 taken 2 times.
✓ Branch 117 → 118 taken 2 times.
|
6 | rejected.insert(QStringLiteral("error"), tr("No calendar matches '%1'.").arg(parts.calendarHint)); |
| 78 | 2 | return rejected; | |
| 79 | } | ||
| 80 | } else { | ||
| 81 |
1/1✓ Branch 125 → 126 taken 4 times.
|
4 | target = defaultTarget(calendars); |
| 82 | } | ||
| 83 |
5/9✓ Branch 127 → 128 taken 11 times.
✗ Branch 127 → 131 not taken.
✓ Branch 128 → 129 taken 11 times.
✗ Branch 128 → 131 not taken.
✓ Branch 129 → 130 taken 11 times.
✗ Branch 130 → 131 not taken.
✓ Branch 130 → 132 taken 11 times.
✗ Branch 133 → 134 not taken.
✓ Branch 133 → 146 taken 11 times.
|
11 | if (!target || target->readOnly || !target->supportsTasks()) { |
| 84 | ✗ | rejected.insert(QStringLiteral("error"), tr("Choose a writable task list in DAV settings.")); return rejected; | |
| 85 | } | ||
| 86 | 11 | CalendarTask task; | |
| 87 | 11 | task.tags = parts.tags; | |
| 88 |
2/2✓ Branch 148 → 149 taken 11 times.
✓ Branch 149 → 150 taken 11 times.
|
11 | task.uid = QUuid::createUuid().toString(QUuid::WithoutBraces); |
| 89 | 11 | task.calendarPath = target->path; | |
| 90 | 11 | task.accountId = target->accountId; | |
| 91 | 11 | task.summary = summary; | |
| 92 | 11 | task.priority = priority; | |
| 93 | 11 | task.status = QStringLiteral("NEEDS-ACTION"); | |
| 94 |
1/1✓ Branch 161 → 162 taken 11 times.
|
11 | task.created = QDateTime::currentDateTimeUtc(); |
| 95 |
10/17✓ Branch 164 → 165 taken 11 times.
✓ Branch 165 → 166 taken 6 times.
✓ Branch 165 → 173 taken 5 times.
✓ Branch 168 → 169 taken 6 times.
✗ Branch 168 → 172 not taken.
✗ Branch 171 → 172 not taken.
✓ Branch 171 → 173 taken 6 times.
✓ Branch 174 → 175 taken 6 times.
✓ Branch 174 → 176 taken 5 times.
✓ Branch 176 → 177 taken 6 times.
✓ Branch 176 → 178 taken 5 times.
✗ Branch 178 → 179 not taken.
✓ Branch 178 → 194 taken 11 times.
✗ Branch 404 → 405 not taken.
✗ Branch 404 → 406 not taken.
✗ Branch 409 → 410 not taken.
✗ Branch 409 → 411 not taken.
|
11 | if (!due.isValid() && (m_tasks.view() == "today" || m_tasks.view() == "week")) { |
| 96 | ✗ | QDate day = QDate::currentDate(); | |
| 97 | ✗ | if (m_tasks.view() == "week") day = day.addDays(6 - (day.dayOfWeek() - m_tasks.weekStart() + 7) % 7); | |
| 98 | ✗ | due = QDateTime(day, QTime(0,0), QTimeZone::utc()); | |
| 99 | } | ||
| 100 | 11 | const auto selected = m_workspace.mailId(); | |
| 101 |
8/11✓ Branch 198 → 199 taken 9 times.
✓ Branch 198 → 202 taken 2 times.
✓ Branch 199 → 200 taken 9 times.
✓ Branch 200 → 201 taken 6 times.
✓ Branch 200 → 202 taken 3 times.
✓ Branch 203 → 204 taken 11 times.
✗ Branch 203 → 205 not taken.
✓ Branch 205 → 206 taken 6 times.
✓ Branch 205 → 217 taken 5 times.
✗ Branch 420 → 421 not taken.
✗ Branch 420 → 422 not taken.
|
11 | if (m_workspace.area() == "mail" && m_deps.acceptsIdentity(selected)) { |
| 102 |
1/1✓ Branch 207 → 208 taken 6 times.
|
6 | const auto header = m_session.cache()->header(selected.folderId, selected.uid); |
| 103 |
1/2✓ Branch 209 → 210 taken 6 times.
✗ Branch 209 → 215 not taken.
|
6 | if (header) |
| 104 |
1/1✓ Branch 211 → 212 taken 6 times.
|
6 | task.relatedMessageId = MessageId::canonicalFirst(header->messageId); |
| 105 | 6 | } | |
| 106 |
3/3✓ Branch 217 → 218 taken 11 times.
✓ Branch 218 → 219 taken 5 times.
✓ Branch 218 → 233 taken 6 times.
|
11 | if (due.isValid()) { |
| 107 |
1/1✓ Branch 221 → 222 taken 5 times.
|
5 | task.dueAllDay = !parts.dueHint.contains(QLatin1Char(':')); |
| 108 |
8/10✓ Branch 222 → 223 taken 4 times.
✓ Branch 222 → 227 taken 1 time.
✓ Branch 223 → 224 taken 4 times.
✓ Branch 224 → 225 taken 4 times.
✓ Branch 225 → 226 taken 4 times.
✓ Branch 226 → 228 taken 4 times.
✓ Branch 230 → 231 taken 4 times.
✓ Branch 230 → 232 taken 1 time.
✗ Branch 432 → 433 not taken.
✗ Branch 432 → 434 not taken.
|
10 | task.due = task.dueAllDay ? QDateTime(due.date(), QTime(0, 0), QTimeZone::utc()) |
| 109 | 5 | : due; | |
| 110 | } | ||
| 111 |
1/1✓ Branch 233 → 234 taken 11 times.
|
11 | addDefaultReminder(&task); |
| 112 |
2/3✓ Branch 234 → 235 taken 11 times.
✗ Branch 235 → 236 not taken.
✓ Branch 235 → 248 taken 11 times.
|
11 | if (!sync->saveTask(task, true)) { |
| 113 | ✗ | rejected.insert(QStringLiteral("error"), tr("Could not save this task. Try again.")); | |
| 114 | ✗ | return rejected; | |
| 115 | } | ||
| 116 |
2/2✓ Branch 248 → 249 taken 11 times.
✓ Branch 249 → 250 taken 11 times.
|
11 | setLastList(calendarSelectionKey(task.accountId, task.calendarPath)); |
| 117 |
4/6✓ Branch 253 → 254 taken 11 times.
✓ Branch 254 → 255 taken 11 times.
✓ Branch 258 → 259 taken 11 times.
✓ Branch 258 → 260 taken 11 times.
✗ Branch 458 → 459 not taken.
✗ Branch 458 → 460 not taken.
|
22 | remember(tr("Task added"), {{task, false}}); |
| 118 | return {{"ok", true}, {"uid", task.uid}, {"accountId", task.accountId}, {"calendarPath", task.calendarPath}, | ||
| 119 |
2/4✓ Branch 269 → 270 taken 66 times.
✓ Branch 269 → 271 taken 11 times.
✗ Branch 472 → 473 not taken.
✗ Branch 472 → 474 not taken.
|
77 | {"taskKey", taskSelectionKey(task)}, {"title", task.summary}}; |
| 120 | 86 | } | |
| 121 | |||
| 122 | 24 | QVariantMap QuickTaskActions::saveTask(const QVariantMap &fields) { | |
| 123 |
2/4✓ Branch 8 → 9 taken 24 times.
✓ Branch 8 → 10 taken 24 times.
✗ Branch 479 → 480 not taken.
✗ Branch 479 → 481 not taken.
|
96 | QVariantMap rejected{{QStringLiteral("ok"), false}}; |
| 124 | 24 | auto *sync = m_session.calendarSync(); | |
| 125 |
1/2✗ Branch 13 → 14 not taken.
✓ Branch 13 → 26 taken 24 times.
|
24 | if (!sync) { |
| 126 | ✗ | rejected.insert(QStringLiteral("error"), tr("No calendar storage available.")); | |
| 127 | ✗ | return rejected; | |
| 128 | } | ||
| 129 |
5/5✓ Branch 27 → 28 taken 24 times.
✓ Branch 28 → 29 taken 24 times.
✓ Branch 29 → 30 taken 24 times.
✓ Branch 35 → 36 taken 6 times.
✓ Branch 35 → 175 taken 18 times.
|
24 | if (fields.value("uid").toString().isEmpty()) { |
| 130 |
2/2✓ Branch 37 → 38 taken 6 times.
✓ Branch 38 → 39 taken 6 times.
|
6 | CalendarTask task; task.uid = QUuid::createUuid().toString(QUuid::WithoutBraces); |
| 131 |
6/6✓ Branch 42 → 43 taken 6 times.
✓ Branch 43 → 44 taken 6 times.
✓ Branch 44 → 45 taken 6 times.
✓ Branch 51 → 52 taken 6 times.
✓ Branch 52 → 53 taken 6 times.
✓ Branch 53 → 54 taken 6 times.
|
6 | task.accountId = fields.value("targetAccountId").toString(); task.calendarPath = fields.value("targetCalendarPath").toString(); |
| 132 |
2/2✓ Branch 59 → 60 taken 6 times.
✓ Branch 60 → 61 taken 6 times.
|
6 | task.status = "NEEDS-ACTION"; task.created = QDateTime::currentDateTimeUtc(); |
| 133 | 6 | bool writable = false; | |
| 134 |
11/16✓ Branch 63 → 64 taken 6 times.
✓ Branch 64 → 65 taken 6 times.
✓ Branch 65 → 66 taken 6 times.
✓ Branch 69 → 70 taken 6 times.
✗ Branch 69 → 76 not taken.
✓ Branch 71 → 72 taken 6 times.
✗ Branch 71 → 76 not taken.
✓ Branch 72 → 73 taken 6 times.
✓ Branch 73 → 74 taken 6 times.
✗ Branch 73 → 76 not taken.
✓ Branch 74 → 75 taken 6 times.
✗ Branch 74 → 76 not taken.
✓ Branch 77 → 78 taken 6 times.
✗ Branch 77 → 79 not taken.
✓ Branch 81 → 67 taken 6 times.
✓ Branch 81 → 82 taken 6 times.
|
12 | for (const auto &list : sync->calendars()) if (list.accountId == task.accountId && list.path == task.calendarPath && list.supportsTasks() && !list.readOnly) writable = true; |
| 135 |
1/6✗ Branch 83 → 84 not taken.
✓ Branch 83 → 93 taken 6 times.
✗ Branch 89 → 90 not taken.
✗ Branch 89 → 91 not taken.
✗ Branch 544 → 545 not taken.
✗ Branch 544 → 546 not taken.
|
6 | if (!writable) return {{"ok", false}, {"error", tr("Choose a writable task list.")}}; |
| 136 |
5/10✓ Branch 94 → 95 taken 6 times.
✓ Branch 95 → 96 taken 6 times.
✓ Branch 96 → 97 taken 6 times.
✓ Branch 97 → 98 taken 6 times.
✗ Branch 104 → 105 not taken.
✓ Branch 104 → 114 taken 6 times.
✗ Branch 110 → 111 not taken.
✗ Branch 110 → 112 not taken.
✗ Branch 571 → 572 not taken.
✗ Branch 571 → 573 not taken.
|
6 | if (fields.value("summary").toString().trimmed().isEmpty()) return {{"ok", false}, {"error", tr("Enter a task title.")}}; |
| 137 |
2/7✓ Branch 114 → 115 taken 6 times.
✗ Branch 115 → 116 not taken.
✓ Branch 115 → 125 taken 6 times.
✗ Branch 121 → 122 not taken.
✗ Branch 121 → 123 not taken.
✗ Branch 585 → 586 not taken.
✗ Branch 585 → 587 not taken.
|
6 | if (!TaskEditorFields::apply(task, fields)) return {{"ok", false}, {"error", tr("Check the title and dates, then try again.")}}; |
| 138 |
2/7✓ Branch 125 → 126 taken 6 times.
✗ Branch 126 → 127 not taken.
✓ Branch 126 → 136 taken 6 times.
✗ Branch 132 → 133 not taken.
✗ Branch 132 → 134 not taken.
✗ Branch 599 → 600 not taken.
✗ Branch 599 → 601 not taken.
|
6 | if (!validParent(task, task.parentUid)) return {{"ok", false}, {"error", tr("Choose a parent in this list without creating a loop.")}}; |
| 139 |
5/5✓ Branch 136 → 137 taken 6 times.
✓ Branch 137 → 138 taken 6 times.
✓ Branch 139 → 140 taken 4 times.
✓ Branch 139 → 141 taken 2 times.
✓ Branch 140 → 141 taken 4 times.
|
6 | if (!fields.contains("alarmTimes")) addDefaultReminder(&task); |
| 140 |
2/7✓ Branch 141 → 142 taken 6 times.
✗ Branch 142 → 143 not taken.
✓ Branch 142 → 152 taken 6 times.
✗ Branch 148 → 149 not taken.
✗ Branch 148 → 150 not taken.
✗ Branch 616 → 617 not taken.
✗ Branch 616 → 618 not taken.
|
6 | if (!sync->saveTask(task, true)) return {{"ok", false}, {"error", tr("Could not save this task. Try again.")}}; |
| 141 |
6/8✓ Branch 152 → 153 taken 6 times.
✓ Branch 153 → 154 taken 6 times.
✓ Branch 157 → 158 taken 6 times.
✓ Branch 158 → 159 taken 6 times.
✓ Branch 162 → 163 taken 6 times.
✓ Branch 162 → 164 taken 6 times.
✗ Branch 638 → 639 not taken.
✗ Branch 638 → 640 not taken.
|
18 | setLastList(calendarSelectionKey(task.accountId, task.calendarPath)); remember(tr("Task added"), {{task, false}}); |
| 142 |
2/4✓ Branch 170 → 171 taken 24 times.
✓ Branch 170 → 172 taken 6 times.
✗ Branch 652 → 653 not taken.
✗ Branch 652 → 654 not taken.
|
30 | return {{"ok", true}, {"uid", task.uid}, {"accountId", task.accountId}, {"calendarPath", task.calendarPath}}; |
| 143 | 6 | } | |
| 144 |
2/2✓ Branch 191 → 192 taken 18 times.
✓ Branch 192 → 193 taken 18 times.
|
54 | const auto stored = sync->taskFor(fields.value(QStringLiteral("uid")).toString(), |
| 145 |
2/2✓ Branch 185 → 186 taken 18 times.
✓ Branch 186 → 187 taken 18 times.
|
54 | fields.value(QStringLiteral("accountId")).toString(), |
| 146 |
3/3✓ Branch 179 → 180 taken 18 times.
✓ Branch 180 → 181 taken 18 times.
✓ Branch 193 → 194 taken 18 times.
|
72 | fields.value(QStringLiteral("calendarPath")).toString()); |
| 147 |
1/2✗ Branch 210 → 211 not taken.
✓ Branch 210 → 222 taken 18 times.
|
18 | if (!stored) { |
| 148 | ✗ | rejected.insert(QStringLiteral("error"), tr("The task no longer exists.")); | |
| 149 | ✗ | return rejected; | |
| 150 | } | ||
| 151 | 18 | CalendarTask task = *stored; | |
| 152 |
3/3✓ Branch 224 → 225 taken 18 times.
✓ Branch 225 → 226 taken 4 times.
✓ Branch 225 → 237 taken 14 times.
|
18 | if (!TaskEditorFields::apply(task, fields)) { |
| 153 |
1/1✓ Branch 231 → 232 taken 4 times.
|
8 | rejected.insert(QStringLiteral("error"), |
| 154 |
1/1✓ Branch 226 → 227 taken 4 times.
|
8 | tr("Check the title and dates, then try again.")); |
| 155 | 4 | return rejected; | |
| 156 | } | ||
| 157 |
2/7✓ Branch 237 → 238 taken 14 times.
✗ Branch 238 → 239 not taken.
✓ Branch 238 → 248 taken 14 times.
✗ Branch 244 → 245 not taken.
✗ Branch 244 → 246 not taken.
✗ Branch 735 → 736 not taken.
✗ Branch 735 → 737 not taken.
|
14 | if (!validParent(task, task.parentUid)) return {{"ok", false}, {"error", tr("Choose a parent in this list without creating a loop.")}}; |
| 158 | // S94.5.1: completing a parent with open subtasks through Save asks first, | ||
| 159 | // like the list action does ("Complete N subtasks too?"). | ||
| 160 | 14 | QList<CalendarTask> openChildren; | |
| 161 |
3/4✓ Branch 249 → 250 taken 3 times.
✓ Branch 249 → 254 taken 11 times.
✓ Branch 252 → 253 taken 3 times.
✗ Branch 252 → 254 not taken.
|
14 | const bool completing = task.status == "COMPLETED" && stored->status != "COMPLETED"; |
| 162 |
2/2✓ Branch 255 → 256 taken 3 times.
✓ Branch 255 → 276 taken 11 times.
|
14 | if (completing) |
| 163 |
5/5✓ Branch 257 → 258 taken 3 times.
✓ Branch 258 → 259 taken 3 times.
✓ Branch 259 → 260 taken 3 times.
✓ Branch 273 → 261 taken 5 times.
✓ Branch 273 → 274 taken 3 times.
|
8 | for (const auto &child : subtree(*stored)) |
| 164 |
7/8✓ Branch 264 → 265 taken 2 times.
✓ Branch 264 → 268 taken 3 times.
✓ Branch 265 → 266 taken 2 times.
✓ Branch 266 → 267 taken 2 times.
✗ Branch 266 → 268 not taken.
✓ Branch 269 → 270 taken 2 times.
✓ Branch 269 → 271 taken 3 times.
✓ Branch 270 → 271 taken 2 times.
|
8 | if (child.uid != stored->uid && TaskListLayout::open(child)) openChildren << child; |
| 165 |
2/2✓ Branch 280 → 281 taken 14 times.
✓ Branch 281 → 282 taken 14 times.
|
28 | const QString childrenMode = fields.value(QStringLiteral("childrenMode")).toString(); |
| 166 |
6/6✓ Branch 287 → 288 taken 2 times.
✓ Branch 287 → 291 taken 12 times.
✓ Branch 289 → 290 taken 1 time.
✓ Branch 289 → 291 taken 1 time.
✓ Branch 292 → 293 taken 1 time.
✓ Branch 292 → 302 taken 13 times.
|
14 | if (!openChildren.isEmpty() && childrenMode.isEmpty()) |
| 167 |
2/4✓ Branch 299 → 300 taken 3 times.
✓ Branch 299 → 301 taken 1 time.
✗ Branch 764 → 765 not taken.
✗ Branch 764 → 766 not taken.
|
4 | return {{"ok", false}, {"needsChildrenChoice", true}, {"childCount", openChildren.size()}}; |
| 168 |
5/8✓ Branch 303 → 304 taken 2 times.
✓ Branch 303 → 310 taken 11 times.
✓ Branch 306 → 307 taken 2 times.
✗ Branch 306 → 310 not taken.
✗ Branch 308 → 309 not taken.
✓ Branch 308 → 310 taken 2 times.
✗ Branch 311 → 312 not taken.
✓ Branch 311 → 324 taken 13 times.
|
13 | if (task.status == "COMPLETED" && stored->status != "COMPLETED" && !task.rrule.isEmpty()) { |
| 169 | ✗ | if (!TaskRecurrence::complete(&task, QDateTime::currentDateTimeUtc())) { | |
| 170 | ✗ | rejected.insert("error", tr("This repeat rule cannot be completed here. Open it in the app that created it.")); return rejected; | |
| 171 | } | ||
| 172 | } | ||
| 173 | const QString targetAccount = | ||
| 174 |
2/2✓ Branch 328 → 329 taken 13 times.
✓ Branch 329 → 330 taken 13 times.
|
26 | fields.value(QStringLiteral("targetAccountId"), task.accountId).toString(); |
| 175 | const QString targetPath = | ||
| 176 |
2/2✓ Branch 338 → 339 taken 13 times.
✓ Branch 339 → 340 taken 13 times.
|
26 | fields.value(QStringLiteral("targetCalendarPath"), task.calendarPath).toString(); |
| 177 |
1/2✓ Branch 345 → 346 taken 13 times.
✗ Branch 345 → 348 not taken.
|
26 | const bool moving = targetAccount != task.accountId || |
| 178 |
2/2✓ Branch 347 → 348 taken 1 time.
✓ Branch 347 → 349 taken 12 times.
|
13 | targetPath != task.calendarPath; |
| 179 |
2/2✓ Branch 350 → 351 taken 1 time.
✓ Branch 350 → 397 taken 12 times.
|
13 | if (moving) { |
| 180 |
7/11✓ Branch 354 → 355 taken 1 time.
✓ Branch 356 → 357 taken 1 time.
✓ Branch 359 → 360 taken 1 time.
✓ Branch 362 → 363 taken 1 time.
✓ Branch 362 → 364 taken 1 time.
✓ Branch 368 → 369 taken 2 times.
✓ Branch 368 → 370 taken 1 time.
✗ Branch 815 → 816 not taken.
✗ Branch 815 → 817 not taken.
✗ Branch 834 → 835 not taken.
✗ Branch 834 → 836 not taken.
|
6 | QVariantMap result = apply({QuickTaskModel::record(*stored)}, "move", {{"accountId", targetAccount}, {"calendarPath", targetPath}}); |
| 181 |
4/5✓ Branch 371 → 372 taken 1 time.
✓ Branch 372 → 373 taken 1 time.
✓ Branch 373 → 374 taken 1 time.
✗ Branch 377 → 378 not taken.
✓ Branch 377 → 379 taken 1 time.
|
1 | if (!result.value("ok").toBool()) return result; |
| 182 | 1 | task.accountId = targetAccount; task.calendarPath = targetPath; task.etag.clear(); task.resourceHref.clear(); | |
| 183 |
2/7✓ Branch 383 → 384 taken 1 time.
✗ Branch 384 → 385 not taken.
✓ Branch 384 → 394 taken 1 time.
✗ Branch 390 → 391 not taken.
✗ Branch 390 → 392 not taken.
✗ Branch 854 → 855 not taken.
✗ Branch 854 → 856 not taken.
|
1 | if (!sync->saveTask(task, false)) return {{"ok", false}, {"error", tr("Could not save this task. Try again.")}}; |
| 184 | 1 | return result; | |
| 185 | 1 | } | |
| 186 |
2/3✓ Branch 397 → 398 taken 12 times.
✗ Branch 398 → 399 not taken.
✓ Branch 398 → 410 taken 12 times.
|
12 | if (!sync->saveTask(task, false)) { |
| 187 | ✗ | rejected.insert(QStringLiteral("error"), tr("Could not save this task. Try again.")); | |
| 188 | ✗ | return rejected; | |
| 189 | } | ||
| 190 |
2/2✓ Branch 410 → 411 taken 12 times.
✓ Branch 411 → 412 taken 12 times.
|
12 | setLastList(calendarSelectionKey(task.accountId, task.calendarPath)); |
| 191 |
2/4✓ Branch 417 → 418 taken 12 times.
✓ Branch 417 → 419 taken 12 times.
✗ Branch 886 → 887 not taken.
✗ Branch 886 → 888 not taken.
|
24 | QList<Snapshot> snapshots{{*stored, true}}; |
| 192 |
2/2✓ Branch 421 → 422 taken 1 time.
✓ Branch 421 → 459 taken 11 times.
|
12 | if (childrenMode == QLatin1String("all")) { |
| 193 |
4/4✓ Branch 422 → 423 taken 1 time.
✓ Branch 423 → 424 taken 1 time.
✓ Branch 457 → 425 taken 1 time.
✓ Branch 457 → 458 taken 1 time.
|
2 | for (CalendarTask child : openChildren) { |
| 194 | 1 | snapshots.append({child, true}); | |
| 195 | 1 | child.previousStatus = child.status; | |
| 196 | 1 | child.previousPercent = child.percentComplete; | |
| 197 | 1 | child.status = QStringLiteral("COMPLETED"); | |
| 198 | 1 | child.percentComplete = 100; | |
| 199 |
1/1✓ Branch 437 → 438 taken 1 time.
|
1 | child.completedAt = QDateTime::currentDateTimeUtc(); |
| 200 |
2/9✓ Branch 440 → 441 taken 1 time.
✗ Branch 441 → 442 not taken.
✓ Branch 441 → 454 taken 1 time.
✗ Branch 442 → 443 not taken.
✗ Branch 446 → 447 not taken.
✗ Branch 447 → 448 not taken.
✗ Branch 448 → 449 not taken.
✗ Branch 452 → 444 not taken.
✗ Branch 452 → 453 not taken.
|
1 | if (!sync->saveTask(child, false)) qCWarning(lcTaskActions) << "Subtask not completed with its parent" << child.uid; |
| 201 | 1 | } | |
| 202 | } | ||
| 203 |
2/2✓ Branch 459 → 460 taken 12 times.
✓ Branch 460 → 461 taken 12 times.
|
12 | remember(tr("Task saved"), snapshots); |
| 204 |
2/4✓ Branch 466 → 467 taken 24 times.
✓ Branch 466 → 468 taken 12 times.
✗ Branch 917 → 918 not taken.
✗ Branch 917 → 919 not taken.
|
36 | return {{"ok", true}, {"stage", "savedLocal"}}; |
| 205 | 143 | } | |
| 206 | |||
| 207 | 2 | QVariantMap QuickTaskActions::reorder(const QVariantMap &identity, const QVariantMap &target, bool after) { | |
| 208 | 2 | auto *sync = m_session.calendarSync(); | |
| 209 |
14/38✓ Branch 9 → 10 taken 2 times.
✓ Branch 10 → 11 taken 2 times.
✓ Branch 11 → 12 taken 2 times.
✓ Branch 13 → 14 taken 2 times.
✓ Branch 14 → 15 taken 2 times.
✓ Branch 15 → 16 taken 2 times.
✓ Branch 18 → 19 taken 2 times.
✗ Branch 18 → 20 not taken.
✓ Branch 20 → 21 taken 2 times.
✗ Branch 20 → 22 not taken.
✓ Branch 22 → 23 taken 2 times.
✗ Branch 22 → 24 not taken.
✓ Branch 24 → 25 taken 2 times.
✗ Branch 24 → 26 not taken.
✓ Branch 26 → 27 taken 2 times.
✗ Branch 26 → 28 not taken.
✓ Branch 28 → 29 taken 2 times.
✗ Branch 28 → 30 not taken.
✓ Branch 30 → 31 taken 2 times.
✗ Branch 30 → 32 not taken.
✓ Branch 32 → 33 taken 2 times.
✗ Branch 32 → 34 not taken.
✗ Branch 225 → 226 not taken.
✗ Branch 225 → 227 not taken.
✗ Branch 229 → 230 not taken.
✗ Branch 229 → 231 not taken.
✗ Branch 233 → 234 not taken.
✗ Branch 233 → 235 not taken.
✗ Branch 237 → 238 not taken.
✗ Branch 237 → 239 not taken.
✗ Branch 241 → 242 not taken.
✗ Branch 241 → 243 not taken.
✗ Branch 245 → 246 not taken.
✗ Branch 245 → 247 not taken.
✗ Branch 249 → 250 not taken.
✗ Branch 249 → 251 not taken.
✗ Branch 253 → 254 not taken.
✗ Branch 253 → 255 not taken.
|
6 | const auto moved = sync ? sync->taskFor(identity.value("uid").toString(), identity.value("accountId").toString(), |
| 210 |
9/22✓ Branch 3 → 4 taken 2 times.
✗ Branch 3 → 17 not taken.
✓ Branch 5 → 6 taken 2 times.
✓ Branch 6 → 7 taken 2 times.
✓ Branch 7 → 8 taken 2 times.
✓ Branch 16 → 18 taken 2 times.
✓ Branch 34 → 35 taken 2 times.
✗ Branch 34 → 36 not taken.
✓ Branch 36 → 37 taken 2 times.
✗ Branch 36 → 38 not taken.
✓ Branch 38 → 39 taken 2 times.
✗ Branch 38 → 40 not taken.
✓ Branch 40 → 41 taken 2 times.
✗ Branch 40 → 42 not taken.
✗ Branch 257 → 258 not taken.
✗ Branch 257 → 259 not taken.
✗ Branch 261 → 262 not taken.
✗ Branch 261 → 263 not taken.
✗ Branch 265 → 266 not taken.
✗ Branch 265 → 267 not taken.
✗ Branch 269 → 270 not taken.
✗ Branch 269 → 271 not taken.
|
8 | identity.value("calendarPath").toString()) : std::nullopt; |
| 211 |
14/38✓ Branch 48 → 49 taken 2 times.
✓ Branch 49 → 50 taken 2 times.
✓ Branch 50 → 51 taken 2 times.
✓ Branch 52 → 53 taken 2 times.
✓ Branch 53 → 54 taken 2 times.
✓ Branch 54 → 55 taken 2 times.
✓ Branch 57 → 58 taken 2 times.
✗ Branch 57 → 59 not taken.
✓ Branch 59 → 60 taken 2 times.
✗ Branch 59 → 61 not taken.
✓ Branch 61 → 62 taken 2 times.
✗ Branch 61 → 63 not taken.
✓ Branch 63 → 64 taken 2 times.
✗ Branch 63 → 65 not taken.
✓ Branch 65 → 66 taken 2 times.
✗ Branch 65 → 67 not taken.
✓ Branch 67 → 68 taken 2 times.
✗ Branch 67 → 69 not taken.
✓ Branch 69 → 70 taken 2 times.
✗ Branch 69 → 71 not taken.
✓ Branch 71 → 72 taken 2 times.
✗ Branch 71 → 73 not taken.
✗ Branch 273 → 274 not taken.
✗ Branch 273 → 275 not taken.
✗ Branch 277 → 278 not taken.
✗ Branch 277 → 279 not taken.
✗ Branch 281 → 282 not taken.
✗ Branch 281 → 283 not taken.
✗ Branch 285 → 286 not taken.
✗ Branch 285 → 287 not taken.
✗ Branch 289 → 290 not taken.
✗ Branch 289 → 291 not taken.
✗ Branch 293 → 294 not taken.
✗ Branch 293 → 295 not taken.
✗ Branch 297 → 298 not taken.
✗ Branch 297 → 299 not taken.
✗ Branch 301 → 302 not taken.
✗ Branch 301 → 303 not taken.
|
6 | const auto anchor = sync ? sync->taskFor(target.value("uid").toString(), target.value("accountId").toString(), |
| 212 |
9/22✓ Branch 42 → 43 taken 2 times.
✗ Branch 42 → 56 not taken.
✓ Branch 44 → 45 taken 2 times.
✓ Branch 45 → 46 taken 2 times.
✓ Branch 46 → 47 taken 2 times.
✓ Branch 55 → 57 taken 2 times.
✓ Branch 73 → 74 taken 2 times.
✗ Branch 73 → 75 not taken.
✓ Branch 75 → 76 taken 2 times.
✗ Branch 75 → 77 not taken.
✓ Branch 77 → 78 taken 2 times.
✗ Branch 77 → 79 not taken.
✓ Branch 79 → 80 taken 2 times.
✗ Branch 79 → 81 not taken.
✗ Branch 305 → 306 not taken.
✗ Branch 305 → 307 not taken.
✗ Branch 309 → 310 not taken.
✗ Branch 309 → 311 not taken.
✗ Branch 313 → 314 not taken.
✗ Branch 313 → 315 not taken.
✗ Branch 317 → 318 not taken.
✗ Branch 317 → 319 not taken.
|
8 | target.value("calendarPath").toString()) : std::nullopt; |
| 213 |
4/12✓ Branch 82 → 83 taken 2 times.
✗ Branch 82 → 87 not taken.
✓ Branch 84 → 85 taken 2 times.
✗ Branch 84 → 87 not taken.
✗ Branch 86 → 87 not taken.
✓ Branch 86 → 88 taken 2 times.
✗ Branch 89 → 90 not taken.
✓ Branch 89 → 99 taken 2 times.
✗ Branch 95 → 96 not taken.
✗ Branch 95 → 97 not taken.
✗ Branch 322 → 323 not taken.
✗ Branch 322 → 324 not taken.
|
2 | if (!moved || !anchor || moved->readOnly) return {{"ok", false}, {"error", tr("This list is read-only or the task is no longer available.")}}; |
| 214 |
3/6✓ Branch 102 → 103 taken 2 times.
✗ Branch 102 → 107 not taken.
✗ Branch 106 → 107 not taken.
✓ Branch 106 → 108 taken 2 times.
✗ Branch 109 → 110 not taken.
✓ Branch 109 → 119 taken 2 times.
|
2 | if (moved->accountId != anchor->accountId || moved->calendarPath != anchor->calendarPath) |
| 215 | ✗ | return {{"ok", false}, {"error", tr("Tasks can only be reordered within their list.")}}; | |
| 216 |
1/1✓ Branch 122 → 123 taken 2 times.
|
2 | const auto listTasks = sync->store()->tasksForCalendar(moved->accountId, moved->calendarPath); |
| 217 |
3/3✓ Branch 124 → 125 taken 2 times.
✓ Branch 126 → 127 taken 2 times.
✓ Branch 128 → 129 taken 2 times.
|
2 | const auto changes = TaskListLayout::manualOrder(listTasks, taskSelectionKey(*moved), taskSelectionKey(*anchor), after); |
| 218 | 2 | QList<Snapshot> snapshots; | |
| 219 |
1/1✓ Branch 132 → 133 taken 2 times.
|
2 | sync->beginTaskBatch(); |
| 220 | 4 | const auto end = qScopeGuard([sync] { sync->endTaskBatch(); }); | |
| 221 |
2/2✓ Branch 190 → 136 taken 2 times.
✓ Branch 190 → 191 taken 2 times.
|
4 | for (const auto &[key, order] : changes) { |
| 222 |
1/2✓ Branch 186 → 141 taken 4 times.
✗ Branch 186 → 187 not taken.
|
4 | for (const auto &listed : listTasks) { |
| 223 |
3/3✓ Branch 142 → 143 taken 4 times.
✓ Branch 145 → 146 taken 2 times.
✓ Branch 145 → 147 taken 2 times.
|
4 | if (taskSelectionKey(listed) != key) continue; |
| 224 |
1/1✓ Branch 147 → 148 taken 2 times.
|
2 | auto task = sync->taskFor(listed.uid, listed.accountId, listed.calendarPath); |
| 225 |
1/2✗ Branch 149 → 150 not taken.
✓ Branch 149 → 151 taken 2 times.
|
2 | if (!task) break; |
| 226 | 2 | const CalendarTask before = *task; | |
| 227 | 2 | task->sortOrder = order; | |
| 228 |
2/3✓ Branch 155 → 156 taken 2 times.
✗ Branch 156 → 157 not taken.
✓ Branch 156 → 172 taken 2 times.
|
2 | if (!sync->saveTask(*task, false)) { |
| 229 | // S106-350: what was written stays undoable (like apply()), so the | ||
| 230 | // message's "Undo" undoes this reorder, not an older action. | ||
| 231 | ✗ | if (!snapshots.isEmpty()) remember(tr("Task moved"), snapshots); | |
| 232 | ✗ | return {{"ok", false}, {"error", tr("Could not save all changes. Undo the saved changes or try again.")}}; | |
| 233 | } | ||
| 234 | 2 | snapshots.append({before, true}); | |
| 235 | 2 | break; | |
| 236 | 4 | } | |
| 237 | } | ||
| 238 |
3/4✓ Branch 192 → 193 taken 2 times.
✗ Branch 192 → 197 not taken.
✓ Branch 193 → 194 taken 2 times.
✓ Branch 194 → 195 taken 2 times.
|
2 | if (!snapshots.isEmpty()) remember(tr("Task moved"), snapshots); |
| 239 |
6/6✓ Branch 197 → 198 taken 2 times.
✓ Branch 201 → 202 taken 2 times.
✓ Branch 202 → 203 taken 2 times.
✓ Branch 204 → 205 taken 2 times.
✓ Branch 208 → 199 taken 2 times.
✓ Branch 208 → 209 taken 2 times.
|
4 | qCInfo(lcTaskActions) << "Reordered task; sort orders written" << snapshots.size(); |
| 240 |
2/4✓ Branch 214 → 215 taken 4 times.
✓ Branch 214 → 216 taken 2 times.
✗ Branch 398 → 399 not taken.
✗ Branch 398 → 400 not taken.
|
6 | return {{"ok", true}, {"changed", snapshots.size()}}; |
| 241 | 6 | } | |
| 242 | |||
| 243 | 1 | QVariantMap QuickTaskActions::moveDue(const QVariantMap &occurrence, int deltaMinutes) { | |
| 244 | 1 | auto *sync = m_session.calendarSync(); | |
| 245 |
14/38✓ Branch 9 → 10 taken 1 time.
✓ Branch 10 → 11 taken 1 time.
✓ Branch 11 → 12 taken 1 time.
✓ Branch 13 → 14 taken 1 time.
✓ Branch 14 → 15 taken 1 time.
✓ Branch 15 → 16 taken 1 time.
✓ Branch 18 → 19 taken 1 time.
✗ Branch 18 → 20 not taken.
✓ Branch 20 → 21 taken 1 time.
✗ Branch 20 → 22 not taken.
✓ Branch 22 → 23 taken 1 time.
✗ Branch 22 → 24 not taken.
✓ Branch 24 → 25 taken 1 time.
✗ Branch 24 → 26 not taken.
✓ Branch 26 → 27 taken 1 time.
✗ Branch 26 → 28 not taken.
✓ Branch 28 → 29 taken 1 time.
✗ Branch 28 → 30 not taken.
✓ Branch 30 → 31 taken 1 time.
✗ Branch 30 → 32 not taken.
✓ Branch 32 → 33 taken 1 time.
✗ Branch 32 → 34 not taken.
✗ Branch 121 → 122 not taken.
✗ Branch 121 → 123 not taken.
✗ Branch 125 → 126 not taken.
✗ Branch 125 → 127 not taken.
✗ Branch 129 → 130 not taken.
✗ Branch 129 → 131 not taken.
✗ Branch 133 → 134 not taken.
✗ Branch 133 → 135 not taken.
✗ Branch 137 → 138 not taken.
✗ Branch 137 → 139 not taken.
✗ Branch 141 → 142 not taken.
✗ Branch 141 → 143 not taken.
✗ Branch 145 → 146 not taken.
✗ Branch 145 → 147 not taken.
✗ Branch 149 → 150 not taken.
✗ Branch 149 → 151 not taken.
|
3 | const auto task = sync ? sync->taskFor(occurrence.value("uid").toString(), occurrence.value("accountId").toString(), |
| 246 |
9/22✓ Branch 3 → 4 taken 1 time.
✗ Branch 3 → 17 not taken.
✓ Branch 5 → 6 taken 1 time.
✓ Branch 6 → 7 taken 1 time.
✓ Branch 7 → 8 taken 1 time.
✓ Branch 16 → 18 taken 1 time.
✓ Branch 34 → 35 taken 1 time.
✗ Branch 34 → 36 not taken.
✓ Branch 36 → 37 taken 1 time.
✗ Branch 36 → 38 not taken.
✓ Branch 38 → 39 taken 1 time.
✗ Branch 38 → 40 not taken.
✓ Branch 40 → 41 taken 1 time.
✗ Branch 40 → 42 not taken.
✗ Branch 153 → 154 not taken.
✗ Branch 153 → 155 not taken.
✗ Branch 157 → 158 not taken.
✗ Branch 157 → 159 not taken.
✗ Branch 161 → 162 not taken.
✗ Branch 161 → 163 not taken.
✗ Branch 165 → 166 not taken.
✗ Branch 165 → 167 not taken.
|
4 | occurrence.value("calendarPath").toString()) : std::nullopt; |
| 247 |
4/11✓ Branch 43 → 44 taken 1 time.
✗ Branch 43 → 47 not taken.
✓ Branch 45 → 46 taken 1 time.
✗ Branch 46 → 47 not taken.
✓ Branch 46 → 48 taken 1 time.
✗ Branch 49 → 50 not taken.
✓ Branch 49 → 59 taken 1 time.
✗ Branch 55 → 56 not taken.
✗ Branch 55 → 57 not taken.
✗ Branch 170 → 171 not taken.
✗ Branch 170 → 172 not taken.
|
1 | if (!task || !task->due.isValid()) return {{"ok", false}, {"error", tr("The task no longer exists.")}}; |
| 248 | // The chip shows overdue tasks on today; the move starts from the shown day. | ||
| 249 |
4/4✓ Branch 60 → 61 taken 1 time.
✓ Branch 61 → 62 taken 1 time.
✓ Branch 62 → 63 taken 1 time.
✓ Branch 63 → 64 taken 1 time.
|
1 | const QDate shown = occurrence.value("start").toDateTime().date(); |
| 250 |
3/5✓ Branch 68 → 69 taken 1 time.
✓ Branch 69 → 70 taken 1 time.
✗ Branch 69 → 71 not taken.
✗ Branch 72 → 73 not taken.
✓ Branch 74 → 75 taken 1 time.
|
1 | const QDate day = (shown.isValid() ? shown : task->due.date()).addDays(qRound(deltaMinutes / 1440.0)); |
| 251 | 1 | const QString due = task->dueAllDay ? day.toString(Qt::ISODate) | |
| 252 |
7/30✓ Branch 76 → 77 taken 1 time.
✗ Branch 76 → 78 not taken.
✓ Branch 77 → 88 taken 1 time.
✗ Branch 79 → 80 not taken.
✗ Branch 80 → 81 not taken.
✗ Branch 81 → 82 not taken.
✗ Branch 82 → 83 not taken.
✗ Branch 85 → 86 not taken.
✗ Branch 86 → 87 not taken.
✗ Branch 87 → 88 not taken.
✗ Branch 88 → 89 not taken.
✓ Branch 88 → 90 taken 1 time.
✗ Branch 90 → 91 not taken.
✓ Branch 90 → 92 taken 1 time.
✗ Branch 92 → 93 not taken.
✓ Branch 92 → 94 taken 1 time.
✗ Branch 94 → 95 not taken.
✓ Branch 94 → 96 taken 1 time.
✗ Branch 96 → 97 not taken.
✓ Branch 96 → 98 taken 1 time.
✗ Branch 196 → 197 not taken.
✗ Branch 196 → 198 not taken.
✗ Branch 200 → 201 not taken.
✗ Branch 200 → 202 not taken.
✗ Branch 206 → 207 not taken.
✗ Branch 206 → 208 not taken.
✗ Branch 210 → 211 not taken.
✗ Branch 210 → 212 not taken.
✗ Branch 215 → 216 not taken.
✗ Branch 215 → 217 not taken.
|
1 | : day.toString(Qt::ISODate) + QLatin1Char(' ') + task->due.toLocalTime().time().toString("HH:mm"); |
| 253 |
7/11✓ Branch 100 → 101 taken 1 time.
✓ Branch 102 → 103 taken 1 time.
✓ Branch 105 → 106 taken 1 time.
✓ Branch 109 → 110 taken 1 time.
✓ Branch 109 → 111 taken 1 time.
✓ Branch 115 → 116 taken 1 time.
✓ Branch 115 → 117 taken 1 time.
✗ Branch 222 → 223 not taken.
✗ Branch 222 → 224 not taken.
✗ Branch 241 → 242 not taken.
✗ Branch 241 → 243 not taken.
|
5 | return apply({QuickTaskModel::record(*task)}, "due", {{"due", due}}); |
| 254 | 4 | } | |
| 255 | |||
| 256 | ✗ | QVariantList QuickTaskActions::dueToday() const { return openTasksFor(QStringLiteral("today")); } | |
| 257 | |||
| 258 | // F94-59, F94-67: the open tasks of a smart list, list ("list:key") or tag | ||
| 259 | // ("tag:name") for the mail view's agenda, narrowed by a filter in the task | ||
| 260 | // search syntax (has:due, has:mail, priority:1, #tag, words) and sorted | ||
| 261 | // (TaskListLayout::sortFlat). | ||
| 262 | 1217 | QVariantList QuickTaskActions::openTasksFor(const QString &view, const QString &filter, const QString &sort) const { | |
| 263 | 1217 | QVariantList result; | |
| 264 | 1217 | auto *sync = m_session.calendarSync(); | |
| 265 |
3/6✓ Branch 3 → 4 taken 1217 times.
✗ Branch 3 → 6 not taken.
✗ Branch 5 → 6 not taken.
✓ Branch 5 → 7 taken 1217 times.
✗ Branch 8 → 9 not taken.
✓ Branch 8 → 10 taken 1217 times.
|
1217 | if (!sync || !sync->store()) return result; |
| 266 |
1/1✓ Branch 10 → 11 taken 1217 times.
|
1217 | const QDate today = QDate::currentDate(); |
| 267 | 1217 | QList<CalendarTask> tasks; | |
| 268 |
5/5✓ Branch 12 → 13 taken 1217 times.
✓ Branch 13 → 14 taken 1217 times.
✓ Branch 14 → 15 taken 1217 times.
✓ Branch 30 → 16 taken 1993 times.
✓ Branch 30 → 31 taken 1217 times.
|
3210 | for (const auto &task : sync->store()->listTasks()) |
| 269 |
8/8✓ Branch 17 → 18 taken 1993 times.
✓ Branch 18 → 19 taken 1894 times.
✓ Branch 18 → 25 taken 99 times.
✓ Branch 20 → 21 taken 1894 times.
✓ Branch 21 → 22 taken 441 times.
✓ Branch 21 → 25 taken 1453 times.
✓ Branch 26 → 27 taken 437 times.
✓ Branch 26 → 28 taken 1556 times.
|
2434 | if (TaskListLayout::open(task) && TaskListLayout::matchesView(task, view, today, m_tasks.weekStart()) && |
| 270 |
3/3✓ Branch 22 → 23 taken 441 times.
✓ Branch 23 → 24 taken 437 times.
✓ Branch 23 → 25 taken 4 times.
|
441 | TaskListLayout::matchesFilter(task, filter)) |
| 271 |
1/1✓ Branch 27 → 28 taken 437 times.
|
1654 | tasks.append(task); |
| 272 |
1/1✓ Branch 32 → 33 taken 1217 times.
|
1217 | TaskListLayout::sortFlat(tasks, sort); |
| 273 |
4/4✓ Branch 33 → 34 taken 1217 times.
✓ Branch 34 → 35 taken 1217 times.
✓ Branch 75 → 36 taken 437 times.
✓ Branch 75 → 76 taken 1217 times.
|
1654 | for (const auto &task : tasks) { |
| 274 |
1/1✓ Branch 37 → 38 taken 437 times.
|
437 | QVariantMap entry = QuickTaskModel::record(task); |
| 275 |
2/2✓ Branch 38 → 39 taken 437 times.
✓ Branch 43 → 44 taken 437 times.
|
874 | entry.insert(QStringLiteral("dueLabel"), TaskListLayout::dueText(task, today)); |
| 276 |
13/15✓ Branch 48 → 49 taken 437 times.
✓ Branch 49 → 50 taken 395 times.
✓ Branch 49 → 58 taken 42 times.
✓ Branch 50 → 51 taken 101 times.
✓ Branch 50 → 52 taken 294 times.
✓ Branch 51 → 54 taken 101 times.
✓ Branch 52 → 53 taken 294 times.
✓ Branch 53 → 54 taken 294 times.
✓ Branch 56 → 57 taken 154 times.
✓ Branch 56 → 58 taken 241 times.
✓ Branch 63 → 64 taken 437 times.
✓ Branch 67 → 68 taken 294 times.
✓ Branch 67 → 69 taken 143 times.
✗ Branch 103 → 104 not taken.
✗ Branch 103 → 105 not taken.
|
874 | entry.insert(QStringLiteral("overdue"), task.due.isValid() && (task.dueAllDay ? task.due.date() : task.due.toLocalTime().date()) < today); |
| 277 |
1/1✓ Branch 70 → 71 taken 437 times.
|
437 | result.append(entry); |
| 278 | 437 | } | |
| 279 | 1217 | return result; | |
| 280 | 1217 | } | |
| 281 | |||
| 282 | ✗ | bool QuickTaskActions::toggleTaskAt(int row) { | |
| 283 | ✗ | const auto task = m_tasks.taskAt(row); | |
| 284 | ✗ | return !task.isEmpty() && toggleTask(task.value(QStringLiteral("uid")).toString(), | |
| 285 | ✗ | task.value(QStringLiteral("accountId")).toString(), | |
| 286 | ✗ | task.value(QStringLiteral("calendarPath")).toString()); | |
| 287 | ✗ | } | |
| 288 | |||
| 289 | 4 | bool QuickTaskActions::toggleTask(const QString &uid, const QString &accountId, const QString &calendarPath) { | |
| 290 |
9/13✓ Branch 3 → 4 taken 4 times.
✓ Branch 10 → 11 taken 4 times.
✓ Branch 12 → 13 taken 4 times.
✓ Branch 13 → 14 taken 4 times.
✓ Branch 14 → 15 taken 4 times.
✓ Branch 22 → 23 taken 4 times.
✓ Branch 22 → 24 taken 4 times.
✓ Branch 26 → 27 taken 12 times.
✓ Branch 26 → 28 taken 4 times.
✗ Branch 45 → 46 not taken.
✗ Branch 45 → 47 not taken.
✗ Branch 51 → 52 not taken.
✗ Branch 51 → 53 not taken.
|
24 | return apply({QVariantMap{{"uid", uid}, {"accountId", accountId}, {"calendarPath", calendarPath}}}, "toggle").value("ok").toBool(); |
| 291 | 4 | } | |
| 292 | |||
| 293 | ✗ | bool QuickTaskActions::removeTaskAt(int row) { | |
| 294 | ✗ | const auto task = m_tasks.taskAt(row); | |
| 295 | ✗ | if (task.isEmpty()) return false; | |
| 296 | ✗ | return removeTask(task.value(QStringLiteral("uid")).toString(), | |
| 297 | ✗ | task.value(QStringLiteral("accountId")).toString(), | |
| 298 | ✗ | task.value(QStringLiteral("calendarPath")).toString()); | |
| 299 | ✗ | } | |
| 300 | |||
| 301 | 1 | bool QuickTaskActions::removeTask(const QString &uid, const QString &accountId, const QString &calendarPath) { | |
| 302 |
9/13✓ Branch 3 → 4 taken 1 time.
✓ Branch 10 → 11 taken 1 time.
✓ Branch 12 → 13 taken 1 time.
✓ Branch 13 → 14 taken 1 time.
✓ Branch 14 → 15 taken 1 time.
✓ Branch 22 → 23 taken 1 time.
✓ Branch 22 → 24 taken 1 time.
✓ Branch 26 → 27 taken 3 times.
✓ Branch 26 → 28 taken 1 time.
✗ Branch 45 → 46 not taken.
✗ Branch 45 → 47 not taken.
✗ Branch 51 → 52 not taken.
✗ Branch 51 → 53 not taken.
|
6 | return apply({QVariantMap{{"uid", uid}, {"accountId", accountId}, {"calendarPath", calendarPath}}}, "delete").value("ok").toBool(); |
| 303 | 1 | } | |
| 304 | |||
| 305 | 798 | void QuickTaskActions::setLastList(const QString &key) { | |
| 306 |
2/2✓ Branch 3 → 4 taken 775 times.
✓ Branch 3 → 5 taken 23 times.
|
798 | if (key == m_lastList) return; |
| 307 | 23 | m_lastList = key; emit stateChanged(); | |
| 308 | } | ||
| 309 | 58 | void QuickTaskActions::remember(const QString &text, const QList<Snapshot> &snapshots) { | |
| 310 | 58 | m_undo = snapshots; m_undoText = text; emit undoChanged(); | |
| 311 | 58 | const quint64 generation = ++m_undoGeneration; | |
| 312 |
1/1✓ Branch 5 → 6 taken 58 times.
|
58 | QTimer::singleShot(8000, this, [this, generation] { |
| 313 | ✗ | if (generation != m_undoGeneration) return; | |
| 314 | ✗ | m_undo.clear(); m_undoText.clear(); emit undoChanged(); | |
| 315 | }); | ||
| 316 | 58 | } | |
| 317 | // F-40 (SPRINT-104): one tasksForCalendar read per list instead of one per | ||
| 318 | // selected task — a bulk action over a large list used to re-read every | ||
| 319 | // row of the list for every marked task, on the GUI thread. | ||
| 320 | 68 | QHash<QString, QList<CalendarTask>> QuickTaskActions::childrenOf(const QString &accountId, | |
| 321 | const QString &calendarPath) const { | ||
| 322 | 68 | QHash<QString, QList<CalendarTask>> children; | |
| 323 | 68 | auto *sync = m_session.calendarSync(); | |
| 324 |
3/6✓ Branch 3 → 4 taken 68 times.
✗ Branch 3 → 6 not taken.
✗ Branch 5 → 6 not taken.
✓ Branch 5 → 7 taken 68 times.
✗ Branch 8 → 9 not taken.
✓ Branch 8 → 10 taken 68 times.
|
68 | if (!sync || !sync->store()) return children; |
| 325 | 68 | ++m_listReads; // F-40: the bulk path asserts this stays at one per list | |
| 326 |
1/1✓ Branch 11 → 12 taken 68 times.
|
68 | const auto tasks = sync->store()->tasksForCalendar(accountId, calendarPath); |
| 327 |
2/2✓ Branch 21 → 14 taken 197 times.
✓ Branch 21 → 22 taken 68 times.
|
265 | for (const auto &item : tasks) |
| 328 |
4/4✓ Branch 16 → 17 taken 20 times.
✓ Branch 16 → 19 taken 177 times.
✓ Branch 17 → 18 taken 20 times.
✓ Branch 18 → 19 taken 20 times.
|
197 | if (!item.parentUid.isEmpty()) children[item.parentUid] << item; |
| 329 | 68 | return children; | |
| 330 | 68 | } | |
| 331 | |||
| 332 | 69 | QList<CalendarTask> QuickTaskActions::subtreeOf(const CalendarTask &task, | |
| 333 | const QHash<QString, QList<CalendarTask>> &children) { | ||
| 334 |
2/4✓ Branch 5 → 6 taken 69 times.
✓ Branch 5 → 7 taken 69 times.
✗ Branch 31 → 32 not taken.
✗ Branch 31 → 33 not taken.
|
138 | QList<CalendarTask> result, stack{task}; QSet<QString> visited; |
| 335 |
2/2✓ Branch 25 → 9 taken 84 times.
✓ Branch 25 → 26 taken 69 times.
|
153 | while (!stack.isEmpty()) { |
| 336 |
2/3✓ Branch 9 → 10 taken 84 times.
✗ Branch 11 → 12 not taken.
✓ Branch 11 → 13 taken 84 times.
|
84 | auto current = stack.takeLast(); if (visited.contains(current.uid)) continue; |
| 337 |
3/3✓ Branch 13 → 14 taken 84 times.
✓ Branch 14 → 15 taken 84 times.
✓ Branch 16 → 17 taken 84 times.
|
84 | visited.insert(current.uid); result << current; stack.append(children.value(current.uid)); |
| 338 | 84 | } | |
| 339 | 69 | return result; | |
| 340 | 138 | } | |
| 341 | |||
| 342 | 48 | QList<CalendarTask> QuickTaskActions::subtree(const CalendarTask &task) const { | |
| 343 |
2/2✓ Branch 2 → 3 taken 48 times.
✓ Branch 3 → 4 taken 48 times.
|
48 | return subtreeOf(task, childrenOf(task.accountId, task.calendarPath)); |
| 344 | } | ||
| 345 | 21 | bool QuickTaskActions::validParent(const CalendarTask &task, const QString &parentUid) const { | |
| 346 |
2/2✓ Branch 3 → 4 taken 20 times.
✓ Branch 3 → 5 taken 1 time.
|
21 | if (parentUid.isEmpty()) return true; |
| 347 |
2/4✓ Branch 9 → 10 taken 1 time.
✓ Branch 9 → 11 taken 1 time.
✗ Branch 37 → 38 not taken.
✗ Branch 37 → 39 not taken.
|
3 | auto *sync = m_session.calendarSync(); QSet<QString> visited{task.uid}; QString current = parentUid; |
| 348 |
1/2✓ Branch 30 → 12 taken 3 times.
✗ Branch 30 → 31 not taken.
|
3 | while (!current.isEmpty()) { |
| 349 |
2/2✓ Branch 13 → 14 taken 1 time.
✓ Branch 13 → 15 taken 2 times.
|
3 | if (visited.contains(current)) return false; |
| 350 |
2/2✓ Branch 15 → 16 taken 2 times.
✓ Branch 16 → 17 taken 2 times.
|
2 | visited.insert(current); const auto parent = sync->taskFor(current, task.accountId, task.calendarPath); |
| 351 |
1/2✗ Branch 18 → 19 not taken.
✓ Branch 18 → 20 taken 2 times.
|
2 | if (!parent) return false; current = parent->parentUid; |
| 352 | 2 | } | |
| 353 | ✗ | return true; | |
| 354 | 2 | } | |
| 355 | 50 | QVariantList QuickTaskActions::subtasks(const QVariantMap &identity) const { | |
| 356 |
10/10✓ Branch 3 → 4 taken 50 times.
✓ Branch 4 → 5 taken 50 times.
✓ Branch 5 → 6 taken 50 times.
✓ Branch 7 → 8 taken 50 times.
✓ Branch 8 → 9 taken 50 times.
✓ Branch 9 → 10 taken 50 times.
✓ Branch 11 → 12 taken 50 times.
✓ Branch 12 → 13 taken 50 times.
✓ Branch 13 → 14 taken 50 times.
✓ Branch 14 → 15 taken 50 times.
|
50 | const auto record = taskRecord(identity.value("uid").toString(), identity.value("accountId").toString(), identity.value("calendarPath").toString()); |
| 357 | 50 | auto *sync = m_session.calendarSync(); | |
| 358 |
23/60✓ Branch 28 → 29 taken 50 times.
✗ Branch 28 → 42 not taken.
✓ Branch 30 → 31 taken 50 times.
✓ Branch 31 → 32 taken 50 times.
✓ Branch 32 → 33 taken 50 times.
✓ Branch 34 → 35 taken 50 times.
✓ Branch 35 → 36 taken 50 times.
✓ Branch 36 → 37 taken 50 times.
✓ Branch 38 → 39 taken 50 times.
✓ Branch 39 → 40 taken 50 times.
✓ Branch 40 → 41 taken 50 times.
✓ Branch 41 → 43 taken 50 times.
✓ Branch 43 → 44 taken 50 times.
✗ Branch 43 → 45 not taken.
✓ Branch 45 → 46 taken 50 times.
✗ Branch 45 → 47 not taken.
✓ Branch 47 → 48 taken 50 times.
✗ Branch 47 → 49 not taken.
✓ Branch 49 → 50 taken 50 times.
✗ Branch 49 → 51 not taken.
✓ Branch 51 → 52 taken 50 times.
✗ Branch 51 → 53 not taken.
✓ Branch 53 → 54 taken 50 times.
✗ Branch 53 → 55 not taken.
✓ Branch 55 → 56 taken 50 times.
✗ Branch 55 → 57 not taken.
✓ Branch 57 → 58 taken 50 times.
✗ Branch 57 → 59 not taken.
✓ Branch 59 → 60 taken 50 times.
✗ Branch 59 → 61 not taken.
✓ Branch 61 → 62 taken 50 times.
✗ Branch 61 → 63 not taken.
✓ Branch 63 → 64 taken 50 times.
✗ Branch 63 → 65 not taken.
✓ Branch 65 → 66 taken 50 times.
✗ Branch 65 → 67 not taken.
✗ Branch 129 → 130 not taken.
✗ Branch 129 → 131 not taken.
✗ Branch 133 → 134 not taken.
✗ Branch 133 → 135 not taken.
✗ Branch 137 → 138 not taken.
✗ Branch 137 → 139 not taken.
✗ Branch 141 → 142 not taken.
✗ Branch 141 → 143 not taken.
✗ Branch 145 → 146 not taken.
✗ Branch 145 → 147 not taken.
✗ Branch 149 → 150 not taken.
✗ Branch 149 → 151 not taken.
✗ Branch 153 → 154 not taken.
✗ Branch 153 → 155 not taken.
✗ Branch 157 → 158 not taken.
✗ Branch 157 → 159 not taken.
✗ Branch 161 → 162 not taken.
✗ Branch 161 → 163 not taken.
✗ Branch 165 → 166 not taken.
✗ Branch 165 → 167 not taken.
✗ Branch 169 → 170 not taken.
✗ Branch 169 → 171 not taken.
✗ Branch 173 → 174 not taken.
✗ Branch 173 → 175 not taken.
|
50 | const auto parent = sync ? sync->taskFor(record.value("uid").toString(), record.value("accountId").toString(), record.value("calendarPath").toString()) : std::nullopt; |
| 359 | 50 | QVariantList result; | |
| 360 |
11/11✓ Branch 68 → 69 taken 45 times.
✓ Branch 68 → 89 taken 5 times.
✓ Branch 70 → 71 taken 45 times.
✓ Branch 71 → 72 taken 45 times.
✓ Branch 72 → 73 taken 45 times.
✓ Branch 77 → 78 taken 1 time.
✓ Branch 77 → 84 taken 45 times.
✓ Branch 78 → 79 taken 1 time.
✓ Branch 80 → 81 taken 1 time.
✓ Branch 86 → 74 taken 46 times.
✓ Branch 86 → 87 taken 45 times.
|
96 | if (parent) for (const auto &task : subtree(*parent)) if (task.uid != parent->uid) result << QuickTaskModel::record(task); |
| 361 | 50 | return result; | |
| 362 | 50 | } | |
| 363 | ✗ | QVariantMap QuickTaskActions::createSubtask(const QVariantMap &identity, const QString &title) { | |
| 364 | ✗ | auto *sync = m_session.calendarSync(); const auto parent = sync ? sync->taskFor(identity.value("uid").toString(), identity.value("accountId").toString(), identity.value("calendarPath").toString()) : std::nullopt; | |
| 365 | ✗ | if (!parent || parent->readOnly || title.trimmed().isEmpty()) return {{"ok", false}, {"error", tr("Enter a title in a writable list.")}}; | |
| 366 | ✗ | CalendarTask task; task.uid = QUuid::createUuid().toString(QUuid::WithoutBraces); | |
| 367 | ✗ | task.accountId = parent->accountId; task.calendarPath = parent->calendarPath; task.parentUid = parent->uid; | |
| 368 | ✗ | task.summary = title.trimmed(); task.status = "NEEDS-ACTION"; task.created = QDateTime::currentDateTimeUtc(); | |
| 369 | ✗ | if (!sync->saveTask(task, true)) return {{"ok", false}, {"error", tr("Could not save this task. Try again.")}}; | |
| 370 | ✗ | remember(tr("Subtask added"), {{task, false}}); return {{"ok", true}, {"task", QuickTaskModel::record(task)}}; | |
| 371 | ✗ | } | |
| 372 | // F94-56: where a new task without "/list" goes — the shown list, else the | ||
| 373 | // last used list, else the first writable task list. | ||
| 374 | 17 | const CalendarInfo *QuickTaskActions::defaultTarget(const QList<CalendarInfo> &calendars) const { | |
| 375 |
7/9✓ Branch 3 → 4 taken 17 times.
✓ Branch 4 → 5 taken 17 times.
✓ Branch 5 → 6 taken 3 times.
✓ Branch 5 → 8 taken 14 times.
✓ Branch 7 → 9 taken 3 times.
✓ Branch 9 → 10 taken 3 times.
✓ Branch 9 → 11 taken 14 times.
✗ Branch 49 → 50 not taken.
✗ Branch 49 → 51 not taken.
|
17 | const QString selectedList = m_tasks.view().startsWith("list:") ? m_tasks.view().mid(5) : m_lastList; |
| 376 |
2/2✓ Branch 30 → 15 taken 16 times.
✓ Branch 30 → 31 taken 10 times.
|
26 | for (const auto &calendar : calendars) |
| 377 |
9/14✓ Branch 16 → 17 taken 16 times.
✓ Branch 18 → 19 taken 7 times.
✓ Branch 18 → 23 taken 9 times.
✓ Branch 19 → 20 taken 7 times.
✗ Branch 19 → 23 not taken.
✓ Branch 20 → 21 taken 7 times.
✓ Branch 21 → 22 taken 7 times.
✗ Branch 21 → 23 not taken.
✓ Branch 24 → 25 taken 16 times.
✗ Branch 24 → 26 not taken.
✓ Branch 26 → 27 taken 7 times.
✓ Branch 26 → 28 taken 9 times.
✗ Branch 59 → 60 not taken.
✗ Branch 59 → 61 not taken.
|
16 | if (calendarSelectionKey(calendar.accountId, calendar.path) == selectedList && !calendar.readOnly && calendar.supportsTasks()) |
| 378 | 7 | return &calendar; | |
| 379 |
2/2✓ Branch 43 → 33 taken 9 times.
✓ Branch 43 → 44 taken 1 time.
|
10 | for (const auto &calendar : calendars) |
| 380 |
4/7✓ Branch 34 → 35 taken 9 times.
✗ Branch 34 → 38 not taken.
✓ Branch 35 → 36 taken 9 times.
✓ Branch 36 → 37 taken 9 times.
✗ Branch 36 → 38 not taken.
✓ Branch 39 → 40 taken 9 times.
✗ Branch 39 → 41 not taken.
|
9 | if (!calendar.readOnly && calendar.supportsTasks()) return &calendar; |
| 381 | 1 | return nullptr; | |
| 382 | 17 | } | |
| 383 | // Sprint 98 D-21: "/List@Account" picks that account's list; else the first list matching by path or name prefix. | ||
| 384 | 29 | const CalendarInfo *QuickTaskActions::matchTaskList(const QList<CalendarInfo> &calendars, const QString &hint) { | |
| 385 | 29 | const int at = hint.lastIndexOf(QLatin1Char('@')); | |
| 386 |
2/4✗ Branch 5 → 6 not taken.
✓ Branch 5 → 7 taken 29 times.
✗ Branch 6 → 8 not taken.
✓ Branch 8 → 9 taken 29 times.
|
29 | const QString name = (at > 0 ? hint.left(at) : hint).toLower(); |
| 387 |
2/8✗ Branch 10 → 11 not taken.
✓ Branch 10 → 13 taken 29 times.
✗ Branch 11 → 12 not taken.
✗ Branch 12 → 14 not taken.
✗ Branch 14 → 15 not taken.
✓ Branch 14 → 16 taken 29 times.
✗ Branch 53 → 54 not taken.
✗ Branch 53 → 55 not taken.
|
29 | const QString account = at > 0 ? hint.mid(at + 1).toLower() : QString{}; |
| 388 |
2/2✓ Branch 43 → 18 taken 30 times.
✓ Branch 43 → 44 taken 5 times.
|
35 | for (const auto &calendar : calendars) { |
| 389 |
3/11✗ Branch 20 → 21 not taken.
✓ Branch 20 → 25 taken 30 times.
✗ Branch 21 → 22 not taken.
✗ Branch 23 → 24 not taken.
✗ Branch 23 → 25 not taken.
✗ Branch 26 → 27 not taken.
✓ Branch 26 → 28 taken 30 times.
✗ Branch 28 → 29 not taken.
✓ Branch 28 → 30 taken 30 times.
✗ Branch 57 → 58 not taken.
✗ Branch 57 → 59 not taken.
|
30 | if (!account.isEmpty() && calendar.accountId.toLower() != account) continue; |
| 390 |
9/12✓ Branch 30 → 31 taken 30 times.
✓ Branch 32 → 33 taken 21 times.
✓ Branch 32 → 35 taken 9 times.
✓ Branch 33 → 34 taken 21 times.
✓ Branch 34 → 35 taken 15 times.
✓ Branch 34 → 36 taken 6 times.
✓ Branch 37 → 38 taken 30 times.
✗ Branch 37 → 39 not taken.
✓ Branch 39 → 40 taken 24 times.
✓ Branch 39 → 41 taken 6 times.
✗ Branch 61 → 62 not taken.
✗ Branch 61 → 63 not taken.
|
30 | if (calendar.path.toLower() == name || calendar.displayName.startsWith(name, Qt::CaseInsensitive)) return &calendar; |
| 391 | } | ||
| 392 | 5 | return nullptr; | |
| 393 | 29 | } | |
| 394 | 33 | QVariantMap QuickTaskActions::previewInput(const QString &input) const { | |
| 395 | // Readable chips for the quick-add row (S94.3.6): the recognised due date | ||
| 396 | // as the list shows it, the priority word, the list hint and #tags. | ||
| 397 |
1/1✓ Branch 2 → 3 taken 33 times.
|
33 | const auto parts = TaskConsoleInput::split(input); |
| 398 | 33 | QString due = parts.dueHint; | |
| 399 |
2/2✓ Branch 5 → 6 taken 15 times.
✓ Branch 5 → 33 taken 18 times.
|
33 | if (!parts.dueHint.isEmpty()) { |
| 400 | 15 | CalendarTask probe; | |
| 401 |
2/2✓ Branch 7 → 8 taken 15 times.
✓ Branch 8 → 9 taken 15 times.
|
15 | probe.due = TaskConsoleInput::parseDue(parts.dueHint); |
| 402 |
1/1✓ Branch 13 → 14 taken 15 times.
|
15 | probe.dueAllDay = !parts.dueHint.contains(QLatin1Char(':')); |
| 403 | 15 | probe.status = QStringLiteral("NEEDS-ACTION"); | |
| 404 |
6/6✓ Branch 20 → 21 taken 15 times.
✓ Branch 21 → 22 taken 5 times.
✓ Branch 21 → 24 taken 10 times.
✓ Branch 22 → 23 taken 5 times.
✓ Branch 23 → 27 taken 5 times.
✓ Branch 26 → 27 taken 10 times.
|
40 | due = probe.due.isValid() ? TaskListLayout::dueText(probe, QDate::currentDate()) |
| 405 |
3/5✓ Branch 24 → 25 taken 10 times.
✓ Branch 29 → 30 taken 10 times.
✓ Branch 29 → 31 taken 5 times.
✗ Branch 150 → 151 not taken.
✗ Branch 150 → 152 not taken.
|
40 | : tr("“%1” is not a date").arg(parts.dueHint); |
| 406 | 15 | } | |
| 407 |
1/1✓ Branch 33 → 34 taken 33 times.
|
33 | const int priority = TaskConsoleInput::parsePriority(parts.priorityHint); |
| 408 | 33 | const QString priorityText = parts.priorityHint.isEmpty() ? QString() | |
| 409 |
3/5✓ Branch 37 → 38 taken 2 times.
✓ Branch 37 → 41 taken 5 times.
✓ Branch 38 → 39 taken 2 times.
✗ Branch 160 → 161 not taken.
✗ Branch 160 → 162 not taken.
|
9 | : priority < 0 ? tr("“%1” is not a priority").arg(parts.priorityHint) |
| 410 |
2/3✓ Branch 41 → 42 taken 5 times.
✓ Branch 42 → 43 taken 5 times.
✗ Branch 42 → 44 not taken.
|
5 | : TaskListLayout::priorityLevel(priority) == 1 ? tr("↑ High") |
| 411 | ✗ | : TaskListLayout::priorityLevel(priority) == 3 ? tr("↓ Low") | |
| 412 |
6/11✓ Branch 35 → 36 taken 26 times.
✓ Branch 35 → 37 taken 7 times.
✓ Branch 40 → 51 taken 2 times.
✓ Branch 43 → 51 taken 5 times.
✗ Branch 46 → 51 not taken.
✗ Branch 47 → 48 not taken.
✗ Branch 48 → 49 not taken.
✗ Branch 48 → 50 not taken.
✗ Branch 49 → 51 not taken.
✓ Branch 51 → 52 taken 2 times.
✓ Branch 51 → 53 taken 31 times.
|
40 | : TaskListLayout::priorityLevel(priority) == 2 ? tr("Normal") : QString(); |
| 413 | 33 | QStringList tags; | |
| 414 |
4/4✓ Branch 59 → 60 taken 1 time.
✓ Branch 60 → 61 taken 1 time.
✓ Branch 66 → 55 taken 1 time.
✓ Branch 66 → 67 taken 33 times.
|
34 | for (const auto &tag : parts.tags) tags << QStringLiteral("#") + tag; |
| 415 | // F94-56: the list the task will land in, shown at the input. | ||
| 416 | 33 | QVariantMap target; | |
| 417 | 33 | bool listOk = true; | |
| 418 |
1/2✓ Branch 68 → 69 taken 33 times.
✗ Branch 68 → 97 not taken.
|
33 | if (auto *sync = m_session.calendarSync()) { |
| 419 |
1/1✓ Branch 69 → 70 taken 33 times.
|
33 | const auto calendars = sync->calendars(); |
| 420 |
3/3✓ Branch 71 → 72 taken 13 times.
✓ Branch 71 → 74 taken 20 times.
✓ Branch 72 → 73 taken 13 times.
|
33 | const CalendarInfo *list = parts.calendarHint.isEmpty() ? defaultTarget(calendars) |
| 421 |
1/1✓ Branch 74 → 75 taken 20 times.
|
20 | : matchTaskList(calendars, parts.calendarHint); |
| 422 |
5/7✓ Branch 76 → 77 taken 29 times.
✓ Branch 76 → 81 taken 4 times.
✓ Branch 77 → 78 taken 29 times.
✗ Branch 77 → 81 not taken.
✓ Branch 78 → 79 taken 29 times.
✓ Branch 79 → 80 taken 29 times.
✗ Branch 79 → 81 not taken.
|
33 | listOk = list && !list->readOnly && list->supportsTasks(); |
| 423 |
2/2✓ Branch 82 → 83 taken 29 times.
✓ Branch 82 → 95 taken 4 times.
|
33 | if (list) |
| 424 |
2/4✓ Branch 91 → 92 taken 87 times.
✓ Branch 91 → 93 taken 29 times.
✗ Branch 175 → 176 not taken.
✗ Branch 175 → 177 not taken.
|
116 | target = {{"key", calendarSelectionKey(list->accountId, list->path)}, {"name", list->displayName}, {"color", list->color}}; |
| 425 | 33 | } | |
| 426 | // Sprint 98 S98.4.6/S98.4.12: the command bar marks parts the parser rejects. | ||
| 427 |
7/9✓ Branch 100 → 101 taken 15 times.
✓ Branch 100 → 105 taken 18 times.
✓ Branch 101 → 102 taken 15 times.
✓ Branch 102 → 103 taken 15 times.
✓ Branch 103 → 104 taken 15 times.
✓ Branch 104 → 105 taken 5 times.
✓ Branch 104 → 106 taken 10 times.
✗ Branch 208 → 209 not taken.
✗ Branch 208 → 210 not taken.
|
48 | return {{"summary", parts.summary}, {"due", due}, {"dueOk", parts.dueHint.isEmpty() || TaskConsoleInput::parseDue(parts.dueHint).isValid()}, |
| 428 | ✗ | {"priority", parts.priorityHint}, {"priorityText", priorityText}, {"priorityOk", priority >= 0}, | |
| 429 |
9/13✓ Branch 112 → 113 taken 13 times.
✓ Branch 112 → 114 taken 20 times.
✓ Branch 117 → 118 taken 20 times.
✓ Branch 132 → 133 taken 20 times.
✓ Branch 132 → 134 taken 13 times.
✓ Branch 134 → 135 taken 20 times.
✓ Branch 134 → 136 taken 13 times.
✓ Branch 136 → 137 taken 15 times.
✓ Branch 136 → 138 taken 18 times.
✗ Branch 198 → 199 not taken.
✗ Branch 198 → 200 not taken.
✗ Branch 202 → 203 not taken.
✗ Branch 202 → 204 not taken.
|
86 | {"list", parts.calendarHint.isEmpty() ? QString() : QStringLiteral("/") + parts.calendarHint}, |
| 430 |
6/8✓ Branch 120 → 121 taken 20 times.
✓ Branch 120 → 122 taken 13 times.
✓ Branch 121 → 122 taken 17 times.
✓ Branch 121 → 123 taken 3 times.
✓ Branch 129 → 130 taken 330 times.
✓ Branch 129 → 131 taken 33 times.
✗ Branch 192 → 193 not taken.
✗ Branch 192 → 194 not taken.
|
429 | {"listOk", parts.calendarHint.isEmpty() || listOk}, {"tags", tags}, {"target", target}}; |
| 431 | 194 | } | |
| 432 | 13 | bool QuickTaskActions::undo() { | |
| 433 |
3/6✓ Branch 3 → 4 taken 13 times.
✗ Branch 3 → 6 not taken.
✗ Branch 5 → 6 not taken.
✓ Branch 5 → 7 taken 13 times.
✗ Branch 8 → 9 not taken.
✓ Branch 8 → 10 taken 13 times.
|
13 | auto *sync = m_session.calendarSync(); if (!sync || m_undo.isEmpty()) return false; |
| 434 |
1/1✓ Branch 11 → 12 taken 13 times.
|
26 | const auto snapshots = m_undo; sync->beginTaskBatch(); const auto end = qScopeGuard([sync] { sync->endTaskBatch(); }); |
| 435 | 13 | bool ok = true; | |
| 436 |
10/13✓ Branch 16 → 17 taken 5 times.
✓ Branch 16 → 21 taken 22 times.
✓ Branch 17 → 18 taken 5 times.
✓ Branch 19 → 20 taken 5 times.
✗ Branch 19 → 21 not taken.
✓ Branch 22 → 23 taken 5 times.
✓ Branch 22 → 24 taken 22 times.
✓ Branch 24 → 25 taken 5 times.
✓ Branch 24 → 31 taken 22 times.
✓ Branch 33 → 15 taken 27 times.
✓ Branch 33 → 34 taken 13 times.
✗ Branch 76 → 77 not taken.
✗ Branch 76 → 78 not taken.
|
40 | for (const auto &snapshot : snapshots) if (!snapshot.existed && sync->taskFor(snapshot.task.uid, snapshot.task.accountId, snapshot.task.calendarPath)) |
| 437 |
3/5✓ Branch 25 → 26 taken 5 times.
✓ Branch 26 → 27 taken 5 times.
✗ Branch 26 → 29 not taken.
✓ Branch 27 → 28 taken 5 times.
✗ Branch 27 → 29 not taken.
|
5 | ok = sync->removeTask(snapshot.task.uid, snapshot.task.accountId, snapshot.task.calendarPath) && ok; |
| 438 |
4/4✓ Branch 37 → 38 taken 22 times.
✓ Branch 37 → 61 taken 5 times.
✓ Branch 63 → 36 taken 27 times.
✓ Branch 63 → 64 taken 13 times.
|
40 | for (const auto &snapshot : snapshots) if (snapshot.existed) { |
| 439 | 22 | auto restored = snapshot.task; | |
| 440 |
11/16✓ Branch 39 → 40 taken 22 times.
✓ Branch 41 → 42 taken 4 times.
✓ Branch 41 → 46 taken 18 times.
✓ Branch 42 → 43 taken 4 times.
✓ Branch 44 → 45 taken 1 time.
✓ Branch 44 → 46 taken 3 times.
✓ Branch 47 → 48 taken 4 times.
✓ Branch 47 → 49 taken 18 times.
✓ Branch 49 → 50 taken 22 times.
✗ Branch 49 → 51 not taken.
✓ Branch 51 → 52 taken 1 time.
✓ Branch 51 → 54 taken 21 times.
✗ Branch 81 → 82 not taken.
✗ Branch 81 → 83 not taken.
✗ Branch 85 → 86 not taken.
✗ Branch 85 → 87 not taken.
|
22 | if (!sync->taskFor(restored.uid, restored.accountId, restored.calendarPath) && !sync->pendingWrite(restored.uid, restored.accountId, restored.calendarPath)) { |
| 441 | 1 | restored.etag.clear(); restored.resourceHref.clear(); | |
| 442 | } | ||
| 443 |
3/5✓ Branch 54 → 55 taken 22 times.
✓ Branch 55 → 56 taken 22 times.
✗ Branch 55 → 58 not taken.
✓ Branch 56 → 57 taken 22 times.
✗ Branch 56 → 58 not taken.
|
22 | ok = sync->saveTask(restored, false) && ok; |
| 444 | 22 | } | |
| 445 |
3/4✓ Branch 64 → 65 taken 13 times.
✗ Branch 64 → 68 not taken.
✓ Branch 65 → 66 taken 13 times.
✓ Branch 67 → 72 taken 13 times.
|
13 | if (ok) { m_undo.clear(); m_undoText.clear(); ++m_undoGeneration; emit undoChanged(); } |
| 446 | ✗ | else m_deps.setStatus(tr("Could not undo this change. Try again.")); | |
| 447 | 13 | return ok; | |
| 448 | 13 | } | |
| 449 | |||
| 450 | 31 | QVariantMap QuickTaskActions::apply(const QVariantList &identities, const QString &action, const QVariantMap &fields) { | |
| 451 | 31 | auto *sync = m_session.calendarSync(); | |
| 452 |
5/11✓ Branch 2 → 3 taken 3 times.
✓ Branch 3 → 4 taken 3 times.
✓ Branch 4 → 5 taken 3 times.
✓ Branch 6 → 7 taken 6 times.
✓ Branch 6 → 8 taken 3 times.
✗ Branch 11 → 12 not taken.
✗ Branch 11 → 13 not taken.
✗ Branch 15 → 16 not taken.
✗ Branch 15 → 19 not taken.
✗ Branch 17 → 18 not taken.
✗ Branch 17 → 19 not taken.
|
9 | const auto reject = [](const QString &error) { return QVariantMap{{"ok", false}, {"error", error}}; }; |
| 453 |
3/8✓ Branch 3 → 4 taken 31 times.
✗ Branch 3 → 6 not taken.
✗ Branch 5 → 6 not taken.
✓ Branch 5 → 7 taken 31 times.
✗ Branch 8 → 9 not taken.
✓ Branch 8 → 14 taken 31 times.
✗ Branch 9 → 10 not taken.
✗ Branch 10 → 11 not taken.
|
31 | if (!sync || identities.isEmpty()) return reject(tr("Select a task first.")); |
| 454 |
3/6✓ Branch 30 → 31 taken 372 times.
✓ Branch 30 → 32 taken 31 times.
✗ Branch 32 → 33 not taken.
✓ Branch 32 → 38 taken 31 times.
✗ Branch 738 → 739 not taken.
✗ Branch 738 → 740 not taken.
|
403 | if (!QStringList{"toggle", "complete", "reopen", "delete", "due", "priority", "move", "parent", "rename", "tags", "order", "progress"}.contains(action)) |
| 455 | ✗ | return reject(tr("Choose an available task action.")); | |
| 456 | 31 | QList<CalendarTask> tasks; QSet<QString> keys; int childCount = 0; | |
| 457 |
4/5✓ Branch 2 → 3 taken 42 times.
✓ Branch 5 → 6 taken 42 times.
✗ Branch 5 → 10 not taken.
✓ Branch 6 → 7 taken 42 times.
✓ Branch 7 → 8 taken 42 times.
|
42 | const auto add = [&](const CalendarTask &task) { if (!keys.contains(taskSelectionKey(task))) { keys.insert(taskSelectionKey(task)); tasks << task; } }; |
| 458 | // F-40 (SPRINT-104): the subtree walk needs every task of a list; load | ||
| 459 | // each list once per action, not once per selected task. | ||
| 460 | 31 | QHash<QString, QHash<QString, QList<CalendarTask>>> childrenByList; | |
| 461 | 21 | const auto subtreeCached = [&](const CalendarTask &task) { | |
| 462 |
2/2✓ Branch 4 → 5 taken 21 times.
✓ Branch 5 → 6 taken 21 times.
|
21 | const QString list = task.accountId + QLatin1Char('|') + task.calendarPath; |
| 463 |
1/1✓ Branch 7 → 8 taken 21 times.
|
21 | auto it = childrenByList.find(list); |
| 464 |
4/4✓ Branch 10 → 11 taken 20 times.
✓ Branch 10 → 15 taken 1 time.
✓ Branch 11 → 12 taken 20 times.
✓ Branch 12 → 13 taken 20 times.
|
21 | if (it == childrenByList.end()) it = childrenByList.insert(list, childrenOf(task.accountId, task.calendarPath)); |
| 465 |
1/1✓ Branch 16 → 17 taken 21 times.
|
42 | return subtreeOf(task, *it); |
| 466 | 21 | }; | |
| 467 |
2/2✓ Branch 151 → 41 taken 33 times.
✓ Branch 151 → 152 taken 30 times.
|
63 | for (const auto &value : identities) { |
| 468 |
11/11✓ Branch 42 → 43 taken 33 times.
✓ Branch 44 → 45 taken 33 times.
✓ Branch 45 → 46 taken 33 times.
✓ Branch 46 → 47 taken 33 times.
✓ Branch 48 → 49 taken 33 times.
✓ Branch 49 → 50 taken 33 times.
✓ Branch 50 → 51 taken 33 times.
✓ Branch 52 → 53 taken 33 times.
✓ Branch 53 → 54 taken 33 times.
✓ Branch 54 → 55 taken 33 times.
✓ Branch 55 → 56 taken 33 times.
|
33 | const auto id = value.toMap(); const auto task = sync->taskFor(id.value("uid").toString(), id.value("accountId").toString(), id.value("calendarPath").toString()); |
| 469 |
7/8✓ Branch 69 → 70 taken 32 times.
✓ Branch 69 → 72 taken 1 time.
✗ Branch 71 → 72 not taken.
✓ Branch 71 → 73 taken 32 times.
✓ Branch 74 → 75 taken 1 time.
✓ Branch 74 → 80 taken 32 times.
✓ Branch 75 → 76 taken 1 time.
✓ Branch 76 → 77 taken 1 time.
|
33 | if (!task || task->readOnly) return reject(tr("This list is read-only or the task is no longer available.")); |
| 470 |
1/1✓ Branch 81 → 82 taken 32 times.
|
32 | add(*task); |
| 471 |
13/13✓ Branch 83 → 84 taken 29 times.
✓ Branch 83 → 93 taken 3 times.
✓ Branch 85 → 86 taken 26 times.
✓ Branch 85 → 93 taken 3 times.
✓ Branch 87 → 88 taken 18 times.
✓ Branch 87 → 93 taken 8 times.
✓ Branch 89 → 90 taken 10 times.
✓ Branch 89 → 94 taken 8 times.
✓ Branch 91 → 92 taken 10 times.
✓ Branch 92 → 93 taken 7 times.
✓ Branch 92 → 94 taken 3 times.
✓ Branch 95 → 96 taken 21 times.
✓ Branch 95 → 140 taken 11 times.
|
32 | if (action == "delete" || action == "move" || action == "complete" || (action == "toggle" && TaskListLayout::open(*task))) { |
| 472 |
1/1✓ Branch 97 → 98 taken 21 times.
|
21 | const auto descendants = subtreeCached(*task); |
| 473 |
4/4✓ Branch 103 → 104 taken 12 times.
✓ Branch 103 → 135 taken 21 times.
✓ Branch 137 → 100 taken 33 times.
✓ Branch 137 → 138 taken 21 times.
|
54 | for (const auto &child : descendants) if (child.uid != task->uid) { |
| 474 |
5/7✓ Branch 105 → 106 taken 10 times.
✓ Branch 105 → 108 taken 2 times.
✓ Branch 106 → 107 taken 10 times.
✓ Branch 107 → 108 taken 10 times.
✗ Branch 107 → 109 not taken.
✓ Branch 110 → 111 taken 12 times.
✗ Branch 110 → 112 not taken.
|
12 | if (action == "delete" || TaskListLayout::open(child)) ++childCount; |
| 475 |
18/26✓ Branch 113 → 114 taken 8 times.
✓ Branch 113 → 120 taken 4 times.
✓ Branch 115 → 116 taken 8 times.
✓ Branch 116 → 117 taken 8 times.
✓ Branch 117 → 118 taken 8 times.
✓ Branch 119 → 120 taken 4 times.
✓ Branch 119 → 121 taken 4 times.
✓ Branch 122 → 123 taken 8 times.
✓ Branch 122 → 124 taken 4 times.
✓ Branch 124 → 125 taken 8 times.
✓ Branch 124 → 126 taken 4 times.
✓ Branch 126 → 127 taken 8 times.
✓ Branch 126 → 128 taken 4 times.
✓ Branch 128 → 129 taken 8 times.
✓ Branch 128 → 130 taken 4 times.
✓ Branch 130 → 131 taken 8 times.
✓ Branch 130 → 132 taken 4 times.
✓ Branch 131 → 135 taken 8 times.
✗ Branch 795 → 796 not taken.
✗ Branch 795 → 797 not taken.
✗ Branch 799 → 800 not taken.
✗ Branch 799 → 801 not taken.
✗ Branch 803 → 804 not taken.
✗ Branch 803 → 805 not taken.
✗ Branch 807 → 808 not taken.
✗ Branch 807 → 809 not taken.
|
12 | if (action == "move" || fields.value("childrenMode").toString() == "all") add(child); |
| 476 |
3/3✓ Branch 133 → 134 taken 2 times.
✓ Branch 133 → 135 taken 2 times.
✓ Branch 134 → 135 taken 2 times.
|
4 | else if (action == "delete") add(child); |
| 477 | } | ||
| 478 | 21 | } | |
| 479 | 34 | } | |
| 480 |
19/27✓ Branch 152 → 153 taken 6 times.
✓ Branch 152 → 162 taken 24 times.
✓ Branch 154 → 155 taken 6 times.
✓ Branch 155 → 156 taken 6 times.
✓ Branch 156 → 157 taken 6 times.
✓ Branch 158 → 159 taken 3 times.
✓ Branch 158 → 162 taken 3 times.
✓ Branch 160 → 161 taken 1 time.
✓ Branch 160 → 162 taken 2 times.
✓ Branch 163 → 164 taken 6 times.
✓ Branch 163 → 165 taken 24 times.
✓ Branch 165 → 166 taken 6 times.
✓ Branch 165 → 167 taken 24 times.
✓ Branch 167 → 168 taken 6 times.
✓ Branch 167 → 169 taken 24 times.
✓ Branch 169 → 170 taken 6 times.
✓ Branch 169 → 171 taken 24 times.
✓ Branch 171 → 172 taken 1 time.
✓ Branch 171 → 181 taken 29 times.
✗ Branch 824 → 825 not taken.
✗ Branch 824 → 826 not taken.
✗ Branch 828 → 829 not taken.
✗ Branch 828 → 830 not taken.
✗ Branch 832 → 833 not taken.
✗ Branch 832 → 834 not taken.
✗ Branch 836 → 837 not taken.
✗ Branch 836 → 838 not taken.
|
30 | if (childCount && fields.value("childrenMode").toString().isEmpty() && action != "move") |
| 481 |
2/4✓ Branch 178 → 179 taken 4 times.
✓ Branch 178 → 180 taken 1 time.
✗ Branch 841 → 842 not taken.
✗ Branch 841 → 843 not taken.
|
5 | return {{"ok", false}, {"needsChildrenChoice", true}, {"childCount", childCount}, {"action", action}}; |
| 482 | 29 | QList<CalendarTask> changed; QList<bool> deletions; | |
| 483 |
3/3✓ Branch 182 → 183 taken 29 times.
✓ Branch 183 → 184 taken 29 times.
✓ Branch 184 → 185 taken 29 times.
|
29 | const QString mode = fields.value("childrenMode").toString(); |
| 484 | 29 | QSet<QString> selected; | |
| 485 |
2/2✓ Branch 228 → 191 taken 31 times.
✓ Branch 228 → 229 taken 29 times.
|
60 | for (const auto &value : identities) { |
| 486 |
12/12✓ Branch 192 → 193 taken 31 times.
✓ Branch 195 → 196 taken 31 times.
✓ Branch 196 → 197 taken 31 times.
✓ Branch 197 → 198 taken 31 times.
✓ Branch 204 → 205 taken 31 times.
✓ Branch 205 → 206 taken 31 times.
✓ Branch 206 → 207 taken 31 times.
✓ Branch 213 → 214 taken 31 times.
✓ Branch 214 → 215 taken 31 times.
✓ Branch 215 → 216 taken 31 times.
✓ Branch 221 → 222 taken 31 times.
✓ Branch 222 → 223 taken 31 times.
|
31 | const auto id = value.toMap(); CalendarTask task; task.uid = id.value("uid").toString(); task.accountId = id.value("accountId").toString(); task.calendarPath = id.value("calendarPath").toString(); selected.insert(taskSelectionKey(task)); |
| 487 | 31 | } | |
| 488 |
4/4✓ Branch 229 → 230 taken 29 times.
✓ Branch 230 → 231 taken 29 times.
✓ Branch 620 → 232 taken 39 times.
✓ Branch 620 → 621 taken 27 times.
|
66 | for (auto task : tasks) { |
| 489 | 39 | bool remove = action == "delete"; | |
| 490 |
11/13✓ Branch 235 → 236 taken 5 times.
✓ Branch 235 → 242 taken 34 times.
✓ Branch 237 → 238 taken 3 times.
✓ Branch 237 → 242 taken 2 times.
✓ Branch 238 → 239 taken 3 times.
✓ Branch 240 → 241 taken 2 times.
✓ Branch 240 → 242 taken 1 time.
✓ Branch 243 → 244 taken 3 times.
✓ Branch 243 → 245 taken 36 times.
✓ Branch 245 → 246 taken 2 times.
✓ Branch 245 → 255 taken 37 times.
✗ Branch 901 → 902 not taken.
✗ Branch 901 → 903 not taken.
|
39 | if (remove && mode == "keep" && !selected.contains(taskSelectionKey(task))) { |
| 491 | 2 | remove = false; | |
| 492 | 2 | CalendarTask parent = task; parent.uid = task.parentUid; | |
| 493 |
3/3✓ Branch 248 → 249 taken 2 times.
✓ Branch 251 → 252 taken 1 time.
✓ Branch 251 → 253 taken 1 time.
|
2 | if (selected.contains(taskSelectionKey(parent))) task.parentUid.clear(); |
| 494 |
7/8✓ Branch 256 → 257 taken 28 times.
✓ Branch 256 → 261 taken 9 times.
✓ Branch 258 → 259 taken 16 times.
✓ Branch 258 → 261 taken 12 times.
✗ Branch 260 → 261 not taken.
✓ Branch 260 → 262 taken 16 times.
✓ Branch 263 → 264 taken 21 times.
✓ Branch 263 → 306 taken 16 times.
|
39 | } else if (action == "toggle" || action == "complete" || action == "reopen") { |
| 495 | 21 | const bool done = task.status == "COMPLETED"; | |
| 496 |
5/6✓ Branch 266 → 267 taken 9 times.
✓ Branch 266 → 270 taken 12 times.
✓ Branch 268 → 269 taken 9 times.
✗ Branch 268 → 271 not taken.
✓ Branch 269 → 270 taken 6 times.
✓ Branch 269 → 271 taken 3 times.
|
21 | const bool completing = action == "complete" || (action == "toggle" && !done); |
| 497 |
4/6✓ Branch 272 → 273 taken 18 times.
✓ Branch 272 → 276 taken 3 times.
✗ Branch 274 → 275 not taken.
✓ Branch 274 → 276 taken 18 times.
✗ Branch 277 → 278 not taken.
✓ Branch 277 → 287 taken 21 times.
|
21 | if (completing && !task.rrule.isEmpty()) { |
| 498 | ✗ | if (!TaskRecurrence::complete(&task, QDateTime::currentDateTimeUtc())) return reject(tr("This repeat rule cannot be completed here. Open it in the app that created it.")); | |
| 499 |
2/2✓ Branch 287 → 288 taken 18 times.
✓ Branch 287 → 296 taken 3 times.
|
21 | } else if (completing) { |
| 500 |
1/2✓ Branch 288 → 289 taken 18 times.
✗ Branch 288 → 291 not taken.
|
18 | if (!done) { task.previousStatus = task.status; task.previousPercent = task.percentComplete; } |
| 501 |
2/2✓ Branch 291 → 292 taken 18 times.
✓ Branch 292 → 293 taken 18 times.
|
18 | task.status = "COMPLETED"; task.percentComplete = 100; task.completedAt = QDateTime::currentDateTimeUtc(); |
| 502 | } else { | ||
| 503 |
3/3✓ Branch 297 → 298 taken 1 time.
✓ Branch 297 → 299 taken 2 times.
✓ Branch 298 → 300 taken 1 time.
|
3 | task.status = task.previousStatus.isEmpty() ? "NEEDS-ACTION" : task.previousStatus; |
| 504 | 3 | task.percentComplete = task.previousPercent; task.completedAt = {}; | |
| 505 | } | ||
| 506 |
2/2✓ Branch 307 → 308 taken 2 times.
✓ Branch 307 → 408 taken 14 times.
|
16 | } else if (action == "due") { |
| 507 |
4/4✓ Branch 309 → 310 taken 2 times.
✓ Branch 310 → 311 taken 2 times.
✓ Branch 311 → 312 taken 2 times.
✓ Branch 315 → 316 taken 2 times.
|
2 | const QString value = fields.value("due").toString(); QDate date = QDate::currentDate(); |
| 508 |
4/6✓ Branch 317 → 318 taken 1 time.
✓ Branch 317 → 319 taken 1 time.
✓ Branch 318 → 322 taken 1 time.
✗ Branch 320 → 321 not taken.
✓ Branch 320 → 322 taken 1 time.
✗ Branch 321 → 322 not taken.
|
2 | if (value == "tomorrow") date = date.addDays(1); else if (value == "nextweek") date = date.addDays(7); |
| 509 |
1/2✗ Branch 323 → 324 not taken.
✓ Branch 323 → 328 taken 2 times.
|
2 | if (value == "none") { task.due = {}; task.dueAllDay = false; } |
| 510 |
4/6✓ Branch 335 → 336 taken 6 times.
✓ Branch 335 → 337 taken 2 times.
✓ Branch 337 → 338 taken 1 time.
✓ Branch 337 → 360 taken 1 time.
✗ Branch 928 → 929 not taken.
✗ Branch 928 → 930 not taken.
|
8 | else if (QStringList{"today", "tomorrow", "nextweek"}.contains(value)) { |
| 511 |
3/5✓ Branch 338 → 339 taken 1 time.
✓ Branch 339 → 340 taken 1 time.
✗ Branch 339 → 342 not taken.
✓ Branch 340 → 341 taken 1 time.
✗ Branch 340 → 342 not taken.
|
1 | const bool timed = task.due.isValid() && !task.dueAllDay; |
| 512 | // F-30 (SPRINT-104): the kept time is the stored due's LOCAL wall | ||
| 513 | // clock — combining the local date with a UTC 23:30 due used to | ||
| 514 | // land the task on the next local day. | ||
| 515 |
4/8✓ Branch 343 → 344 taken 1 time.
✗ Branch 343 → 348 not taken.
✓ Branch 345 → 346 taken 1 time.
✓ Branch 346 → 347 taken 1 time.
✓ Branch 347 → 351 taken 1 time.
✗ Branch 350 → 351 not taken.
✗ Branch 943 → 944 not taken.
✗ Branch 943 → 945 not taken.
|
4 | task.due = timed ? QDateTime(date, task.due.toLocalTime().time(), |
| 516 |
2/5✓ Branch 344 → 345 taken 1 time.
✓ Branch 357 → 358 taken 1 time.
✗ Branch 357 → 359 not taken.
✗ Branch 947 → 948 not taken.
✗ Branch 947 → 949 not taken.
|
2 | QTimeZone::systemTimeZone()) |
| 517 |
2/8✗ Branch 348 → 349 not taken.
✗ Branch 349 → 350 not taken.
✗ Branch 353 → 354 not taken.
✓ Branch 353 → 355 taken 1 time.
✓ Branch 355 → 356 taken 1 time.
✗ Branch 355 → 357 not taken.
✗ Branch 939 → 940 not taken.
✗ Branch 939 → 941 not taken.
|
2 | : QDateTime(date, QTime(0,0), QTimeZone::utc()); |
| 518 | 1 | task.dueAllDay = !timed; | |
| 519 |
4/9✓ Branch 362 → 363 taken 1 time.
✓ Branch 365 → 366 taken 1 time.
✓ Branch 365 → 367 taken 1 time.
✗ Branch 367 → 368 not taken.
✓ Branch 367 → 373 taken 1 time.
✗ Branch 368 → 369 not taken.
✗ Branch 369 → 370 not taken.
✗ Branch 955 → 956 not taken.
✗ Branch 955 → 957 not taken.
|
2 | } else if (!TaskEditorFields::apply(task, {{"due", value}})) return reject(tr("Choose a valid due date.")); |
| 520 | // S106-349: the editor's rules hold for these shortcuts too. | ||
| 521 |
2/10✗ Branch 374 → 375 not taken.
✓ Branch 374 → 380 taken 2 times.
✗ Branch 375 → 376 not taken.
✗ Branch 376 → 377 not taken.
✗ Branch 376 → 380 not taken.
✗ Branch 377 → 378 not taken.
✗ Branch 378 → 379 not taken.
✗ Branch 378 → 380 not taken.
✗ Branch 381 → 382 not taken.
✓ Branch 381 → 387 taken 2 times.
|
2 | if (!task.rrule.isEmpty() && !task.due.isValid() && !task.dtStart.isValid()) |
| 522 | ✗ | return reject(tr("A repeating task needs a due or start date.")); | |
| 523 |
5/11✓ Branch 387 → 388 taken 2 times.
✓ Branch 388 → 389 taken 2 times.
✗ Branch 388 → 395 not taken.
✓ Branch 389 → 390 taken 2 times.
✗ Branch 390 → 391 not taken.
✓ Branch 390 → 395 taken 2 times.
✗ Branch 391 → 392 not taken.
✗ Branch 393 → 394 not taken.
✗ Branch 393 → 395 not taken.
✗ Branch 396 → 397 not taken.
✓ Branch 396 → 402 taken 2 times.
|
2 | if (task.due.isValid() && task.dtStart.isValid() && task.due < task.dtStart) |
| 524 | ✗ | return reject(tr("The due date cannot be before the start date.")); | |
| 525 |
3/4✓ Branch 404 → 405 taken 2 times.
✗ Branch 404 → 407 not taken.
✓ Branch 409 → 410 taken 1 time.
✓ Branch 409 → 429 taken 13 times.
|
16 | } else if (action == "priority") { |
| 526 |
6/11✓ Branch 411 → 412 taken 1 time.
✓ Branch 412 → 413 taken 1 time.
✓ Branch 415 → 416 taken 1 time.
✓ Branch 418 → 419 taken 1 time.
✓ Branch 418 → 420 taken 1 time.
✗ Branch 423 → 424 not taken.
✓ Branch 423 → 612 taken 1 time.
✗ Branch 424 → 425 not taken.
✗ Branch 425 → 426 not taken.
✗ Branch 980 → 981 not taken.
✗ Branch 980 → 982 not taken.
|
3 | if (!TaskEditorFields::apply(task, {{"priority", fields.value("priority")}})) return reject(tr("Choose High, Normal, Low or None.")); |
| 527 |
2/2✓ Branch 430 → 431 taken 2 times.
✓ Branch 430 → 450 taken 11 times.
|
13 | } else if (action == "rename") { |
| 528 |
6/11✓ Branch 432 → 433 taken 2 times.
✓ Branch 433 → 434 taken 2 times.
✓ Branch 436 → 437 taken 2 times.
✓ Branch 439 → 440 taken 2 times.
✓ Branch 439 → 441 taken 2 times.
✗ Branch 444 → 445 not taken.
✓ Branch 444 → 612 taken 2 times.
✗ Branch 445 → 446 not taken.
✗ Branch 446 → 447 not taken.
✗ Branch 1005 → 1006 not taken.
✗ Branch 1005 → 1007 not taken.
|
6 | if (!TaskEditorFields::apply(task, {{"summary", fields.value("summary")}})) return reject(tr("Enter a task title.")); |
| 529 |
1/5✗ Branch 451 → 452 not taken.
✓ Branch 451 → 462 taken 11 times.
✗ Branch 453 → 454 not taken.
✗ Branch 454 → 455 not taken.
✗ Branch 455 → 456 not taken.
|
11 | } else if (action == "tags") task.tags = fields.value("tags").toStringList(); |
| 530 |
2/2✓ Branch 463 → 464 taken 2 times.
✓ Branch 463 → 501 taken 9 times.
|
11 | else if (action == "progress") { |
| 531 | // F94-28: progress from the row; 1–99 % means "In progress", 0 % reopens | ||
| 532 | // an in-progress task, 100 % completes it (D-4). | ||
| 533 |
4/4✓ Branch 465 → 466 taken 2 times.
✓ Branch 466 → 467 taken 2 times.
✓ Branch 467 → 468 taken 2 times.
✓ Branch 468 → 469 taken 2 times.
|
2 | const int percent = qBound(0, fields.value("percent").toInt(), 100); |
| 534 | // F-30 (SPRINT-104): 100 % on a repeating task ADVANCES the series | ||
| 535 | // (TaskRecurrence::complete) instead of ending it — the row action | ||
| 536 | // used to mark the whole series completed. | ||
| 537 |
4/6✓ Branch 472 → 473 taken 1 time.
✓ Branch 472 → 476 taken 1 time.
✗ Branch 474 → 475 not taken.
✓ Branch 474 → 476 taken 1 time.
✗ Branch 477 → 478 not taken.
✓ Branch 477 → 487 taken 2 times.
|
2 | if (percent == 100 && !task.rrule.isEmpty()) { |
| 538 | ✗ | if (!TaskRecurrence::complete(&task, QDateTime::currentDateTimeUtc())) | |
| 539 | ✗ | return reject(tr("This repeat rule cannot be completed here. Open it in the app that created it.")); | |
| 540 | } else { | ||
| 541 | 2 | task.percentComplete = percent; | |
| 542 |
2/2✓ Branch 487 → 488 taken 1 time.
✓ Branch 487 → 496 taken 1 time.
|
2 | if (percent == 100) { |
| 543 |
1/1✓ Branch 490 → 491 taken 1 time.
|
1 | task.previousStatus = task.status; task.previousPercent = tasks[changed.size()].percentComplete; |
| 544 |
2/2✓ Branch 491 → 492 taken 1 time.
✓ Branch 492 → 493 taken 1 time.
|
1 | task.status = "COMPLETED"; task.completedAt = QDateTime::currentDateTimeUtc(); |
| 545 |
2/3✓ Branch 496 → 497 taken 1 time.
✗ Branch 496 → 498 not taken.
✓ Branch 497 → 612 taken 1 time.
|
1 | } else if (percent > 0) task.status = "IN-PROCESS"; |
| 546 | ✗ | else if (task.status == "IN-PROCESS") task.status = "NEEDS-ACTION"; | |
| 547 | } | ||
| 548 | } | ||
| 549 |
1/5✗ Branch 502 → 503 not taken.
✓ Branch 502 → 511 taken 9 times.
✗ Branch 504 → 505 not taken.
✗ Branch 505 → 506 not taken.
✗ Branch 506 → 507 not taken.
|
9 | else if (action == "order") task.sortOrder = fields.value("sortOrder").toLongLong(); |
| 550 |
2/2✓ Branch 512 → 513 taken 1 time.
✓ Branch 512 → 529 taken 8 times.
|
9 | else if (action == "parent") { |
| 551 |
3/3✓ Branch 514 → 515 taken 1 time.
✓ Branch 515 → 516 taken 1 time.
✓ Branch 516 → 517 taken 1 time.
|
1 | task.parentUid = fields.value("parentUid").toString(); |
| 552 |
4/5✓ Branch 522 → 523 taken 1 time.
✓ Branch 523 → 524 taken 1 time.
✗ Branch 523 → 612 not taken.
✓ Branch 524 → 525 taken 1 time.
✓ Branch 525 → 526 taken 1 time.
|
1 | if (!validParent(task, task.parentUid)) return reject(tr("Choose a parent in this list without creating a loop.")); |
| 553 |
2/2✓ Branch 530 → 531 taken 5 times.
✓ Branch 530 → 612 taken 3 times.
|
8 | } else if (action == "move") { |
| 554 |
6/6✓ Branch 532 → 533 taken 5 times.
✓ Branch 533 → 534 taken 5 times.
✓ Branch 534 → 535 taken 5 times.
✓ Branch 539 → 540 taken 5 times.
✓ Branch 540 → 541 taken 5 times.
✓ Branch 541 → 542 taken 5 times.
|
5 | const QString account = fields.value("accountId").toString(), path = fields.value("calendarPath").toString(); |
| 555 | 5 | bool allowed = false; | |
| 556 |
13/16✓ Branch 545 → 546 taken 5 times.
✓ Branch 546 → 547 taken 5 times.
✓ Branch 547 → 548 taken 5 times.
✓ Branch 551 → 552 taken 10 times.
✗ Branch 551 → 558 not taken.
✓ Branch 553 → 554 taken 4 times.
✓ Branch 553 → 558 taken 6 times.
✓ Branch 554 → 555 taken 4 times.
✓ Branch 555 → 556 taken 4 times.
✗ Branch 555 → 558 not taken.
✓ Branch 556 → 557 taken 4 times.
✗ Branch 556 → 558 not taken.
✓ Branch 559 → 560 taken 4 times.
✓ Branch 559 → 561 taken 6 times.
✓ Branch 563 → 549 taken 10 times.
✓ Branch 563 → 564 taken 5 times.
|
15 | for (const auto &calendar : sync->calendars()) if (calendar.accountId == account && calendar.path == path && calendar.supportsTasks() && !calendar.readOnly) allowed = true; |
| 557 |
4/4✓ Branch 565 → 566 taken 1 time.
✓ Branch 565 → 571 taken 4 times.
✓ Branch 566 → 567 taken 1 time.
✓ Branch 567 → 568 taken 1 time.
|
5 | if (!allowed) return reject(tr("Choose a writable task list.")); |
| 558 |
3/6✓ Branch 572 → 573 taken 4 times.
✗ Branch 572 → 575 not taken.
✓ Branch 574 → 575 taken 4 times.
✗ Branch 574 → 576 not taken.
✓ Branch 577 → 578 taken 4 times.
✗ Branch 577 → 602 not taken.
|
4 | if (task.accountId != account || task.calendarPath != path) { |
| 559 |
2/5✓ Branch 578 → 579 taken 4 times.
✗ Branch 581 → 582 not taken.
✓ Branch 581 → 587 taken 4 times.
✗ Branch 582 → 583 not taken.
✗ Branch 583 → 584 not taken.
|
4 | if (sync->taskFor(task.uid, account, path)) return reject(tr("This task already exists in the target list.")); |
| 560 | 4 | task.accountId = account; task.calendarPath = path; task.etag.clear(); task.resourceHref.clear(); | |
| 561 |
1/1✓ Branch 592 → 593 taken 4 times.
|
4 | CalendarTask parent = tasks[changed.size()]; parent.uid = parent.parentUid; |
| 562 |
3/3✓ Branch 595 → 596 taken 4 times.
✓ Branch 598 → 599 taken 2 times.
✓ Branch 598 → 600 taken 2 times.
|
4 | if (!keys.contains(taskSelectionKey(parent))) task.parentUid.clear(); |
| 563 | 4 | } | |
| 564 | 6 | } | |
| 565 |
2/2✓ Branch 612 → 613 taken 37 times.
✓ Branch 613 → 614 taken 37 times.
|
37 | changed << task; deletions << remove; |
| 566 | 39 | } | |
| 567 |
1/1✓ Branch 621 → 623 taken 27 times.
|
54 | QList<Snapshot> snapshots; sync->beginTaskBatch(); const auto end = qScopeGuard([sync] { sync->endTaskBatch(); }); |
| 568 |
2/2✓ Branch 677 → 625 taken 37 times.
✓ Branch 677 → 678 taken 27 times.
|
64 | for (int i = 0; i < tasks.size(); ++i) { |
| 569 |
2/2✓ Branch 625 → 626 taken 37 times.
✓ Branch 626 → 627 taken 37 times.
|
37 | const auto &before = tasks[i]; const auto &next = changed[i]; |
| 570 |
2/2✓ Branch 627 → 628 taken 37 times.
✓ Branch 628 → 629 taken 37 times.
|
37 | const bool moved = taskSelectionKey(before) != taskSelectionKey(next); |
| 571 |
10/11✓ Branch 632 → 633 taken 37 times.
✓ Branch 633 → 634 taken 34 times.
✓ Branch 633 → 639 taken 3 times.
✓ Branch 635 → 636 taken 19 times.
✓ Branch 635 → 639 taken 15 times.
✓ Branch 637 → 638 taken 19 times.
✗ Branch 637 → 639 not taken.
✓ Branch 640 → 641 taken 19 times.
✓ Branch 640 → 645 taken 18 times.
✓ Branch 641 → 642 taken 19 times.
✓ Branch 642 → 643 taken 19 times.
|
37 | if (!deletions[i] && next.status == "COMPLETED" && before.status != "COMPLETED") m_tasks.holdCompleted(taskSelectionKey(before)); |
| 572 |
8/8✓ Branch 645 → 646 taken 37 times.
✓ Branch 646 → 647 taken 3 times.
✓ Branch 646 → 649 taken 34 times.
✓ Branch 647 → 648 taken 3 times.
✓ Branch 649 → 650 taken 4 times.
✓ Branch 649 → 652 taken 30 times.
✓ Branch 650 → 651 taken 4 times.
✓ Branch 652 → 653 taken 30 times.
|
37 | const bool ok = deletions[i] ? sync->removeTask(before.uid, before.accountId, before.calendarPath) : moved ? sync->moveTask(before, next) : sync->saveTask(next, false); |
| 573 |
1/2✗ Branch 655 → 656 not taken.
✓ Branch 655 → 667 taken 37 times.
|
37 | if (!ok) { |
| 574 | ✗ | if (!snapshots.isEmpty()) remember(tr("Task changes"), snapshots); | |
| 575 | ✗ | return reject(tr("Could not save all changes. Undo the saved changes or try again.")); | |
| 576 | } | ||
| 577 |
4/8✓ Branch 668 → 669 taken 37 times.
✓ Branch 670 → 671 taken 4 times.
✓ Branch 670 → 675 taken 33 times.
✓ Branch 672 → 673 taken 4 times.
✗ Branch 1135 → 1136 not taken.
✗ Branch 1135 → 1137 not taken.
✗ Branch 1141 → 1142 not taken.
✗ Branch 1141 → 1143 not taken.
|
37 | snapshots << Snapshot{before, true}; if (moved) snapshots << Snapshot{next, false}; |
| 578 | } | ||
| 579 |
12/12✓ Branch 679 → 680 taken 3 times.
✓ Branch 679 → 681 taken 24 times.
✓ Branch 680 → 691 taken 3 times.
✓ Branch 682 → 683 taken 2 times.
✓ Branch 682 → 684 taken 22 times.
✓ Branch 683 → 691 taken 2 times.
✓ Branch 685 → 686 taken 16 times.
✓ Branch 685 → 688 taken 6 times.
✓ Branch 687 → 688 taken 9 times.
✓ Branch 687 → 690 taken 7 times.
✓ Branch 688 → 689 taken 15 times.
✓ Branch 690 → 691 taken 7 times.
|
27 | const QString text = action == "delete" ? tr("Task deleted") : action == "move" ? tr("Task moved") : action == "complete" || action == "toggle" ? tr("Task completed") : tr("Task changed"); |
| 580 |
1/1✓ Branch 691 → 692 taken 27 times.
|
27 | remember(text, snapshots); |
| 581 |
6/6✓ Branch 693 → 694 taken 2 times.
✓ Branch 693 → 700 taken 25 times.
✓ Branch 694 → 695 taken 2 times.
✓ Branch 695 → 696 taken 2 times.
✓ Branch 696 → 697 taken 2 times.
✓ Branch 697 → 698 taken 2 times.
|
27 | if (action == "move") setLastList(calendarSelectionKey(changed.first().accountId, changed.first().calendarPath)); |
| 582 |
8/8✓ Branch 700 → 701 taken 27 times.
✓ Branch 704 → 705 taken 27 times.
✓ Branch 705 → 706 taken 27 times.
✓ Branch 706 → 707 taken 27 times.
✓ Branch 707 → 708 taken 27 times.
✓ Branch 709 → 710 taken 27 times.
✓ Branch 713 → 702 taken 27 times.
✓ Branch 713 → 714 taken 27 times.
|
54 | qCInfo(lcTaskActions) << "Applied task action" << action << "count" << tasks.size(); |
| 583 |
2/4✓ Branch 720 → 721 taken 81 times.
✓ Branch 720 → 722 taken 27 times.
✗ Branch 1158 → 1159 not taken.
✗ Branch 1158 → 1160 not taken.
|
108 | return {{"ok", true}, {"undoText", text}, {"count", tasks.size()}}; |
| 584 | 99 | } | |
| 585 | |||
| 586 | 4 | bool QuickTaskActions::resolveConflict(const QVariantMap &identity, const QString &choice) { | |
| 587 | 4 | auto *sync = m_session.calendarSync(); | |
| 588 |
29/76✓ Branch 3 → 4 taken 4 times.
✗ Branch 3 → 25 not taken.
✓ Branch 9 → 10 taken 4 times.
✗ Branch 9 → 25 not taken.
✓ Branch 11 → 12 taken 4 times.
✓ Branch 12 → 13 taken 4 times.
✓ Branch 13 → 14 taken 4 times.
✓ Branch 15 → 16 taken 4 times.
✓ Branch 16 → 17 taken 4 times.
✓ Branch 17 → 18 taken 4 times.
✓ Branch 19 → 20 taken 4 times.
✓ Branch 20 → 21 taken 4 times.
✓ Branch 21 → 22 taken 4 times.
✓ Branch 22 → 23 taken 4 times.
✓ Branch 23 → 24 taken 4 times.
✗ Branch 23 → 25 not taken.
✓ Branch 27 → 28 taken 4 times.
✗ Branch 27 → 29 not taken.
✓ Branch 29 → 30 taken 4 times.
✗ Branch 29 → 31 not taken.
✓ Branch 31 → 32 taken 4 times.
✗ Branch 31 → 33 not taken.
✓ Branch 33 → 34 taken 4 times.
✗ Branch 33 → 35 not taken.
✓ Branch 35 → 36 taken 4 times.
✗ Branch 35 → 37 not taken.
✓ Branch 37 → 38 taken 4 times.
✗ Branch 37 → 39 not taken.
✓ Branch 39 → 40 taken 4 times.
✗ Branch 39 → 41 not taken.
✓ Branch 41 → 42 taken 4 times.
✗ Branch 41 → 43 not taken.
✓ Branch 43 → 44 taken 4 times.
✗ Branch 43 → 45 not taken.
✓ Branch 45 → 46 taken 4 times.
✗ Branch 45 → 47 not taken.
✓ Branch 47 → 48 taken 4 times.
✗ Branch 47 → 49 not taken.
✓ Branch 49 → 50 taken 4 times.
✗ Branch 49 → 51 not taken.
✓ Branch 51 → 52 taken 4 times.
✗ Branch 51 → 53 not taken.
✓ Branch 53 → 54 taken 4 times.
✗ Branch 53 → 57 not taken.
✓ Branch 55 → 56 taken 12 times.
✓ Branch 55 → 57 taken 4 times.
✗ Branch 59 → 60 not taken.
✗ Branch 59 → 61 not taken.
✗ Branch 63 → 64 not taken.
✗ Branch 63 → 65 not taken.
✗ Branch 67 → 68 not taken.
✗ Branch 67 → 69 not taken.
✗ Branch 71 → 72 not taken.
✗ Branch 71 → 73 not taken.
✗ Branch 75 → 76 not taken.
✗ Branch 75 → 77 not taken.
✗ Branch 79 → 80 not taken.
✗ Branch 79 → 81 not taken.
✗ Branch 83 → 84 not taken.
✗ Branch 83 → 85 not taken.
✗ Branch 87 → 88 not taken.
✗ Branch 87 → 89 not taken.
✗ Branch 91 → 92 not taken.
✗ Branch 91 → 93 not taken.
✗ Branch 95 → 96 not taken.
✗ Branch 95 → 97 not taken.
✗ Branch 99 → 100 not taken.
✗ Branch 99 → 101 not taken.
✗ Branch 103 → 104 not taken.
✗ Branch 103 → 105 not taken.
✗ Branch 107 → 108 not taken.
✗ Branch 107 → 109 not taken.
✗ Branch 111 → 112 not taken.
✗ Branch 111 → 115 not taken.
✗ Branch 113 → 114 not taken.
✗ Branch 113 → 115 not taken.
|
20 | return sync && QStringList{"mine", "server", "both"}.contains(choice) && sync->resolveEventConflict(identity.value("uid").toString(), identity.value("accountId").toString(), identity.value("calendarPath").toString(), choice); |
| 589 | 4 | } | |
| 590 | ✗ | bool QuickTaskActions::retryWrite(const QVariantMap &identity) { | |
| 591 | ✗ | auto *sync = m_session.calendarSync(); | |
| 592 | ✗ | return sync && sync->outbox() && sync->outbox()->retryWrite(identity.value("uid").toString(), identity.value("accountId").toString(), identity.value("calendarPath").toString()); | |
| 593 | } | ||
| 594 | ✗ | bool QuickTaskActions::discardChange(const QVariantMap &identity) { | |
| 595 | ✗ | auto *sync = m_session.calendarSync(); | |
| 596 | ✗ | return sync && sync->outbox() && sync->outbox()->discardLocalChange(identity.value("uid").toString(), identity.value("accountId").toString(), identity.value("calendarPath").toString()); | |
| 597 | } | ||
| 598 | ✗ | QVariantMap QuickTaskActions::convertChecklist(const QVariantMap &identity, const QString &description, int line) { | |
| 599 | ✗ | auto *sync = m_session.calendarSync(); | |
| 600 | ✗ | const auto original = sync ? sync->taskFor(identity.value("uid").toString(), identity.value("accountId").toString(), identity.value("calendarPath").toString()) : std::nullopt; | |
| 601 | ✗ | QStringList lines = description.split('\n'); | |
| 602 | ✗ | if (!original || original->readOnly || line < 0 || line >= lines.size()) return {{"ok", false}, {"error", tr("Choose a checklist item in a writable list.")}}; | |
| 603 | ✗ | const auto match = QRegularExpression("^\\s*[-*+]\\s+\\[([ xX])\\]\\s*(.+)$").match(lines[line]); | |
| 604 | ✗ | if (!match.hasMatch()) return {{"ok", false}, {"error", tr("Enter a checklist title before converting it.")}}; | |
| 605 | ✗ | CalendarTask parent = *original, child; | |
| 606 | ✗ | child.uid = QUuid::createUuid().toString(QUuid::WithoutBraces); child.accountId = parent.accountId; child.calendarPath = parent.calendarPath; | |
| 607 | ✗ | child.parentUid = parent.uid; child.summary = match.captured(2).trimmed(); child.created = QDateTime::currentDateTimeUtc(); | |
| 608 | ✗ | const bool done = match.captured(1) != " "; child.status = done ? "COMPLETED" : "NEEDS-ACTION"; child.percentComplete = done ? 100 : 0; | |
| 609 | ✗ | if (done) child.completedAt = child.created; | |
| 610 | ✗ | lines.removeAt(line); parent.description = lines.join('\n'); | |
| 611 | ✗ | sync->beginTaskBatch(); const auto end = qScopeGuard([sync] { sync->endTaskBatch(); }); | |
| 612 | ✗ | if (!sync->saveTask(child, true)) return {{"ok", false}, {"error", tr("Could not create the subtask. Try again.")}}; | |
| 613 | ✗ | remember(tr("Checklist item converted"), {{child, false}, {*original, true}}); | |
| 614 | ✗ | if (!sync->saveTask(parent, false)) return {{"ok", false}, {"error", tr("Could not update the checklist. Undo or try again.")}}; | |
| 615 | ✗ | return {{"ok", true}, {"description", parent.description}, {"uid", child.uid}}; | |
| 616 | ✗ | } | |
| 617 | |||
| 618 | 1 | bool QuickTaskActions::open(const QVariantMap &identity) { | |
| 619 |
10/10✓ Branch 3 → 4 taken 1 time.
✓ Branch 4 → 5 taken 1 time.
✓ Branch 5 → 6 taken 1 time.
✓ Branch 7 → 8 taken 1 time.
✓ Branch 8 → 9 taken 1 time.
✓ Branch 9 → 10 taken 1 time.
✓ Branch 11 → 12 taken 1 time.
✓ Branch 12 → 13 taken 1 time.
✓ Branch 13 → 14 taken 1 time.
✓ Branch 14 → 15 taken 1 time.
|
1 | const auto task = taskRecord(identity.value("uid").toString(),identity.value("accountId").toString(),identity.value("calendarPath").toString()); |
| 620 |
2/3✓ Branch 27 → 28 taken 1 time.
✗ Branch 28 → 29 not taken.
✓ Branch 28 → 30 taken 1 time.
|
1 | if (task.isEmpty()) return false; |
| 621 |
1/1✓ Branch 30 → 31 taken 1 time.
|
1 | emit openRequested(task); return true; |
| 622 | 1 | } | |
| 623 | |||
| 624 | 774 | void QuickTaskActions::setDefaultReminder(int value) { | |
| 625 |
5/6✓ Branch 2 → 3 taken 7 times.
✓ Branch 2 → 4 taken 767 times.
✓ Branch 3 → 4 taken 7 times.
✗ Branch 3 → 5 not taken.
✓ Branch 4 → 5 taken 763 times.
✓ Branch 4 → 6 taken 11 times.
|
774 | if ((value != 0 && value != 1) || value == m_defaultReminder) return; |
| 626 | 11 | m_defaultReminder = value; emit stateChanged(); | |
| 627 | } | ||
| 628 | 15 | void QuickTaskActions::addDefaultReminder(CalendarTask *task) const { | |
| 629 |
7/10✓ Branch 2 → 3 taken 1 time.
✓ Branch 2 → 9 taken 14 times.
✓ Branch 3 → 4 taken 1 time.
✗ Branch 3 → 9 not taken.
✓ Branch 5 → 6 taken 1 time.
✗ Branch 5 → 9 not taken.
✓ Branch 7 → 8 taken 1 time.
✗ Branch 7 → 9 not taken.
✓ Branch 10 → 11 taken 1 time.
✓ Branch 10 → 22 taken 14 times.
|
15 | if (m_defaultReminder == 1 && task && task->due.isValid() && task->alarmTimes.isEmpty()) |
| 630 |
6/12✓ Branch 11 → 12 taken 1 time.
✓ Branch 12 → 13 taken 1 time.
✗ Branch 12 → 14 not taken.
✓ Branch 13 → 16 taken 1 time.
✗ Branch 14 → 15 not taken.
✗ Branch 15 → 16 not taken.
✓ Branch 16 → 17 taken 1 time.
✓ Branch 17 → 18 taken 1 time.
✗ Branch 19 → 20 not taken.
✓ Branch 19 → 21 taken 1 time.
✗ Branch 25 → 26 not taken.
✗ Branch 25 → 27 not taken.
|
1 | task->alarmTimes << QDateTime(task->dueAllDay ? task->due.date() : task->due.toLocalTime().date(),QTime(9,0)); |
| 631 | 15 | } | |
| 632 |