Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0022864Openbravo ERP04. Warehouse managementpublic2013-01-23 10:052013-01-31 16:22
ReporterVictorVillar 
Assigned Tojonalegriaesarte 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target Version3.0MP20Fixed in Version 
Merge Request Status
Review Assigned ToVictorVillar
OBNetwork customerOBPS
Web browser
ModulesCore
Support ticket19097
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0022864: Wrong bracket on existsTransactions function in CostingRuleProcess.java
DescriptionWrong bracket on existsTransactions function in CostingRuleProcess.java
besides =p

where.append(" where " + MaterialTransaction.PROPERTY_PRODUCT + " = p)");
it should be moved to the next line as follows:

where.append(" and " + MaterialTransaction.PROPERTY_ORGANIZATION + " .id in (:childOrgs))");
Steps To ReproduceWrong bracket on existsTransactions function in CostingRuleProcess.java
besides =p

where.append(" where " + MaterialTransaction.PROPERTY_PRODUCT + " = p)");
it should be moved to the next line as follows:

where.append(" and " + MaterialTransaction.PROPERTY_ORGANIZATION + " .id in (:childOrgs))");
Proposed SolutionPlease see the diff attached
Additional Information
TagsNo tags attached.
Relationships
Attached Filesdiff diffcostingrule.diff (1,006) 2013-01-23 10:05
https://issues.openbravo.com/file_download.php?file_id=5910&type=bug
Issue History
Date ModifiedUsernameFieldChange
2013-01-23 10:05VictorVillarNew Issue
2013-01-23 10:05VictorVillarAssigned To => dmiguelez
2013-01-23 10:05VictorVillarFile Added: diffcostingrule.diff
2013-01-23 10:05VictorVillarModules => Core
2013-01-23 10:05VictorVillarOBNetwork customer => Yes
2013-01-23 10:05VictorVillarSupport ticket => 19097
2013-01-23 10:05VictorVillarResolution time => 1361660400
2013-01-23 10:05VictorVillarTriggers an Emergency Pack => No
2013-01-23 10:05VictorVillarIssue Monitored: networkb
2013-01-23 11:38dmiguelezAssigned Todmiguelez => jonalegriaesarte
2013-01-24 12:09hgbotCheckin
2013-01-24 12:09hgbotNote Added: 0055807
2013-01-24 12:09hgbotStatusnew => resolved
2013-01-24 12:09hgbotResolutionopen => fixed
2013-01-24 12:09hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/7814d070ffe372b303caadd46c0951985f462ac6 [^]
2013-01-24 21:41hudsonbotCheckin
2013-01-24 21:41hudsonbotNote Added: 0055827
2013-01-31 16:22VictorVillarReview Assigned To => VictorVillar
2013-01-31 16:22VictorVillarNote Added: 0056018
2013-01-31 16:22VictorVillarStatusresolved => closed

Notes
(0055807)
hgbot   
2013-01-24 12:09   
Repository: erp/devel/pi
Changeset: 7814d070ffe372b303caadd46c0951985f462ac6
Author: Jon Alegría <jon.alegria <at> openbravo.com>
Date: Thu Jan 24 12:08:50 2013 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/7814d070ffe372b303caadd46c0951985f462ac6 [^]

Fixes issue 0022864: Wrong bracket on existsTransactions function in
CostingRuleProcess.java

---
M src/org/openbravo/costing/CostingRuleProcess.java
---
(0055827)
hudsonbot   
2013-01-24 21:41   
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/fbbe21bdfeff [^]

Maturity status: Test
(0056018)
VictorVillar   
2013-01-31 16:22   
Verified