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

send_alerts fails when processing saved search which uses xapian engine #169

Open
drtjmb opened this issue Nov 29, 2013 · 0 comments
Open

send_alerts fails when processing saved search which uses xapian engine #169

drtjmb opened this issue Nov 29, 2013 · 0 comments

Comments

@drtjmb
Copy link
Member

@drtjmb drtjmb commented Nov 29, 2013

Only occurs if xapian search engine is used.

To reproduce:

Carry out simple search
Save simple search
Turn simple search into alert (eg. set frequency = daily)
Run send_alerts

Analysis:

Serialised saved search includes search engine identifier

?plugin=Xapian&searchid=simple&dataset=archive&exp=0%7C1%7C-date%2Fcreators_name%2Ftitle%7Carchive%7C-%7Cq%3A%3AALL%3AIN%3Ayellow%7C-%7C

DataObj/SavedSearch::send_out_alert tries to add_field to the xapian search plugin which doesn't make sense for xapian (internal search plugin has some wrapping in order to inherit the add_field from EPrint::Search).

send_alerts fails - any unprocessed saved searches will not processed.

sebastfr pushed a commit that referenced this issue Nov 29, 2013
sebastfr pushed a commit that referenced this issue Nov 29, 2013
drtjmb pushed a commit that referenced this issue Dec 4, 2013
…169

Email alerts will not work for saved 'simple' searches that are run with the xapian engine. This changeset shows a warning when editing a saved search.
drtjmb pushed a commit that referenced this issue Dec 4, 2013
…169

Email alerts will not work for saved 'simple' searches that are run with the xapian engine. This changeset shows a warning when editing a saved search.
@jiadiyao jiadiyao added the bug label Aug 25, 2017
@jiadiyao jiadiyao added enhancement and removed bug labels Nov 28, 2017
drn05r added a commit to eprints/eprints3.4 that referenced this issue Feb 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants