API Enable single-column fulltext filter search as fallback

docs/en/02_Developer_Guides/12_Search/02_FulltextSearch.md

@@ -39,6 +39,43 @@ records and cannot easily be adapted to include custom `DataObject` instances. T
 default site search, have a look at those extensions and modify as required.
 </div>
 
+### Fulltext Filter
+
+SilverStripe provides a `[api:FulltextFiler]` which you can use to perform custom fulltext searches on
+`[api:DataList]`'s.
+
+Example DataObject:
+
+	:::php
+	class SearchableDataObject extends DataObject {
+		
+		private static $db = array(
+			"Title" => "Varchar(255)",
+			"Content" => "HTMLText",
+		);
+
+		private static $indexes = array(
+			'SearchFields' => array(
+				'type' => 'fulltext',
+				'name' => 'SearchFields',
+				'value' => '"Title", "Content"',
+			)
+		);
+
+		private static $create_table_options = array(
+			'MySQLDatabase' => 'ENGINE=MyISAM'
+		);
+
+	}
+
+Performing the search:
+
+	:::php
+	SearchableDataObject::get()->filter('SearchFields:fulltext', 'search term');
+
+If your search index is a single field size, then you may also specify the search filter by the name of the
+field instead of the index.
+
 ## API Documentation
 
 * [api:FulltextSearchable]

search/filters/FulltextFilter.php

@@ -68,18 +68,16 @@ class FulltextFilter extends SearchFilter {
 			} else {
 				// Parse a fulltext string (eg. fulltext ("ColumnA", "ColumnB")) to figure out which columns
 				// we need to search.
-				if(preg_match('/^fulltext\ \((.+)\)$/i', $index, $matches)) {
+				if(preg_match('/^fulltext\s+\((.+)\)$/i', $index, $matches)) {
 					return $matches[1];
 				} else {
 					throw new Exception("Invalid fulltext index format for '" . $this->getName()
 						. "' on '" . $this->model . "'");
-					return;
 				}
 			}
-			return $columns;
 		}
 
-		throw new Exception($this->getName() . ' is not a fulltext index on ' . $this->model . '.');
+		return parent::getDbName();
 	}
 
 }

search/filters/SearchFilter.php

@@ -168,7 +168,9 @@ abstract class SearchFilter extends Object {
 		
 		// Ensure that we're dealing with a DataObject.
 		if (!is_subclass_of($this->model, 'DataObject')) {
-			throw new Exception("Model supplied to " . get_class($this) . " should be an instance of DataObject.");
+			throw new InvalidArgumentException(
+				"Model supplied to " . get_class($this) . " should be an instance of DataObject."
+			);
 		}
 
 		$candidateClass = ClassInfo::table_for_object_field(
@@ -183,7 +185,7 @@ abstract class SearchFilter extends Object {
 			return '"' . implode('"."', $parts) . '"';
 		}
 		
-		return "\"{$candidateClass}\".\"{$this->name}\"";
+		return sprintf('"%s"."%s"', $candidateClass, $this->name);
 	}
 
 	/**
@@ -200,7 +202,6 @@ abstract class SearchFilter extends Object {
 		return $dbField->RAW();
 	}
 
-	
 	/**
 	 * Apply filter criteria to a SQL query.
 	 *

tests/search/FulltextFilterTest.php

@@ -2,13 +2,16 @@
 
 class FulltextFilterTest extends SapphireTest {
 	
-	protected static $fixture_file = "FulltextFilterTest.yml";
+	protected $extraDataObjects = array(
+		'FulltextFilterTest_DataObject'
+	);
 
+	protected static $fixture_file = "FulltextFilterTest.yml";
 
 	public function testFilter() {
 		if(DB::getConn() instanceof MySQLDatabase) {
-			$baseQuery = FulltextDataObject::get();
-			$this->assertEquals(3, $baseQuery->count(), "FulltextDataObject count does not match.");
+			$baseQuery = FulltextFilterTest_DataObject::get();
+			$this->assertEquals(3, $baseQuery->count(), "FulltextFilterTest_DataObject count does not match.");
 
 			// First we'll text the 'SearchFields' which has been set using an array
 			$search = $baseQuery->filter("SearchFields:fulltext", 'SilverStripe');
@@ -25,24 +28,64 @@ class FulltextFilterTest extends SapphireTest {
 			$search = $baseQuery->exclude("OtherSearchFields:fulltext", "SilverStripe");
 			$this->assertEquals(2, $search->count());
 
-			// Edgecase
-			$this->setExpectedException("Exception");
-			$search = $baseQuery->exclude("Madeup:fulltext", "SilverStripe");
+			// Search on a single field
+			$search = $baseQuery->filter("ColumnE:fulltext", 'Dragons');
+			$this->assertEquals(1, $search->count());
+
+			$search = $baseQuery->exclude("ColumnE:fulltext", "Dragons");
+			$this->assertEquals(2, $search->count());
 		} else {
 			$this->markTestSkipped("FulltextFilter only supports MySQL syntax.");
 		}
 	}
 
+	public function testGenerateQuery() {
+		// Test SearchFields
+		$filter1 = new FulltextFilter('SearchFields', 'SilverStripe');
+		$filter1->setModel('FulltextFilterTest_DataObject');
+		$query1 = FulltextFilterTest_DataObject::get()->dataQuery();
+		$filter1->apply($query1);
+		$this->assertEquals('"ColumnA", "ColumnB"', $filter1->getDbName());
+		$this->assertEquals(
+			array("MATCH (\"ColumnA\", \"ColumnB\") AGAINST ('SilverStripe')"),
+			$query1->query()->getWhere()
+		);
+
+
+		// Test Other searchfields
+		$filter2 = new FulltextFilter('OtherSearchFields', 'SilverStripe');
+		$filter2->setModel('FulltextFilterTest_DataObject');
+		$query2 = FulltextFilterTest_DataObject::get()->dataQuery();
+		$filter2->apply($query2);
+		$this->assertEquals('"ColumnC", "ColumnD"', $filter2->getDbName());
+		$this->assertEquals(
+			array("MATCH (\"ColumnC\", \"ColumnD\") AGAINST ('SilverStripe')"),
+			$query2->query()->getWhere()
+		);
+
+		// Test fallback to single field
+		$filter3 = new FulltextFilter('ColumnA', 'SilverStripe');
+		$filter3->setModel('FulltextFilterTest_DataObject');
+		$query3 = FulltextFilterTest_DataObject::get()->dataQuery();
+		$filter3->apply($query3);
+		$this->assertEquals('"FulltextFilterTest_DataObject"."ColumnA"', $filter3->getDbName());
+		$this->assertEquals(
+			array("MATCH (\"FulltextFilterTest_DataObject\".\"ColumnA\") AGAINST ('SilverStripe')"),
+			$query3->query()->getWhere()
+		);
+	}
+
 }
 
 
-class FulltextDataObject extends DataObject {
+class FulltextFilterTest_DataObject extends DataObject implements TestOnly {
 
 	private static $db = array(
 		"ColumnA" => "Varchar(255)",
 		"ColumnB" => "HTMLText",
 		"ColumnC" => "Varchar(255)",
 		"ColumnD" => "HTMLText",
+		"ColumnE" => 'Varchar(255)'
 	);
 
 	private static $indexes = array(
@@ -52,6 +95,7 @@ class FulltextDataObject extends DataObject {
 			'value' => '"ColumnA", "ColumnB"',
 		),
 		'OtherSearchFields' => 'fulltext ("ColumnC", "ColumnD")',
+		'SingleIndex' => 'fulltext ("ColumnE")'
 	);
 	
 	private static $create_table_options = array(

tests/search/FulltextFilterTest.yml

@@ -1,16 +1,19 @@
-FulltextDataObject:
+FulltextFilterTest_DataObject:
   object1:
     ColumnA: 'SilverStripe'
-    CluumnB: <p>Some content about SilverStripe.</p>
+    CluumnB: '<p>Some content about SilverStripe.</p>'
     ColumnC: 'SilverStripe'
-    ColumnD: '<p>Some content about SilverStripe.</p>
+    ColumnD: '<p>Some content about SilverStripe.</p>'
+    ColumnE: 'Dragons be here'
   object2:
     ColumnA: 'Test Row'
     ColumnB: '<p>Some information about this test row.</p>'
     ColumnC: 'Test Row'
     ColumnD: '<p>Some information about this test row.</p>'
+    ColumnE: 'No'
   object3:
     ColumnA: 'Fulltext Search'
     ColumnB: '<p>Testing fulltext search.</p>'
-    ColumnC: 'Fulltext Search
+    ColumnC: 'Fulltext Search'
     ColumnD: '<p>Testing fulltext search.</p>'
+    ColumnE: ''