From f88976916dd8e2ce11df77f7068176c975035621 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julien=20Fastr=C3=A9?= <julien@fastre.info>
Date: Mon, 29 Dec 2014 00:43:33 +0100
Subject: [PATCH] implements parsing of search pattern

refs #223
tests must be finished
---
 Search/SearchProvider.php           | 53 +++++++++++++++++++++++++++--
 Tests/Search/SearchProviderTest.php |  9 +++--
 2 files changed, 57 insertions(+), 5 deletions(-)

diff --git a/Search/SearchProvider.php b/Search/SearchProvider.php
index f8ab1a636..f173cd729 100644
--- a/Search/SearchProvider.php
+++ b/Search/SearchProvider.php
@@ -48,14 +48,19 @@ class SearchProvider
      */
     public function parse($pattern)
     {
-        $terms = array();
+        //reset must be extracted
+        $this->mustBeExtracted = array();
+        //filter to lower and remove accentued
+        $filteredPattern = mb_strtolower($pattern);
         
-        $terms['_domain'] = $this->getDomain($pattern);
+        $terms = $this->extractTerms($filteredPattern);      
+        $terms['_domain'] = $this->extractDomain($filteredPattern);
+        $terms['_default'] = $this->extractDefault($filteredPattern);
         
         return $terms;
     }
     
-    private function getDomain($subject)
+    private function extractDomain(&$subject)
     {
         preg_match_all( '/@([a-z]+)/', $subject, $terms);
         
@@ -63,9 +68,51 @@ class SearchProvider
             throw new ParsingException('You should not have more than one domain');
         }
         
+        //add pattern to be extracted
+        if (isset($terms[0][0])) {
+            $this->mustBeExtracted[] = $terms[0][0];
+        }
+        
         return isset($terms[1][0]) ? $terms[1][0] : NULL;
     }
     
+    private function extractTerms(&$subject)
+    {
+        $terms = array();
+        preg_match_all('/([a-z]+):([\w\-]+|\([^\(\r\n]+\))/', $subject, $matches);
+        
+        foreach ($matches[2] as $key => $match) {
+            //remove from search pattern
+            $this->mustBeExtracted[] = $matches[0][$key];
+             //strip parenthesis
+            if (mb_substr($match, 0, 1) === '(' && 
+                  mb_substr($match, mb_strlen($match) - 1) === ')') {
+                $match = mb_substr($match, 1, mb_strlen($match)-2);
+            }
+            $terms[$matches[1][$key]] = $match;
+        }
+        
+        return $terms;
+    }
+    
+    /**
+     * store string which must be extracted to find default arguments
+     * 
+     * @var string[]
+     */
+    private $mustBeExtracted = array();
+    
+    /**
+     * extract default (residual) arguments
+     * 
+     * @param string $subject
+     * @return string
+     */
+    private function extractDefault($subject) {
+        return trim(str_replace($this->mustBeExtracted, '', $subject));
+    }
+    
+    
     /**
      * search through services which supports domain and give
      * results as html string
diff --git a/Tests/Search/SearchProviderTest.php b/Tests/Search/SearchProviderTest.php
index e80465f23..d6cbad829 100644
--- a/Tests/Search/SearchProviderTest.php
+++ b/Tests/Search/SearchProviderTest.php
@@ -39,9 +39,14 @@ class SearchProviderTest extends \PHPUnit_Framework_TestCase
     
     public function testDomain()
     {
-        $term = $this->p("@person is not my name");
+        $terms = $this->p("@person birthdate:2014-01-02 name:(my name) is not my name");
         
-        $this->assertEquals('person', $term['_domain']);
+        $this->assertEquals(array(
+           '_domain' => 'person',
+           'birthdate' => '2014-01-02',
+           '_default' => 'is not my name',
+           'name' => 'my name'
+        ), $terms);
     }
     
     /**