divio / aldryn-search

Haystack 2.0 search index for django CMS
Other
48 stars 78 forks source link

Fix page title indexing #6

Closed qris closed 10 years ago

qris commented 10 years ago

Page titles should be indexed too!

Refactor get_search_data to avoid unnecessary string concatenation, which is wasteful and produces unexpected spaces in the output.

czpython commented 10 years ago

@qris page titles can be indexed by setting the INDEX_TITLE attribute to True, I set this to True by default. Regarding the tests, not sure if the page title tests are necessary as haystack already has it's own tests, maybe we just test theINDEX_TITLE flag. @stefanfoulis thoughts ?