presentation/QuickContactActionsMet.cpp
| Line | Branch | Exec | Source |
|---|---|---|---|
| 1 | // "Met someone…" of QuickContactActions (Sprint 103 S103.3.5, I-4, | ||
| 2 | // interaction.md §5): the prefill from the calendar (the event running now | ||
| 3 | // or ended in the last six hours, its place and people), the duplicate | ||
| 4 | // guard and the creation of a card with its first meeting — or only the | ||
| 5 | // meeting on an existing card. Introducers are never guessed: the event's | ||
| 6 | // people are offered, not filled in. | ||
| 7 | |||
| 8 | #include "QuickContactActions.h" | ||
| 9 | #include "QuickContactModel.h" | ||
| 10 | |||
| 11 | #include "app/core/ContactSyncCoordinator.h" | ||
| 12 | #include "data/CalendarStore.h" | ||
| 13 | #include "data/ContactIndex.h" | ||
| 14 | #include "data/ContactQueries.h" | ||
| 15 | #include "data/ContactStore.h" | ||
| 16 | #include "data/ICalEventCodec.h" | ||
| 17 | #include "data/ICalendar.h" | ||
| 18 | #include "data/RecurrenceExpansion.h" | ||
| 19 | |||
| 20 | #include <QDate> | ||
| 21 | #include <QDateTime> | ||
| 22 | #include <QLocale> | ||
| 23 | #include <QLoggingCategory> | ||
| 24 | #include <QRegularExpression> | ||
| 25 | #include <QSqlQuery> | ||
| 26 | #include <limits> | ||
| 27 | |||
| 28 | Q_DECLARE_LOGGING_CATEGORY(lcContactRelationActions) | ||
| 29 | |||
| 30 | namespace { | ||
| 31 | |||
| 32 |
3/9✓ Branch 10 → 11 taken 1 time.
✓ Branch 12 → 13 taken 2 times.
✓ Branch 12 → 14 taken 1 time.
✗ Branch 21 → 22 not taken.
✗ Branch 21 → 23 not taken.
✗ Branch 37 → 38 not taken.
✗ Branch 37 → 41 not taken.
✗ Branch 39 → 40 not taken.
✗ Branch 39 → 41 not taken.
|
5 | QVariantMap failure(const QString &error) { return {{QStringLiteral("ok"), false}, {QStringLiteral("error"), error}}; } |
| 33 | |||
| 34 | // The event of the moment: running now, else the one that ended last within | ||
| 35 | // six hours. All-day events are no occasion. | ||
| 36 | // S106-265: occurrences, not stored rows — a running repetition of a series | ||
| 37 | // has the master's first dates in its row and was never "of the moment". | ||
| 38 | 3 | std::optional<CalendarEvent> eventOfTheMoment(const CalendarStore &store, const QDateTime &now) { | |
| 39 | 3 | std::optional<CalendarEvent> best; | |
| 40 | const QList<CalendarEvent> occurrences = expandEventsForDisplay( | ||
| 41 |
6/6✓ Branch 3 → 4 taken 3 times.
✓ Branch 4 → 5 taken 3 times.
✓ Branch 5 → 6 taken 3 times.
✓ Branch 6 → 7 taken 3 times.
✓ Branch 7 → 8 taken 3 times.
✓ Branch 8 → 9 taken 3 times.
|
6 | store.eventsForDateRange(now.addDays(-1), now.addSecs(60)), now.date().addDays(-1), now.date(), false, |
| 42 |
1/1✓ Branch 9 → 10 taken 3 times.
|
6 | {kDisplayOccurrencesPerSeries, {}}); |
| 43 |
2/2✓ Branch 86 → 16 taken 3 times.
✓ Branch 86 → 87 taken 3 times.
|
6 | for (const CalendarEvent &event : occurrences) { |
| 44 |
6/10✓ Branch 17 → 18 taken 3 times.
✗ Branch 17 → 23 not taken.
✓ Branch 18 → 19 taken 3 times.
✓ Branch 19 → 20 taken 3 times.
✗ Branch 19 → 23 not taken.
✓ Branch 20 → 21 taken 3 times.
✗ Branch 22 → 23 not taken.
✓ Branch 22 → 24 taken 3 times.
✗ Branch 25 → 26 not taken.
✓ Branch 25 → 27 taken 3 times.
|
3 | if (event.allDay || !event.dtStart.isValid() || event.dtStart > now) |
| 45 | 1 | continue; | |
| 46 |
2/3✓ Branch 27 → 28 taken 3 times.
✓ Branch 28 → 29 taken 3 times.
✗ Branch 28 → 30 not taken.
|
3 | const QDateTime end = event.dtEnd.isValid() ? event.dtEnd : event.dtStart; |
| 47 |
4/4✓ Branch 32 → 33 taken 3 times.
✓ Branch 33 → 34 taken 3 times.
✓ Branch 36 → 37 taken 1 time.
✓ Branch 36 → 38 taken 2 times.
|
3 | if (end < now.addSecs(-6 * 3600)) |
| 48 | 1 | continue; | |
| 49 |
1/1✓ Branch 38 → 39 taken 2 times.
|
2 | const bool running = end >= now; |
| 50 |
1/8✗ Branch 41 → 42 not taken.
✓ Branch 41 → 53 taken 2 times.
✗ Branch 43 → 44 not taken.
✗ Branch 44 → 45 not taken.
✗ Branch 44 → 47 not taken.
✗ Branch 49 → 50 not taken.
✗ Branch 51 → 52 not taken.
✗ Branch 51 → 53 not taken.
|
2 | const bool bestRunning = best && (best->dtEnd.isValid() ? best->dtEnd : best->dtStart) >= now; |
| 51 |
2/12✗ Branch 55 → 56 not taken.
✓ Branch 55 → 74 taken 2 times.
✗ Branch 56 → 57 not taken.
✗ Branch 56 → 58 not taken.
✗ Branch 57 → 58 not taken.
✗ Branch 57 → 74 not taken.
✗ Branch 58 → 59 not taken.
✗ Branch 58 → 75 not taken.
✗ Branch 59 → 60 not taken.
✗ Branch 59 → 64 not taken.
✓ Branch 76 → 77 taken 2 times.
✗ Branch 76 → 78 not taken.
|
2 | if (!best || (running && !bestRunning) || |
| 52 | ✗ | (running == bestRunning && (running ? event.dtStart > best->dtStart | |
| 53 | ✗ | : end > (best->dtEnd.isValid() ? best->dtEnd : best->dtStart)))) | |
| 54 | 2 | best = event; | |
| 55 | 3 | } | |
| 56 | 3 | return best; | |
| 57 | 6 | } | |
| 58 | |||
| 59 | } // namespace | ||
| 60 | |||
| 61 | 7 | QString QuickContactActions::cardKeyForUid(const QString &uid, qint64 preferredBook) const { | |
| 62 |
1/1✓ Branch 2 → 3 taken 7 times.
|
7 | ContactSyncCoordinator *sync = m_coordinator(); |
| 63 |
3/6✓ Branch 3 → 4 taken 7 times.
✗ Branch 3 → 6 not taken.
✗ Branch 5 → 6 not taken.
✓ Branch 5 → 7 taken 7 times.
✗ Branch 8 → 9 not taken.
✓ Branch 8 → 10 taken 7 times.
|
7 | if (!sync || uid.isEmpty()) |
| 64 | ✗ | return {}; | |
| 65 |
2/2✓ Branch 11 → 12 taken 7 times.
✓ Branch 12 → 13 taken 7 times.
|
7 | QSqlQuery q(sync->store()->database()); |
| 66 | // The calendar's people index keeps UIDs folded to lower case (S103.1.10): | ||
| 67 | // an exact match first, then one without regard to case. | ||
| 68 |
1/2✓ Branch 51 → 16 taken 7 times.
✗ Branch 51 → 52 not taken.
|
7 | for (const QLatin1String match : {QLatin1String("c.uid = ?"), QLatin1String("c.uid = ? COLLATE NOCASE")}) { |
| 69 |
1/1✓ Branch 21 → 22 taken 7 times.
|
21 | q.prepare(QStringLiteral("SELECT c.book_id, c.uid FROM cards c LEFT JOIN address_books b ON b.id = c.book_id " |
| 70 | "WHERE %1 ORDER BY c.book_id <> ?, coalesce(b.read_only, 1), c.id LIMIT 1") | ||
| 71 |
1/1✓ Branch 20 → 21 taken 7 times.
|
21 | .arg(match)); |
| 72 |
1/1✓ Branch 27 → 28 taken 7 times.
|
7 | q.addBindValue(uid); |
| 73 |
1/1✓ Branch 31 → 32 taken 7 times.
|
7 | q.addBindValue(preferredBook); |
| 74 |
5/8✓ Branch 33 → 34 taken 7 times.
✓ Branch 34 → 35 taken 7 times.
✗ Branch 34 → 38 not taken.
✓ Branch 35 → 36 taken 7 times.
✓ Branch 36 → 37 taken 7 times.
✗ Branch 36 → 38 not taken.
✓ Branch 39 → 40 taken 7 times.
✗ Branch 39 → 50 not taken.
|
7 | if (q.exec() && q.next()) |
| 75 |
5/5✓ Branch 40 → 41 taken 7 times.
✓ Branch 41 → 42 taken 7 times.
✓ Branch 42 → 43 taken 7 times.
✓ Branch 43 → 44 taken 7 times.
✓ Branch 44 → 45 taken 7 times.
|
7 | return ContactQueries::cardKey(q.value(0).toLongLong(), q.value(1).toString()); |
| 76 | } | ||
| 77 | ✗ | return {}; | |
| 78 | 7 | } | |
| 79 | |||
| 80 | 2 | QVariantMap QuickContactActions::personForUid(const QString &uid) const { | |
| 81 |
1/1✓ Branch 2 → 3 taken 2 times.
|
2 | const QString key = cardKeyForUid(uid); |
| 82 |
2/3✗ Branch 4 → 5 not taken.
✓ Branch 4 → 6 taken 2 times.
✓ Branch 6 → 7 taken 2 times.
|
4 | return key.isEmpty() ? QVariantMap{} : personEntry(key); |
| 83 | 2 | } | |
| 84 | |||
| 85 | 8 | QVariantMap QuickContactActions::introductionFor(const QString &senderHeader, const QString &address, | |
| 86 | const QString &dateIso) const { | ||
| 87 |
4/7✓ Branch 2 → 3 taken 2 times.
✓ Branch 2 → 15 taken 6 times.
✓ Branch 4 → 5 taken 2 times.
✗ Branch 4 → 15 not taken.
✓ Branch 9 → 10 taken 2 times.
✗ Branch 147 → 148 not taken.
✗ Branch 147 → 149 not taken.
|
10 | static const QRegularExpression angle(QStringLiteral("<([^<>@\\s]+@[^<>\\s]+)>")); |
| 88 |
1/1✓ Branch 16 → 17 taken 8 times.
|
8 | const QRegularExpressionMatch match = angle.match(senderHeader); |
| 89 |
5/5✓ Branch 17 → 18 taken 8 times.
✓ Branch 18 → 19 taken 7 times.
✓ Branch 18 → 20 taken 1 time.
✓ Branch 19 → 21 taken 7 times.
✓ Branch 21 → 22 taken 8 times.
|
8 | const QString sender = (match.hasMatch() ? match.captured(1) : senderHeader).trimmed(); |
| 90 |
2/2✓ Branch 23 → 24 taken 8 times.
✓ Branch 24 → 25 taken 8 times.
|
8 | const QDate day = QDateTime::fromString(dateIso, Qt::ISODate).date(); |
| 91 |
11/14✓ Branch 27 → 28 taken 8 times.
✗ Branch 27 → 33 not taken.
✓ Branch 28 → 29 taken 8 times.
✓ Branch 29 → 30 taken 7 times.
✓ Branch 29 → 33 taken 1 time.
✓ Branch 30 → 31 taken 7 times.
✓ Branch 32 → 33 taken 3 times.
✓ Branch 32 → 34 taken 4 times.
✓ Branch 35 → 36 taken 7 times.
✓ Branch 35 → 37 taken 1 time.
✓ Branch 37 → 38 taken 4 times.
✓ Branch 37 → 39 taken 4 times.
✗ Branch 157 → 158 not taken.
✗ Branch 157 → 159 not taken.
|
8 | if (sender.isEmpty() || !day.isValid() || sender.compare(address.trimmed(), Qt::CaseInsensitive) == 0) |
| 92 | 4 | return {}; | |
| 93 |
1/1✓ Branch 39 → 40 taken 4 times.
|
4 | const QVariantMap introducer = personForAddress(sender); |
| 94 |
2/2✓ Branch 44 → 45 taken 4 times.
✓ Branch 45 → 46 taken 4 times.
|
8 | const QString key = introducer.value(QStringLiteral("key")).toString(); |
| 95 | 4 | qint64 bookId = 0; | |
| 96 | 4 | QString uid; | |
| 97 |
6/7✓ Branch 51 → 52 taken 3 times.
✓ Branch 51 → 54 taken 1 time.
✓ Branch 52 → 53 taken 3 times.
✗ Branch 53 → 54 not taken.
✓ Branch 53 → 55 taken 3 times.
✓ Branch 56 → 57 taken 1 time.
✓ Branch 56 → 58 taken 3 times.
|
4 | if (key.isEmpty() || !ContactQueries::splitKey(key, &bookId, &uid)) |
| 98 | 1 | return {}; | |
| 99 |
2/2✓ Branch 62 → 63 taken 3 times.
✓ Branch 63 → 64 taken 3 times.
|
6 | const QString name = introducer.value(QStringLiteral("name")).toString(); |
| 100 |
1/1✓ Branch 71 → 72 taken 3 times.
|
6 | return {{QStringLiteral("date"), day.toString(Qt::ISODate)}, |
| 101 | 9 | {QStringLiteral("channel"), QStringLiteral("mail")}, | |
| 102 | 15 | {QStringLiteral("via"), QVariantList{QVariantMap{{QStringLiteral("uid"), uid}, | |
| 103 | 6 | {QStringLiteral("name"), name}, | |
| 104 |
4/8✓ Branch 111 → 112 taken 3 times.
✓ Branch 111 → 113 taken 3 times.
✓ Branch 115 → 116 taken 9 times.
✓ Branch 115 → 117 taken 3 times.
✗ Branch 198 → 199 not taken.
✗ Branch 198 → 200 not taken.
✗ Branch 204 → 205 not taken.
✗ Branch 204 → 206 not taken.
|
24 | {QStringLiteral("key"), key}}}}, |
| 105 |
2/4✓ Branch 105 → 106 taken 12 times.
✓ Branch 105 → 107 taken 3 times.
✗ Branch 186 → 187 not taken.
✗ Branch 186 → 188 not taken.
|
18 | {QStringLiteral("suggestedBy"), name}}; |
| 106 | 32 | } | |
| 107 | |||
| 108 | 4 | QVariantMap QuickContactActions::personForAddress(const QString &address) const { | |
| 109 |
1/1✓ Branch 2 → 3 taken 4 times.
|
4 | ContactSyncCoordinator *sync = m_coordinator(); |
| 110 |
5/11✓ Branch 3 → 4 taken 4 times.
✗ Branch 3 → 7 not taken.
✓ Branch 4 → 5 taken 4 times.
✗ Branch 6 → 7 not taken.
✓ Branch 6 → 8 taken 4 times.
✓ Branch 9 → 10 taken 4 times.
✗ Branch 9 → 11 not taken.
✗ Branch 11 → 12 not taken.
✓ Branch 11 → 13 taken 4 times.
✗ Branch 26 → 27 not taken.
✗ Branch 26 → 28 not taken.
|
4 | if (!sync || address.trimmed().isEmpty()) |
| 111 | ✗ | return {}; | |
| 112 |
3/3✓ Branch 13 → 14 taken 4 times.
✓ Branch 15 → 16 taken 4 times.
✓ Branch 16 → 17 taken 4 times.
|
4 | const ContactQueries::Lookup found = ContactQueries::lookup(sync->store()->database(), address.trimmed()); |
| 113 |
3/3✓ Branch 19 → 20 taken 3 times.
✓ Branch 19 → 21 taken 1 time.
✓ Branch 20 → 22 taken 3 times.
|
4 | return found.known ? personEntry(found.key) : QVariantMap{}; |
| 114 | 4 | } | |
| 115 | |||
| 116 | 5 | QVariantMap QuickContactActions::metContext(const QString &eventUid) const { | |
| 117 |
1/1✓ Branch 2 → 3 taken 5 times.
|
5 | const QDateTime now = QDateTime::currentDateTime(); |
| 118 |
2/2✓ Branch 6 → 7 taken 5 times.
✓ Branch 7 → 8 taken 5 times.
|
10 | QVariantMap context{{QStringLiteral("date"), QDate::currentDate().toString(Qt::ISODate)}, |
| 119 |
5/7✓ Branch 12 → 13 taken 5 times.
✓ Branch 13 → 14 taken 5 times.
✓ Branch 14 → 15 taken 5 times.
✓ Branch 18 → 19 taken 10 times.
✓ Branch 18 → 20 taken 5 times.
✗ Branch 231 → 232 not taken.
✗ Branch 231 → 233 not taken.
|
20 | {QStringLiteral("dateText"), QLocale().toString(QDate::currentDate(), QLocale::LongFormat)}}; |
| 120 |
1/2✗ Branch 28 → 29 not taken.
✓ Branch 28 → 30 taken 5 times.
|
5 | if (m_calendarPath.isEmpty()) |
| 121 | ✗ | return context; | |
| 122 |
1/1✓ Branch 30 → 31 taken 5 times.
|
5 | CalendarStore store; |
| 123 |
2/3✓ Branch 31 → 32 taken 5 times.
✗ Branch 32 → 33 not taken.
✓ Branch 32 → 34 taken 5 times.
|
5 | if (!store.open(m_calendarPath)) |
| 124 | ✗ | return context; | |
| 125 | 5 | std::optional<CalendarEvent> event; | |
| 126 |
2/2✓ Branch 35 → 36 taken 2 times.
✓ Branch 35 → 78 taken 3 times.
|
5 | if (!eventUid.isEmpty()) { |
| 127 | // The occurrence of that event nearest to now (S106-265: the series | ||
| 128 | // master's first dates used to stand for every repetition). | ||
| 129 | 2 | QList<CalendarEvent> rows; | |
| 130 |
7/7✓ Branch 36 → 37 taken 2 times.
✓ Branch 37 → 38 taken 2 times.
✓ Branch 38 → 39 taken 2 times.
✓ Branch 41 → 42 taken 2 times.
✓ Branch 42 → 43 taken 2 times.
✓ Branch 50 → 44 taken 3 times.
✓ Branch 50 → 51 taken 2 times.
|
5 | for (const CalendarEvent &candidate : store.eventsForDateRange(now.addDays(-400), now.addDays(400))) |
| 131 |
3/3✓ Branch 46 → 47 taken 2 times.
✓ Branch 46 → 48 taken 1 time.
✓ Branch 47 → 48 taken 2 times.
|
5 | if (candidate.uid == eventUid) rows.append(candidate); |
| 132 | 2 | qint64 nearest = std::numeric_limits<qint64>::max(); | |
| 133 |
4/4✓ Branch 53 → 54 taken 2 times.
✓ Branch 54 → 55 taken 2 times.
✓ Branch 55 → 56 taken 2 times.
✓ Branch 56 → 57 taken 2 times.
|
4 | for (const CalendarEvent &occurrence : expandEventsForDisplay(rows, now.date().addDays(-400), now.date().addDays(400), |
| 134 |
5/5✓ Branch 57 → 58 taken 2 times.
✓ Branch 59 → 60 taken 2 times.
✓ Branch 60 → 61 taken 2 times.
✓ Branch 74 → 62 taken 61 times.
✓ Branch 74 → 75 taken 2 times.
|
65 | false, {kDisplayOccurrencesPerSeries, {}})) { |
| 135 |
1/1✓ Branch 63 → 64 taken 61 times.
|
61 | const qint64 distance = qAbs(occurrence.dtStart.secsTo(now)); |
| 136 |
6/7✓ Branch 65 → 66 taken 61 times.
✓ Branch 66 → 67 taken 61 times.
✗ Branch 66 → 69 not taken.
✓ Branch 67 → 68 taken 4 times.
✓ Branch 67 → 69 taken 57 times.
✓ Branch 70 → 71 taken 4 times.
✓ Branch 70 → 72 taken 57 times.
|
61 | if (occurrence.dtStart.isValid() && distance < nearest) { |
| 137 | 4 | nearest = distance; | |
| 138 | 4 | event = occurrence; | |
| 139 | } | ||
| 140 | 2 | } | |
| 141 | 2 | } else { | |
| 142 |
1/1✓ Branch 78 → 79 taken 3 times.
|
3 | event = eventOfTheMoment(store, now); |
| 143 | } | ||
| 144 |
2/2✓ Branch 83 → 84 taken 1 time.
✓ Branch 83 → 85 taken 4 times.
|
5 | if (!event) |
| 145 | 1 | return context; | |
| 146 |
1/1✓ Branch 88 → 89 taken 4 times.
|
4 | const auto full = store.eventFor(event->uid, event->accountId, event->calendarPath); |
| 147 |
4/11✓ Branch 90 → 91 taken 4 times.
✗ Branch 90 → 96 not taken.
✓ Branch 95 → 97 taken 4 times.
✓ Branch 97 → 98 taken 4 times.
✗ Branch 97 → 99 not taken.
✓ Branch 99 → 100 taken 4 times.
✗ Branch 99 → 101 not taken.
✗ Branch 286 → 287 not taken.
✗ Branch 286 → 288 not taken.
✗ Branch 290 → 291 not taken.
✗ Branch 290 → 292 not taken.
|
8 | const ICal::Component vevent = full ? ICal::parseFirst(full->icalRaw, QStringLiteral("VEVENT")) : ICal::Component{}; |
| 148 | // "Also at this event": named people and attendees, without the user. | ||
| 149 | 4 | QVariantList people; | |
| 150 | // Cards by key, people without a card by name; the user is never offered. | ||
| 151 | 4 | QStringList seen; | |
| 152 |
2/3✓ Branch 101 → 102 taken 4 times.
✗ Branch 103 → 104 not taken.
✓ Branch 103 → 105 taken 4 times.
|
4 | if (const QString me = meKey(); !me.isEmpty()) |
| 153 |
0/1✗ Branch 104 → 105 not taken.
|
4 | seen.append(me); |
| 154 | 4 | const auto offer = [this, &people, &seen](const QString &key, const QString &name) { | |
| 155 |
7/11✓ Branch 3 → 4 taken 2 times.
✓ Branch 3 → 8 taken 2 times.
✓ Branch 7 → 9 taken 2 times.
✓ Branch 9 → 10 taken 2 times.
✓ Branch 9 → 11 taken 2 times.
✓ Branch 11 → 12 taken 2 times.
✓ Branch 11 → 13 taken 2 times.
✗ Branch 63 → 64 not taken.
✗ Branch 63 → 65 not taken.
✗ Branch 67 → 68 not taken.
✗ Branch 67 → 69 not taken.
|
6 | const QString id = key.isEmpty() ? QStringLiteral("name:") + name : key; |
| 156 |
5/8✓ Branch 14 → 15 taken 4 times.
✗ Branch 14 → 19 not taken.
✓ Branch 16 → 17 taken 2 times.
✓ Branch 16 → 20 taken 2 times.
✗ Branch 18 → 19 not taken.
✓ Branch 18 → 20 taken 2 times.
✗ Branch 21 → 22 not taken.
✓ Branch 21 → 23 taken 4 times.
|
4 | if (seen.contains(id) || (key.isEmpty() && name.isEmpty())) |
| 157 | ✗ | return; | |
| 158 |
1/1✓ Branch 23 → 24 taken 4 times.
|
4 | seen.append(id); |
| 159 |
6/14✓ Branch 43 → 44 taken 2 times.
✓ Branch 43 → 45 taken 2 times.
✓ Branch 45 → 46 taken 2 times.
✓ Branch 45 → 47 taken 2 times.
✓ Branch 47 → 48 taken 2 times.
✓ Branch 47 → 49 taken 2 times.
✗ 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.
|
18 | QVariantMap entry = key.isEmpty() ? QVariantMap{{QStringLiteral("name"), name}, {QStringLiteral("textOnly"), true}} |
| 160 |
7/11✓ Branch 25 → 26 taken 2 times.
✓ Branch 25 → 35 taken 2 times.
✓ Branch 35 → 36 taken 2 times.
✓ Branch 36 → 37 taken 2 times.
✓ Branch 36 → 41 taken 2 times.
✓ Branch 38 → 39 taken 4 times.
✓ Branch 38 → 41 taken 2 times.
✗ Branch 71 → 72 not taken.
✗ Branch 71 → 76 not taken.
✗ Branch 73 → 74 not taken.
✗ Branch 73 → 76 not taken.
|
12 | : personEntry(key); |
| 161 |
2/3✓ Branch 49 → 50 taken 4 times.
✓ Branch 50 → 51 taken 4 times.
✗ Branch 50 → 55 not taken.
|
4 | if (!entry.isEmpty()) |
| 162 |
1/1✓ Branch 52 → 53 taken 4 times.
|
4 | people.append(entry); |
| 163 |
4/11✓ Branch 34 → 36 taken 2 times.
✓ Branch 41 → 42 taken 2 times.
✓ Branch 41 → 43 taken 2 times.
✓ Branch 58 → 59 taken 4 times.
✗ Branch 58 → 61 not taken.
✗ Branch 95 → 96 not taken.
✗ Branch 95 → 100 not taken.
✗ Branch 96 → 97 not taken.
✗ Branch 96 → 100 not taken.
✗ Branch 98 → 99 not taken.
✗ Branch 98 → 100 not taken.
|
14 | }; |
| 164 |
5/5✓ Branch 106 → 107 taken 4 times.
✓ Branch 107 → 108 taken 4 times.
✓ Branch 108 → 109 taken 4 times.
✓ Branch 116 → 110 taken 2 times.
✓ Branch 116 → 117 taken 4 times.
|
6 | for (const EventPerson &person : ICalEventCodec::peopleOf(vevent)) |
| 165 |
2/2✓ Branch 111 → 112 taken 2 times.
✓ Branch 112 → 113 taken 2 times.
|
6 | offer(cardKeyForUid(person.uid), person.name); |
| 166 |
3/3✓ Branch 119 → 120 taken 2 times.
✓ Branch 119 → 121 taken 2 times.
✓ Branch 120 → 122 taken 2 times.
|
4 | const QStringList own = m_ownAddresses ? m_ownAddresses() : QStringList{}; |
| 167 |
1/1✓ Branch 122 → 123 taken 4 times.
|
4 | ContactSyncCoordinator *sync = m_coordinator(); |
| 168 |
5/5✓ Branch 123 → 124 taken 4 times.
✓ Branch 124 → 125 taken 4 times.
✓ Branch 125 → 126 taken 4 times.
✓ Branch 157 → 127 taken 4 times.
✓ Branch 157 → 158 taken 4 times.
|
8 | for (const EventAttendee &attendee : ICalEventCodec::attendeesOf(vevent)) { |
| 169 | 4 | const bool room = attendee.cutype.compare(QLatin1String("ROOM"), Qt::CaseInsensitive) == 0; | |
| 170 |
5/6✓ Branch 130 → 131 taken 4 times.
✗ Branch 130 → 133 not taken.
✓ Branch 132 → 133 taken 2 times.
✓ Branch 132 → 134 taken 2 times.
✓ Branch 135 → 136 taken 2 times.
✓ Branch 135 → 137 taken 2 times.
|
4 | if (room || own.contains(attendee.address, Qt::CaseInsensitive)) |
| 171 | 2 | continue; | |
| 172 | const ContactQueries::Lookup found = | ||
| 173 |
4/8✓ Branch 137 → 138 taken 2 times.
✗ Branch 137 → 141 not taken.
✓ Branch 139 → 140 taken 2 times.
✓ Branch 140 → 142 taken 2 times.
✓ Branch 142 → 143 taken 2 times.
✗ Branch 142 → 144 not taken.
✗ Branch 303 → 304 not taken.
✗ Branch 303 → 305 not taken.
|
2 | sync ? ContactQueries::lookup(sync->store()->database(), attendee.address) : ContactQueries::Lookup{}; |
| 174 |
3/5✗ Branch 145 → 146 not taken.
✓ Branch 145 → 147 taken 2 times.
✗ Branch 148 → 149 not taken.
✓ Branch 148 → 150 taken 2 times.
✓ Branch 151 → 152 taken 2 times.
|
2 | offer(found.known ? found.key : QString(), attendee.name.isEmpty() ? attendee.address : attendee.name); |
| 175 | 6 | } | |
| 176 |
1/1✓ Branch 164 → 165 taken 4 times.
|
8 | context.insert(QStringLiteral("eventUid"), event->uid); |
| 177 |
1/1✓ Branch 173 → 174 taken 4 times.
|
8 | context.insert(QStringLiteral("eventTitle"), event->summary); |
| 178 |
1/1✓ Branch 182 → 183 taken 4 times.
|
8 | context.insert(QStringLiteral("place"), event->location); |
| 179 |
3/3✓ Branch 187 → 188 taken 4 times.
✓ Branch 188 → 189 taken 4 times.
✓ Branch 193 → 194 taken 4 times.
|
8 | context.insert(QStringLiteral("date"), event->dtStart.date().toString(Qt::ISODate)); |
| 180 |
4/4✓ Branch 198 → 199 taken 4 times.
✓ Branch 200 → 201 taken 4 times.
✓ Branch 201 → 202 taken 4 times.
✓ Branch 206 → 207 taken 4 times.
|
8 | context.insert(QStringLiteral("dateText"), QLocale().toString(event->dtStart.date(), QLocale::LongFormat)); |
| 181 |
1/1✓ Branch 216 → 217 taken 4 times.
|
8 | context.insert(QStringLiteral("people"), people); |
| 182 | 4 | return context; | |
| 183 | 22 | } | |
| 184 | |||
| 185 | 4 | QVariantMap QuickContactActions::existingPerson(const QString &name, const QString &email) const { | |
| 186 |
1/1✓ Branch 2 → 3 taken 4 times.
|
4 | ContactSyncCoordinator *sync = m_coordinator(); |
| 187 |
1/2✗ Branch 3 → 4 not taken.
✓ Branch 3 → 5 taken 4 times.
|
4 | if (!sync) |
| 188 | ✗ | return {}; | |
| 189 |
3/3✓ Branch 5 → 6 taken 4 times.
✓ Branch 8 → 9 taken 2 times.
✓ Branch 8 → 24 taken 2 times.
|
4 | if (!email.trimmed().isEmpty()) { |
| 190 |
3/3✓ Branch 9 → 10 taken 2 times.
✓ Branch 11 → 12 taken 2 times.
✓ Branch 12 → 13 taken 2 times.
|
2 | const ContactQueries::Lookup found = ContactQueries::lookup(sync->store()->database(), email.trimmed()); |
| 191 |
2/2✓ Branch 15 → 16 taken 1 time.
✓ Branch 15 → 18 taken 1 time.
|
2 | if (found.known) |
| 192 |
1/1✓ Branch 16 → 17 taken 1 time.
|
1 | return personEntry(found.key); |
| 193 | 2 | } | |
| 194 |
2/2✓ Branch 24 → 25 taken 3 times.
✓ Branch 25 → 26 taken 3 times.
|
3 | const QString folded = ContactIndex::foldForSort(name.simplified()); |
| 195 |
2/2✓ Branch 28 → 29 taken 1 time.
✓ Branch 28 → 30 taken 2 times.
|
3 | if (folded.isEmpty()) |
| 196 | 1 | return {}; | |
| 197 | // S106-268: the exact folded name, looked up directly — the search's | ||
| 198 | // results are cut (page, ranking, top 8) before an exact-name filter could | ||
| 199 | // see them, so a same-named card could be missed and created twice. | ||
| 200 |
2/2✓ Branch 31 → 32 taken 2 times.
✓ Branch 32 → 33 taken 2 times.
|
2 | QSqlQuery q(sync->store()->database()); |
| 201 |
1/1✓ Branch 37 → 38 taken 2 times.
|
2 | q.prepare(QStringLiteral("SELECT book_id, uid FROM cards WHERE sort_display = ? " |
| 202 | "AND kind NOT IN ('group', 'family') ORDER BY id LIMIT 8")); | ||
| 203 |
1/1✓ Branch 42 → 43 taken 2 times.
|
2 | q.addBindValue(folded); |
| 204 |
2/3✓ Branch 44 → 45 taken 2 times.
✓ Branch 45 → 46 taken 2 times.
✗ Branch 45 → 69 not taken.
|
2 | if (q.exec()) |
| 205 |
3/3✓ Branch 67 → 68 taken 2 times.
✓ Branch 68 → 47 taken 1 time.
✓ Branch 68 → 69 taken 1 time.
|
2 | while (q.next()) { |
| 206 |
6/6✓ Branch 47 → 48 taken 1 time.
✓ Branch 48 → 49 taken 1 time.
✓ Branch 49 → 50 taken 1 time.
✓ Branch 50 → 51 taken 1 time.
✓ Branch 51 → 52 taken 1 time.
✓ Branch 52 → 53 taken 1 time.
|
1 | const QVariantMap entry = personEntry(ContactQueries::cardKey(q.value(0).toLongLong(), q.value(1).toString())); |
| 207 |
2/3✓ Branch 57 → 58 taken 1 time.
✓ Branch 58 → 59 taken 1 time.
✗ Branch 58 → 61 not taken.
|
1 | if (!entry.isEmpty()) return entry; |
| 208 | 1 | } | |
| 209 | 1 | return {}; | |
| 210 | 3 | } | |
| 211 | |||
| 212 | 5 | QVariantMap QuickContactActions::createMet(const QVariantMap &fields) { | |
| 213 |
1/1✓ Branch 9 → 10 taken 5 times.
|
15 | QVariantMap met{{QStringLiteral("date"), fields.value(QStringLiteral("date"))}, |
| 214 |
1/1✓ Branch 18 → 19 taken 5 times.
|
15 | {QStringLiteral("place"), fields.value(QStringLiteral("place"))}, |
| 215 |
1/1✓ Branch 27 → 28 taken 5 times.
|
15 | {QStringLiteral("eventUid"), fields.value(QStringLiteral("eventUid"))}, |
| 216 |
3/5✓ Branch 36 → 37 taken 5 times.
✓ Branch 40 → 41 taken 20 times.
✓ Branch 40 → 42 taken 5 times.
✗ Branch 342 → 343 not taken.
✗ Branch 342 → 344 not taken.
|
35 | {QStringLiteral("via"), fields.value(QStringLiteral("via"))}}; |
| 217 |
2/2✓ Branch 70 → 71 taken 5 times.
✓ Branch 71 → 72 taken 5 times.
|
10 | const QString existing = fields.value(QStringLiteral("existingKey")).toString(); |
| 218 |
2/2✓ Branch 77 → 78 taken 2 times.
✓ Branch 77 → 156 taken 3 times.
|
5 | if (!existing.isEmpty()) { |
| 219 | // S106-270: a card that already says how you met keeps that record (the | ||
| 220 | // first meeting; date, place and introducers may be richer than this | ||
| 221 | // quick entry) — it used to be replaced. | ||
| 222 |
8/8✓ Branch 78 → 79 taken 2 times.
✓ Branch 83 → 84 taken 2 times.
✓ Branch 84 → 85 taken 2 times.
✓ Branch 89 → 90 taken 2 times.
✓ Branch 90 → 91 taken 2 times.
✓ Branch 91 → 92 taken 2 times.
✓ Branch 103 → 104 taken 1 time.
✓ Branch 103 → 131 taken 1 time.
|
6 | if (!m_model.card(existing).value(QStringLiteral("people")).toMap().value(QStringLiteral("met")).toMap().isEmpty()) { |
| 223 |
5/5✓ Branch 104 → 105 taken 1 time.
✓ Branch 108 → 109 taken 1 time.
✓ Branch 109 → 110 taken 1 time.
✓ Branch 113 → 106 taken 1 time.
✓ Branch 113 → 114 taken 1 time.
|
2 | qCInfo(lcContactRelationActions) << "Met someone: the existing card keeps its meeting"; |
| 224 |
2/4✓ Branch 124 → 125 taken 2 times.
✓ Branch 124 → 126 taken 1 time.
✗ Branch 474 → 475 not taken.
✗ Branch 474 → 476 not taken.
|
6 | return {{QStringLiteral("ok"), true}, {QStringLiteral("key"), existing}}; |
| 225 | } | ||
| 226 | // The duplicate guard's choice: only the meeting goes to that card. | ||
| 227 |
3/5✓ Branch 136 → 137 taken 1 time.
✓ Branch 139 → 140 taken 1 time.
✓ Branch 139 → 141 taken 1 time.
✗ Branch 499 → 500 not taken.
✗ Branch 499 → 501 not taken.
|
3 | const QVariantMap saved = save(existing, {{QStringLiteral("met"), met}}, true); |
| 228 |
5/5✓ Branch 143 → 144 taken 1 time.
✓ Branch 147 → 148 taken 1 time.
✓ Branch 148 → 149 taken 1 time.
✓ Branch 152 → 145 taken 1 time.
✓ Branch 152 → 153 taken 1 time.
|
2 | qCInfo(lcContactRelationActions) << "Met someone: meeting added to an existing card"; |
| 229 | 1 | return saved; | |
| 230 | 1 | } | |
| 231 |
3/3✓ Branch 160 → 161 taken 3 times.
✓ Branch 161 → 162 taken 3 times.
✓ Branch 162 → 163 taken 3 times.
|
6 | const QString name = fields.value(QStringLiteral("name")).toString().simplified(); |
| 232 |
2/2✓ Branch 169 → 170 taken 1 time.
✓ Branch 169 → 175 taken 2 times.
|
3 | if (name.isEmpty()) |
| 233 |
2/2✓ Branch 170 → 171 taken 1 time.
✓ Branch 171 → 172 taken 1 time.
|
1 | return failure(tr("Enter a name.")); |
| 234 | 2 | const int split = int(name.lastIndexOf(QLatin1Char(' '))); | |
| 235 | 4 | QVariantMap card{{QStringLiteral("formattedName"), name}, | |
| 236 |
2/3✓ Branch 185 → 186 taken 2 times.
✗ Branch 185 → 187 not taken.
✓ Branch 186 → 188 taken 2 times.
|
4 | {QStringLiteral("given"), split > 0 ? name.left(split) : name}, |
| 237 |
2/3✓ Branch 192 → 193 taken 2 times.
✗ Branch 192 → 194 not taken.
✓ Branch 193 → 195 taken 2 times.
|
4 | {QStringLiteral("family"), split > 0 ? name.mid(split + 1) : QString()}, |
| 238 |
2/4✓ Branch 202 → 203 taken 8 times.
✓ Branch 202 → 204 taken 2 times.
✗ Branch 542 → 543 not taken.
✗ Branch 542 → 544 not taken.
|
12 | {QStringLiteral("met"), met}}; |
| 239 |
5/5✓ Branch 218 → 219 taken 2 times.
✓ Branch 219 → 220 taken 2 times.
✓ Branch 220 → 221 taken 2 times.
✓ Branch 227 → 228 taken 1 time.
✓ Branch 227 → 271 taken 1 time.
|
4 | if (const QString email = fields.value(QStringLiteral("email")).toString().trimmed(); !email.isEmpty()) |
| 240 |
1/1✓ Branch 250 → 251 taken 1 time.
|
2 | card.insert(QStringLiteral("emails"), |
| 241 | 6 | QVariantList{QVariantMap{{QStringLiteral("value"), email}, {QStringLiteral("type"), QStringLiteral("home")}, | |
| 242 |
4/8✓ Branch 256 → 257 taken 1 time.
✓ Branch 256 → 258 taken 1 time.
✓ Branch 260 → 261 taken 3 times.
✓ Branch 260 → 262 taken 1 time.
✗ Branch 607 → 608 not taken.
✗ Branch 607 → 609 not taken.
✗ Branch 613 → 614 not taken.
✗ Branch 613 → 615 not taken.
|
9 | {QStringLiteral("sourceIndex"), -1}}}); |
| 243 |
2/2✓ Branch 276 → 277 taken 2 times.
✓ Branch 277 → 278 taken 2 times.
|
4 | const qint64 bookId = fields.value(QStringLiteral("bookId")).toLongLong(); |
| 244 |
4/4✓ Branch 282 → 283 taken 1 time.
✓ Branch 282 → 285 taken 1 time.
✓ Branch 283 → 284 taken 1 time.
✓ Branch 286 → 287 taken 2 times.
|
2 | const QVariantMap created = create(bookId > 0 ? bookId : defaultBookId(), card, true); |
| 245 |
3/4✓ Branch 291 → 292 taken 2 times.
✓ Branch 292 → 293 taken 2 times.
✗ Branch 297 → 298 not taken.
✓ Branch 297 → 299 taken 2 times.
|
4 | if (!created.value(QStringLiteral("ok")).toBool()) |
| 246 | ✗ | return created; | |
| 247 |
2/2✓ Branch 306 → 307 taken 2 times.
✓ Branch 309 → 310 taken 2 times.
|
4 | pushUndo(tr("Undo new contact"), {QStringLiteral("created"), {}, created.value(QStringLiteral("key")).toString(), 0}); |
| 248 |
3/3✓ Branch 317 → 318 taken 2 times.
✓ Branch 319 → 320 taken 2 times.
✓ Branch 320 → 321 taken 2 times.
|
4 | report(tr("%1 added").arg(name), true); |
| 249 |
5/5✓ Branch 323 → 324 taken 2 times.
✓ Branch 327 → 328 taken 2 times.
✓ Branch 328 → 329 taken 2 times.
✓ Branch 332 → 325 taken 2 times.
✓ Branch 332 → 333 taken 2 times.
|
4 | qCInfo(lcContactRelationActions) << "Met someone: card created"; |
| 250 | 2 | return created; | |
| 251 | 56 | } | |
| 252 |