Bug 9263

Summary: Trim barcodes in offline circulation
Product: Koha Reporter: Fridolin Somers <fridolin.somers>
Component: CirculationAssignee: Fridolin Somers <fridolin.somers>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P5 - low CC: gmcharlt, kyle.m.hall, kyle, m.de.rooy
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Proposed patch
[SIGNED-OFF] Bug 9263: Trim barcodes in offline circulation
Bug 9263 - Followup - Improve speed of substitution.
Proposed patch (revisited)
Bug 9263: Trim barcodes in offline circulation

Description Fridolin Somers 2012-12-11 16:32:19 UTC
In normal circulation page, when a barcode starts or ends with spaces, they are ignored and circulation is performed.
Do the same in barcodes coming form offline circulation.
Comment 1 Fridolin Somers 2012-12-11 16:35:00 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2012-12-12 15:40:41 UTC Comment hidden (obsolete)
Comment 3 Kyle M Hall 2012-12-12 15:42:26 UTC Comment hidden (obsolete)
Comment 4 Fridolin Somers 2012-12-12 16:03:51 UTC
(In reply to comment #3)
> Bug 9263 - Followup - Improve speed of substitution.


Thanks.
I think also 'g' regexp option was missing in my patch.
Comment 5 Fridolin Somers 2012-12-12 16:20:56 UTC Comment hidden (obsolete)
Comment 6 Kyle M Hall 2012-12-12 17:24:43 UTC
Created attachment 14051 [details] [review]
Bug 9263: Trim barcodes in offline circulation

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 7 Marcel de Rooy 2013-01-20 17:37:45 UTC
QA Comment:
Trivial patch. Did not test offline circ. But this patch is just housekeeping and trims spaces from barcode and cardnumber.
Looks good to me. Will not hurt anyone.
Passed QA
Comment 8 Jared Camins-Esakov 2013-01-23 00:12:06 UTC
This patch has been pushed to master.

Please try to remember to sign off on patches when QAing.
Comment 9 Marcel de Rooy 2013-01-23 11:50:25 UTC
(In reply to comment #8)
> This patch has been pushed to master.
> 
> Please try to remember to sign off on patches when QAing.

Hi Jared,
I explicitly did not add an additional signoff line because I did not test the offline circ as mentioned in my QA comment. Would a Passed-QA-by line in such exceptional (!) cases be of use? (Also note that this is a trivial patch.)
Marcel
Comment 10 Marcel de Rooy 2013-01-23 11:50:25 UTC
(In reply to comment #8)
> This patch has been pushed to master.
> 
> Please try to remember to sign off on patches when QAing.

Hi Jared,
I explicitly did not add an additional signoff line because I did not test the offline circ as mentioned in my QA comment. Would a Passed-QA-by line in such exceptional (!) cases be of use? (Also note that this is a trivial patch.)
Marcel
Comment 11 Jared Camins-Esakov 2013-01-23 13:03:36 UTC
(In reply to comment #10)
> (In reply to comment #8)
> > This patch has been pushed to master.
> > 
> > Please try to remember to sign off on patches when QAing.
> 
> Hi Jared,
> I explicitly did not add an additional signoff line because I did not test
> the offline circ as mentioned in my QA comment. Would a Passed-QA-by line in
> such exceptional (!) cases be of use? (Also note that this is a trivial
> patch.)
> Marcel

Yes, a Passed-QA-by line would be much appreciated.