Loading...
 
Skip to main content

full text search is busted

Status
Closed
Subject
full text search is busted
Version
1.9.x
Category
  • Usability
  • Feature request
Feature
Search
Resolution status
Fixed or Solved
Submitted by
Peter
Volunteered to solve
Peter
Lastmod by
amette
Rating
(0)
Description

CLIP from my whining about why the search doesn’t work in TW.


  1. 1 - why isn’t there a proper SEARCH ?


- to start this off can we at least agree on a couple things:
o standard search for - RED BALL would search on RED AND BALL
o TW does RED OR BALL – which is WRONG!!

- BASIC search should be capable of:
o RED BALL = RED AND BALL
o “red ball” = only match “red ball” as quoted
o and then, maybe RED or BALL = RED OR BALL


with a little more digging and some input from marclaporte, redflow and mose I think I have figured out the problem:

- TW supports the concept of FULL TEXT SEARCH (FT)
- this allows user to (from example above) do a search like “+red +ball” to achieve “red AND ball”.
- but these is a bug in concept in searchlib - function _find()
- if FT is set then _FIND does the right thing by doing a MATCH AGAINST sql query
- HOWEVER – if it doesn’t find any results it ALSO will do a simple (i.e. non-FT) search
o in concept this is silly since it defeats whole point of doing FT in the first place
o also, it generates an sql error because for simple search a REGEXP query is done which will choke on the + signs in the search string

Solution

BUG FIX SUGGESTION:

- remove code which does “simple search if no results”
o at least make it optional

OTHER improvements

- as I have stated before – the “NORMAL” concept of search IS my fixed version of FT search
- because of that the default search should be FT
o by that I mean “red ball” should automatically do  FT “+red +ball” without having to put + signs

Importance
6
Priority
30
Demonstrate Bug on Tiki 19+
Please demonstrate your bug on show2.tiki.org
Demonstrate Bug (older Tiki versions)
Ticket ID
166
Created
Monday 09 May, 2005 03:20:18 UTC
by Unknown
LastModif
Friday 24 September, 2021 12:23:39 UTC


Collapse/expand modules below
Show PHP error messages