Skip to content
This repository was archived by the owner on Apr 28, 2022. It is now read-only.

Commit 6fa9714

Browse files
committed
[nemo-systemsettings] Fix apgs pending agreement update. Fixes JB#48765
Setting location mode and then accepting the agreement wasn't removing it from the pending list.
1 parent 498d326 commit 6fa9714

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

src/locationsettings.cpp

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -288,9 +288,14 @@ bool LocationSettingsPrivate::updateProvider(const QString &name, const Location
288288
m_pendingAgreements.removeOne(name);
289289
emit q->pendingAgreementsChanged();
290290
}
291-
} else if (provider.hasAgreement && !provider.agreementAccepted && !m_pendingAgreements.contains(name)) {
292-
m_pendingAgreements.append(name);
293-
emit q->pendingAgreementsChanged();
291+
} else if (provider.hasAgreement) {
292+
if (!provider.agreementAccepted && !m_pendingAgreements.contains(name)) {
293+
m_pendingAgreements.append(name);
294+
emit q->pendingAgreementsChanged();
295+
} else if (provider.agreementAccepted && m_pendingAgreements.contains(name)) {
296+
m_pendingAgreements.removeOne(name);
297+
emit q->pendingAgreementsChanged();
298+
}
294299
}
295300
}
296301

0 commit comments

Comments
 (0)