Bug 13875

Summary: Columns configuration - table name cannot contain underscore
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: josef.moravec, tomascohen
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:
Bug Depends on:    
Bug Blocks: 13864    
Attachments: Bug 13875: Columns configuration - Allow underscore in table name
[SIGNED-OFF] Bug 13875: Columns configuration - Allow underscore in table name
[PASSED QA] Bug 13875: Columns configuration - Allow underscore in table name
[PASSED QA] Bug 13864: Columns configuration - Late orders
[PASSED QA] Bug 13875: Columns configuration - Allow underscore in table name

Description Jonathan Druart 2015-03-19 15:39:11 UTC
This limitation is useless, the underscore is used to separate the page, the table and the column.
Comment 1 Jonathan Druart 2015-03-19 15:41:44 UTC Comment hidden (obsolete)
Comment 2 Josef Moravec 2015-03-31 11:43:01 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2015-04-06 09:18:25 UTC Comment hidden (obsolete)
Comment 4 Katrin Fischer 2015-04-06 09:21:32 UTC Comment hidden (obsolete)
Comment 5 Katrin Fischer 2015-04-06 10:05:21 UTC
Created attachment 37523 [details] [review]
[PASSED QA] Bug 13875: Columns configuration - Allow underscore in table name

This is a bug fix for bug the columns configuration feature, and
specialy for bug 13864.

Test plan:
Apply patches on bug 13864 and save columns for the late orders (in the
columns configuration admin page).
Confirm that they are correctly saved.

Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
works as expected

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Tomás Cohen Arazi 2015-04-08 18:03:59 UTC
Patch pushed to master.

Thanks Jonathan!