Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0036091Openbravo ERP04. Warehouse managementpublic2017-05-24 16:482017-06-08 17:12
Reporterngarcia 
Assigned ToAtulOpenbravo 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR17Q3 
Merge Request Status
Review Assigned Toaferraz
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0036091: ProcessID parameter is not available in the extension points of M_GET_STOCK_PARAM function
DescriptionProcessID parameter is not available in the extension points of M_GET_STOCK_PARAM function.

Steps To ReproduceBefore 3.0PR15Q2 the ProcessID value was inserted in AD_PINSTANCE_PARA table before calling the M_GET_STOCK function from some database functions such as:

src-db/database/model/functions/MA_PRODUCTIONRUN_STANDARD.xml
src-db/database/model/functions/MRP_RUN_INITIALIZE.xml
src-db/database/model/functions/M_INOUT_CREATE.xml
src-db/database/model/functions/M_PRODUCTION_RUN.xml

After 3.0PR15Q2, the parameter is passed to the M_GET_STOCK_PARAM function but is not used there and the extension points of that function don't receive that value:

M_Get_Stock - Finish Process - Deprecated
M_Get_Stock - Finish Process

The following module has a function that uses the processID value:

Agile Production Reservation in Production Run (com.openbravo.gps.agileproduction.reservation)
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
related to defect 0027160 closed eduardo_Argal C_Order_Post poor performance 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-05-24 16:48ngarciaNew Issue
2017-05-24 16:48ngarciaAssigned To => Triage Finance
2017-05-24 16:48ngarciaModules => Core
2017-05-24 16:48ngarciaTriggers an Emergency Pack => No
2017-05-24 16:48ngarciaRelationship addedrelated to 0027160
2017-05-24 16:48ngarciaIssue Monitored: networkb
2017-05-24 16:52ngarciaResolution time => 1497391200
2017-05-26 12:46aferrazAssigned ToTriage Finance => AtulOpenbravo
2017-06-02 07:28AtulOpenbravoStatusnew => scheduled
2017-06-05 18:34hgbotCheckin
2017-06-05 18:34hgbotNote Added: 0097144
2017-06-05 18:34hgbotStatusscheduled => resolved
2017-06-05 18:34hgbotResolutionopen => fixed
2017-06-05 18:34hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/6a40dea2c7739dcd4cb83f9c7913835e2d5a0d2e [^]
2017-06-05 18:34hgbotCheckin
2017-06-05 18:34hgbotNote Added: 0097145
2017-06-06 11:20hgbotCheckin
2017-06-06 11:20hgbotNote Added: 0097160
2017-06-06 11:21aferrazReview Assigned To => aferraz
2017-06-06 11:21aferrazNote Added: 0097161
2017-06-06 11:21aferrazStatusresolved => closed
2017-06-06 11:21aferrazFixed in Version => 3.0PR17Q3
2017-06-08 17:12hudsonbotCheckin
2017-06-08 17:12hudsonbotNote Added: 0097243
2017-06-08 17:12hudsonbotCheckin
2017-06-08 17:12hudsonbotNote Added: 0097244
2017-06-08 17:12hudsonbotCheckin
2017-06-08 17:12hudsonbotNote Added: 0097245

Notes
(0097144)
hgbot   
2017-06-05 18:34   
Repository: erp/devel/pi
Changeset: 6a40dea2c7739dcd4cb83f9c7913835e2d5a0d2e
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Fri Jun 02 23:55:38 2017 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/6a40dea2c7739dcd4cb83f9c7913835e2d5a0d2e [^]

Fixes Issue 36091:ProcessID parameter is not available in the extension
points of M_GET_STOCK_PARAM function

Each DB procedure calling M_GET_STOCK_PARAM send its own process id to
M_GET_STOCK_PARAM. v_ProcessID is sent insert as PInstance Parameter
instead of '---'.
To avoid unique constraint ad_pinstance_para_pinstance_un error on
table ad_pinstance_para for columns ad_pinstance_id and seqno, as 160
seqno is already used for "Available" parameter, 190 seqno is set for
v_ProcessID parameter.

---
M src-db/database/model/functions/M_GET_STOCK_PARAM.xml
---
(0097145)
hgbot   
2017-06-05 18:34   
Repository: erp/devel/pi
Changeset: 9a60d02449edd9ca3b4b65e12ab60fbdd085ea06
Author: Alvaro Ferraz <alvaro.ferraz <at> openbravo.com>
Date: Mon Jun 05 18:11:34 2017 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/9a60d02449edd9ca3b4b65e12ab60fbdd085ea06 [^]

Related to issue 36091: Code review improvements

---
M src-db/database/model/functions/M_GET_STOCK_PARAM.xml
---
(0097160)
hgbot   
2017-06-06 11:20   
Repository: erp/devel/pi
Changeset: 60c767f6ecdd889d09139a0b9ce8f58cbd00044a
Author: Atul Gaware <atul.gaware <at> openbravo.com>
Date: Tue Jun 06 03:19:53 2017 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/60c767f6ecdd889d09139a0b9ce8f58cbd00044a [^]

Related to Issue 36091:Set proper process id in M_Inout_Create while calling
M_Get_Stock_Param

Process Id of M_Inout_Create is '199' as this should be passed as ProcessID
parameter while calling M_Get_Stock_Param

---
M src-db/database/model/functions/M_INOUT_CREATE.xml
---
(0097161)
aferraz   
2017-06-06 11:21   
Code review OK
(0097243)
hudsonbot   
2017-06-08 17:12   
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/fda0f16ab470 [^]
Maturity status: Test
(0097244)
hudsonbot   
2017-06-08 17:12   
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/fda0f16ab470 [^]
Maturity status: Test
(0097245)
hudsonbot   
2017-06-08 17:12   
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/fda0f16ab470 [^]
Maturity status: Test