Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0032084Openbravo ERP00. Application dictionarypublic2016-02-02 11:522016-03-17 10:56
Reportervmromanos 
Assigned Tomarkmm82 
PriorityimmediateSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Todmiguelez
OBNetwork customer
Web browser
ModulesAdvanced Payables and Receivables Mngmt
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2014-08-22
Regression introduced in release3.0PR14Q4
Regression introduced by commithttps://code.openbravo.com/erp/devel/main/rev/ae0addbe904c [^]
Triggers an Emergency PackNo
Summary0032084: Process Price Difference adjustment breaks menu
DescriptionThe "Process Price Difference adjustment" menu entry has 3 problems:

1. It should be included into menu folder Warehouse Management | Setup
2. It should be renamed to "Process Price Difference Adjustment" to follow the capitalization rule
3. Process should include a description and help
Steps To ReproduceLogin as Openbravo Admin
Open the Application menu
An entry called "Process Price Difference adjustment" is shown in the menu root
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00320853.0PR16Q1 closed Triage Finance Process Price Difference adjustment breaks menu 
depends on backport 00320863.0PR15Q4.3 closed Triage Finance Process Price Difference adjustment breaks menu 
depends on backport 00320873.0PR15Q3.6 closed Triage Finance Process Price Difference adjustment breaks menu 
Attached Filespng menu.png (60,349) 2016-02-02 11:52
https://issues.openbravo.com/file_download.php?file_id=8994&type=bug
png
Issue History
Date ModifiedUsernameFieldChange
2016-02-02 11:52vmromanosNew Issue
2016-02-02 11:52vmromanosAssigned To => Triage Finance
2016-02-02 11:52vmromanosFile Added: menu.png
2016-02-02 11:52vmromanosModules => Advanced Payables and Receivables Mngmt
2016-02-02 11:52vmromanosRegression level => Production - Confirmed Stable
2016-02-02 11:52vmromanosRegression date => 2014-08-22
2016-02-02 11:52vmromanosRegression introduced in release => 3.0PR14Q4
2016-02-02 11:52vmromanosRegression introduced by commit => https://code.openbravo.com/erp/devel/main/rev/ae0addbe904c [^]
2016-02-02 11:52vmromanosTriggers an Emergency Pack => No
2016-02-02 11:53vmromanosStatusnew => scheduled
2016-02-02 11:53vmromanosAssigned ToTriage Finance => markmm82
2016-02-02 12:54dmiguelezNote Added: 0083842
2016-02-02 21:49markmm82Note Added: 0083863
2016-02-05 10:33psanjuanNote Added: 0083957
2016-02-05 10:34psanjuanNote Edited: 0083957bug_revision_view_page.php?bugnote_id=0083957#r10954
2016-02-05 10:34psanjuanNote Edited: 0083957bug_revision_view_page.php?bugnote_id=0083957#r10955
2016-02-05 10:49psanjuanNote Added: 0083958
2016-02-05 14:17psanjuanNote Edited: 0083958bug_revision_view_page.php?bugnote_id=0083958#r10969
2016-02-05 15:10hgbotCheckin
2016-02-05 15:10hgbotNote Added: 0083984
2016-02-05 15:10hgbotStatusscheduled => resolved
2016-02-05 15:10hgbotResolutionopen => fixed
2016-02-05 15:10hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/42c991e213babb9a78fcbd7fedce459ccc72c2e8 [^]
2016-02-05 15:10dmiguelezReview Assigned To => dmiguelez
2016-02-05 15:10dmiguelezNote Added: 0083986
2016-02-05 15:10dmiguelezStatusresolved => closed
2016-03-17 10:56hudsonbotCheckin
2016-03-17 10:56hudsonbotNote Added: 0085110

Notes
(0083842)
dmiguelez   
2016-02-02 12:54   
Description: This process allows to adjust the Cost of a Transaction based on the Price difference between the Order and the Invoice.
Help: Usually this process is done in the Costing Background Process, but when the Invoice is registered after the Goods Shipment/Receipt it is necessary to either launch this process manually or to schedule it as a background process.
(0083863)
markmm82   
2016-02-02 21:49   
Test plan

As Group Admin
-Go to Menu folder: Warehouse Management | Setup
Notice that the "Process Price Difference Adjustment" entry was renamed and included inside.

As System Admin
-Go to the Process Definition
-Search by Search key: "PriceDifferenceByDateProcess"
Notice:
-Name was Capitalized.
-Help and Description were added.
(0083957)
psanjuan   
2016-02-05 10:33   
(edited on: 2016-02-05 10:34)
As a suggestion of enhancement:

Description: This process allows to adjust the cost of a transaction, caused by a price difference between the order and the corresponding invoice.

Help/Comment: Price difference adjustments can be performed "automatically" by scheduling the "Price Correction Background Process" or manually for a given product or set of products by running this process.
(0083958)
psanjuan   
2016-02-05 10:49   
(edited on: 2016-02-05 14:17)
User guide also updated. To be reviewed once user guide is generated again.

Issue can be code reviewed, pushed and closed.
(0083984)
hgbot   
2016-02-05 15:10   
Repository: erp/devel/pi
Changeset: 42c991e213babb9a78fcbd7fedce459ccc72c2e8
Author: Mark <m.molina <at> nectus.com>
Date: Tue Feb 02 13:45:46 2016 -0500
URL: http://code.openbravo.com/erp/devel/pi/rev/42c991e213babb9a78fcbd7fedce459ccc72c2e8 [^]

Fixes issue 32084: Process Price Difference adjustment breaks menu

Were fixed the 3 problems of "Process Price Difference adjustment" menu entry found:

1. It is included into menu folder Warehouse Management | Setup
2. Was renamed to "Process Price Difference Adjustment" to follow the capitalization rule
3. Was included a description and help.

---
M modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_MENU.xml
M modules/org.openbravo.advpaymentmngt/src-db/database/sourcedata/AD_TREENODE.xml
M src-db/database/sourcedata/OBUIAPP_PROCESS.xml
---
(0083986)
dmiguelez   
2016-02-05 15:10   
Code Review + Testing Ok
(0085110)
hudsonbot   
2016-03-17 10:56   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/b22fb0500156 [^]
Maturity status: Test