c7f47ea36a
This fixes the regression that multiplies the amount entered by 100 when CurrencyFormat is set to FR. It replaces the last dot with a comma before dealing with the value of ActualCost and ReplacementCost. Test Plan: a)Replicate the issue: 0- Set CurrencyFormat to FR 1- Go to Acquisitions 2- Search for a Vendor 3- Click on "New basket" 4- Give basket a name and click "Save" 5- Click on "Add to basket" 6- Add an order through preferred method 7- In Accounting details, enter a vendor price with dot decimal (i.e. 19.44) 8- Save your order line 9- Click on "Close this basket" 10- Confirm closing of basket by clicking on "Yes, close" 11- Click on "Receive shipment" 12- Enter an invoice number and click "Next" 13- Click on "Receive" to the right of your order 14- In Accounting details, notice the Actual cost is written with a decimal dot. 15- Change the Actual cost, using a dot decimal (i.e 20.99) 16- Receive the order 17- Click on "Save" 18- In "Already received" notice the price is multiplied by 100 (i.e. 2099,00) 19- Click on "Cancel receipt" 20- Click on "Receive" to the right of your order 21- In Accounting details, change the Actual cost, using a comma decimal (i.e. 20,99) 22- Receive the order 23- Click on "Save" 24- In "Already received", notice the price is correct. b)Apply the patch c)Test the patch: 1- Click on "Cancel receipt" 2- Click on "Receive to the right of your order 3- Change the Actual cost/Replacement cost, using a dot decimal (21.99) 4- Receive the order 5- Click on "Save" 6- Notice that the Actual cost and the Replacement cost use commas 7- Change the Actual cost, using a comma decimal (21,99) 8- Click on "Save" 9- In "Already received", notice the price is still correct. Signed-off-by: Victor Grousset <victor.grousset@biblibre.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> |
||
---|---|---|
.. | ||
acqui-home.pl | ||
add_user_search.pl | ||
addorder.pl | ||
addorderiso2709.pl | ||
ajax-getauthvaluedropbox.pl | ||
basket.pl | ||
basketgroup.pl | ||
basketheader.pl | ||
booksellers.pl | ||
cancelorder.pl | ||
check_budget_total.pl | ||
check_uniqueness.pl | ||
duplicate_orders.pl | ||
edi_ean.pl | ||
edifactmsgs.pl | ||
edimsg.pl | ||
finishreceive.pl | ||
histsearch.pl | ||
invoice-files.pl | ||
invoice.pl | ||
invoices.pl | ||
lateorders-export.pl | ||
lateorders.pl | ||
modordernotes.pl | ||
neworderbiblio.pl | ||
neworderempty.pl | ||
newordersubscription.pl | ||
newordersuggestion.pl | ||
ordered.pl | ||
orderreceive.pl | ||
parcel.pl | ||
parcels.pl | ||
showorder.pl | ||
spent.pl | ||
supplier.pl | ||
transferorder.pl | ||
uncertainprice.pl | ||
updatesupplier.pl | ||
z3950_search.pl |