<table><tr><td style="">ahmadsamir created this revision.<br />ahmadsamir added a reviewer: hindenburg.<br />ahmadsamir created this object with edit policy "Administrators".<br />Restricted Application added a subscriber: Konsole.<br />Restricted Application added a project: Konsole.
</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/D9448" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>This should fix Bug 348478 [1].</p>

<p>To reproduce the issue:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">open a konsole window and open more than one tab</li>
<li class="remarkup-list-item">detach one of them</li>
<li class="remarkup-list-item">close the detached tab</li>
<li class="remarkup-list-item">try to use e.g. the Edit menu, it doesn't work</li>
</ul>

<p>The issue is that _pluggedController in sessionFinished() after closing the detached tab doesn't point to the SessionController of the "session" from the sender, but actually points to the SessionController of the session of  the tab that's active in the original window; so calling unplugController on it messes up the menus.</p>

<p>[1] <a href="https://bugs.kde.org/show_bug.cgi?id=348478" class="remarkup-link" target="_blank" rel="noreferrer">https://bugs.kde.org/show_bug.cgi?id=348478</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9448" rel="noreferrer">https://phabricator.kde.org/D9448</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/ViewManager.cpp</div></div></div><br /><div><strong>To: </strong>ahmadsamir, hindenburg<br /><strong>Cc: </strong>Konsole, ngraham, hindenburg<br /></div>