There are several places on the patron entry form where we could use patron-title.inc instead of outputting the patron name information directly in the template.
Created attachment 92583 [details] [review] Bug 23534: Use patron-title.inc on patron entry page There are a few instances where we can simplify the patron entry template by using the patron-title include file instead of outputting patron name variables one by one. This patch does so in the page title, page breadcrumbs, and page heading. To test, apply the patch and edit a patron record. The page title, breadcrumbs, and main heading should all look correct.
Created attachment 92662 [details] [review] Bug 23534: Use patron-title.inc on patron entry page There are a few instances where we can simplify the patron entry template by using the patron-title include file instead of outputting patron name variables one by one. This patch does so in the page title, page breadcrumbs, and page heading. To test, apply the patch and edit a patron record. The page title, breadcrumbs, and main heading should all look correct. Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Blocked by bug 23533 currently not applying - please switch back once this is resolved!
Created attachment 99767 [details] [review] Bug 23534: Use patron-title.inc on patron entry page There are a few instances where we can simplify the patron entry template by using the patron-title include file instead of outputting patron name variables one by one. This patch does so in the page title, page breadcrumbs, and page heading. To test, apply the patch and edit a patron record. The page title, breadcrumbs, and main heading should all look correct. Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Created attachment 99826 [details] [review] Bug 23534: Use patron-title.inc on patron entry page There are a few instances where we can simplify the patron entry template by using the patron-title include file instead of outputting patron name variables one by one. This patch does so in the page title, page breadcrumbs, and page heading. To test, apply the patch and edit a patron record. The page title, breadcrumbs, and main heading should all look correct. Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Nice work everyone! Pushed to master for 20.05
Created attachment 100036 [details] [review] Bug 23534: Fix selenium tests The title of the page now contains lot of info (is it what we really want?)
This added an untranslatable "Print receipt" button.
(In reply to paxed from comment #9) > This added an untranslatable "Print receipt" button. Hi paxed - I don't see it in the patch - can you point it out?
(In reply to Katrin Fischer from comment #10) > (In reply to paxed from comment #9) > > This added an untranslatable "Print receipt" button. > > Hi paxed - I don't see it in the patch - can you point it out? Blah, comment ended up on the wrong one, meant bug 23355
not backported to 19.11.x branch due to dependencies