Changeset 28066 for trunk

Timestamp:

Apr 4, 2014, 6:14:38 AM (11 years ago)

Author:

rvelices

Message:

bug 3056: quick search - fixes and better numeric range searches examples:
ratio:0.9..1.1
ratio:0.9..
ratio:>0.9
ratio:<1.1
ratio:>3/2
ratio:16/9

File:

• trunk/include/functions_search.inc.php (modified) (8 diffs)

trunk/include/functions_search.inc.php

@@ -316 +316 @@
 class QNumericRangeScope extends QSearchScope
 {
-  function __construct($id, $aliases, $allow_empty=false)
+  private $epsilon;
+  function __construct($id, $aliases, $allow_empty=false, $epsilon=0)
   {
     parent::__construct($id, $aliases, $allow_empty, false);
+    $this->epsilon = $epsilon;
   }
 
@@ -327 +329 @@
       $range = array( substr($str,0,$pos), substr($str, $pos+2));
     else
-      $range = array($str, $str);
+    {
+      if ('>' == @$str[0])// ratio:>1
+        $range = array( substr($str,1), '');
+      elseif ('<' == @$str[0]) // size:<5mp
+        $range = array('', substr($str,1));
+      else
+        $range = array($str, $str);
+    }
+
     foreach ($range as $i =>&$val)
     {
-      if (preg_match('/^([0-9.]+)([km])?/i', $val, $matches))
+      if (preg_match('#^([0-9.]+)/([0-9.]+)$#i', $val, $matches))
+      {
+        $val = floatval($matches[1]/$matches[2]);
+      }
+      elseif (preg_match('/^([0-9.]+)([km])?/i', $val, $matches))
       {
         $val = floatval($matches[1]);
@@ -349 +363 @@
       else
         $val = '';
+      if (is_numeric($val))
+      {
+        if ($i)
+          $val += $this->epsilon;
+        else
+          $val -= $this->epsilon;
+      }
     }
 
@@ -584 +605 @@
             {
               if ($i+1 < count($this->tokens))
-                $this->token[$i+1]->modifier |= QST_OR;
+                $this->tokens[$i+1]->modifier |= QST_OR;
               $token->term = "";
             }
@@ -619 +640 @@
   }
 
+  private function apply_scope(QSearchScope $scope)
+  {
+    for ($i=0; $i<count($this->tokens); $i++)
+    {
+      if ($this->tokens[$i]->is_single)
+      {
+        if (!isset($this->tokens[$i]->scope))
+          $this->tokens[$i]->scope = $scope;
+      }
+      else
+        $this->tokens[$i]->aooky_scope($scope);
+    }
+  }
+
   private static function priority($modifier)
   {
@@ -727 +762 @@
 function qsearch_get_images(QExpression $expr, QResults $qsr)
 {
-  $qsr->images_iids = array_fill(0, count($expr->tokens), array());
+  $qsr->images_iids = array_fill(0, count($expr->stokens), array());
 
   $inflector = null;
@@ -830 +865 @@
   foreach ($tokens as $token)
   {
-    if (!isset($token->scope) || 'tag' == $token->scope)
+    if (!isset($token->scope) || 'tag' == $token->scope->id)
     {
       $transliterated_tokens[] = transliterate($token->term);
@@ -1088 +1123 @@
   $scopes[] = new QNumericRangeScope('width', array());
   $scopes[] = new QNumericRangeScope('height', array());
-  $scopes[] = new QNumericRangeScope('ratio', array());
+  $scopes[] = new QNumericRangeScope('ratio', array(), false, 0.001);
   $scopes[] = new QNumericRangeScope('size', array());
   $scopes[] = new QNumericRangeScope('filesize', array());