<table><tr><td style="">dfaure created this revision.<br />dfaure added reviewers: ochurlaud, dkurz, dvratil.<br />dfaure requested review of this revision.
</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/D16508">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This was introduced by <a href="https://phabricator.kde.org/R76:1ec2e08a83c074ebf4dad554c32495e0bf6eb8ed" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">1ec2e08</a> which only called the color-generator<br />
code when there *is* a ColorAttribute on the collection (with an<br />
invalid color, which seems unlikely), rather than when the attribute<br />
isn't there.</p>

<p>I thought this was the bug that made all new resources red (with caldav)<br />
but actually that seems to come from the resource itself. So now I'm not<br />
100% sure about this commit anymore, although it still seems to make<br />
sense to me -- for other resources, possibly.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Disabling the code that reads from the ColorAttribute<br />
and removing all "Resources Colors" entries from ~/.config/eventviewsrc,<br />
the colors are properly generated.</p></div></div><br /><div><strong>BRANCH</strong><div><div>2018_10_color_generation (branched from Applications/18.08)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16508">https://phabricator.kde.org/D16508</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/helper.cpp</div></div></div><br /><div><strong>To: </strong>dfaure, ochurlaud, dkurz, dvratil<br /><strong>Cc: </strong>KDE PIM<br /></div>