Bug 8166 - Adding new currencies & exchange rates if not fill any field it save blank record
Summary: Adding new currencies & exchange rates if not fill any field it save blank re...
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: 3.8
Hardware: All All
: P1 - high minor
Assignee: Amit Gupta
QA Contact: Ian Walls
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-30 06:53 UTC by Amit Gupta
Modified: 2013-12-05 19:59 UTC (History)
3 users (show)

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

Attachments
Adding new currencies & exchange rates if not fill any field it save blank record (1.16 KB, patch)
2012-05-30 07:07 UTC, Amit Gupta 		Details | Diff | Splinter Review
Bug 8166 - Adding new currencies & exchange rates (1.27 KB, patch)
2012-06-09 20:03 UTC, Jared Camins-Esakov 		Details | Diff | Splinter Review
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Amit Gupta 2012-05-30 06:53:46 UTC 
While adding new currencies & exchange rates if we not fill any field for ex: currency, rate etc

One dialog is coming and say

Form not submitted because of the following problem(s)
------------------------------------------------------------------------------------
- Currency name missing
- Rate missing
- Symbol missing

if we click OK it save a blank record with exchange rate 0.
Comment 1 Amit Gupta 2012-05-30 07:07:38 UTC Comment hidden (obsolete)
Comment 2 Jared Camins-Esakov 2012-06-09 20:03:10 UTC 
Created attachment 10055 [details] [review]
Bug 8166 - Adding new currencies & exchange rates

if not fill any field it save blank record

To Test:
1) Go to Currencies & Exchange rates.
2) Create New currency
3) If you not fill any field for ex: (Currency, rate, symbol)
   it will not save the record.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
Comment 3 Paul Poulain 2012-06-10 11:58:10 UTC 
QA comment: tiny patch, passed QA
Comment 4 Chris Cormack 2012-06-14 06:46:14 UTC 
Pushed to 3.8.x, will be in 3.8.2