| Time |
Nick |
Message |
| 06:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |
| 07:28 |
|
rjackson_isl_hom joined #evergreen |
| 07:41 |
|
collum joined #evergreen |
| 08:38 |
|
mmorgan joined #evergreen |
| 08:43 |
|
mantis joined #evergreen |
| 09:03 |
|
Dyrcona joined #evergreen |
| 09:21 |
* Dyrcona |
searches for the minutes of the meeting where we voted on the web staff client. |
| 09:29 |
Dyrcona |
OK. I found it, but the vote wasn't recorded: https://wiki.evergreen-ils.org/doku.php?id=dev:hackfest:hackaway-2013 (I remember that it was unanimous.) |
| 09:31 |
Dyrcona |
Also, some notes of the discussion are available for those interested in history: https://docs.google.com/document/d/1QALWLD-vAjDV8HseQyE5j0Yc-93Nrl-1B00xeP_npak/edit#heading=h.qkhgf919wj2j |
| 09:41 |
|
rfrasur joined #evergreen |
| 09:46 |
Bmagic |
Dyrcona: that was a fun read |
| 09:57 |
|
jvwoolf joined #evergreen |
| 10:34 |
mantis |
What does the "error -1" status mean in Patron > Holds? |
| 10:36 |
Dyrcona |
mantis: It usually means the hold cannot possibly be filled under current circumstances. Perhaps all available copies were deleted, etc. |
| 10:37 |
mantis |
Dyrcona++ |
| 10:37 |
Dyrcona |
Basically, the code to set the hold status went through all of its conditions and nothing matched. |
| 10:43 |
pastebot |
"Dyrcona" at 168.25.130.30 pasted "Hold status" (12 lines) at http://paste.evergreen-ils.org/14435 |
| 10:44 |
Dyrcona |
mantis ^^ |
| 10:45 |
mantis |
Thanks! |
| 10:46 |
|
jvwoolf joined #evergreen |
| 10:49 |
Dyrcona |
Looking at the code, I'd say the most likely culprit is that the hold has a current copy plus the capture time is set but the current copy does not have a status of in transit nor on the hold shelf. |
| 10:50 |
Dyrcona |
So, somebody messed up handling the copy/hold. |
| 10:52 |
Dyrcona |
I'm not sure what happens if a permission or other error occurs while looking up the status. I'd expect the staff client to pop up a dialog, but I didn't check that code. |
| 11:16 |
mmorgan |
mantis: bug 1526605 |
| 11:16 |
pinesol |
Launchpad bug 1526605 in Evergreen "Holds can get stuck with a hold status of -1" [Undecided,New] https://launchpad.net/bugs/1526605 |
| 11:16 |
mmorgan |
Also bug 1469287 |
| 11:16 |
pinesol |
Launchpad bug 1469287 in Evergreen "Retargeting a cancelled hold can cause problems with uncancelled captured holds" [Medium,Invalid] https://launchpad.net/bugs/1469287 |
| 11:16 |
berick |
-1 means "something ain't right" |
| 11:17 |
* mmorgan |
hasn't looked at those in awhile. |
| 11:21 |
abneiman |
oh, that 1526605 - I have heard rumors of it from many sources, but never able to reproduce it myself. mantis if you can shed any light on that one it would be helpful! |
| 11:23 |
* mmorgan |
's theory is that the status of the item gets changed by some other process to a status that doesn't fit with the state of the hold |
| 11:28 |
mantis |
Unfortunately I don't have much information on the item hold but I'll let you all know if I get a response from the library that reported |
| 11:29 |
mmorgan |
mantis: Retargeting the hold should fix it so it can capture another item |
| 11:57 |
csharp_ |
mantis: if you're comfortable reading perl, checkout open-ils.circ.hold.status.retrieve around line 1503 or so in Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm |
| 11:58 |
csharp_ |
it goes through several checks and if fails all of them, it returns the -1 status |
| 11:59 |
csharp_ |
the bug would be whatever is resulting in not landing on one of the other statuses |
| 12:00 |
|
jihpringle joined #evergreen |
| 12:34 |
|
collum joined #evergreen |
| 12:59 |
|
jihpringle joined #evergreen |
| 13:11 |
|
collum joined #evergreen |
| 16:22 |
|
jihpringle joined #evergreen |
| 16:45 |
|
jvwoolf left #evergreen |
| 17:12 |
|
mmorgan left #evergreen |
| 18:01 |
pinesol |
News from qatests: Testing Success <http://testing.evergreen-ils.org/~live> |