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

Jump to bottom

EPrints::Index::update_ordervalues should check existence of ordervalues #48

Closed

sebastfr opened this issue Mar 6, 2013 · 1 comment

Labels

bug

Milestone

sebastfr commented Mar 6, 2013

bin/epadmin reorder calls EPrints::Index::update_ordervalues but this doesn't check if there are ordervalues already.

So if there are no ordervalues, the EPrints::Index::update_ordervalues call will fail silently and won't insert the new order values.

Author

sebastfr commented Mar 6, 2013

Seen on 3.3.10 - there's some new code in trunk's epadmin but the behaviour seems the same if a list of ids is provided.

sebastfr pushed a commit that referenced this issue


          Fixes #48 - deletes ordervalues then insert them

b639539

sebastfr closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment