Ticket #1709 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

Simple_search fails with all packages

Reported by: dread Owned by: dread
Priority: major Milestone: ckan-sprint-2012-01-23
Component: ckan Keywords:
Cc: Repository: ckan
Theme: none

Description (last modified by dread) (diff)

The home page always shows the total number of packages as 0, because the query is '*:*' and SQL search doesn't know how to handle it.

Also, simple_search configuration is not documented.

Also, no tests for sql search.

Change History

comment:1 Changed 2 years ago by dread

  • Description modified (diff)

comment:2 Changed 2 years ago by rgrp

I have already fixed the issue with counts (in a branch to be merged) :-)

Also this is opened in an already ended sprint :-)

IMO we don't bother testing simple_search since intended to be hacky and unsupported. Worth documenting the option but highlighting non-supported nature.

comment:3 Changed 2 years ago by rgrp

  • Owner changed from dread to rgrp
  • Status changed from new to assigned

comment:4 Changed 2 years ago by rgrp

I also note there is an error with how importing was working (re top level imports in lib/search.py) that meant simple_search was broken. I've fixed this locally and raised with Ross (whose commit triggered the problem) ...

comment:5 Changed 2 years ago by dread

  • Owner changed from rgrp to dread

I have checked in a similar fix and with a simple test.

comment:6 Changed 2 years ago by dread

  • Status changed from assigned to closed
  • Resolution set to fixed

comment:7 Changed 2 years ago by dread

Note: imports issue was previously dealt with in #1708 and was originally caused by Ian's commit - Ross just merged it in to master.

Note: See TracTickets for help on using tickets.