xapian-haystack icon indicating copy to clipboard operation
xapian-haystack copied to clipboard

Limit content search

Open asedeno opened this issue 3 years ago • 1 comments

As per the haystack documentation [1] the content field is a special term for the field marked document=True. This change implements that behavior.

[1] https://django-haystack.readthedocs.io/en/master/searchqueryset_api.html#the-content-shortcut

This reopens #182 on a topic branch.

NB: This has been rebased onto master as of now, but has not been re-tested at all. Consider it a draft starting point to consider the issue.

asedeno avatar Sep 20 '21 15:09 asedeno

Rebased so we can get all the test coverage stuff running on the branch.

As mentioned in the previous iteration of this PR, it breaks tests. I believe those tests are broken by design because the content field was being misused. That said, fixing it would not be backwards compatible, so doing so probably warrants a new major version or something. I'm not sure when I'll get around to examining the broken tests yet though, so for now this should linger as a draft unless someone else wants to work on it.

asedeno avatar Sep 27 '21 04:09 asedeno