Incorrect variable causes trouble if an attempted claim return action fails

[Michael Peters]

Discovered that we get the error below when marking an item claimed returned that prevented the actual failure reason from being displayed. After changing the "E" to the correct "robj" variable we were then able to see the actual reason why the attempted claim return action failed (in this case, the asset.copy had been deleted!).

Please open a helpdesk ticket and include the following text:

Thu Aug 04 2011 17:34:38 GMT-0400 (Eastern Daylight Time)

The items were not likely marked Claimed Returned.

{
"message":"E is not defined",
"fileName":"http://evergreen.lib.in.us/xul/rel_2_0_4/server/patron/items.js",
"lineNumber":525,
"stack":"(1)@http://evergreen.lib.in.us/xul/rel_2_0_4/server/patron/items.js:525\n(
[
object XULCommandEvent
]
)@http://evergreen.lib.in.us/xul/rel_2_0_4/server/patron/items.js:156\n",
"name":"ReferenceError

[Michael Peters (mrpeters)]

Thanks much to Jason Etheridge for committing the fix and thanks to Thomas Berezansky for discovering the incorrect variable!

http://git.evergreen-ils.org/?p=Evergreen.git;a=commitdiff;h=04008a5bece9c12d2e6ce48ed7ed11eb5d537289

[Jason Etheridge (phasefx)]

Also pushed to rel_2_1 and rel_2_0