]> git.evergreen-ils.org Git - working/SIPServer.git/commit
Dan Scott's patch:
authordjfiander <djfiander>
Wed, 13 Jan 2010 01:39:30 +0000 (01:39 +0000)
committerdjfiander <djfiander>
Wed, 13 Jan 2010 01:39:30 +0000 (01:39 +0000)
commit5ace8f64f8ae2cd190ecdbf4e805a5d95560ab53
tree47da7465819ee3e5f3ab6013298e883ae72e628f
parent80735e01c0568ae7f096d25dd6d7b331b9305837
Dan Scott's patch:

Item status response 18 was returning CJ0|CM0| which are invalid
according to the SIP2 specification; they should either be
18-char fixed fields or not there at all (as they're optional).

It looks like it was just a reversed test. In looking through
Sip/MsgType.pm in that area, I put together a patch which
prevents those fields from being generated if the incoming value
is 0 - and separates out the value assignment & test into
separate statements, which should make the code a bit easier to
read.

As an aside, I believe the same problem was preventing the due
date (AH) from being returned in the item status response.
Sip/MsgType.pm