https://bugs.kde.org/show_bug.cgi?id=458933

Stefano Crocco <stefano.cro...@alice.it> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/netw |https://invent.kde.org/netw
                   |ork/konqueror/commit/c9f3d1 |ork/konqueror/commit/429936
                   |957f7b6c168a025b5b4014e46a5 |da48ebd8bcc6f75dacd9180eb08
                   |f49bb63                     |6e4add3

--- Comment #7 from Stefano Crocco <stefano.cro...@alice.it> ---
Git commit 429936da48ebd8bcc6f75dacd9180eb086e4add3 by Stefano Crocco.
Committed on 22/09/2022 at 13:06.
Pushed by stefanocrocco into branch 'cherry-pick-c9f3d195'.

Fix crash caused by using original view rather than actual view

UrlLoader::decideEmbedOrSave used the original view when deciding
whether it could be reused to open the URL, instead of using the view
provided using UrlLoader::setView. If the URL was opened by clicking on
a sidebar item, this caused a sidebar view to be used, rather than the
correct dolphin part. This lead to a crash because the sidebar view is
a passive view, which caused KonqViewManager::chooseNextView to return
nullptr


(cherry picked from commit c9f3d1957f7b6c168a025b5b4014e46a5f49bb63)

M  +1    -1    src/konqmainwindow.cpp
M  +6    -8    src/urlloader.cpp

https://invent.kde.org/network/konqueror/commit/429936da48ebd8bcc6f75dacd9180eb086e4add3

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to