Bug 23539

Summary: accountlines.accounttype should match authorised_values.authorised_value in size
Product: Koha Reporter: Kyle M Hall <kyle>
Component: Architecture, internals, and plumbingAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: minor    
Priority: P5 - low CC: fridolin.somers, m.de.rooy, martin.renvoize
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
19.11.00,19.05.04
Attachments: Bug 23539: accountlines.accounttype should match authorised_values.authorised_value in size
Bug 23539: accountlines.accounttype should match authorised_values.authorised_value in size
Bug 23539: accountlines.accounttype should match authorised_values.authorised_value in size

Description Kyle M Hall 2019-09-04 17:43:59 UTC
Account type is a varchar(16), but the authorised value filling it ( MANUAL_INVOICE type ) is a varchar(80). Account type should be increased to this size to avoid truncation of authorised values.
Comment 1 Kyle M Hall 2019-09-04 17:49:31 UTC
Created attachment 92591 [details] [review]
Bug 23539: accountlines.accounttype should match authorised_values.authorised_value in size
Comment 2 Owen Leonard 2019-09-05 12:24:17 UTC
Created attachment 92605 [details] [review]
Bug 23539: accountlines.accounttype should match authorised_values.authorised_value in size

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Marcel de Rooy 2019-09-06 05:57:58 UTC
Created attachment 92614 [details] [review]
Bug 23539: accountlines.accounttype should match authorised_values.authorised_value in size

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Martin Renvoize 2019-09-06 12:09:13 UTC
I've been meaning to sort this one out for ages.. thanks so much for getting to it :).
Comment 5 Martin Renvoize 2019-09-06 12:12:49 UTC
Nice work!

Pushed to master for 19.11.00
Comment 6 Fridolin Somers 2019-09-09 10:35:30 UTC
Pushed to 19.05.x for 19.05.04
Comment 7 Fridolin Somers 2019-09-09 10:36:16 UTC
(In reply to Kyle M Hall from comment #0)
> Account type is a varchar(16), but the authorised value filling it (
> MANUAL_INVOICE type ) is a varchar(80). Account type should be increased to
> this size to avoid truncation of authorised values.

Authorised values category is call 'MANUAL_INV'
Comment 8 Kyle M Hall 2019-09-17 19:17:52 UTC
(In reply to Fridolin SOMERS from comment #7)
> (In reply to Kyle M Hall from comment #0)
> > Account type is a varchar(16), but the authorised value filling it (
> > MANUAL_INVOICE type ) is a varchar(80). Account type should be increased to
> > this size to avoid truncation of authorised values.
> 
> Authorised values category is call 'MANUAL_INV'

Thanks for the correction Frido!