Summary: | xml parsing error when the webservice GetPatronInfo/show_loans is called | ||
---|---|---|---|
Product: | Koha | Reporter: | Koha Team University Lyon 3 <koha> |
Component: | Circulation | Assignee: | Fridolin Somers <fridolin.somers> |
Status: | CLOSED FIXED | QA Contact: | Mason James <mtj> |
Severity: | normal | ||
Priority: | P5 - low | CC: | chris, gmcharlt, kyle.m.hall, mtj, paul.poulain |
Version: | 3.8 | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Proposed patch (dates with hour)
Bug 8275: xml parsing error when the webservice GetPatronInfo/show_loans is called |
Description
Koha Team University Lyon 3
2012-06-20 14:26:22 UTC
Created attachment 10999 [details] [review] Proposed patch (dates with hour) Proposed patch. Since issue and due date have now hour and minutes, I propose to keep this information in the webservice. So I formatted dates using '%Y-%m-%d %H:%M'. Note that it's the same format as 'timestamp' field. Created attachment 11822 [details] [review] Bug 8275: xml parsing error when the webservice GetPatronInfo/show_loans is called Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> > Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz> patch looks good, passing QA… $ koha-qa.pl testing 1 commit(s) (applied to commit 0acebb8) * 3414757 Bug 8275: xml parsing error when the webservice GetPatronInfo/sho C4/ILSDI/Services.pm * C4/ILSDI/Services.pm OK Koha team Lyon 3, do you agree with the counter fix proposed by Fridolyn ? (In reply to comment #4) > Koha team Lyon 3, do you agree with the counter fix proposed by Fridolyn ? Seems that makes sense, it's ok for us Patch pushed to master Pushed to 3.8.x, will be in 3.8.5 |