Bug 10572

Summary: Add phone to message_transport_types table for new installs
Product: Koha Reporter: Marcel de Rooy <m.de.rooy>
Component: Installation and upgrade (web-based installer)Assignee: Marcel de Rooy <m.de.rooy>
Status: CLOSED FIXED QA Contact:
Severity: trivial    
Priority: P5 - low CC: gmcharlt, jonathan.druart, katrin.fischer
Version: Main   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=4246
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 10520    
Bug Blocks:    
Attachments: Bug 10572: Add phone to message_transport_types table for new installs
Bug 10572: Add phone to message_transport_types table for new installs
Bug 10572: Followup: dbrev for phone message_transport_type
Bug 10572 - Add phone to message_transport_types table for new installs- QA Followup
Bug 10572: Add phone to message_transport_types table for new installs
Bug 10572: Followup: dbrev for phone message_transport_type
Bug 10572 - Add phone to message_transport_types table for new installs- QA Followup

Description Marcel de Rooy 2013-07-11 08:02:43 UTC
In connection to bug 4246 commit 591bae324d77d285b27dfa6e10dc7001671f53cc added phone to the message_transport_types table in updatedatabase.pl

This should be done also for new installs via the .sql files of the web installer.

I am adding it only to English now, basing this change upon the work in reports 10509 and 10520. (I discovered this tiny omission there too.)
Comment 1 Marcel de Rooy 2013-07-11 08:08:51 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2013-08-02 17:43:20 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2013-08-04 13:26:54 UTC
Hi Kyle, this could go in as a bugfix to earlier versions as well, but the new web installer translations probably can't. Could you provide a follow-up please?
Comment 4 Katrin Fischer 2013-08-04 13:29:39 UTC
Also, I think to fix this properly we shoudl include a database update. It looks like 4246 has been released with 3.10, so there is a good chance of people installing 3.10 or 3.12 will have missed the database update and will have the problem.
Comment 5 Marcel de Rooy 2013-08-07 12:31:59 UTC Comment hidden (obsolete)
Comment 6 Marcel de Rooy 2013-08-07 12:35:32 UTC
Back to Signed off.
Comment 7 Katrin Fischer 2013-08-25 19:25:08 UTC
Kyle, could you address comment 3 please?
Comment 8 Jonathan Druart 2013-09-12 14:48:03 UTC
(In reply to Katrin Fischer from comment #7)
> Kyle, could you address comment 3 please?

Is not it fixed with the second patch?
Comment 9 Kyle M Hall 2013-09-12 17:22:44 UTC Comment hidden (obsolete)
Comment 10 Kyle M Hall 2013-09-12 17:25:50 UTC
Created attachment 21052 [details] [review]
Bug 10572: Add phone to message_transport_types table for new installs

In connection to bug 4246 commit 591bae324d77d285b27dfa6e10dc7001671f53cc added
phone to the message_transport_types table in updatedatabase.pl

This should be done also for new installs via the .sql files of the web
installer.

I am adding it only to English now, basing this change upon the work in reports
10509 and 10520. (I discovered this tiny omission there too.)

Test plan:
Apply patches 10509/10520.
Run new install or run this particular sql file from command line.
Note: If you only run this particular file, you do not need patch 10509.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 11 Kyle M Hall 2013-09-12 17:25:57 UTC
Created attachment 21053 [details] [review]
Bug 10572: Followup: dbrev for phone message_transport_type

Test plan:
Run the updatestructure step of web installer.
Check if you have a phone message_transport_type.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 12 Kyle M Hall 2013-09-12 17:26:12 UTC
Created attachment 21054 [details] [review]
Bug 10572 - Add phone to message_transport_types table for new installs- QA Followup

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 13 Galen Charlton 2013-10-31 14:46:18 UTC
Pushed to master.  Thanks, Marcel!