Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0013301Openbravo ERPA. Platformpublic2010-05-12 10:202010-05-19 00:01
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target Version2.50MP18Fixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0013301: [OBCONTEXT] Warehouse not present in OBContext
DescriptionThe DAL OBContext has user, role, client, org but not yet the warehouse stored. This should be added to be more in line with the session variables.
Steps To Reproduce
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2010-05-12 10:20mtaalNew Issue
2010-05-12 10:20mtaalAssigned To => mtaal
2010-05-12 10:20mtaalOBNetwork customer => No
2010-05-12 10:21hgbotCheckin
2010-05-12 10:21hgbotNote Added: 0027162
2010-05-12 10:21hgbotStatusnew => resolved
2010-05-12 10:21hgbotResolutionopen => fixed
2010-05-12 10:21hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d75a4058f957a9c4a9b7d9e8c763d6e20b0a9e0d [^]
2010-05-14 13:27shuehnerNote Added: 0027253
2010-05-14 13:27shuehnerStatusresolved => new
2010-05-14 13:27shuehnerResolutionfixed => open
2010-05-16 21:54hgbotCheckin
2010-05-16 21:54hgbotNote Added: 0027341
2010-05-16 21:54hgbotStatusnew => resolved
2010-05-16 21:54hgbotResolutionopen => fixed
2010-05-16 21:54hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/d75a4058f957a9c4a9b7d9e8c763d6e20b0a9e0d [^] => http://code.openbravo.com/erp/devel/pi/rev/6b25d2db763af6a7925b896dfbd229458654d5b2 [^]
2010-05-18 12:12gorka_gilTarget Version2.50MP16 => 2.50MP18
2010-05-18 13:36shuehnerNote Added: 0027422
2010-05-18 13:36shuehnerStatusresolved => closed
2010-05-18 21:44hudsonbotCheckin
2010-05-18 21:44hudsonbotNote Added: 0027515
2010-05-18 21:44hudsonbotCheckin
2010-05-18 21:44hudsonbotNote Added: 0027536
2010-05-19 00:01anonymoussf_bug_id0 => 3003688

Notes
(0027162)
hgbot   
2010-05-12 10:21   
Repository: erp/devel/pi
Changeset: d75a4058f957a9c4a9b7d9e8c763d6e20b0a9e0d
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed May 12 10:20:57 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/d75a4058f957a9c4a9b7d9e8c763d6e20b0a9e0d [^]

fixes issue 13301: [OBCONTEXT] Warehouse not present in OBContext

---
M src-test/org/openbravo/test/dal/OBContextTest.java
M src/org/openbravo/base/secureApp/LoginUtils.java
M src/org/openbravo/dal/core/OBContext.java
---
(0027253)
shuehner   
2010-05-14 13:27   
Reopened: ,found logic bug while reviewing the diff:

in this url
https://code.openbravo.com/erp/devel/pi/rev/d75a4058f957a9c4a9b7d9e8c763d6e20b0a9e0d [^]

in line 3.76

we are loosing languageCode as null,null is used instead of languageCode,null
(0027341)
hgbot   
2010-05-16 21:54   
Repository: erp/devel/pi
Changeset: 6b25d2db763af6a7925b896dfbd229458654d5b2
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sun May 16 21:53:44 2010 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/6b25d2db763af6a7925b896dfbd229458654d5b2 [^]

fixes issue 13301: [OBCONTEXT] Warehouse not present in OBContext
Added extra testcase for language in OBContext
Change in previous mode computation in OBContext

---
M src-test/org/openbravo/test/dal/OBContextTest.java
M src/org/openbravo/dal/core/OBContext.java
---
(0027422)
shuehner   
2010-05-18 13:36   
Tested in pi/pg. Required additional fix is now done, also a needed fix for 12594/12660 was added here.
(0027515)
hudsonbot   
2010-05-18 21:44   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/d75a4058f957 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/aa11838d5f80 [^]
Tests: http://builds.openbravo.com/view/devel-int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.17380.obx [^]
(0027536)
hudsonbot   
2010-05-18 21:44   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/6b25d2db763a [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/aa11838d5f80 [^]
Tests: http://builds.openbravo.com/view/devel-int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.17380.obx [^]