Bug 23315

Summary: Some system preferences are no longer editable
Product: Koha Reporter: Chris Cormack <chris>
Component: Staff interfaceAssignee: Chris Cormack <chris>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: major    
Priority: P5 - low CC: fridolin.somers, gmcharlt, lucas, m.de.rooy
Version: master   
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:
19.11.00,19.05.03
Bug Depends on: 21582    
Bug Blocks:    
Attachments: Proposed patch
Bg 23315 : Some systempreferences can not be edited
Bug 23315: (follow-up) Add configuration to enable CodeMirror
Bug 23315 : Some systempreferences can not be edited
Bug 23315: (follow-up) Add configuration to enable CodeMirror
Bug 23315 : Some systempreferences can not be edited
Bug 23315: (follow-up) Add configuration to enable CodeMirror

Description Chris Cormack 2019-07-15 02:25:57 UTC
Bug 21582 removed htmlarea in favour or textarea in the system preferences.
However 5 instances of html area remain

one in circulation.pref
and 4 in staff_client.pref

This renders those 5 systempreferences uneditable.
Comment 1 Chris Cormack 2019-07-15 02:31:52 UTC
Created attachment 91521 [details] [review]
Proposed patch
Comment 2 Owen Leonard 2019-07-15 13:18:48 UTC
Created attachment 91526 [details] [review]
Bg 23315 : Some systempreferences can not be edited

To test
1/ Go to systempreferences
2/ Notice that SelfCheckHelpMessage IntranetmainUserblock
IntranetCirculationHomeHTML IntranetReportsHomeHTML
StaffLoginInstructions are not editable
3/ Apply patch, refresh page
4/ Now they can be edited

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Owen Leonard 2019-07-15 13:18:51 UTC
Created attachment 91527 [details] [review]
Bug 23315: (follow-up) Add configuration to enable CodeMirror

This patch adds the "syntax" parameter to the corrected system
preferences so that the textareas will be displayed as CodeMirror
editors.

To test, apply the patch and test the affected system preferences:

Circulation -> SelfCheckHelpMessage
Staff client -> IntranetmainUserblock
Staff client -> IntranetCirculationHomeHTML
Staff client -> IntranetReportsHomeHTML
Staff client -> StaffLoginInstructions

With UseWYSIWYGinSystemPreferences disabled, the preferences should be
displayed as CodeMirror editors. With  UseWYSIWYGinSystemPreferences
enabled they should be WYSIWYG editors.
Comment 4 Chris Cormack 2019-07-15 20:07:05 UTC
Created attachment 91541 [details] [review]
Bug 23315 : Some systempreferences can not be edited

To test
1/ Go to systempreferences
2/ Notice that SelfCheckHelpMessage IntranetmainUserblock
IntranetCirculationHomeHTML IntranetReportsHomeHTML
StaffLoginInstructions are not editable
3/ Apply patch, refresh page
4/ Now they can be edited

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 5 Chris Cormack 2019-07-15 20:07:24 UTC
Created attachment 91542 [details] [review]
Bug 23315: (follow-up) Add configuration to enable CodeMirror

This patch adds the "syntax" parameter to the corrected system
preferences so that the textareas will be displayed as CodeMirror
editors.

To test, apply the patch and test the affected system preferences:

Circulation -> SelfCheckHelpMessage
Staff client -> IntranetmainUserblock
Staff client -> IntranetCirculationHomeHTML
Staff client -> IntranetReportsHomeHTML
Staff client -> StaffLoginInstructions

With UseWYSIWYGinSystemPreferences disabled, the preferences should be
displayed as CodeMirror editors. With  UseWYSIWYGinSystemPreferences
enabled they should be WYSIWYG editors.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Comment 6 Marcel de Rooy 2019-07-19 06:37:22 UTC
Created attachment 91611 [details] [review]
Bug 23315 : Some systempreferences can not be edited

To test
1/ Go to systempreferences
2/ Notice that SelfCheckHelpMessage IntranetmainUserblock
IntranetCirculationHomeHTML IntranetReportsHomeHTML
StaffLoginInstructions are not editable
3/ Apply patch, refresh page
4/ Now they can be edited

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

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Marcel de Rooy 2019-07-19 06:37:27 UTC
Created attachment 91612 [details] [review]
Bug 23315: (follow-up) Add configuration to enable CodeMirror

This patch adds the "syntax" parameter to the corrected system
preferences so that the textareas will be displayed as CodeMirror
editors.

To test, apply the patch and test the affected system preferences:

Circulation -> SelfCheckHelpMessage
Staff client -> IntranetmainUserblock
Staff client -> IntranetCirculationHomeHTML
Staff client -> IntranetReportsHomeHTML
Staff client -> StaffLoginInstructions

With UseWYSIWYGinSystemPreferences disabled, the preferences should be
displayed as CodeMirror editors. With  UseWYSIWYGinSystemPreferences
enabled they should be WYSIWYG editors.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 8 Martin Renvoize 2019-07-19 09:10:17 UTC
Nice work!

Pushed to master for 19.11.00
Comment 9 Fridolin Somers 2019-08-06 13:46:29 UTC
Pushed to 19.05.x for 19.05.03
Comment 10 Lucas Gass 2019-08-13 19:50:39 UTC
not needed in 18.11.x, no backport