Bug 7123 - barcode should be one word
Summary: barcode should be one word
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Templates (show other bugs)
Version: 3.6
Hardware: All All
: PATCH-Sent (DO NOT USE) minor (vote)
Assignee: Owen Leonard
QA Contact: Bugs List
URL: /cgi-bin/koha/circ/branchtransfers.pl
Keywords:
Depends on:
Blocks:
 
Reported: 2011-11-01 01:31 UTC by Nicole C. Engard
Modified: 2013-12-05 19:59 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Proposed fix (12.79 KB, patch)
2011-11-03 15:46 UTC, Owen Leonard
Details | Diff | Splinter Review
Fix for Bug 7123 - barcode should be one word (12.80 KB, patch)
2011-12-12 09:27 UTC, Chris Cormack
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nicole C. Engard 2011-11-01 01:31:48 UTC
on this page the word barcode is spelled "bar code" - it should be one word
Comment 1 Owen Leonard 2011-11-03 15:46:54 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2011-12-12 09:27:10 UTC
Created attachment 6720 [details] [review]
Fix for Bug 7123 - barcode should be one word

Other fixes thrown in for good measure:

- Adding data to the results table to match output of returns
- Better checking of error codes in the script to control
  output of error messages (don't show error box when there
  are no errors).
- Move some errors/messages from the right column onto the
  top of the page to match Koha's established pattern.

Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Comment 3 Paul Poulain 2011-12-13 16:51:14 UTC
QA comment:
* works as announced
* prove xt/author/valid-templates.t 
xt/author/valid-templates.t .. ok     
* prove xt/author/translatable-templates.t
xt/author/translatable-templates.t .. ok   
* perlcritic circ/branchtransfers.pl 
circ/branchtransfers.pl source OK


passed QA

patch pushed, please test
Comment 4 Jared Camins-Esakov 2012-05-23 12:38:26 UTC
This was included in the 3.6 branch prior to 3.6.4.