[tracker/needle: 5/7] needle: Support searching by title as well as content

[Martyn James Russell <mr src gnome org>]

commit 623b48988bee5d3ec789b2ca5a4eb593fdb481f1
Author: Martyn Russell <martyn lanedo com>
Date:   Tue Jun 29 22:00:07 2010 +0100

    needle: Support searching by title as well as content

 src/needle/needle.ui   |   43 +++++++++++++++++++++++++++++++++++++++----
 src/needle/needle.vala |   40 +++++++++++++++++++++++++++++++++++-----
 2 files changed, 74 insertions(+), 9 deletions(-)
---
diff --git a/src/needle/needle.ui b/src/needle/needle.ui
index 3e752f7..da23252 100644
--- a/src/needle/needle.ui
+++ b/src/needle/needle.ui
@@ -53,9 +53,8 @@
                 <property name="visible">True</property>
                 <property name="tooltip_text" translatable="yes">Display results in a list view</property>
                 <property name="is_important">True</property>
-                <property name="label" translatable="yes">_List</property>
                 <property name="use_underline">True</property>
-                <property name="stock_id">gtk-index</property>
+                <property name="icon_name">gtk-index</property>
                 <property name="active">True</property>
                 <property name="group">toolbutton_view_icons</property>
               </object>
@@ -69,9 +68,8 @@
                 <property name="visible">True</property>
                 <property name="tooltip_text" translatable="yes">Display results in an icon view</property>
                 <property name="is_important">True</property>
-                <property name="label" translatable="yes">_Icons</property>
                 <property name="use_underline">True</property>
-                <property name="stock_id">gtk-orientation-portrait</property>
+                <property name="icon_name">gtk-orientation-portrait</property>
                 <property name="active">True</property>
               </object>
               <packing>
@@ -89,6 +87,43 @@
               </packing>
             </child>
             <child>
+              <object class="GtkRadioToolButton" id="toolbutton_find_in_contents">
+                <property name="visible">True</property>
+                <property name="tooltip_text" translatable="yes">Find search criteria inside files</property>
+                <property name="label" translatable="yes">toolbutton3</property>
+                <property name="use_underline">True</property>
+                <property name="stock_id">gtk-find</property>
+                <property name="active">True</property>
+              </object>
+              <packing>
+                <property name="expand">False</property>
+                <property name="homogeneous">True</property>
+              </packing>
+            </child>
+            <child>
+              <object class="GtkRadioToolButton" id="toolbutton_find_in_titles">
+                <property name="visible">True</property>
+                <property name="tooltip_text" translatable="yes">Find search criteria in file titles</property>
+                <property name="label" translatable="yes">toolbutton4</property>
+                <property name="use_underline">True</property>
+                <property name="stock_id">gtk-bold</property>
+                <property name="group">toolbutton_find_in_contents</property>
+              </object>
+              <packing>
+                <property name="expand">False</property>
+                <property name="homogeneous">True</property>
+              </packing>
+            </child>
+            <child>
+              <object class="GtkSeparatorToolItem" id="&lt;separator&gt;">
+                <property name="visible">True</property>
+              </object>
+              <packing>
+                <property name="expand">False</property>
+                <property name="homogeneous">True</property>
+              </packing>
+            </child>
+            <child>
               <object class="GtkToolItem" id="toolbutton_search_label">
                 <property name="visible">True</property>
                 <property name="is_important">True</property>
diff --git a/src/needle/needle.vala b/src/needle/needle.vala
index 0e6a208..67702f7 100644
--- a/src/needle/needle.vala
+++ b/src/needle/needle.vala
@@ -40,6 +40,8 @@ public class Needle {
 	private ToolButton forward;
 	private ToggleToolButton view_list;
 	private ToggleToolButton view_icons;
+	private ToggleToolButton find_in_contents;
+	private ToggleToolButton find_in_titles;
 	private Entry search;
 	private ScrolledWindow sw_treeview;
 	private TreeView treeview;
@@ -114,6 +116,12 @@ public class Needle {
 		view_icons = builder.get_object ("toolbutton_view_icons") as ToggleToolButton;
 		view_icons.toggled.connect (view_toggled);
 
+		find_in_contents = builder.get_object ("toolbutton_find_in_contents") as ToggleToolButton;
+		find_in_contents.toggled.connect (find_in_toggled);
+
+		find_in_titles = builder.get_object ("toolbutton_find_in_titles") as ToggleToolButton;
+		find_in_titles.toggled.connect (find_in_toggled);
+
 		search = builder.get_object ("entry_search") as Entry;
 		search.changed.connect (search_changed);
 
@@ -231,9 +239,9 @@ public class Needle {
 		var diff_days = diff_sec / secs_per_day;
 		var diff_days_abs = diff_days.abs ();
 		
-		stdout.printf ("timeval now:%ld, then:%ld, diff secs:%ld, diff days:%ld, abs: %ld, seconds per day:%d\n", tv_now.tv_sec, tv_then.tv_sec, diff_sec, diff_days, diff_days_abs, secs_per_day);
+		// stdout.printf ("timeval now:%ld, then:%ld, diff secs:%ld, diff days:%ld, abs: %ld, seconds per day:%d\n", tv_now.tv_sec, tv_then.tv_sec, diff_sec, diff_days, diff_days_abs, secs_per_day);
 			
-		/* if it's more than a week, use the default date format */
+		// if it's more than a week, use the default date format
 		if (diff_days_abs > 7) { 	    
 			return t.format ("%x");
 		}					
@@ -266,8 +274,21 @@ public class Needle {
 	private bool search_run () {
 		// Need to escape this string
 		string query;
+		string criteria;
+		
+		criteria = search.get_text ();
+		
+		if (criteria.length < 1) {
+			last_search_id = 0;
+			return false;
+		}
 
-		query = "SELECT ?u nie:url(?u) tracker:coalesce(nie:title(?u), nfo:fileName(?u), \"Unknown\") nfo:fileLastModified(?u) nfo:fileSize(?u) nie:url(?c) WHERE { ?u fts:match \"%s\" . ?u nfo:belongsToContainer ?c } ORDER BY DESC(fts:rank(?u)) OFFSET 0 LIMIT 100".printf ((search).text);
+		if (find_in_contents.active) {
+			query = @"SELECT ?u nie:url(?u) tracker:coalesce(nie:title(?u), nfo:fileName(?u), \"Unknown\") nfo:fileLastModified(?u) nfo:fileSize(?u) nie:url(?c) WHERE { ?u fts:match \"$criteria\" . ?u nfo:belongsToContainer ?c } ORDER BY DESC(fts:rank(?u)) OFFSET 0 LIMIT 100";
+		} else {
+			query = @"SELECT ?u nie:url(?u) tracker:coalesce(nfo:fileName(?u), \"Unknown\") nfo:fileLastModified(?u) nfo:fileSize(?u) nie:url(?c) WHERE { ?u a nfo:FileDataObject ; nfo:belongsToContainer ?c . FILTER(fn:contains(nfo:fileName(?u), \"$criteria\")) } ORDER BY DESC(nfo:fileName(?u)) OFFSET 0 LIMIT 100";
+		}
+	
 		debug ("Query:'%s'", query);
 
 		try {
@@ -279,10 +300,10 @@ public class Needle {
 			var theme = IconTheme.get_for_screen (screen);
 
 			var size_small = 0;
-			Gtk.icon_size_lookup (Gtk.IconSize.DND, out size_small, null) ;  //24; // Small
+			Gtk.icon_size_lookup (Gtk.IconSize.DND, out size_small, null);
 
 			var size_big = 0;
-			Gtk.icon_size_lookup (Gtk.IconSize.DIALOG, out size_big, null) ;  //32; // Big
+			Gtk.icon_size_lookup (Gtk.IconSize.DIALOG, out size_big, null);
 
 			for (int i = 0; i < result.length[0]; i++) {
 				debug ("--> %s", result[i,0]);
@@ -301,6 +322,7 @@ public class Needle {
 				TreeIter iter;
 				store.append (out iter);
 				
+				// FIXME: should optimise this a bit more, inserting 2 images into a list eek
 				store.set (iter, 
 						   0, pixbuf_small,
 						   1, pixbuf_big,
@@ -339,6 +361,14 @@ public class Needle {
 		}
 	}
 
+	private void find_in_toggled () {
+		if (find_in_contents.active) {
+			// TODO: Re-run query		
+		} else {
+			// TODO: Re-run query		
+		}
+	}
+
 	private void view_row_selected (TreeView view, TreePath path, TreeViewColumn column) {
 		TreeIter iter;