Fix/multiple manymany filters

README.md

@@ -44,7 +44,7 @@ Name: search
 Before:
     - '#site'
 ---
-Jaedb\Search\SearchPageController:
+PlasticStudio\Search\SearchPageController:
   types:
     docs:
       Label: 'Documents'

_config/config.yml

@@ -5,8 +5,8 @@ Before:
 ---
 SilverStripe\Control\Controller:
   extensions:
-    - Jaedb\Search\SearchControllerExtension
-Jaedb\Search\SearchPageController:
+    - PlasticStudio\Search\SearchControllerExtension
+PlasticStudio\Search\SearchPageController:
   # types:
   #   docs:
   #     Label: 'Documents'

composer.json

@@ -1,8 +1,8 @@
 {
-	"name": "jaedb/search",
+	"name": "plasticstudio/search",
     "type": "silverstripe-vendormodule",
 	"description": "SilverStripe search engine",
-	"homepage": "http://jamesbarnsley.co.nz",
+	"homepage": "https://psdigital.co.nz",
 	"keywords": ["silverstripe"],
 	"license": "BSD-3-Clause",
 	"authors": [
@@ -13,7 +13,7 @@
 		}
 	],
 	"support": {
-		"issues": "http://github.com/jaedb/search/issues"
+		"issues": "http://github.com/plasticstudio/search/issues"
 	},
     "extra": {
         "expose": [

package.json

@@ -3,7 +3,7 @@
   "version": "1.1.0",
   "author": "James Barnsley",
   "description": "SilverStripe Search Engine",
-  "repository": "https://github.com/jaedb/search",
+  "repository": "https://github.com/plasticstudio/search",
   "licenses": {
     "type": "Apache License",
     "url": "http://www.apache.org/licenses/LICENSE-2.0.txt"

src/SearchControllerExtension.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Jaedb\Search;
+namespace PlasticStudio\Search;
 
 use Exception;
 use SilverStripe\ORM\DataExtension;

src/SearchPage.php

@@ -1,6 +1,6 @@
 <?php
 
-namespace Jaedb\Search;
+namespace PlasticStudio\Search;
 
 use Page;
 

src/SearchPageController.php

@@ -1,6 +1,6 @@
 <?php  
 
-namespace Jaedb\Search;
+namespace PlasticStudio\Search;
 
 use PageController;
 use SilverStripe\ORM\ArrayList;
@@ -22,11 +22,11 @@ class SearchPageController extends PageController {
 	public function index($request){
 
 		if (Director::isLive()){
-			Requirements::css('/resources/jaedb/search/client/Search.min.css');
-			Requirements::javascript('/resources/jaedb/search/client/Search.min.js');
+			Requirements::css('/resources/plasticstudio/search/client/Search.min.css');
+			Requirements::javascript('/resources/plasticstudio/search/client/Search.min.js');
 		} else {
-			Requirements::css('/resources/jaedb/search/client/Search.css');
-			Requirements::javascript('/resources/jaedb/search/client/Search.js');
+			Requirements::css('/resources/plasticstudio/search/client/Search.css');
+			Requirements::javascript('/resources/plasticstudio/search/client/Search.js');
 		}
 
 		// get the parameters and variables of this request (ie the query and filters)
@@ -59,7 +59,7 @@ public function index($request){
 	 **/
 
 	public static function get_types_available(){
-		$types = Config::inst()->get('Jaedb\Search\SearchPageController', 'types');
+		$types = Config::inst()->get('PlasticStudio\Search\SearchPageController', 'types');
 		$array = [];
 
 		if ($types){
@@ -73,7 +73,7 @@ public static function get_types_available(){
 	}
 
 	public static function get_filters_available(){
-		$filters = Config::inst()->get('Jaedb\Search\SearchPageController', 'filters');
+		$filters = Config::inst()->get('PlasticStudio\Search\SearchPageController', 'filters');
 		$array = [];
 
 		if ($filters){
@@ -87,7 +87,7 @@ public static function get_filters_available(){
 	}
 
 	public static function get_sorts_available(){
-		$sorts = Config::inst()->get('Jaedb\Search\SearchPageController', 'sorts');
+		$sorts = Config::inst()->get('PlasticStudio\Search\SearchPageController', 'sorts');
 		$array = [];
 
 		if ($sorts){
@@ -327,7 +327,7 @@ public function PerformSearch(){
 							$tables_to_check[] = $type['Table'];
 
 							foreach ($tables_to_check as $table_to_check){
-								$column_exists_query = DB::query( "SHOW COLUMNS FROM \"".$table_to_check."\" LIKE '".$filter['Column']."'" );				
+								$column_exists_query = DB::query( "SHOW COLUMNS FROM \"".$table_to_check."\" LIKE '".$filter['Column']."'" );
 
 								foreach ($column_exists_query as $column){
 									$table_with_column = $table_to_check;
@@ -426,8 +426,8 @@ public function PerformSearch(){
 
 								$filter_join = $filter['JoinTables'][$type['Key']];
 
-								$joins.= "LEFT JOIN \"".$filter_join['Table']."\" ON \"".$type['Table']."\".\"ID\" = \"".$filter_join['Column']."\"";
-							
+								$joins.= "LEFT JOIN \"".$filter_join['Table']."\" ON \"".$type['Table']."\".\"ID\" = \"".$filter_join['Table']."\".\"".$filter_join['Column']."\"";
+
 								if (is_array($filter['Value'])){
 									$ids = '';
 									foreach ($filter['Value'] as $id){
@@ -440,6 +440,9 @@ public function PerformSearch(){
 									$ids = $filter['Value'];
 								}
 
+								if ($relations_sql !== ''){
+									$relations_sql.= " AND ";
+								}
 								$relations_sql.= "\"".$filter_join['Table']."\".\"".$filter['Table']."ID\" IN (". $ids .")";
 							}
 
@@ -459,7 +462,7 @@ public function PerformSearch(){
 			$sql.= $where;
 
 			// Debugging
-			//echo '<h3 style="position: relative; padding: 20px; background: #EEEEEE; z-index: 999;">'.$sql.'</h3>';
+			//echo '<h3 style="position: relative; padding: 20px; background: #EEEEEE; z-index: 999;">'.str_replace('"', '`', $sql).'</h3>';
 
 			// Eexecutioner enter stage left
 			$results = DB::query($sql);
@@ -482,14 +485,18 @@ public function PerformSearch(){
 		}
 
 		// Apply sorting
-		$sort = self::get_mapped_sort()['Sort'];		
-		$sort = str_replace("'", "\'", $sort);
-		$sort = str_replace('"', '\"', $sort);
-		$sort = str_replace('`', '\`', $sort);
+		if(isset(self::get_mapped_sort()['Sort'])){
+			$sort = self::get_mapped_sort()['Sort'];		
+			$sort = str_replace("'", "\'", $sort);
+			$sort = str_replace('"', '\"', $sort);
+			$sort = str_replace('`', '\`', $sort);
+		}else{
+			$sort = 'Title ASC';
+		}
 		$allResults = $allResults->Sort($sort);
 
 		// Remove duplicates
-		//$allResults->removeDuplicates('ID');
+		$allResults->removeDuplicates('ID');
 
 		// load into a paginated list. To change the items per page, set via the template (ie Results.setPageLength(20))
 		$paginatedItems = PaginatedList::create($allResults, $this->request);