Bug 363494 - closing a Patch Review via (git) commit opens the documentation view
Summary: closing a Patch Review via (git) commit opens the documentation view
Status: CLOSED WAITINGFORINFO
Alias: None
Product: kdevplatform
Classification: Developer tools
Component: patchreview (other bugs)
Version First Reported In: git master
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kdevelop-bugs-null
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-05-25 09:05 UTC by RJVB
Modified: 2016-10-02 16:11 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
screen recording showing the issue (3.75 MB, video/mp4)
2016-09-07 11:30 UTC, RJVB
Details

Note You need to log in before you can comment on or make changes to this bug.
Description RJVB 2016-05-25 09:05:56 UTC
When exiting from a patch review by committing (to git), the documentation toolview is opened even if it was not when the patch review plugin was activated. Also happens when the commit is cancelled.

Reproducible: Always

Steps to Reproduce:
1. Edit a project that is a git working copy
2. Commit the changes via KDevelop's built-in git/commit/commit, or cancel the commit


Actual Results:  
Upon exit from the patch review plugin/toolview I find myself looking at the documentation view, which I didn't request (and which typically doesn't show anything useful in this case).

Expected Results:  
One expects that the view is restored exactly as it was before, with the same files open, the same document active, and no newly activated views.

This was also an issue with KDevelop 4.7 until at some point it stopped. I've never been able to determine if that was because of a code change or because of something I did locally (to the runtime state). I'd hope it's the former, though.
Comment 1 Aleix Pol 2016-09-06 19:17:09 UTC
It doesn't make sense.
Also I can't reproduce.
Also documentation isn't mentioned on patch review at all.

Please reopen if you find any pointers on what can be going on.
Comment 2 RJVB 2016-09-06 20:07:00 UTC
I agree it doesn't make any sense, but still it is what's happening.

With all due respect, what's the point in accepting bug reports if you leave it to the reporter to figure out what's going on, and *close* the report in the meantime? You cannot expect everyone who's ever made small contributions to the code to know where to start looking for this kind of weirdness if the codebase is as huge and complex as kdevplatform + kdevelop ...

Do I have to film my screen to prove I'm not talking out the side of my neck?
Comment 3 RJVB 2016-09-07 07:57:34 UTC
Another thing:

where in the code could I set a breakpoint to catch whatever it is that causes the documentation view to open, or indeed to be reactivated after it's been removed from the visible toolviews?
Comment 4 RJVB 2016-09-07 11:30:43 UTC
Created attachment 100966 [details]
screen recording showing the issue

Sorry for the slightly nauseating hand-held video; I don't do this often enough to have proper screencasting tools
Comment 5 Aleix Pol 2016-09-07 16:52:40 UTC
Could be a problem upon restoring the session. Do you get the same problem when debugging and getting back to code?
Comment 6 RJVB 2016-09-07 18:20:03 UTC
Indeed, same thing when the debugged application terminates and the debug view is closed.

Could this be something like a missing variable initialisation? I've already encountered situations where that led to issues on only some platforms, but not with development builds. As I mentioned in the original report, I did see similar behaviour in KDevelop4 on OS X, and somehow it was always the documentation toolview that kept opening.

At least it seems I can no longer reproduce the reactivation of that toolview when I've removed it from the toolbar. That's an acceptable workaround for me but probably not for everyone.
Comment 7 RJVB 2016-09-11 09:38:45 UTC
Well, I thought I could simply work around this issue by removing the Documentation button from my toolbars. While that's true I have to repeat the action each time I restart KDevelop because that damn. button keeps coming back...
Comment 8 RJVB 2016-09-11 09:40:14 UTC
(posted too fast)

Wouldn't that give a hint at what's going here, i.e. "simply" a lack of persistence of the documentation view's settings?
Comment 9 RJVB 2016-10-02 16:11:33 UTC
I think that what happens is that the review and debug areas have its own setting for the documentation visibility, and that somehow the visibility isn't restored properly when going back to the code area.

Why the documentation view actually opens when it wasn't in the review (or debug) area beats me...