Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033382Openbravo ERP07. Sales managementpublic2016-06-29 22:062018-02-22 17:21
Reporterrbianchini 
Assigned ToSanjota 
PriorityhighSeveritymajorReproducibilitysometimes
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0PR16Q1.3 
Target VersionFixed in Version3.0PR16Q4 
Merge Request Status
Review Assigned Toaferraz
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2015-10-28
Regression introduced in release3.0PR16Q1.3
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/e1c26dc9a819 [^]
Triggers an Emergency PackNo
Summary0033382: Wrong Storage Bin in Goods Receipt Lines
DescriptionIn some cases, when you create a goods receipt line the storage bin's defaul value don't belongs to the warehouse defined in the header.
It works fine in 15Q4.6 but don't in 16Q1.3 or 16Q2
Steps To Reproduce1 - Log in ERP as Openbravo with the following configuration:
Role: The White Valley Group Admin
Client: The White Valley Group
Organization: Vall Blanca Store
Warehouse: Vall Blanca Store Warehouse

2 - Open the Goods Receipt window and create a new header (in form view) as follows:
Organization: Vall Blanca Store
Document Type: Spain MM Receipt
Warehouse: White Valley Spain Distribution Center
BP: Mountains and Snow S.L

3 - Save the header and create a new line in form view
4A - In 15Q4.6 The storage bin loaded as default is WVS/001 0-0-0 --> OK
4B - In 16Q1.3 and 16Q2 The storage bin loaded as default is CMS/001 0-0-0. The storage bin is wrong since it don't belongs to the White Valley Spain Distribution Center warehouse defined in the header


Steps to Reproduce with F & B Internation Group Admin Role:
1 - Log in ERP as Openbravo with the following configuration:
Role: F & B International group Admin
Client: F & B International group

2. - Open Warehouse and Storage bin:
--Create New Warehouse :
Organization : "F&B España, S.A"
Name : "Test"
Search Key : "Test"
Save
--Create Storage bin Under newly created warehouse "Test"
Search Key : "NewTestBin"

3 - Open the Goods Receipt window and create a new header (in form view) as follows:
Organization: "F&B España - Región Norte"
Warehouse: "Test"

4 - Save the header and create a new line in form view
Check that in Storage Bin field there is no default value means it's blank.

Note: It should show newly created "NewTestBin" in Storage Bin field.
Proposed SolutionReview why the default value OnHandLocatorDefault is not executing, since if you execute the OnHandLocatorDefault's query, it returns the correct value
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00333873.0PR16Q2.1 closed Sanjota Wrong Storage Bin in Goods Receipt Lines 
depends on backport 00333883.0PR16Q1.4 closed Sanjota Wrong Storage Bin in Goods Receipt Lines 
depends on backport 00334023.0PR16Q3 closed Sanjota Wrong Storage Bin in Goods Receipt Lines 
caused by defect 00311763.0PR16Q1 closed dmiguelez it is possible to select wrong storage bins in the goods shipment lines 
related to defect 0034483 closed vmromanos Mark Incoming/Outgoing shipment windows as not supported 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-06-29 22:06rbianchiniNew Issue
2016-06-29 22:06rbianchiniAssigned To => Triage Finance
2016-06-29 22:06rbianchiniModules => Core
2016-06-29 22:06rbianchiniResolution time => 1467774000
2016-06-29 22:06rbianchiniRegression level => Production - Confirmed Stable
2016-06-29 22:06rbianchiniRegression introduced in release => 3.0PR16Q1.3
2016-06-29 22:06rbianchiniTriggers an Emergency Pack => No
2016-06-30 12:51aferrazResolution time1467774000 => 1467756000
2016-06-30 12:51aferrazRegression date => 2015-10-28
2016-06-30 12:51aferrazRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/e1c26dc9a819 [^]
2016-06-30 12:51aferrazRelationship addedcaused by 0031176
2016-06-30 12:52aferrazStatusnew => scheduled
2016-06-30 12:52aferrazAssigned ToTriage Finance => Sanjota
2016-06-30 17:10SanjotaSteps to Reproduce Updatedbug_revision_view_page.php?rev_id=12561#r12561
2016-06-30 19:09SanjotaNote Added: 0088116
2016-07-01 08:53aferrazStatusscheduled => feedback
2016-07-01 08:53aferrazStatusfeedback => scheduled
2016-07-01 09:24hgbotCheckin
2016-07-01 09:24hgbotNote Added: 0088127
2016-07-01 09:24hgbotStatusscheduled => resolved
2016-07-01 09:24hgbotResolutionopen => fixed
2016-07-01 09:24hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/2abf88eb9008e6368a7464074b1f15903dd6ff18 [^]
2016-07-01 09:26aferrazReview Assigned To => aferraz
2016-07-01 09:26aferrazNote Added: 0088128
2016-07-01 09:26aferrazStatusresolved => closed
2016-07-01 09:26aferrazFixed in Version => 3.0PR16Q4
2016-07-01 10:54SanjotaNote Edited: 0088116bug_revision_view_page.php?bugnote_id=0088116#r12569
2016-08-11 18:40hudsonbotCheckin
2016-08-11 18:40hudsonbotNote Added: 0088978
2016-11-14 10:13vmromanosRelationship addedrelated to 0034483
2018-02-22 17:21hgbotCheckin
2018-02-22 17:21hgbotNote Added: 0102605

Notes
(0088116)
Sanjota   
2016-06-30 19:09   
(edited on: 2016-07-01 10:54)
Test Plan:
1 - Log in ERP as Openbravo with the following configuration:
Role: The White Valley Group Admin
Client: The White Valley Group
Organization: Vall Blanca Store
Warehouse: Vall Blanca Store Warehouse

2 - Open the Goods Receipt window and create a new header (in form view) as follows:
Organization: Vall Blanca Store
Document Type: Spain MM Receipt
Warehouse: White Valley Spain Distribution Center
BP: Mountains and Snow S.L

3 - Save the header and create a new line in form view
4A - In 15Q4.6 The storage bin loaded as default is WVS/001 0-0-0 --> OK
4B - In 16Q1.3 and 16Q2 The storage bin loaded as default is WVS/001 0-0-0 not CMS/001 0-0-0.




Another Test Plan:
1 - Log in ERP as Openbravo with the following configuration:
Role: F & B International group Admin
Client: F & B International group

2. - Open Warehouse and Storage bin:
--Create New Warehouse :
Organization : "F&B España, S.A"
Name : "Test"
Search Key : "Test"
Save
--Create Storage bin Under newly created warehouse "Test"
Search Key : "NewTestBin"

3 - Open the Goods Receipt window and create a new header (in form view) as follows:
Organization: "F&B España - Región Norte"
Warehouse: "Test"

4 - Save the header and create a new line in form view
Check that in Storage Bin field default value will be "NewTestBin".


Also includes test plan of 31176.
(0088127)
hgbot   
2016-07-01 09:24   
Repository: erp/devel/pi
Changeset: 2abf88eb9008e6368a7464074b1f15903dd6ff18
Author: Sanjota <sanjota.nelagi <at> promantia.com>
Date: Thu Jun 30 18:04:33 2016 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/2abf88eb9008e6368a7464074b1f15903dd6ff18 [^]

Fixes issue 33382: Wrong Storage Bin in Goods Receipt Lines

In some cases, when you create a goods receipt line the storage bin's defaul value don't belongs to the warehouse defined in the header.

---
M src-db/database/sourcedata/AD_AUXILIARINPUT.xml
---
(0088128)
aferraz   
2016-07-01 09:26   
Code review + Testing OK
(0088978)
hudsonbot   
2016-08-11 18:40   
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/edaa05b1fb18 [^]
Maturity status: Test
(0102605)
hgbot   
2018-02-22 17:21   
Repository: tools/automation/int
Changeset: cc1d31e2ef07a1dc6373baa9b0a2c864a41b08b1
Author: Rafael Queralta Pozo <rqueralta <at> nauta.cu>
Date: Tue Oct 18 14:03:06 2016 -0400
URL: http://code.openbravo.com/tools/automation/int/rev/cc1d31e2ef07a1dc6373baa9b0a2c864a41b08b1 [^]

Related to issue 33382: Added some automated tests

---
M src-test/com/openbravo/test/integration/erp/data/warehouse/setup/warehouseandstoragebins/StorageBinData.java
M src-test/com/openbravo/test/integration/erp/data/warehouse/setup/warehouseandstoragebins/WarehouseData.java
M src-test/com/openbravo/test/integration/erp/gui/general/enterprise/organization/OrganizationTab.java
M src-test/com/openbravo/test/integration/erp/gui/popups/AddressSelectorPopUp.java
M src-test/com/openbravo/test/integration/erp/modules/functionalteam/procurement/testsuites/PRO_Regressions/PRO_RegressionSuite.java
M src-test/com/openbravo/test/integration/erp/modules/functionalteam/procurement/testsuites/PRO_Regressions/build.xml
M src-test/com/openbravo/test/integration/erp/modules/functionalteam/sales/testsuites/SAL_Regressions/SAL_RegressionSuite.java
M src-test/com/openbravo/test/integration/erp/modules/functionalteam/sales/testsuites/SAL_Regressions/build.xml
M src-test/com/openbravo/test/integration/erp/testscripts/generalsetup/enterprise/Organization.java
A src-test/com/openbravo/test/integration/erp/gui/general/enterprise/organization/WarehouseTab.java
A src-test/com/openbravo/test/integration/erp/modules/functionalteam/procurement/testsuites/PRO_Regressions/PRORegression33382.java
A src-test/com/openbravo/test/integration/erp/modules/functionalteam/sales/testsuites/SAL_Regressions/SALRegression33382.java
---