Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

subject indexed for searchable_text #194

Closed
agitator opened this Issue Dec 10, 2014 · 3 comments

Comments

Projects
None yet
3 participants
@agitator
Copy link
Member

agitator commented Dec 10, 2014

is there a reason why keywords are not index for searchable_text?

def SearchableText(obj, text=False):
    return u" ".join((
        safe_unicode(obj.id),
        safe_unicode(obj.title) or u"",
        safe_unicode(obj.description) or u"",
    ))

@indexer(IDocument)
def SearchableText_document(obj):
    if obj.text is None or obj.text.output is None:
        return SearchableText(obj)
    return _unicode_save_string_concat(SearchableText(obj), obj.text.output)
@pbauer

This comment has been minimized.

Copy link
Member

pbauer commented Dec 10, 2014

@agitator no reason, please create a pr

@pbauer pbauer added the 01 type: bug label Dec 10, 2014

@gforcada

This comment has been minimized.

Copy link
Contributor

gforcada commented Nov 26, 2015

Just stumbled upon this as well :-)

@agitator

This comment has been minimized.

Copy link
Member Author

agitator commented Nov 26, 2015

do it ;-)

gforcada added a commit that referenced this issue Nov 26, 2015

@pbauer pbauer closed this in #303 Nov 26, 2015

pbauer added a commit to plone/buildout.coredev that referenced this issue Nov 26, 2015

[fc] Repository: plone.app.contenttypes
Branch: refs/heads/master
Date: 2015-11-26T14:27:53+01:00
Author: Gil Forcada (gforcada) <gforcada@gnome.org>
Commit: plone/plone.app.contenttypes@9d3b9de

Index subject field on the catalog so that is searchable

Fixes plone/plone.app.contenttypes#194

Files changed:
M CHANGES.rst
M plone/app/contenttypes/indexers.py
M plone/app/contenttypes/tests/test_indexes.py
Repository: plone.app.contenttypes
Branch: refs/heads/master
Date: 2015-11-26T15:41:54+01:00
Author: Philip Bauer (pbauer) <bauer@starzel.de>
Commit: plone/plone.app.contenttypes@054b653

Merge pull request #303 from plone/index-subject-194

Index subject field on the catalog so that is searchable

Files changed:
M CHANGES.rst
M plone/app/contenttypes/indexers.py
M plone/app/contenttypes/tests/test_indexes.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.