Bug 41842: Use space-dash-space separator between course name and section

The staff interface templates were using a bare hyphen with no spaces
when displaying course name and section together, while the OPAC templates
(Bug 42654) use ' - ' (space-dash-space). This brings the staff interface
into alignment with the OPAC convention for consistency and readability.

Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Pedro Amorim <pedro.amorim@openfifth.co.uk>
This commit is contained in:
Martin Renvoize 2026-06-25 10:55:05 +01:00 committed by Pedro Amorim
parent 303ba57409
commit 3cd648d52f
No known key found for this signature in database
GPG Key ID: 06B75BA7F3894D55
5 changed files with 14 additions and 14 deletions

View File

@ -7,7 +7,7 @@
<title
>[% FILTER collapse %]
[% t("Add reserves") | html %]
&rsaquo; [% course.course_name | html %][%- IF course.section -%]-[% course.section | html %][%- END -%] &rsaquo; [% t("Course reserves") | html %] &rsaquo; [% t("Koha") | html %]
&rsaquo; [% course.course_name | html %][%- IF course.section -%] - [% course.section | html %][%- END -%] &rsaquo; [% t("Course reserves") | html %] &rsaquo; [% t("Koha") | html %]
[% END %]</title
>
[% INCLUDE 'doc-head-close.inc' %]
@ -24,7 +24,7 @@
<a href="/cgi-bin/koha/course_reserves/course-reserves.pl">Course reserves</a>
[% END %]
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/course_reserves/course-details.pl?course_id=[% course.course_id | uri %]">[% course.course_name | html %][%- IF course.section -%]-[% course.section | html %][%- END -%]</a>
<a href="/cgi-bin/koha/course_reserves/course-details.pl?course_id=[% course.course_id | uri %]">[% course.course_name | html %][%- IF course.section -%] - [% course.section | html %][%- END -%]</a>
[% END %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Add reserves</span>
@ -33,7 +33,7 @@
[% END #/ WRAPPER sub-header.inc %]
[% WRAPPER 'main-container.inc' %]
[% course_em = ( course.section ? course.course_name _"-"_ course.section : course.course_name ) | html | $HtmlTags tag="em" %]
[% course_em = ( course.section ? course.course_name _" - "_ course.section : course.course_name ) | html | $HtmlTags tag="em" %]
<h1> [% tx("Add reserves for course {course_name}", { course_name = course_em }) | $raw %] </h1>
[% IF ERROR_ITEM_NOT_FOUND %]

View File

@ -12,7 +12,7 @@
[% ELSE %]
[% t("Adding item to course") | html %]
[% END %]
&rsaquo; [% course.course_name | html %][%- IF course.section -%]-[% course.section | html %][%- END -%] &rsaquo; [% t("Course reserves") | html %] &rsaquo; [% t("Koha") | html %]
&rsaquo; [% course.course_name | html %][%- IF course.section -%] - [% course.section | html %][%- END -%] &rsaquo; [% t("Course reserves") | html %] &rsaquo; [% t("Koha") | html %]
[% END %]</title
>
[% INCLUDE 'doc-head-close.inc' %]
@ -30,7 +30,7 @@
<a href="/cgi-bin/koha/course_reserves/course-reserves.pl">Course reserves</a>
[% END %]
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/course_reserves/course-details.pl?course_id=[% course.course_id | uri %]">[% course.course_name | html %][%- IF course.section -%]-[% course.section | html %][%- END -%]</a>
<a href="/cgi-bin/koha/course_reserves/course-details.pl?course_id=[% course.course_id | uri %]">[% course.course_name | html %][%- IF course.section -%] - [% course.section | html %][%- END -%]</a>
[% END %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
[% item_title = biblio.title | html | $HtmlTags tag="em" %]
@ -45,7 +45,7 @@
[% WRAPPER 'main-container.inc' %]
<h1>
[% course_em = ( course.section ? course.course_name _"-"_ course.section : course.course_name ) | html | $HtmlTags tag="em" %]
[% course_em = ( course.section ? course.course_name _" - "_ course.section : course.course_name ) | html | $HtmlTags tag="em" %]
[% IF is_edit || course_reserve %]
[% tx("Editing item in course {course_name}", { course_name = course_em }) | $raw %]
[% ELSE %]

View File

@ -11,7 +11,7 @@
<title
>[% FILTER collapse %]
[% t("Add reserves") | html %]
&rsaquo; [% course.course_name | html %][%- IF course.section -%]-[% course.section | html %][%- END -%] &rsaquo; [% t("Course reserves") | html %] &rsaquo; [% t("Koha") | html %]
&rsaquo; [% course.course_name | html %][%- IF course.section -%] - [% course.section | html %][%- END -%] &rsaquo; [% t("Course reserves") | html %] &rsaquo; [% t("Koha") | html %]
[% END %]</title
>
[% INCLUDE 'doc-head-close.inc' %]
@ -28,7 +28,7 @@
<a href="/cgi-bin/koha/course_reserves/course-reserves.pl">Course reserves</a>
[% END %]
[% WRAPPER breadcrumb_item %]
<a href="/cgi-bin/koha/course_reserves/course-details.pl?course_id=[% course.course_id | uri %]">[% course.course_name | html %][%- IF course.section -%]-[% course.section | html %][%- END -%]</a>
<a href="/cgi-bin/koha/course_reserves/course-details.pl?course_id=[% course.course_id | uri %]">[% course.course_name | html %][%- IF course.section -%] - [% course.section | html %][%- END -%]</a>
[% END %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Add reserves</span>
@ -38,7 +38,7 @@
[% WRAPPER 'main-container.inc' %]
<h1>
[% course_em = ( course.section ? course.course_name _"-"_ course.section : course.course_name ) | html | $HtmlTags tag="em" %]
[% course_em = ( course.section ? course.course_name _" - "_ course.section : course.course_name ) | html | $HtmlTags tag="em" %]
[% tx("Add reserves for course {course_name}", { course_name = course_em }) | $raw %]
</h1>
[% IF action == 'invalid_course' %]

View File

@ -12,7 +12,7 @@
[% INCLUDE 'doc-head-open.inc' %]
<title
>[% FILTER collapse %]
[% tx("Course details for {course_name}", { course_name = course.course_name }) | html %][%- IF course.section -%]-[% course.section | html %][%- END -%]
[% tx("Course details for {course_name}", { course_name = course.course_name }) | html %][%- IF course.section -%] - [% course.section | html %][%- END -%]
&rsaquo; [% t("Course reserves") | html %] &rsaquo; [% t("Koha") | html %]
[% END %]</title
>
@ -54,13 +54,13 @@
<a href="/cgi-bin/koha/course_reserves/course-reserves.pl">Course reserves</a>
[% END %]
[% WRAPPER breadcrumb_item bc_active= 1 %]
<span>Course details for [% course.course_name | html | $HtmlTags tag="em" %][%- IF course.section -%]-[% course.section | html | $HtmlTags tag="em" %][%- END -%]</span>
<span>Course details for [% course.course_name | html | $HtmlTags tag="em" %][%- IF course.section -%] - [% course.section | html | $HtmlTags tag="em" %][%- END -%]</span>
[% END %]
[% END #/ WRAPPER breadcrumbs %]
[% END #/ WRAPPER sub-header.inc %]
[% WRAPPER 'main-container.inc' %]
[% course_em = ( course.section ? course.course_name _"-"_ course.section : course.course_name ) | html | $HtmlTags tag="em" %]
[% course_em = ( course.section ? course.course_name _" - "_ course.section : course.course_name ) | html | $HtmlTags tag="em" %]
<h1>[% tx("Course details for {course_name}", { course_name = course_em }) | $raw %]</h1>
[% IF CAN_user_coursereserves_add_reserves OR CAN_user_coursereserves_manage_courses OR CAN_user_coursereserves_manage_courses %]

View File

@ -7,7 +7,7 @@
<title
>[% FILTER collapse %]
[% IF ( course_name ) %]
[% tx("Edit course {course_name}", { course_name = course_name }) | html %][% IF section %]-[% section | html %][% END %]
[% tx("Edit course {course_name}", { course_name = course_name }) | html %][% IF section %] - [% section | html %][% END %]
[% ELSE %]
[% t("New course") | html %]
[% END %]
@ -52,7 +52,7 @@
[% WRAPPER 'main-container.inc' %]
<h1>
[% IF course_name %]
[% course_em = ( section ? course_name _"-"_ section : course_name ) | html | $HtmlTags tag="em" %]
[% course_em = ( section ? course_name _" - "_ section : course_name ) | html | $HtmlTags tag="em" %]
[% tx("Edit course {course_name}", { course_name = course_em }) | $raw %]
[% ELSE %]
<span>New course</span>