← Back to team overview

zeitgeist team mailing list archive

[Merge] lp:~jbicha/activity-log-manager/clear-recently-used into lp:activity-log-manager

 

Jeremy Bicha has proposed merging lp:~jbicha/activity-log-manager/clear-recently-used into lp:activity-log-manager.

Requested reviews:
  Activity Log Manager (activity-log-manager)

For more details, see:
https://code.launchpad.net/~jbicha/activity-log-manager/clear-recently-used/+merge/173391

Here's the other half of the bug fix.

It doesn't look like GTK supports more advanced features, such as blacklists or an easy way to only clear events older than a certain date.

http://www.valadoc.org/#!api=gtk+-3.0/Gtk.RecentManager
-- 
https://code.launchpad.net/~jbicha/activity-log-manager/clear-recently-used/+merge/173391
Your team Activity Log Manager is requested to review the proposed merge of lp:~jbicha/activity-log-manager/clear-recently-used into lp:activity-log-manager.
=== modified file 'src/unified-privacy.vala'
--- src/unified-privacy.vala	2013-07-07 22:43:51 +0000
+++ src/unified-privacy.vala	2013-07-08 01:24:27 +0000
@@ -831,6 +831,7 @@
 			int res = this.run();
 			if(res == ResponseType.OK) {
 				Zeitgeist.TimeRange tr;
+				var recent = new RecentManager();
 				var index = this.get_active_radio_button();
 				if (index < 3) {
 					int range = 60*60*1000;
@@ -852,6 +853,7 @@
 				else if (index == 4){
 					tr = new Zeitgeist.TimeRange.anytime();
 					get_history.begin (tr);
+					recent.purge_items();
 				}
 			}
 			


Follow ups