D22424: TextDocument: remove actions from contextmenu on hide already

[Friedrich W. H. Kossebau]

kossebau added a comment.


  In D22424#496112 <https://phabricator.kde.org/D22424#496112>, @kossebau wrote:
  
  > In D22424#494980 <https://phabricator.kde.org/D22424#494980>, @rjvbb wrote:
  >
  > > However, the patch has a side-effect: I'm getting messages like this each time I switch or open documents:
  > >
  > >   kdevplatform.shell: Broken text-document:  QUrl("file:///opt/local/var/tmp/kdevelop-tmp-505-%7Bf793a513-f14e-47b9-8448-06ca72900c04%7D/kdevelop_Q27955.patch")
  > >
  > >
  > > This example is for a file that indeed doesn't exist, but that I also do not have open (so WTH?!) but I also get it for files that open perfectly.
  > >
  > > I fail to see an immediate reason for this, so I'm back to running my own patch.
  >
  >
  > Interesting. Not seen here, but giving this some stress testing now. The kdevelop_xxxxx.patch is coming from the diff review system creating text document objects for displaying the diffs, but I also see no direct relation ship to this patch here. Will poke around a bit,
  
  
  Playing around I finally managed to reproduce this as well. Though also for 5.3 and without this patch: seems something to do with workingsets code being eager to create TextDocument objects for any urls they have in the sessionrc file, Dumped my findings as https://bugs.kde.org/show_bug.cgi?id=409858

REPOSITORY
  R32 KDevelop

REVISION DETAIL
  https://phabricator.kde.org/D22424

To: kossebau, #kdevelop
Cc: anthonyfieroni, rjvbb, kdevelop-devel, hmitonneau, christiant, glebaccon, domson, antismap, iodelay, alexeymin, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20190716/bcb272ef/attachment.html>