Bug 13128

Summary: Add span with id for logged in branch name, add hidden span for branchcode
Product: Koha Reporter: Kyle M Hall (khall) <kyle>
Component: TemplatesAssignee: Kyle M Hall (khall) <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: tomascohen
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:
Attachments: Bug 13128 - Add span with id for logged in branch name, add hidden span for branchcode
[SIGNED-OFF] Bug 13128 - Add span with id for logged in branch name, add hidden span for branchcode
[PASSED QA] Bug 13128 - Add span with id for logged in branch name, add hidden span for branchcode

Description Kyle M Hall (khall) 2014-10-22 13:07:58 UTC
It would be nice if there were a simple way to access the name and branchcode of the currently logged in library, as well as the logged in user via JavaScript ( via intranetuserjs ).

I propose the following:
1) Remove the whitespace surrounding the loggedinusername span
2) Wrap the logged in branch name with a span that has an id
3) Add the branchcode in a hidden span that has an id
Comment 1 Kyle M Hall (khall) 2014-10-22 13:10:46 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2014-10-22 14:00:04 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-10-25 19:10:20 UTC
Created attachment 32741 [details] [review]
[PASSED QA] Bug 13128 - Add span with id for logged in branch name, add hidden span for branchcode

It would be nice if there were a simple way to access the name and
branchcode of the currently logged in library, as well as the logged in
user via JavaScript ( via intranetuserjs ).

I propose the following:
1) Remove the whitespace surrounding the loggedinusername span
2) Wrap the logged in branch name with a span that has an id
3) Add the branchcode in a hidden span that has an id

Test Plan:
1) Apply this patch
2) Log into Koha
3) Inspect the html source code
4) Note the lack of whitespace within the tag <span class="loggedinusername">[% loggedinusername %]</span>
5) Note the logged in branch name is now surrounded by a span with an id
6) Note below that is a span containing the branchcode, also with an id.
7) Verify the branchcode span is hidden by default.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Small template change, no problems found.
Comment 4 Tomás Cohen Arazi (tcohen) 2014-10-27 15:32:58 UTC
Patch pushed to master.

Thanks Kyle!