Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0037119Openbravo ERP03. Procurement managementpublic2017-10-19 17:092018-01-08 18:21
Reportermaite 
Assigned Tononofrancisco 
PriorityimmediateSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version3.0PR18Q1 
Merge Request Status
Review Assigned Todmiguelez
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2016-10-13
Regression introduced in release3.0PR17Q1
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/rev/9ec41af386ae [^]
Triggers an Emergency PackNo
Summary0037119: "Copy Lines" process not properly copying "organization" dimension when defined at invoice line level
Description"Copy lines" process is properly copying another accounting dimensions defined at line level, such as Cost Center, but it nos properly copying Organization accounting dimension
Steps To Reproduce0. Access "Client" window and set "Organization" as "Show in lines" and "Cost Center" as "Show in lines"
1. Access Dimensions tab, filter by "AR Invoice" and "AP Invoice" and set "Organization" as "Show in lines"
2. Access Cost Center window and define 2 records: cc1 and cc2
3. Define Conversion rate between EUR and USD for toda's date
4. Logout and login again
5. Create new Purchase Invoice for "España" organization and today's date
6. Add line for any product, set Organization dimension "España norte" and Cost Center "a"
7. Add line for any other product, set Organization dimension "España sur" and Cost Center "b". Process invoice
8. Create new Purchase Invoice for "España" organization and today's date. Run "Copy lines" and select previous invoice

Realize that Cost Center information has been correctly copied but NOT Organization information
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
depends on backport 00371203.0PR17Q4 closed nonofrancisco "Copy Lines" process not properly copying "organization" dimension when defined at invoice line level 
depends on backport 00371213.0PR17Q3.1 closed nonofrancisco "Copy Lines" process not properly copying "organization" dimension when defined at invoice line level 
depends on backport 00371223.0PR17Q2.4 closed nonofrancisco "Copy Lines" process not properly copying "organization" dimension when defined at invoice line level 
caused by defect 0034195 closed markmm82 Error using copy lines in sales invoice 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-10-19 17:09maiteNew Issue
2017-10-19 17:09maiteAssigned To => Triage Finance
2017-10-19 17:09maiteModules => Core
2017-10-19 17:09maiteResolution time => 1510182000
2017-10-19 17:09maiteTriggers an Emergency Pack => No
2017-10-19 17:09maiteIssue Monitored: networkb
2017-10-19 17:35nonofranciscoAssigned ToTriage Finance => nonofrancisco
2017-10-19 17:36nonofranciscoStatusnew => scheduled
2017-10-19 21:51nonofranciscoNote Added: 0099972
2017-10-19 23:41aferrazRelationship addedcaused by 0034195
2017-10-24 10:04dmiguelezRegression level => Production - Confirmed Stable
2017-10-24 10:04dmiguelezRegression date => 2016-10-13
2017-10-24 10:04dmiguelezRegression introduced in release => 3.0PR17Q1
2017-10-24 10:04dmiguelezRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/rev/9ec41af386ae [^]
2017-10-24 16:21hgbotCheckin
2017-10-24 16:21hgbotNote Added: 0100035
2017-10-24 16:21hgbotStatusscheduled => resolved
2017-10-24 16:21hgbotResolutionopen => fixed
2017-10-24 16:21hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/34517e931d716616241929f25b708370b290e55c [^]
2017-10-24 16:24dmiguelezReview Assigned To => dmiguelez
2017-10-24 16:24dmiguelezNote Added: 0100042
2017-10-24 16:24dmiguelezStatusresolved => closed
2018-01-03 12:38hudsonbotCheckin
2018-01-03 12:38hudsonbotNote Added: 0101344
2018-01-08 18:21dmiguelezFixed in Version => 3.0PR18Q1

Notes
(0099972)
nonofrancisco   
2017-10-19 21:51   
Test Plan

0. Access "Client" window and set "Organization" as "Show in lines" and "Cost Center" as "Show in lines"
1. Access Dimensions tab, filter by "AR Invoice" and "AP Invoice" and set "Organization" as "Show in lines"
2. Access Cost Center window and define 2 records: cc1 and cc2
3. Define Conversion rate between EUR and USD for toda's date
4. Logout and login again
5. Create new Purchase Invoice for "España" organization and today's date, for Bebidas Alegres S.L
6. Add line for Cerveza Ale, set Organization dimension "España norte" and Cost Center "cc1"
7. Add line for Vino Tinto , set Organization dimension "España sur" and Cost Center "cc2". Process invoice
8. Create new Purchase Invoice for "España" organization and today's date. Run "Copy lines" and select previous invoice

Realize that Line for Cerveza Ale has Organization dimension set to "España norte" and Cost Center "cc1"
Realize that Line for Vino Tinto has Organization dimension set to "España sur" and Cost Center "cc2"
(0100035)
hgbot   
2017-10-24 16:21   
Repository: erp/devel/pi
Changeset: 34517e931d716616241929f25b708370b290e55c
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Tue Oct 24 10:15:35 2017 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/34517e931d716616241929f25b708370b290e55c [^]

Fixes Issue 37119. Sets correct Organization in new Invoice line

* If the organization of the original invoice line belongs to the child
  tree of the organization of the document header of the new line,
  use the organization of the original invoice line
* If not, use the organization of the document header of the new line

---
M src/org/openbravo/erpCommon/ad_actionButton/CopyFromInvoice.java
---
(0100042)
dmiguelez   
2017-10-24 16:24   
Code Review + Testing Ok
(0101344)
hudsonbot   
2018-01-03 12:38   
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/c81e0d3cbab5 [^]
Maturity status: Test