<table><tr><td style="">bruns created this revision.<br />bruns added a reviewer: dvratil.<br />Restricted Application added a project: KDE PIM.<br />bruns 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/D12474">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>EwsConfigDialog::fullUsername() always adds the domain delimiter when<br />
the "Domain" checkbox is set, even if the actual domain name is empty,<br />
while the login procedure in EwsResource::passwordRequestFinished(...)<br />
only adds a domain delimiter if the domain is non-empty. Make sure<br />
login in the config dialog and during normal operation is equivalent.</p>

<p>Not sending an (empty) domain causes login failures for some servers,<br />
while sending it seems to be unproblematic, and can be forced off by<br />
unselecting the checkbox.</p>

<p>This fixes a regression caused by commit<br />
<a href="https://phabricator.kde.org/R44:7c74258355cd91106d7563142177dd7f17801256" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">7c74258355cd</a> ("EWS: Refactor server connection and password retrieval"),<br />
which changed the check for sending the domain from mSettings->hasDomain()<br />
to mSettings->domain().isEmpty().</p>

<p>BUG: 388496<br />
CCBUG: 390798</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Create a new folder (either using web client or kontact)<br />
Make sure it appears in kontact<br />
Delete it in the web client (or using kontact on another computer)</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R44 KDE PIM Runtime</div></div></div><br /><div><strong>BRANCH</strong><div><div>delimiter_with_empty_domain</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12474">https://phabricator.kde.org/D12474</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>resources/ews/ewsresource.cpp</div></div></div><br /><div><strong>To: </strong>bruns, dvratil<br /><strong>Cc: </strong>KDE PIM, dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>