138f14c2b9
This followup answer QA remarks : - neworderempty.pl updated so that the 2 new variables are passed to the template - modordernotes.tt fixed to make the translation easier - in CSV headers, to make clear that no change are made for the moment, rename "note" to "internal note" Additionnaly, "Publisher code" was wrong in the csv headers. I changed it to "Publisher" (the field in database is publishercode, but the content is a real publisher name, not a code) I did not change "Note:" in modordernotes.tt, because it is just under a h1 tag which specifies the type of note the librarian is editing. Test plan : - edit an existing order, and try to change/add/delete the vendor note, and the internal note. Check the changes are properly saved - export a basket and a basketgroup in CSV. Check the columns headers are "Publisher" and "Vendor note" Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> Fixed some tabs. Passes QA script and tests. Tested: - add notes when creating an order - edit notes modifying an order line - edit notes using the links on the basket summary - check basket CSV export - close basket - check basket group CSV export - edit notes on order receive page using the links - edit notes on receive Note: Translatability of templates could be improved by a follow-up. It's better not to divide up sentences with if/else structures. Signed-off-by: Galen Charlton <gmc@esilibrary.com> |
||
---|---|---|
.. | ||
csv_headers/acqui | ||
virtualshelves/merge | ||
acquisitions-add-to-basket.inc | ||
acquisitions-menu.inc | ||
acquisitions-search.inc | ||
acquisitions-toolbar.inc | ||
additem.js.inc | ||
admin-menu.inc | ||
auth-finder-search.inc | ||
authorities-search-results.inc | ||
authorities-search.inc | ||
authorities-toolbar.inc | ||
authorities.inc | ||
authorities_js.inc | ||
biblio-default-view.inc | ||
biblio-view-menu.inc | ||
borrower_debarments.inc | ||
browser-strings.inc | ||
budgets-active-currency.inc | ||
budgets-admin-search.inc | ||
budgets-admin-toolbar.inc | ||
calendar.inc | ||
cat-menu.inc | ||
cat-search.inc | ||
cat-toolbar.inc | ||
cataloging-search.inc | ||
checkin-search.inc | ||
checkouts-table-footer.inc | ||
circ-menu.inc | ||
circ-menu.tt | ||
circ-search.inc | ||
cities-admin-search.inc | ||
contracts-admin-search.inc | ||
currencies-admin-search.inc | ||
datatables-strings.inc | ||
datatables.inc | ||
date-format.inc | ||
doc-head-close-receipt.inc | ||
doc-head-close.inc | ||
doc-head-open.inc | ||
facets.inc | ||
file-upload.inc | ||
form-blocks.inc | ||
greybox.inc | ||
guided-reports-view.inc | ||
header.inc | ||
help-bottom.inc | ||
help-top.inc | ||
home-search.inc | ||
installer-doc-head-close.inc | ||
intranet-bottom.inc | ||
intranetstylesheet.inc | ||
labels-menu.inc | ||
labels-toolbar.inc | ||
letters-search.inc | ||
members-menu.inc | ||
members-toolbar.inc | ||
merge-record-strings.inc | ||
merge-record.inc | ||
messaging-preference-form.inc | ||
page-numbers.inc | ||
patron-search-box.inc | ||
patron-search.inc | ||
patron-title.inc | ||
patron-toolbar.inc | ||
patroncards-errors.inc | ||
patroncards-menu.inc | ||
patroncards-toolbar.inc | ||
patrons-admin-search.inc | ||
popup-bottom.inc | ||
prefs-admin-search.inc | ||
prefs-menu.inc | ||
printers-admin-search.inc | ||
quotes-toolbar.inc | ||
quotes-upload-toolbar.inc | ||
reports-menu.inc | ||
reports-toolbar.inc | ||
resort_form.inc | ||
search_indexes.inc | ||
serials-menu.inc | ||
serials-search.inc | ||
serials-toolbar.inc | ||
stopwords-admin-search.inc | ||
subscriptions-search.inc | ||
subtypes_unimarc.inc | ||
suggestions-add-search.inc | ||
timepicker.inc | ||
tools-item-action.inc | ||
tools-menu.inc | ||
tools-nomatch-action.inc | ||
tools-overlay-action.inc | ||
validator-strings.inc | ||
vendor-menu.inc | ||
virtualshelves-toolbar.inc | ||
z3950-admin-search.inc |