Problem with frameworks 5.86.0 and Konqueror

[Stefano Crocco]

On domenica 19 settembre 2021 13:16:28 CEST Stefano Crocco wrote:
> On domenica 19 settembre 2021 12:42:22 CEST David Faure wrote:
> > On dimanche 19 septembre 2021 10:22:38 CEST Stefano Crocco wrote:
> > > This problem doesn't only affect http(s) URLs: if I enter, for example,
> > > a
> > > man: URL, the man page isn't opened in Konqueror, as it used to, but in
> > > KHelpCenter, which is something which shouldn't happen at all.
> > 
> > Ah, that part isn't fixed.
> > 
> > My fix with "disabling external browsers" is about http(s).
> > For other things like man and info, the logic is really "this KRun is
> > being
> > called from an application which can display HTML itself, don't launch
> > another process for that". We could add yet another setter, but it's
> > getting ugly, especially for a deprecated class.
> > 
> > I'm thinking more and more that we need to copy KRun and
> > KParts::BrowserRun
> > into konqueror, merging it all into KonqRun. Not just because of this
> > issue
> > of course, but because KRun is deprecated (in favour of OpenUrlJob) for
> > most use cases, while the konqueror use case is different (opening things
> > in-process rather than launching an external application, in many cases).
> > 
> > What do you think?
> 
> I had thought about that too, but I hoped there was a better alternative. If
> KRun is deprecated, it makes sense. I'll see what I can do.
> 
> Thanks
> 
> Stefano

I started copying most of the code from KRun::init and 
KParts::BrowserRun::init into KonqRun::init to fix the incompatibility with 
kio-5.86.0 (the merge request is at https://invent.kde.org/network/
konqueror/-/merge_requests/88). I'll go on with merging the three classes in 
the future.

By the way, do you know whether there is a way for me to get notified about bug 
reports for Konqueror?

Thanks

Stefano