<table><tr><td style="">kossebau created this revision.<br />Restricted Application added a subscriber: kdevelop-devel.
</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/D5536" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>The disconnect was broken since it was added:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">disconnect( this, SLOT(validateData()) );</pre></div>

<p>(added in <a href="https://phabricator.kde.org/R33:9f46f693e743ad84032c7e820c61f2e45fc630e2" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">9f46f693e743ad84032c7e820c61f2e45fc630e2</a>)<br />
which will call</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">QObject::disconnect(const QObject *receiver, const char *method)
"Disconnects all signals in this object from receiver's method."</pre></div>

<p>and thus not do what has been intended, to disconnect from the signal<br />
of the previous importWidget.<br />
The port to new signal/slot syntax did not improve the wrong logic<br />
even if the very disconnect call was discussed for other reasons<br />
in the review :)<br />
(see commit <a href="https://phabricator.kde.org/R33:f626743803ab900038890b04b5613b871d2c7ee1" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">f626743803ab900038890b04b5613b871d2c7ee1</a> and<br />
<a href="https://git.reviewboard.kde.org/r/121045" class="remarkup-link" target="_blank" rel="noreferrer">https://git.reviewboard.kde.org/r/121045</a>)</p>

<p>As it seems needed to track the current import widget, to know whose<br />
signal to disconnect from, the new member m_currentImportWidget<br />
can be also used to drop code for dynamic lookup of that widget<br />
in some the property access methods of ProjectVcsPage.<br />
(will be a separate commit)</p>

<p>The broken disconnect did not result in a real bug from what I saw,<br />
but still added noise in the debug log, so one reason to fix it.</p>

<p>Target: 5.1</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R33 KDevPlatform</div></div></div><br /><div><strong>BRANCH</strong><div><div>fixbrokendisconnect</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D5536" rel="noreferrer">https://phabricator.kde.org/D5536</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/appwizard/projectvcspage.cpp<br />
plugins/appwizard/projectvcspage.h</div></div></div><br /><div><strong>To: </strong>kossebau, KDevelop<br /><strong>Cc: </strong>kdevelop-devel<br /></div>