Comment 6 for bug 1629108

Revision history for this message
Kathy Lussier (klussier) wrote :

Thank you for the guidance Mike! After talking it over with Blake, I would like to create a new bug to address the copy count issue and consider merging this code sooner so that we can address all of the bugs that will be fixed by changing the search path.

After we add a test to this code, I plan to sign off on it, but not merge it immediately. It works well in my testing, but I welcome additional eyeballs on the code.

I also am curious if this code would be eligible for backporting. On the one hand, the change in search path for metarecord searches feels like a new feature to me. However, it also fixes several outstanding bugs related to limiting, the ability to correctly retrieve e-resources, and the ability to page through the metarecord results. Any opinions on that?