Bug 9238 - make fund pull down say 'select fund', not 'select budget'
When adding a new order to a basket the fund selection field is labeled "Fund:" but the default option is "Select a budget." This patch changes this string to "Select a fund" and also corrects the associated JavaScript error message text displayed when one doesn't choose a fund. To test, add an order to basket. The "New order" form should show "Select a fund" as the default option for "Fund" in the "Accounting details" section. If you submit the form without selecting a fund the error message should read "You must select a fund." Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com> tests pass Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> String updates only, passes koha-qa.pl Signed-off-by: Galen Charlton <gmc@esilibrary.com>
This commit is contained in:
parent
40aa42f31f
commit
dbe233a0e5
1 changed files with 3 additions and 3 deletions
|
@ -40,7 +40,7 @@ function Check(ff) {
|
|||
|
||||
if(isNull(ff.budget_id,1)){
|
||||
ok=1;
|
||||
_alertString += "\n- "+ _("You must select a budget");
|
||||
_alertString += "\n- "+ _("You must select a fund");
|
||||
}
|
||||
|
||||
if (!(isNum(ff.quantity,0)) || ff.quantity.value == 0){
|
||||
|
@ -383,7 +383,7 @@ $(document).ready(function()
|
|||
[% END %][%# UNLESS subscriptionid %]
|
||||
[% END %][%# IF (AcqCreateItemOrdering) %]
|
||||
<fieldset class="rows">
|
||||
<legend>Accounting Details</legend>
|
||||
<legend>Accounting details</legend>
|
||||
<ol>
|
||||
<li>
|
||||
[% IF ( close ) %]
|
||||
|
@ -415,7 +415,7 @@ $(document).ready(function()
|
|||
[% ELSE %]
|
||||
<label class="required" for="budget_id">Fund: </label>
|
||||
<select id="budget_id" onchange="fetchSortDropbox(this.form)" size="1" name="budget_id">
|
||||
<option value="">Select a budget</option>
|
||||
<option value="">Select a fund</option>
|
||||
[% FOREACH budget_loo IN budget_loop %]
|
||||
[% IF ( budget_loo.b_sel ) %]
|
||||
<option value="[% budget_loo.b_id %]" selected="selected">[% budget_loo.b_txt %]</option>
|
||||
|
|
Loading…
Reference in a new issue