https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7654

--- Comment #23 from Katrin Fischer <[email protected]> ---
(In reply to M. Tompsett from comment #22)
> Comment on attachment 58869 [details] [review]
> Bug 7654: Add location at opac subscription view
> 
> Review of attachment 58869 [details] [review]:
> -----------------------------------------------------------------
> 
> Here are some untested thoughts. :)
> 
> ::: koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt
> @@ +796,4 @@
> >                              <p>There are [% subscriptionsnumber %] 
> > subscription(s) associated with this title.</p>
> >                              [% FOREACH subscription IN subscriptions %]
> >                                  [% IF ( subscription.branchcode ) %]
> > +                                    <h3>At library: [% Branches.GetName( 
> > subscription.branchcode ) %]</h3>
> 
> What if it gets a branch, but someone failed to enter a description or it is
> blank? Displaying the branchcode in that case would make sense. I do not
> think this correction is correct.

That's actually not a problem. The branchcode and branchname are both
mandatory. It's not possible to have a branch without a branchname in Koha.
Using the TT plugin like this is common in Koha.

-- 
You are receiving this mail because:
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to