aboutsummaryrefslogtreecommitdiffstats
path: root/include/functions_search.inc.php
diff options
context:
space:
mode:
authorrvelices <rv-github@modusoptimus.com>2014-05-31 18:50:30 +0000
committerrvelices <rv-github@modusoptimus.com>2014-05-31 18:50:30 +0000
commit8b4857c7730a8c55396ac254c50a4ff1e0d8016b (patch)
tree480de7e1f2f81b54b95f4bd557926965f35cb996 /include/functions_search.inc.php
parentbd73ad9679430a1bc27beb78b24a1d64ab0509ec (diff)
quick search - small fixes & improvements
available number of comments per user do not use visible_categories (only forbidden) git-svn-id: http://piwigo.org/svn/trunk@28579 68402e56-0260-453c-a942-63ccdbb3a9ee
Diffstat (limited to '')
-rw-r--r--include/functions_search.inc.php24
1 files changed, 14 insertions, 10 deletions
diff --git a/include/functions_search.inc.php b/include/functions_search.inc.php
index 6280c4d9d..1b3e521ea 100644
--- a/include/functions_search.inc.php
+++ b/include/functions_search.inc.php
@@ -606,7 +606,7 @@ class QMultiToken
$crt_token .= $ch;
break;
}
- if (strlen($crt_token) && preg_match('/[0-9]/', substr($crt_token,-1))
+ if (strlen($crt_token) && preg_match('/[0-9]/', substr($crt_token,-1))
&& $qi+1<strlen($q) && preg_match('/[0-9]/', $q[$qi+1]))
{// dot between digits is not a separator e.g. F2.8
$crt_token .= $ch;
@@ -707,7 +707,7 @@ class QMultiToken
}
/**
- * Applies recursively a search scope to all sub single tokens. We allow 'tag:(John Bill)' but we cannot evaluate
+ * Applies recursively a search scope to all sub single tokens. We allow 'tag:(John Bill)' but we cannot evaluate
* scopes on expressions so we rewrite as '(tag:John tag:Bill)'
*/
private function apply_scope(QSearchScope $scope)
@@ -843,8 +843,8 @@ function qsearch_get_text_token_search_sql($token, $fields)
if ($use_ft)
{
- $max = max( array_map( 'mb_strlen',
- preg_split('/['.preg_quote('!"#$%&()*+,./:;<=>?@[\]^`{|}~','/').']+/', $variant, PREG_SPLIT_NO_EMPTY)
+ $max = max( array_map( 'mb_strlen',
+ preg_split('/['.preg_quote('-\'!"#$%&()*+,./:;<=>?@[\]^`{|}~','/').']+/', $variant)
) );
if ($max<4)
$use_ft = false;
@@ -966,7 +966,7 @@ WHERE ('. implode("\n OR ",$clauses) .')';
// check adjacent short words
for ($i=0; $i<count($expr->stokens)-1; $i++)
{
- if ( (strlen($expr->stokens[$i])<=3 || strlen($expr->stokens[$i+1])<=3)
+ if ( (strlen($expr->stokens[$i]->term)<=3 || strlen($expr->stokens[$i+1]->term)<=3)
&& (($expr->stoken_modifiers[$i] & (QST_QUOTED|QST_WILDCARD)) == 0)
&& (($expr->stoken_modifiers[$i+1] & (QST_BREAK|QST_QUOTED|QST_WILDCARD)) == 0) )
{
@@ -995,11 +995,16 @@ SELECT image_id FROM '.IMAGE_TAG_TABLE.'
if ($expr->stoken_modifiers[$i]&QST_NOT)
$not_ids = array_merge($not_ids, $tag_ids);
else
- $positive_ids = array_merge($positive_ids, $tag_ids);
+ {
+ if (strlen($token->term)>2 || count($expr->stokens)==1 || isset($token->scope) || ($token->modifier&(QST_WILDCARD|QST_QUOTED)) )
+ {// add tag ids to list only if the word is not too short (such as de / la /les ...)
+ $positive_ids = array_merge($positive_ids, $tag_ids);
+ }
+ }
}
elseif (isset($token->scope) && 'tag' == $token->scope->id && strlen($token->term)==0)
{
- if ($tokens[$i]->modifier & QST_WILDCARD)
+ if ($token->modifier & QST_WILDCARD)
{// eg. 'tag:*' returns all tagged images
$qsr->tag_iids[$i] = query2array('SELECT DISTINCT image_id FROM '.IMAGE_TAG_TABLE, null, 'image_id');
}
@@ -1095,7 +1100,7 @@ function get_quick_search_results($q, $options)
$cache_key = $persistent_cache->make_key( array(
strtolower($q),
$conf['order_by'],
- $user['id'],$user['cache_update_time'],
+ $user['id'],$user['cache_update_time'],
isset($options['permissions']) ? (boolean)$options['permissions'] : true,
isset($options['images_where']) ? $options['images_where'] : '',
) );
@@ -1221,8 +1226,7 @@ function get_quick_search_results_no_cache($q, $options)
array
(
'forbidden_categories' => 'category_id',
- 'visible_categories' => 'category_id',
- 'visible_images' => 'i.id'
+ 'forbidden_images' => 'i.id'
),
null,true
);