Attachment 20426 Details for Bug 9238 – Bug 9238 - fund pull down says 'select budget'

[patch] Bug 9238 - fund pull down says 'select budget'

Bug-9238---fund-pull-down-says-select-budget.patch (text/plain), 2.52 KB, created by Kyle M Hall (khall) on 2013-08-16 16:05:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2013-08-16 16:05:03 UTC

Size: 2.52 KB

patch

obsolete

>From 813e36eb11b9fcde8d911d2c4da2441898f5a10d Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Tue, 13 Aug 2013 10:05:58 -0400
>Subject: [PATCH] Bug 9238 - fund pull down says '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
>---
> .../prog/en/modules/acqui/neworderempty.tt         |    6 +++---
> 1 files changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt
>index 52def80..da8f88a 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt
>@@ -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>
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 9238: 20312 | 20319 | 20426