Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
0036917Openbravo ERP04. Warehouse managementpublic2017-09-21 13:392017-09-26 15:25
ngarcia 
markmm82 
urgentminoralways
closedfixed 
5
 
3.0PR17Q4 
aferraz
Core
No
0036917: Default values are not selected in Return Material Receipt window if the organization does not have an on hand warehouse defined
Default values are not selected in Return Material Receipt window if the organization does not have an on hand warehouse defined

As group admin role:
   Remove 'España Región Norte' warehouse to 'F&B España - Región Norte' organization (Organization - Warehouse tab)
   Create a new record in Return Material Receipt
   Save it
   Create a new record and check all the fields remain empty

   The log shows:
      ERROR org.openbravo.client.application.window.FormInitializationComponent - TabId:30576C6ABD12419F9D19D497216FC9B8 - Mode:NEW - rowId:null
org.openbravo.base.exception.OBException: Couldn't execute callout (class org.openbravo.erpCommon.ad_callouts.SE_InOut_Warehouse)
    at org.openbravo.client.application.window.FormInitializationComponent.runCallouts(FormInitializationComponent.java:1555)
...
Caused by: java.lang.NullPointerException
    at org.openbravo.erpCommon.ad_callouts.SE_InOut_Warehouse.execute(SE_InOut_Warehouse.java:41)
No tags attached.
related to design defect 00323663.0PR16Q4 closed inigosanchez callout invocation ineficiencies 
depends on backport 00369213.0PR17Q3.1 closed markmm82 Default values are not selected in Return Material Receipt window if the organization does not have an on hand warehouse defined 
depends on backport 00369223.0PR17Q2.4 closed markmm82 Default values are not selected in Return Material Receipt window if the organization does not have an on hand warehouse defined 
Issue History
2017-09-21 13:39ngarciaNew Issue
2017-09-21 13:39ngarciaAssigned To => Triage Finance
2017-09-21 13:39ngarciaModules => Core
2017-09-21 13:39ngarciaResolution time => 1507759200
2017-09-21 13:39ngarciaTriggers an Emergency Pack => No
2017-09-21 13:40ngarciaIssue Monitored: networkb
2017-09-21 14:06eugeniIssue Monitored: eugeni
2017-09-21 17:18caristuRelationship addedrelated to 0032366
2017-09-21 18:02aferrazNote Added: 0099434
2017-09-21 18:02aferrazSeveritymajor => minor
2017-09-21 18:03aferrazStatusnew => scheduled
2017-09-21 18:03aferrazAssigned ToTriage Finance => markmm82
2017-09-25 10:41hgbotCheckin
2017-09-25 10:41hgbotNote Added: 0099517
2017-09-25 10:41hgbotStatusscheduled => resolved
2017-09-25 10:41hgbotResolutionopen => fixed
2017-09-25 10:41hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/170d69d2ab888f84bcf34501f96b240da1eb421b [^]
2017-09-25 11:00aferrazReview Assigned To => aferraz
2017-09-25 11:00aferrazNote Added: 0099524
2017-09-25 11:00aferrazStatusresolved => closed
2017-09-25 11:00aferrazFixed in Version => 3.0PR17Q4
2017-09-26 15:25hudsonbotCheckin
2017-09-26 15:25hudsonbotNote Added: 0099587

Notes
(0099434)
aferraz   
2017-09-21 18:02   
Moved to minor as per [1]: Default Value missing for the fields required.
[1] http://wiki.openbravo.com/wiki/Bug_Reporting_Guidelines#How_to_Choose_the_Right_Severity [^]
(0099517)
hgbot   
2017-09-25 10:41   
Repository: erp/devel/pi
Changeset: 170d69d2ab888f84bcf34501f96b240da1eb421b
Author: Mark <markmm82 <at> gmail.com>
Date: Fri Sep 22 11:35:32 2017 -0400
URL: http://code.openbravo.com/erp/devel/pi/rev/170d69d2ab888f84bcf34501f96b240da1eb421b [^]

Fixes issue 36917: Default values are not selected in Return Material Receipt
window if the organization does not have an on hand warehouse defined

A NPE is fired if Warehouse is not selected, because was trying to access a method
of null object.

To fix that is avoided a NPE, only creating a OBDAL object if a warehouse is selected.

---
M src/org/openbravo/erpCommon/ad_callouts/SE_InOut_Warehouse.java
---
(0099524)
aferraz   
2017-09-25 11:00   
Code review + Testing OK
(0099587)
hudsonbot   
2017-09-26 15:25   
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/8fa643b4437e [^]
Maturity status: Test