Bug 20223

Summary: Merge members-menu and circ-menu inc files
Product: Koha Reporter: Josef Moravec <josef.moravec>
Component: TemplatesAssignee: Owen Leonard <oleonard>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: fridolin.somers, jonathan.druart, martin.renvoize, nick
Version: Main   
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:
Circulation function:
Bug Depends on: 18789    
Bug Blocks:    
Attachments: Bug 20223: Merge members-menu and circ-menu inc files
Bug 20223: Merge members-menu and circ-menu inc files
Bug 20223: Merge members-menu and circ-menu inc files
Bug 20223: Merge members-menu and circ-menu inc files

Description Josef Moravec 2018-02-16 07:41:52 UTC
We have two similer patron menus in templates, almost everywhere is used circ-menu.inc.

Only when editing patron record, the members-menu.inc is used. It means the menu items are not compete, because members-menu is not synchronized with updates of circ-menu.

There is also reference to members-menu in member.tt template (the template used in patron search ) which is probably never taken into effect.
Comment 1 Owen Leonard 2018-06-29 14:41:36 UTC
Created attachment 76575 [details] [review]
Bug 20223: Merge members-menu and circ-menu inc files

This patch removes members-menu.inc and replaces the last functional use
of it with a call to circ-menu.inc.

An invalid use of members-menu.inc has been removed from member.tt.

To test, apply the patch and open a patron record for editing. The
sidebar menu should look correct and all sidebar links should work
correctly.

View the patrons home page and confirm that nothing has broken.

Search the Koha codebase for references to members-menu.inc. There
should be none.
Comment 2 Owen Leonard 2018-09-04 15:47:09 UTC
Created attachment 78411 [details] [review]
Bug 20223: Merge members-menu and circ-menu inc files

This patch removes members-menu.inc and replaces the last functional use
of it with a call to circ-menu.inc.

An invalid use of members-menu.inc has been removed from member.tt.

To test, apply the patch and open a patron record for editing. The
sidebar menu should look correct and all sidebar links should work
correctly.

View the patrons home page and confirm that nothing has broken.

Search the Koha codebase for references to members-menu.inc. There
should be none.
Comment 3 Michal Denar 2018-09-14 12:32:29 UTC
Created attachment 78647 [details] [review]
Bug 20223: Merge members-menu and circ-menu inc files

This patch removes members-menu.inc and replaces the last functional use
of it with a call to circ-menu.inc.

An invalid use of members-menu.inc has been removed from member.tt.

To test, apply the patch and open a patron record for editing. The
sidebar menu should look correct and all sidebar links should work
correctly.

View the patrons home page and confirm that nothing has broken.

Search the Koha codebase for references to members-menu.inc. There
should be none.

Michal Denar Signed-off-by:  <black23@gmail.com>
Comment 4 Jonathan Druart 2018-09-21 21:30:02 UTC
Created attachment 79261 [details] [review]
Bug 20223: Merge members-menu and circ-menu inc files

This patch removes members-menu.inc and replaces the last functional use
of it with a call to circ-menu.inc.

An invalid use of members-menu.inc has been removed from member.tt.

To test, apply the patch and open a patron record for editing. The
sidebar menu should look correct and all sidebar links should work
correctly.

View the patrons home page and confirm that nothing has broken.

Search the Koha codebase for references to members-menu.inc. There
should be none.

Signed-off-by: Michal Denar <black23@gmail.com>

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 5 Jonathan Druart 2018-09-21 21:31:02 UTC
Changing to bugfix.
Comment 6 Jonathan Druart 2018-09-21 21:34:57 UTC
Found bug 21397 while testing.
Comment 7 Nick Clemens (kidclamp) 2018-09-26 19:26:47 UTC
Awesome work all!

Pushed to master for 18.11
Comment 8 Martin Renvoize (ashimema) 2018-09-27 12:21:01 UTC
Pushed to 18.05.x for 18.05.05
Comment 9 Fridolin Somers 2018-10-03 11:55:08 UTC
Depends on Bug 18789 not in 17.11.x