KDE4 patch: recently used documents

Bug #441562 reported by Daniel Riedemann
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
BleachBit
Fix Released
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

Revision history for this message
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.

Revision history for this message
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

Revision history for this message
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
Revision history for this message
Andrew Ziem (ahziem1) wrote :

Committed in SVN r1205:1206

Changed in bleachbit:
milestone: none → 0.7.0
status: Confirmed → Fix Committed
Revision history for this message
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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.