Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0008161
TypeCategorySeverityReproducibilityDate SubmittedLast Update
feature request[Openbravo ERP] 07. Sales managementmajoralways2009-03-17 16:532009-07-03 14:26
ReporterjaimetorreView Statuspublic 
Assigned Togmauleon 
PriorityurgentResolutionfixedFixed in Version2.50MP1
StatusclosedFix in branchpiFixed in SCM revisiondb44a8f264fe
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionpiSCM revision 
Review Assigned To
Web browser
ModulesCore
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0008161: Not possible not to apply price adjustments for a specific order line

DescriptionWhen one enters an order line and a price adjustment applies for it, a new register is entered in Price Adjustments tab. There is no possibility to say that for this line, even if price adjustments applies, we do not want the price adjustment to be calculated.
Steps To Reproduce1) Create a price adjustment in 'Master Data Management || Pricing || Price Adjustments || Adjustments' with values:
* Starting date = 01-01-2000
* Discount % = 50
* Business Partner = All Selected
* Product = All Selected
2) Move to Business Partner tab and select one business partner
3) Move to Product tab and select one product
4) Create a new Sales or Purchase Order for the business partner of the price adjustment
5) Create one line for the product of the price adjustment.

Notice that the Discount field is correctly calculated as:
* Standard Price = SP (hidden)
* List Price = LP
* Discount = D = 100*(1-(SP/LP))

6) Save this line

7) Go to price adjustment tabs and see that the price adjustment has been correctly applied.
Proposed SolutionShort summary on different prices:
* List price: price used as reference in a Price list
* Standard price: price applied to customers after discount in Price list (this price is not shown in the sales order window, it is hidden)
* Discount: percentage (List price - Standard price)/List price
* Unit price: price finally applied after price adjustments
* Price adjustments: zero, one or many changes in the price applied in an order line. All of them together are the difference between Standard Price and Unit price

For further clarification see attached PricesRelations.png
 
From a technical perspective:
* There is no need to decouple price adjustments and order completion since they are already decoupled. Price adjustments are calculated when the line is entered through a trigger (c_orderline_trg2).
* Currently it is not possible to cancel (not to apply) price adjustments
* To achieve it, add a new column and a make a change to change c_orderline_trg2 trigger.
* This will be implemented using a check box in the order line ("Cancel Price Adjustments"), by default set to false (N).
* If the user wants avoid applying price adjustments in a particular line he has to check this check-box and save the line.
* The trigger would remove all the changes over the standard price through price adjustements.
* Visibility of price adjustments will also be improved by showing 'Standard Price' right-only field. Right now Standard price is hidden, so it is not easy to understand how the numbers are calculated. It will be clearer for the user how the discount is calculated and what lines have price adjustements (Unit price different from Standard Price)
 
Eg.
List price: 10
Unit price: 5
Discount: 20%
Standard price: 8 (not shown)

The distribution of fields in Order Lines would be like OrderLines.png.
* 'Standard Price' text field would display below 'Unit Price' field and will be read-only.
* 'Cancel Price Adjustment' check-box would display below Currency' field.
* These two fields will be shown based on the following display logic: UP != SP OR CPA = Y (UP = Unit Price; SP = Standard Price; CPA = Cancel Price Adjustments)
Tagstictech
Attached Filespng file icon PricesRelation.png [^] (19,705 bytes) 2009-03-25 10:17


png file icon OrderLines.png [^] (83,362 bytes) 2009-03-25 10:17

- Relationships [ Relation Graph ] [ Dependency Graph ]
related to defect 0008275 acknowledgedjonalegriaesarte Correctly document Prices relations 
related to feature request 0009005pi acknowledgedrmorley Delete one Price Adjustment for Sales Order line 
related to feature request 0029623 newTriage Omni OMS It would be fine to have the Cancel Promotions option in invoice lines not to apply the Price Adjustments 

-  Notes
(0015115)
hgbot (developer)
2009-04-01 16:53

 Repository: erp/devel/pi
Changeset: 0342347da77fe894ee3253dd0897057dc17bba24
Author: Gorka Mauleon <gorka.mauleon <at> openbravo.com>
Date: Wed Apr 01 16:52:58 2009 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/0342347da77fe894ee3253dd0897057dc17bba24 [^]

fixes 0008161: Not possible not to apply price adjustments for a specific order line

---
M src-db/database/model/tables/C_ORDERLINE.xml
M src-db/database/model/triggers/C_ORDERLINE_TRG2.xml
M src-db/database/sourcedata/AD_COLUMN.xml
M src-db/database/sourcedata/AD_ELEMENT.xml
M src-db/database/sourcedata/AD_FIELD.xml
M src/org/openbravo/erpCommon/ad_callouts/SL_Order_Amt.java
M src/org/openbravo/erpCommon/ad_callouts/SL_Order_Product.java
---
(0015545)
hgbot (developer)
2009-04-22 16:20

 Repository: erp/devel/pi
Changeset: db44a8f264fecd0cae7f76cc56c5e966638240d6
Author: Gorka Mauleon <gorka.mauleon <at> openbravo.com>
Date: Wed Apr 22 16:20:10 2009 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/db44a8f264fecd0cae7f76cc56c5e966638240d6 [^]

fixes bug 0008161: Not possible not to apply price adjustments for a specific order line

---
M referencedata/sampledata/Accounting_Test.xml
M referencedata/sampledata/SmallBazaar.xml
M src-db/database/model/tables/C_ORDERLINE.xml
---

- Issue History
Date Modified Username Field Change
2009-03-17 16:53 jaimetorre New Issue
2009-03-17 16:53 jaimetorre Assigned To => rafaroda
2009-03-17 16:53 jaimetorre Regression testing => No
2009-03-17 17:07 jaimetorre Tag Attached: tictech
2009-03-25 09:38 rafaroda Relationship added related to 0008275
2009-03-25 10:17 rafaroda Assigned To rafaroda => gmauleon
2009-03-25 10:17 rafaroda Status new => acknowledged
2009-03-25 10:17 rafaroda Summary Not possible to deactivate price adjustments for a specific order line => Not possible not to apply price adjustments for a specific order line
2009-03-25 10:17 rafaroda Description Updated
2009-03-25 10:17 rafaroda Proposed Solution updated
2009-03-25 10:17 rafaroda Steps to Reproduce Updated
2009-03-25 10:17 rafaroda File Added: PricesRelation.png
2009-03-25 10:17 rafaroda File Added: OrderLines.png
2009-04-01 15:47 gmauleon version => pi
2009-04-01 16:51 gmauleon Status acknowledged => scheduled
2009-04-01 16:51 gmauleon fix_in_branch => pi
2009-04-01 16:53 hgbot Checkin
2009-04-01 16:53 hgbot Note Added: 0015115
2009-04-01 16:53 hgbot Status scheduled => resolved
2009-04-01 16:53 hgbot Resolution open => fixed
2009-04-01 16:53 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/0342347da77fe894ee3253dd0897057dc17bba24 [^]
2009-04-22 16:20 hgbot Checkin
2009-04-22 16:20 hgbot Note Added: 0015545
2009-04-22 16:20 hgbot Fixed in SCM revision http://code.openbravo.com/erp/devel/pi/rev/0342347da77fe894ee3253dd0897057dc17bba24 [^] => http://code.openbravo.com/erp/devel/pi/rev/db44a8f264fecd0cae7f76cc56c5e966638240d6 [^]
2009-05-12 14:04 rafaroda Relationship added related to 0009005
2009-06-05 18:53 pjuvara Status resolved => closed
2009-06-06 00:00 anonymous sf_bug_id 0 => 2802024
2009-07-03 14:26 pjuvara Priority immediate => urgent
2015-04-17 13:27 ngarcia Relationship added related to 0029623

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker