Skip to content
This repository has been archived by the owner on Dec 21, 2017. It is now read-only.

v0.3.0 #6

Open
wants to merge 23 commits into
base: master
Choose a base branch
from
Open

v0.3.0 #6

wants to merge 23 commits into from

Conversation

emmanuel-keller
Copy link
Contributor

No description provided.

emmanuel-keller and others added 23 commits September 2, 2013 14:42
IndexAnalyzer and the DecimalAnalyzer. Filtering (with, without) and
sorting (order_by) use now the right field name.
documents were extracted from the ActiveRecord instance. Two related
test cases have been added.
tests was correct when each model item was checked separately. Now the
test checks the full model.
- Loading all records from the database at once + manual reordering to match OSS result.
- Added :include option to #oss_search & #find_results to eager load associations.
- Using after_commit instead of after_save for indexing documents.

Other minor things:

- Renamed #search to #oss_search (with alias) for simpler custom searches in models.
- Overall ruby syntax & functional code refactoring.
Fixes & optimizations to Searchable
Conflicts:
	lib/oss_active_record/searchable.rb
FIXME: refactor IndexInstance to either receive index as dependency or
lazy build it on each public method.
…ts_for_nil_class

Create index object when needed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants