Bug 252483

Thanks for the bug report! I could confirm the issue in trunk.

Still present in 4.5.2.

I can add this other bug (linked to the previous I guess):

0) there are files in the Trash => Go to the trash in dolphin: OK: the icon is "full trash"
1) delete the files *by selecting them then press the "delete" button*
2) as a result, the trash is emptied, but here again, the trash icon remains "full trash"

Still present in 4.6.1.

have the same problem in 4.6.90

If that deleted file is recovered through Edit->Undo, then the trash icon will become empty as expected.

Still present in 4.7.3.

Created attachment 66404 [details]
patch for konq_operations.cpp file in kde-baseapps/lib/konq

when we restore a file or delete a file from trash, we need to emit dataChanged() signal from our custom model so that corresponding view gets updated.

I tried filesremoved slot of kdirnotify but it was not working so i emitted filesadded instead of that which in turn emits datachanged ().[ because i don't have access to kdirnotify.cpp ]

Thanks for the patch! It looks fine from my point of view but I cannot decide whether it should get pushed: Would it be possible that you make an "official" merge-request by posting the patch to https://git.reviewboard.kde.org/dashboard/ and put David Faure (dfaure@kde.org) to the reviewers? The patched code is maintained by David and it is impossible for David to keep an overview about all the open issues/patches in bugs.kde.org... (but David keeps an eye on the patches from the reviewboard).

hi,
what is the login for review board. By the way can you tell why emitfilesremoved is not working.
(In reply to comment #7)
> Thanks for the patch! It looks fine from my point of view but I cannot decide
> whether it should get pushed: Would it be possible that you make an "official"
> merge-request by posting the patch to
> https://git.reviewboard.kde.org/dashboard/ and put David Faure (dfaure@kde.org)
> to the reviewers? The patched code is maintained by David and it is impossible
> for David to keep an overview about all the open issues/patches in
> bugs.kde.org... (but David keeps an eye on the patches from the reviewboard).

> what is the login for review board.

You need to request an account at https://identity.kde.org/

> By the way can you tell why emitfilesremoved is not working.

No sorry - I'm not familiar with the code of konq_operations...

Resetting assignee to default as per bug #305719

[Part of the Dolphin Bug Weeks 2012]
I can confirm this on still exists in Dolphin 2.1/KDE SC 4.9.2 (openSUSE 12.2.6 KR49 packages). The icon also stays "full" when you delete an item from the trash without using the "Empty Trash" option.

(In reply to comment #11)
> [Part of the Dolphin Bug Weeks 2012]
> I can confirm this on still exists in Dolphin 2.1/KDE SC 4.9.2 (openSUSE
> 12.2.6 KR49 packages). The icon also stays "full" when you delete an item
> from the trash without using the "Empty Trash" option.

Oops. Missed that this is apparently a libkonq issue.

@David: could you have a look at the patch from comment 6?

I can confirm that this issue is still present in 4.10 .

Distribution: Gentoo
KDE: 4.10

Git commit bf7cdfca99e4d28d6f40d019b34f7fe5679c1715 by David Faure.
Committed on 19/02/2013 at 11:17.
Pushed by dfaure into branch 'KDE/4.10'.

Update views showing trash:/ when restoring trashed files.

Thanks to A.J.Reddy for the investigation and initial patch, and thanks
to Frank Reininghaus for the testing.
FIXED-IN: 4.10.1

M  +3    -3    lib/konq/konq_operations.cpp

http://commits.kde.org/kde-baseapps/bf7cdfca99e4d28d6f40d019b34f7fe5679c1715

OK thanks. Does the correction also work when you delete files from trash (see comment #2) ?
Because in the patch (from comment #6) I see the test "|| m_method == DEL" which is not present in the lats commit ?

I can confirm that the commit from comment 15 fixes the issue when restoring but not when deleting.

Reopen the bug since not completely solved.
=> see comment #2

Thanks for testing. It's extremely easy to miss things like that, especially if there are comments in the report that add new, similar bugs to the existing one, but I hope that a clean solution for the remaining one can also be found. I guess the main question is if there is a better solution than updating the trash for every single 'delete' operation.

*** Bug 365754 has been marked as a duplicate of this bug. ***

Confirmed that the Trash icon still doesn't update when you use a Delete operation to render it empty.

*** Bug 251620 has been marked as a duplicate of this bug. ***

*** Bug 356981 has been marked as a duplicate of this bug. ***

*** Bug 366662 has been marked as a duplicate of this bug. ***

Git commit c0797e8adef58bc57254d86d6f1abaa46d732f19 by Roman Inflianskas.
Committed on 11/03/2018 at 16:47.
Pushed by romaninflianskas into branch 'master'.

Update trash icon after restore operation

Summary:
After restoring all the files from the trash, the trash icon remain 'full'. This patch fixes this.

Reviewers: elvisangelaccio

Reviewed By: elvisangelaccio

Subscribers: elvisangelaccio, #dolphin

Differential Revision: https://phabricator.kde.org/D11216

M  +1    -0    src/trash/dolphintrash.cpp

https://commits.kde.org/dolphin/c0797e8adef58bc57254d86d6f1abaa46d732f19

not completely fixed in Dolphin 18.04, Arch Linux.

open dolphin (if info panel is not enabled, press F11 to do it)
send some file to trash
click trash icon under places in sidebar (observe red trash icon in info panel)
click "empty trash" button
trash icon in info panel is still red

I can't reproduce your issue, Dr Chapatin. Those steps to reproduce work fine for me with 18.04.0 as well as current git master.

(In reply to Nate Graham from comment #27)
> I can't reproduce your issue, Dr Chapatin. Those steps to reproduce work
> fine for me with 18.04.0 as well as current git master.

Weird. I opened bug 392882 about this issue.
On my neon dev unstable trash icon have other two bugs.