<table><tr><td style="">dkurz marked an inline comment as done.<br />dkurz added a comment.
</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/D7548" rel="noreferrer">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D7548#140147" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D7548#140147</a>, <a href="https://phabricator.kde.org/p/dvratil/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@dvratil</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I created an event in KOrganizer, gave it a bunch of categories and saved it. Then I edited the event, changed some parts and added some more categories and I did not see any GID-named categories being created. Did I miss some steps?</p></div>
</blockquote>

<p>Are you trying to reproduce the bug in an unpatched version? Some reasons why it might not have triggered for you:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">you did not close the incidence dialog between the changes</li>
<li class="remarkup-list-item">you opened the category dialog when making your other changes, too</li>
<li class="remarkup-list-item">you assigned completely new categories to your tag (I think it should trigger in that case, but I'm not sure)</li>
</ul>

<p>The bug does not happen as soon as the category dialog is confirmed. In contrast to IncidenceCategories, which gets its Akonadi::Item from the EditorItemManager with incomplete Tags (no attributes fetched), the category dialog has its own set of Tags complete with the TAG attribute.</p>

<p>Try the following:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">open an existing event</li>
<li class="remarkup-list-item">only change its description</li>
<li class="remarkup-list-item">click OK</li>
<li class="remarkup-list-item">open the same event again</li>
</ol>

<p>That should do the trick.</p></div></div><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D7548#inline-30797" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">dvratil</span> wrote in <span style="color: #4b4d51; font-weight: bold;">incidencecategories.cpp:69</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">So who now pre-selects the tags that the Item has?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">We already meet incidence (stored in mLoadedIncidence) in the other load method. KCalCore::Incidence does not know about Akonadi::Items, but only has a set of strings for its categories, so there's no risk of confusing the two. The incidence's categories are then used as soon as we have complete information about the tags to match them in onTagsFetched.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R78 PIM: Incidence Editor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7548" rel="noreferrer">https://phabricator.kde.org/D7548</a></div></div><br /><div><strong>To: </strong>dkurz, KDE PIM<br /><strong>Cc: </strong>dvratil, dvasin, winterz, vkrause, mlaurent, knauss<br /></div>