Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0010729Openbravo ERP04. Warehouse managementpublic2009-09-24 10:532012-05-25 12:10
Reporterarunkumar 
Assigned Todmiguelez 
PrioritylowSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version2.50MP6 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0010729: Traceability Report displays two tables
DescriptionTraceability Report displays two tables for a product that is being traced.
Steps To ReproduceLogin as userA/userA
Go to Warehouse Management || Analysis Tools || Traceability Report || Traceability Report
Select the product "A Final Goods"
Select the radio button "outgoing".
Click "HTML Format".

A report is displayed with two tables.

One table displays the outgoing information related to the product A Final Goods with attribute L111.
Another table displays nothing(A Final Goods - Attribute L123). What is the use of another table?
Proposed Solution
Additional Information
TagsOB3-Reviewed, VMA-Reviewed
Relationships
Attached Filesjpg TraceabilityReport.JPG (173,675) 2009-09-24 10:53
https://issues.openbravo.com/file_download.php?file_id=1778&type=bug
jpg
Issue History
Date ModifiedUsernameFieldChange
2009-09-24 10:53arunkumarNew Issue
2009-09-24 10:53arunkumarAssigned To => rafaroda
2009-09-24 10:53arunkumarFile Added: TraceabilityReport.JPG
2009-09-24 16:46psarobeStatusnew => scheduled
2009-09-24 16:46psarobeNote Added: 0020344
2009-09-24 16:46psarobefix_in_branch => pi
2009-09-25 08:32arunkumarfix_in_branchpi =>
2009-09-25 08:32arunkumarProposed Solution updated
2009-09-25 08:37rafarodaNote Added: 0020354
2010-02-11 18:05rafarodaAssigned Torafaroda => adrianromero
2011-06-03 11:00dalsasuaAssigned Toadrianromero => dalsasua
2011-07-20 18:11dalsasuaAssigned Todalsasua => jonalegriaesarte
2011-10-29 20:42psarobeTag Attached: OB3-Reviewed
2012-02-15 19:04iciordiaAssigned Tojonalegriaesarte => vmromanos
2012-02-16 17:28vmromanosTag Attached: VMA-Reviewed
2012-05-04 08:35dmiguelezAssigned Tovmromanos => dmiguelez
2012-05-04 12:48dmiguelezNote Added: 0048375
2012-05-09 10:01hgbotCheckin
2012-05-09 10:01hgbotNote Added: 0048488
2012-05-09 10:01hgbotStatusscheduled => resolved
2012-05-09 10:01hgbotResolutionopen => fixed
2012-05-09 10:01hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/cafefe0b3587d73423ec70900c626bd79cdb8032 [^]
2012-05-09 10:01hgbotCheckin
2012-05-09 10:01hgbotNote Added: 0048489
2012-05-09 10:14miruritaNote Added: 0048491
2012-05-09 10:14miruritaStatusresolved => closed
2012-05-25 12:10hudsonbotCheckin
2012-05-25 12:10hudsonbotNote Added: 0049073
2012-05-25 12:10hudsonbotCheckin
2012-05-25 12:10hudsonbotNote Added: 0049074

Notes
(0020344)
psarobe   
2009-09-24 16:46   
When there is no movements for a product the table should not be shown
(0020354)
rafaroda   
2009-09-25 08:37   
The second table is for showing information of the same product, but with different attribute set value.

I do agree with Pablo's solution: only show new tables only if the product with this attribute set has data to be shown.
(0048375)
dmiguelez   
2012-05-04 12:48   
Test Plan:

Login as Openbravo/openbravo
Change rol to QAAdmin

Go to Warehouse Management || Analysis Tools || Traceability Report || Traceability Report
Select the product "Final Goods A"
Select the radio button "outgoing".
Click "HTML Format".

Realize that there is only one table shown, and that this one has values in it.
(0048488)
hgbot   
2012-05-09 10:01   
Repository: erp/devel/pi
Changeset: cafefe0b3587d73423ec70900c626bd79cdb8032
Author: David Miguelez <david.miguelez <at> openbravo.com>
Date: Fri May 04 12:40:39 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/cafefe0b3587d73423ec70900c626bd79cdb8032 [^]

Fixes issue 0010729: Now it displays only tables with information inside.

---
M src/org/openbravo/erpCommon/ad_reports/MInOutTraceReports.java
---
(0048489)
hgbot   
2012-05-09 10:01   
Repository: erp/devel/pi
Changeset: 1ac6a16a51d738bbf5e392550e1bf97d0cf2ab52
Author: Mikel Irurita <mikel.irurita <at> openbravo.com>
Date: Wed May 09 09:57:57 2012 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/1ac6a16a51d738bbf5e392550e1bf97d0cf2ab52 [^]

Related to issue 10729: update license text year

---
M src/org/openbravo/erpCommon/ad_reports/MInOutTraceReports.java
---
(0048491)
mirurita   
2012-05-09 10:14   
code review + testing OK

// Removing the data structure inside the for loop works fine
String[] data = { "", "", "a", "", "b", "", "c", "", "" };
for (int i = 0; i < data.length; i++) {
  if ("".equals(data[i])) {
    data = (String[]) ArrayUtils.removeElement(data, data[i]);
    i--;
  }
}
//Output: ["a", "b", "c"]
(0049073)
hudsonbot   
2012-05-25 12:10   
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/5401e185a8b0 [^]

Maturity status: Test
(0049074)
hudsonbot   
2012-05-25 12:10   
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/5401e185a8b0 [^]

Maturity status: Test