Nick Clemens
96eed0f5e0
This patch simply removes a discrepancty where when a discount is provided on the order form
we add the discount to the price, and when it comes form the vendor we subtract the discount.
Discounts should always descrease the estimated cost
To test:
0 - Have a vendor with a discount of 10% specified
1 - Export a record from your Koha
2 - Stage the record for import and match on biblionumber
3 - Add to a basket in acq from the staged file
4 - Select the title, and set order price to $10 and do not fill the discount field
5 - Add the order - note $10 RRP and $9 actual/budgeted cost
6 - Repeat 2&3
7 - Select the tile, set price $10 and discount 50%
8 - Add the order - note $20 RRP and $10 actual/budgeted cost! FAIL
9 - Apply patch, restart all
10 - Repeat 2-7
11 - Add the order, note $10 RRP and $5 actual.budgeted cost
Signed-off-by: Kelly McElligott <kelly@bywatersolutions.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
(cherry picked from commit
|
||
---|---|---|
.. | ||
acqui-home.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 | ||
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 |