Bug 14537

Summary: Rename system preference 'OverdueNoticeBcc' to 'NoticeBcc'
Product: Koha Reporter: Barton Chittenden <barton>
Component: Architecture, internals, and plumbingAssignee: Aleisha Amohia <aleisha>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P5 - low CC: aleisha, katrin.fischer, kyle, m.de.rooy, veron, ztajoli
Version: master   
Hardware: All   
OS: All   
Change sponsored?: Sponsored Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
The system preference 'OverdueNoticeBcc' is renamed to 'NoticeBcc' as it does not only apply to overdue notices, but to notices in general.
Version(s) released in:
Attachments: Bug 14537: Renaming OverdueNoticeBcc to NoticeBcc
Bug 14537: Renaming OverdueNoticeBcc to NoticeBcc
Bug 14537: Renaming OverdueNoticeBcc to NoticeBcc
Bug 14537: [QA Follow-up] Remove one overdues residue in pref description

Description Barton Chittenden 2015-07-16 12:39:36 UTC
The system prefrerence 'OverdueNoticeBcc' seems, by its name, to imply that a BCC will be added to all overdue notices -- the description says that the address will be added to all notices. The behavior of the system follows the description, not the system preference name.

The system preference should be re-named to NoticeBcc.

See also 

  * bug 12490
  * discussion on the topic in #koha: http://irc.koha-community.org/koha/2015-07-16#i_1703637
Comment 1 Aleisha Amohia 2017-04-05 04:21:27 UTC
Created attachment 61862 [details] [review]
Bug 14537: Renaming OverdueNoticeBcc to NoticeBcc

To test:
1) Apply patch and update database
2) Run 'git grep "OverdueNoticeBcc"' and confirm there are no other
instances

Sponsored-by: Catalyst IT
Comment 2 Marc Véron 2017-04-05 10:21:50 UTC
Created attachment 61872 [details] [review]
Bug 14537: Renaming OverdueNoticeBcc to NoticeBcc

To test:
1) Apply patch and update database
2) Run 'git grep "OverdueNoticeBcc"' and confirm there are no other
instances

Sponsored-by: Catalyst IT

Signed-off-by: Marc Véron <veron@veron.ch>
Comment 3 Marcel de Rooy 2017-04-07 09:18:02 UTC
Created attachment 61961 [details] [review]
Bug 14537: Renaming OverdueNoticeBcc to NoticeBcc

To test:
1) Apply patch and update database
2) Run 'git grep "OverdueNoticeBcc"' and confirm there are no other
instances

Sponsored-by: Catalyst IT

Signed-off-by: Marc Véron <veron@veron.ch>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Marcel de Rooy 2017-04-07 09:18:07 UTC
Created attachment 61962 [details] [review]
Bug 14537: [QA Follow-up] Remove one overdues residue in pref description

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 5 Kyle M Hall 2017-04-13 13:15:18 UTC
Pushed to master for 17.05, thanks Aleisha, Marcel!
Comment 6 Katrin Fischer 2017-04-16 12:27:35 UTC
+1 

This won't get ported back to 16.11.x as it is an enhancement.
Comment 7 Marc Véron 2017-05-17 19:17:40 UTC
Fixed small typo in text for release notes (prefrence)