KDE4 patch: recently used documents

Bug #441562 reported by Daniel Riedemann on 2009-10-03
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
BleachBit
Medium
Andrew Ziem

Bug Description

Hello,

At my zsh contribution I threatened you that I am going to contribute more. So here I am again... ;-)
I patched the KDE Cleaner so it includes now the recently used documents list.

Tested again at Arch Linux with KDE 4.3.1
It deletes 4KB for each entry in the recently used list.

If I contribute more like this, do you prefer diffs or should I attach the whole new cleaner?

Best regards

Andrew Ziem (ahziem1) wrote :

darie,

What are the file names in the directory? Do they have a common pattern such as an extension? Let's make the glob more specific.

For existing cleaners, diff is best. For new cleaners, whole file is best.

Daniel Riedemann (darie) wrote :

Hello Andrew,

You're right, better to be more specific with deleting files. The files are all stored as *.desktop
I added a new diff with *.desktop. :-)

Regards,
Daniel

Andrew Ziem (ahziem1) wrote :

Thank you. Looks good. I will merge. I notice I have these files on my Fedora 11 system (KDE 4.3.1 also):

~/.kde/share/apps/RecentDocuments/*.desktop

Changed in bleachbit:
assignee: nobody → Andrew Ziem (ahziem1)
importance: Undecided → Medium
status: New → Confirmed
Andrew Ziem (ahziem1) wrote :

Committed in SVN r1205:1206

Changed in bleachbit:
milestone: none → 0.7.0
status: Confirmed → Fix Committed
Andrew Ziem (ahziem1) wrote :
Changed in bleachbit:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Patches