<table><tr><td style="">justusw created this revision.<br />Restricted Application added a project: KDE PIM.<br />Restricted Application added a subscriber: KDE PIM.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D6516" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Invert the condition.  Previously, filters were ignored that presented<br />
a valid color.  Now those with a valid color are selected.  This was<br />
likely a copy&paste error from 'get_string'.</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>Refactor compliance code</p>

<p>Refactor the code checking whether a key is compliant into a new<br />
function and export that.</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>New function checking if a key is valid</p>

<p>A key is valid if all UIDs have at least the validity 'full'.</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>Tweak complianceStringForKey</p>

<p>According to the design spec, a key should only be usable for<br />
VS-compliant conversations, if it is valid.</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>Separate filter code from method to helper function</p>

<p>In the same way that 'bgColor' uses 'get_color'.  This makes it<br />
possible to search through two sets of filters.</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>Add and use a second set of key filters</p>

<p>The second set is not part of the model (i.e. it will not be displayed<br />
in Kleopatras drop down list in the default view), and will only be<br />
used for styling keys.</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>Add filters for keys that are (not) VS-compliant</p>

<p>Add a filter that selects VS-compliant keys, and two that highlight<br />
the validity of the keys using a background color.</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>Add and use function providing a short compliance string for keys</p>

<p>Include short compliance information in key lists and combo boxes.</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>Add a field 'Validity' to the key model</p>

<p>Add a new field for the validity of keys, and populate it with<br />
validity and compliance information.</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>Rework the keyfilter model</p>

<p>Rework the 'data(..)' method, use the filter ids as UserRole, so that<br />
we can match on them.</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>Add a filter for keys that are not yet certified</p>

<p>And also plug it into the list it if the trust model is "direct".</p>

<p>Signed-off-by: Justus Winter <justus@gnupg.org></p>

<p>Here arc, butcher this commit...</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R90 PIM: Kleo Library</div></div></div><br /><div><strong>BRANCH</strong><div><div>justus/compliance</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D6516" rel="noreferrer">https://phabricator.kde.org/D6516</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/kleo/keyfiltermanager.cpp<br />
src/models/keylistmodel.cpp<br />
src/models/keylistmodelinterface.h<br />
src/ui/keyselectioncombo.cpp<br />
src/utils/formatting.cpp<br />
src/utils/formatting.h</div></div></div><br /><div><strong>To: </strong>justusw, aheinecke<br /><strong>Cc: </strong>KDE PIM, dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>