Comment 14 for bug 1653001

I just took a look at this branch. The default sort is now Copy Location -> Call number, and the call number appears to be sorting by sortkey. However, I found a few issues.

The Copy location sorting is a little wonky. I've attached a screenshot that shows it starts going by the copy location order defined in the local admin interface. Easy Reader is one of the first copy locations defined in that interface, which is why it shows first. However, if you look further down in the sort order, we see Easy Reader titles showing up again.

The barcode link is now embedding the barcode instead of the copy id in the URL, which leads to an empty item status screen.

I would consider merging the code anyway and filing bugs to address the above two issues later, but I'm also having trouble retrieving all of the titles that should be on the pull list. Logged into BR1 on a database with the Concerto data, I should see 41 titles in the pull list. However, I'm only retrieving 16 titles. I do see the 41 in the xul interface and also when I print the pull list using the Print button.

At first, I thought I was seeing a bug I already reported in bug 1643697. However, in that bug, I was always able to retrieve all holds after paging through the list. In this case, I'm unable to see more than the 16 holds.