Bug 437161

Summary: konqueror not loading anything from address bar
Product: [Frameworks and Libraries] frameworks-kcompletion Reporter: Markus <m4rkusxxl>
Component: generalAssignee: David Gil <davidgiloliva>
Status: CONFIRMED ---    
Severity: major CC: arojas, asturm, danwint370, faure, grahamperrin, jjm, kdelibs-bugs, klaus-dieter.fietze, kollix, luigi.toscano, mwoehlke.floss, norbert, sven, tandake
Priority: NOR    
Version: 5.82.0   
Target Milestone: ---   
Platform: Gentoo Packages   
OS: Linux   
Latest Commit: Version Fixed In:
Sentry Crash Report:

Description Markus 2021-05-15 15:40:16 UTC
SUMMARY
Since the last update of frameworks, konqueror is not loading anything from the address bar.
I can enter urls or shortcuts, but neither pressing enter nor pressing the goto button has any reaction. (Shortcuts are not resolved to urls.)
I can open urls from commandline and bookmarks. Web pages seem to work normal as well.


STEPS TO REPRODUCE
1. Open konqueror.
2. Enter url.
3. Press enter or "go to" button.

OBSERVED RESULT
Nothing happens.

EXPECTED RESULT
Url should load.

SOFTWARE/OS VERSIONS
KDE Plasma Version: 5.21.5
KDE Frameworks Version: 5.82.0
KDE Apps Version: 21.04.1
Qt Version: 5.15.2
Comment 1 Antonio Rojas 2021-05-15 16:50:27 UTC
Confirmed, culprit is kcompletion. It also breaks address autocompletion in falkon
Comment 2 Antonio Rojas 2021-05-15 17:50:39 UTC
Somehow the DQT_DISABLE_DEPRECATED_BEFORE change in https://invent.kde.org/frameworks/kcompletion/-/commit/b223e3e20e64dcd767c09e6071c41c7968f6f5b2 causes this
Comment 3 Markus 2021-05-15 18:04:18 UTC
I can confirm  Antonio Rojas findings:
* Downgrade to kcompletion-5.81.0 works.
* Revert QT_DISABLE_DEPRECATED_BEFORE=0x050e00 in kcompletion-5.82.0 works.
Comment 4 Klaus-Dieter Fietze 2021-05-16 12:49:22 UTC
I confirm. Same behaviour with this configuration:

Operating System: openSUSE Tumbleweed 20210513
KDE Plasma Version: 5.21.5
KDE Frameworks Version: 5.82.0
Qt Version: 5.15.2
Konqueror Version: 21.04.0
Kernel Version: 5.12.2-1-default

Glad to see that there are still people out there using konqueror!
Comment 5 Jonathan Marten 2021-05-16 15:00:33 UTC
If you have are using Qt 5.15 and can compile from source, please test with
https://invent.kde.org/network/konqueror/-/merge_requests/71 merged.
Comment 6 Markus 2021-05-16 15:20:05 UTC
Unpatched kcompletion-5.82.0 with patched konqueror-21.04.1 works.
Comment 7 Antonio Rojas 2021-05-30 08:00:29 UTC
*** Bug 437846 has been marked as a duplicate of this bug. ***
Comment 8 Antonio Rojas 2021-06-02 08:38:34 UTC
*** Bug 437615 has been marked as a duplicate of this bug. ***
Comment 9 Luigi Toscano 2021-06-02 12:24:14 UTC
The mere request(In reply to Jonathan Marten from comment #5)
> If you have are using Qt 5.15 and can compile from source, please test with
> https://invent.kde.org/network/konqueror/-/merge_requests/71 merged.

This has been merged but we need a fix for the release/21.04 branch. Can't that be backported?
Comment 10 Norbert Preining 2021-06-03 09:51:35 UTC
I tried the mentioned PR on Debian/experimental (21.04.1, frameworks 5.82) and it did *not* fix the issue for me.
Comment 11 Norbert Preining 2021-06-03 09:53:58 UTC
For full details

frameworks 5.82
konqueror 21.04.1
qt 5.15.2
plasma 5.21.90

Debian/sid+experimental
Comment 12 Antonio Rojas 2021-06-03 09:57:53 UTC
Regardless of the konqueror PR, if a change in kcompletion breaks applications, shouldn't if be fixed in kcompletion? are signals considered public API?
Comment 13 Andreas Sturmlechner 2021-06-03 11:13:11 UTC
(In reply to Norbert Preining from comment #10)
> I tried the mentioned PR on Debian/experimental (21.04.1, frameworks 5.82)
> and it did *not* fix the issue for me.

Worked fine for me and fixed the problem.

> >>> Preparing source in /../work/konqueror-21.04.1 ...
> * Applying konqueror-21.04.1-use-new-QComboBox-signal.patch ...                                                                                                [ ok ]
> * User patches applied.
Comment 14 Norbert Preining 2021-06-03 19:47:05 UTC
Sorry for the noise, my patch set was incomplete.

I can confirm that the MR fixes the issue.

THanks
Comment 15 Antonio Rojas 2021-06-11 21:13:49 UTC
*** Bug 438480 has been marked as a duplicate of this bug. ***
Comment 16 Klaus-Dieter Fietze 2021-06-16 09:08:43 UTC
To my great joy, this feature works again. Thank you !

Operating System: openSUSE Tumbleweed 20210614  
KDE Plasma Version: 5.22.0  
KDE Frameworks Version: 5.82.0  
Qt Version: 5.15.2  
Kernel Version: 5.12.9-1-default (64-bit)  

I am glad that I can use Konqueror again. It deserves more attention and a bigger user community.  
How about spelling? Is there some work in progress to make work?
Comment 17 Antonio Rojas 2021-06-22 14:58:24 UTC
*** Bug 439028 has been marked as a duplicate of this bug. ***
Comment 18 Antonio Rojas 2021-06-25 22:03:54 UTC
*** Bug 439167 has been marked as a duplicate of this bug. ***
Comment 19 Antonio Rojas 2021-07-09 19:07:48 UTC
*** Bug 439683 has been marked as a duplicate of this bug. ***
Comment 20 Markus 2021-09-17 12:52:16 UTC
Should be fixed in 21.04.2 and 21.08.0, so this can be closed, right?
Comment 21 Klaus-Dieter Fietze 2021-09-20 07:55:23 UTC
(In reply to Markus from comment #20)
> Should be fixed in 21.04.2 and 21.08.0, so this can be closed, right?

As I already said above, it is fixed.
From my point of view, this bug can be closed.
Thanks again for your work!