This allows to sort by and show (both as additional data in icon view and column in column view) deletion date of files in Trash.
CHANGELOG: It is now possible to view and sort by "Deletion Date" in Trash
BUG: 153492
FIXED-IN: 17.08.0
This allows to sort by and show (both as additional data in icon view and column in column view) deletion date of files in Trash.
CHANGELOG: It is now possible to view and sort by "Deletion Date" in Trash
BUG: 153492
FIXED-IN: 17.08.0
Added "Deletion Date" as column in my Trash and it worked and I could sort by it by clicking the column header. Additional info also shows it in the view.
It doesn't store the enum values anywhere (.directory file), I hope, as I added a value inbetween.
I didn't find any other KIO besides trash that uses Extra Fields so I didn't bother rewriting all the things to generically support this feature ;)
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
Nice! Kai-Uwe, you rock ;)
I'll let the Dolphin people review the implementation, but this is a definite +1 from me, for finally implementing a very old feature request.
A very big +2!
You can wait for Emanuel to chime in and give his opinion.
It seems a bit unlikely that he would chime in anytime soon though. His last mail in the KFM list (either by mail or by comment on reviewboard or here) is from Feb. 18. You can wait for him, but i think he would be fine with fine with the change.
Does trash:// KIO really only record the deletion date vs. the time?
It's a proper timestamp, it's just the naming I took from there. I was thinking of calling it deletiontime to be in with modificationtime and accesstime, I wouldn't mind either of them.
If trash KIO is already inconsistent we can indeed pick being inconsistent with either it or with the rest of Dolphin.
But I am concerned about the UI string.
Special handling of DeletionDateRole is missing in:
Otherwise the sorting and grouping by deletion date would be wrong/inconsistent.
I don't know what causes this, though :)