Changeset View
Changeset View
Standalone View
Standalone View
src/metacontact.cpp
Show First 20 Lines • Show All 42 Lines • ▼ Show 20 Line(s) | 42 | public: | |||
---|---|---|---|---|---|
43 | MetaContactProxy(const AbstractContact::List &contacts) | 43 | MetaContactProxy(const AbstractContact::List &contacts) | ||
44 | : m_contacts(contacts) | 44 | : m_contacts(contacts) | ||
45 | {} | 45 | {} | ||
46 | 46 | | |||
47 | QVariant customProperty(const QString &key) const override | 47 | QVariant customProperty(const QString &key) const override | ||
48 | { | 48 | { | ||
49 | if (key.startsWith(QLatin1String("all-"))) { | 49 | if (key.startsWith(QLatin1String("all-"))) { | ||
50 | QVariantList ret; | 50 | QVariantList ret; | ||
51 | Q_FOREACH (const AbstractContact::Ptr &contact, m_contacts) { | 51 | for (const AbstractContact::Ptr &contact : qAsConst(m_contacts)) { | ||
dfaure: (the qAsConst isn't technically needed since this method is const; but it doesn't hurt either) | |||||
52 | QVariant val = contact->customProperty(key); | 52 | QVariant val = contact->customProperty(key); | ||
53 | Q_ASSERT(val.canConvert<QVariantList>() || val.isNull()); | 53 | Q_ASSERT(val.canConvert<QVariantList>() || val.isNull()); | ||
54 | 54 | | |||
55 | if (!val.isNull()) { | 55 | if (!val.isNull()) { | ||
56 | ret.append(val.toList()); | 56 | ret.append(val.toList()); | ||
57 | } | 57 | } | ||
58 | } | 58 | } | ||
59 | return ret; | 59 | return ret; | ||
60 | } else { | 60 | } else { | ||
61 | Q_FOREACH (const AbstractContact::Ptr &contact, m_contacts) { | 61 | for (const AbstractContact::Ptr &contact : qAsConst(m_contacts)) { | ||
62 | QVariant val = contact->customProperty(key); | 62 | QVariant val = contact->customProperty(key); | ||
63 | if (val.isValid()) { | 63 | if (val.isValid()) { | ||
64 | return val; | 64 | return val; | ||
65 | } | 65 | } | ||
66 | } | 66 | } | ||
67 | return QVariant(); | 67 | return QVariant(); | ||
68 | } | 68 | } | ||
69 | } | 69 | } | ||
▲ Show 20 Lines • Show All 143 Lines • Show Last 20 Lines |
(the qAsConst isn't technically needed since this method is const; but it doesn't hurt either)