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

Akseli Lahtinen <akse...@akselmo.dev> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |https://invent.kde.org/fram
                   |                            |eworks/kio/-/commit/2cf5289
                   |                            |d9e994f830749eb30791aabe171
                   |                            |9406f9

--- Comment #3 from Akseli Lahtinen <akse...@akselmo.dev> ---
Git commit 2cf5289d9e994f830749eb30791aabe1719406f9 by Akseli Lahtinen.
Committed on 28/07/2025 at 09:45.
Pushed by akselmo into branch 'master'.

KCoreDirLister: Do not list files again if the new url has same job

When getting a redirection URL, we check if the URL already has a job or
not. We also check that it's not the current job we're working on.

If it has same job as given to the slot as parameter, we are not
working on a new directory, but an old one. This can happen
when we are in `sftp://foo@bar/home/foo`, then type `sftp://foo@bar`
which in turn redirects us back to `sftp://foo@bar/home/foo`.

In these cases, we know we're not working on a new directory,
so we handle it through cache or refreshing it.

This fixes crashes with the oldJob assert and also
gives back to Qt the trailing slash handling, while
keeping expected behavior.

This also implements the mock test of commit 0808afd7
Related: bug 451050, bug 507185

M  +4    -0    CMakeLists.txt
M  +70   -8    autotests/kdirlistertest.cpp
M  +3    -2    autotests/kdirlistertest.h
M  +9    -15   src/core/kcoredirlister.cpp
M  +4    -0    src/core/worker.cpp
M  +4    -0    src/core/worker_p.h

https://invent.kde.org/frameworks/kio/-/commit/2cf5289d9e994f830749eb30791aabe1719406f9

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

Reply via email to