0008189: 250 VM: Pareto Report: Update ABC is executed incorrectly

(0014767)
RenateNieuwkoop (reporter)
2009-03-18 11:14

Also when executing a PI based on the ABC code 'A', the boots do appear (see snapshot).

Warehouse Management || Transactions || Physical Inventory || Header >> Lines

(0015073)
psarobe (manager)
2009-03-30 18:54

Sorry Renate but still not clear for me.

Do you mean that after pressing the button Update ABC everything works fine? What is the problem?

(0015077)
RenateNieuwkoop (reporter)
2009-03-30 19:47

Hi Pablo,

Please check the snapshots. What you can see is that the Pareto report gives incorrect information. It's saying that the boots are 'C', yet when you update the codes, an A appears.

So the Pareto report gives incorrect information, that's the problem.

(0015549)
psarobe (manager)
2009-04-22 16:31

still not clear to me

(0015554)
RenateNieuwkoop (reporter)
2009-04-22 16:44

Hi Pablo,

As you know in the Training environment we always have the same data and the same exercises with the same output. Now, in 2.50, we are running the Pareto report and based on our outcome in 2.40 were expecting certain parts to be classified as an 'A' part. However, on the Pareto report they appear as a class 'C', which is not what we expected. The we comment the codes to the product and now on the product the class 'A' is assigned (as we expected it to be in the first place).

So the display in the Pareto report of the values is incorrect, but when it actually assigns the classification to the products, it does have the correct classification.
So that's why I am saying that just the display in the Pareto report seems to be incorrect.

(0020907)
networkb (developer)
2009-10-09 13:23

Changed the target version to the next release

(0021227)
hgbot (developer)
2009-10-20 19:05

Repository: erp/devel/pi
Changeset: 92157df159bf8fe63ede03f42fd62a49d52eac09
Author: Gorka Ion Damián <gorkaion.damian <at> openbravo.com>
Date: Tue Oct 20 19:04:22 2009 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/92157df159bf8fe63ede03f42fd62a49d52eac09 [^]

Fixed issue 8189. Fixed the Pareto Report Product
The ABC categories are now calculated based on the total amount stocked and
not on the cost of one unit.
Removed the attributes from the report that were not needed.

---
M src-db/database/model/functions/M_GET_PARETO_ABC.xml
M src-db/database/model/functions/M_UPDATE_PARETO_PRODUCT.xml
M src/org/openbravo/erpCommon/ad_reports/ReportParetoProduct.html
M src/org/openbravo/erpCommon/ad_reports/ReportParetoProduct.java
M src/org/openbravo/erpCommon/ad_reports/ReportParetoProduct.xml
M src/org/openbravo/erpCommon/ad_reports/ReportParetoProduct_data.xsql
---

(0021361)
arunkumar (reporter)
2009-10-23 13:19

The code 'A' has been assigned for Boots in both places
(1) Product -> org specifis
(2) After running Pareto report product

Tested & Working fine.

(0022132)
RenateNieuwkoop (reporter)
2009-11-24 10:35

Well, it looks a lot better now, most products are updated correctly.
The strange thing is that on the whole list, still one product was updated incorrectly: product shirt appeared on the Pareto report wtih a code 'B', yet it was updated with a code 'C' in the Org Specific tab.

(0022509)
psarobe (manager)
2009-12-07 20:18

Renate, are you sure that that product is not a problem in your environment?

(0022653)
rafaroda (developer)
2009-12-10 17:43

Reminder sent to: RenateNieuwkoop

Renate,

Please comment on this issue otherwise it will be closed as properly resolved.

Thanks.

(0022657)
RenateNieuwkoop (reporter)
2009-12-10 17:51

Sorry Rafa, I will retest it tomorrow!

thanks for your patience
Renate

(0022715)
RenateNieuwkoop (reporter)
2009-12-11 17:00

Hi Rafa,

I reran the report and now it was updated correctly. I'll keep an eye on it, but for now you can close this issue

thanks
Renate

(0023920)
RenateNieuwkoop (reporter)
2010-02-03 10:29

Hi

As I am testing MP11 this bug is reappearing. Please fix this again.
See snapshots.

thanks
Renate

(0023921)
RenateNieuwkoop (reporter)
2010-02-03 10:43

The ABC report states that the code for Boots is A (which is correct, since we always use the same data in Training). However, when committing the data, the ABC code in the product setup for boots appears as C. Also when creating the Physical Inventory count list based on the ABC code, the boots are considered C instead of A.

(0023928)
RenateNieuwkoop (reporter)
2010-02-03 15:57

Actually I had MP10. Now testing in MP11 it appears to be correct again.

Issue History
Date Modified	Username	Field	Change
2009-03-18 10:48	RenateNieuwkoop	New Issue
2009-03-18 10:48	RenateNieuwkoop	Assigned To	=> rafaroda
2009-03-18 10:48	RenateNieuwkoop	File Added: 180309ParetoReport250.jpg
2009-03-18 10:49	RenateNieuwkoop	File Added: 180309ABCcodeProduct250.jpg
2009-03-18 11:14	RenateNieuwkoop	Note Added: 0014767
2009-03-30 18:54	psarobe	Note Added: 0015073
2009-03-30 18:54	psarobe	Status	new => feedback
2009-03-30 19:47	RenateNieuwkoop	Note Added: 0015077
2009-04-22 16:31	psarobe	Note Added: 0015549
2009-04-22 16:44	RenateNieuwkoop	Note Added: 0015554
2009-04-23 13:06	psarobe	Status	feedback => scheduled
2009-04-23 13:06	psarobe	fix_in_branch	=> pi
2009-10-09 13:23	networkb	Note Added: 0020907
2009-10-09 13:23	networkb	Reporter	RenateNieuwkoop => networkb
2009-10-09 13:23	networkb	Target Version	=> 2.50MP7
2009-10-09 13:23	networkb	fix_in_branch	pi =>
2009-10-13 12:14	psarobe	Assigned To	rafaroda => gorkaion
2009-10-20 19:05	hgbot	Checkin
2009-10-20 19:05	hgbot	Note Added: 0021227
2009-10-20 19:05	hgbot	Status	scheduled => resolved
2009-10-20 19:05	hgbot	Resolution	open => fixed
2009-10-20 19:05	hgbot	Fixed in SCM revision	=> http://code.openbravo.com/erp/devel/pi/rev/92157df159bf8fe63ede03f42fd62a49d52eac09 [^]
2009-10-22 19:26	psarobe	Tag Attached: main
2009-10-22 19:33	psarobe	Tag Attached: 250MP8
2009-10-23 13:19	arunkumar	Status	resolved => closed
2009-10-23 13:19	arunkumar	Note Added: 0021361
2009-10-23 13:19	arunkumar	Fixed in Version	=> 2.50MP8
2009-10-24 00:00	anonymous	sf_bug_id	0 => 2885053
2009-11-24 10:35	RenateNieuwkoop	Note Added: 0022132
2009-11-24 10:37	psarobe	Status	closed => new
2009-11-24 10:37	psarobe	Resolution	fixed => open
2009-11-24 10:37	rafaroda	Issue Monitored: rafaroda
2009-11-24 10:37	psarobe	Fixed in Version	2.50MP8 =>
2009-11-24 10:39	RenateNieuwkoop	File Added: 231109ParetoReportasterix.jpg
2009-11-24 10:40	RenateNieuwkoop	File Added: 231109ParetoReportABC.jpg
2009-12-07 20:18	psarobe	Note Added: 0022509
2009-12-07 20:18	psarobe	Status	new => feedback
2009-12-10 17:43	rafaroda	Issue Monitored: RenateNieuwkoop
2009-12-10 17:43	rafaroda	Note Added: 0022653
2009-12-10 17:51	RenateNieuwkoop	Note Added: 0022657
2009-12-11 17:00	RenateNieuwkoop	Note Added: 0022715
2009-12-11 17:22	psarobe	Status	feedback => closed
2009-12-11 17:22	psarobe	Resolution	open => no change required
2010-02-03 10:29	RenateNieuwkoop	Note Added: 0023920
2010-02-03 10:43	RenateNieuwkoop	Note Added: 0023921
2010-02-03 15:57	RenateNieuwkoop	Note Added: 0023928
2010-08-16 12:00	adrianromero	Relationship added	related to 0014161

Notes
(0014767) RenateNieuwkoop (reporter) 2009-03-18 11:14	Also when executing a PI based on the ABC code 'A', the boots do appear (see snapshot). Warehouse Management \|\| Transactions \|\| Physical Inventory \|\| Header >> Lines

(0015073) psarobe (manager) 2009-03-30 18:54	Sorry Renate but still not clear for me. Do you mean that after pressing the button Update ABC everything works fine? What is the problem?

(0015077) RenateNieuwkoop (reporter) 2009-03-30 19:47	Hi Pablo, Please check the snapshots. What you can see is that the Pareto report gives incorrect information. It's saying that the boots are 'C', yet when you update the codes, an A appears. So the Pareto report gives incorrect information, that's the problem.

(0015549) psarobe (manager) 2009-04-22 16:31	still not clear to me

(0015554) RenateNieuwkoop (reporter) 2009-04-22 16:44	Hi Pablo, As you know in the Training environment we always have the same data and the same exercises with the same output. Now, in 2.50, we are running the Pareto report and based on our outcome in 2.40 were expecting certain parts to be classified as an 'A' part. However, on the Pareto report they appear as a class 'C', which is not what we expected. The we comment the codes to the product and now on the product the class 'A' is assigned (as we expected it to be in the first place). So the display in the Pareto report of the values is incorrect, but when it actually assigns the classification to the products, it does have the correct classification. So that's why I am saying that just the display in the Pareto report seems to be incorrect.

(0020907) networkb (developer) 2009-10-09 13:23	Changed the target version to the next release

(0021227) hgbot (developer) 2009-10-20 19:05	Repository: erp/devel/pi Changeset: 92157df159bf8fe63ede03f42fd62a49d52eac09 Author: Gorka Ion Damián <gorkaion.damian <at> openbravo.com> Date: Tue Oct 20 19:04:22 2009 +0200 URL: http://code.openbravo.com/erp/devel/pi/rev/92157df159bf8fe63ede03f42fd62a49d52eac09 [^] Fixed issue 8189. Fixed the Pareto Report Product The ABC categories are now calculated based on the total amount stocked and not on the cost of one unit. Removed the attributes from the report that were not needed. --- M src-db/database/model/functions/M_GET_PARETO_ABC.xml M src-db/database/model/functions/M_UPDATE_PARETO_PRODUCT.xml M src/org/openbravo/erpCommon/ad_reports/ReportParetoProduct.html M src/org/openbravo/erpCommon/ad_reports/ReportParetoProduct.java M src/org/openbravo/erpCommon/ad_reports/ReportParetoProduct.xml M src/org/openbravo/erpCommon/ad_reports/ReportParetoProduct_data.xsql ---

(0021361) arunkumar (reporter) 2009-10-23 13:19	The code 'A' has been assigned for Boots in both places (1) Product -> org specifis (2) After running Pareto report product Tested & Working fine.

(0022132) RenateNieuwkoop (reporter) 2009-11-24 10:35	Well, it looks a lot better now, most products are updated correctly. The strange thing is that on the whole list, still one product was updated incorrectly: product shirt appeared on the Pareto report wtih a code 'B', yet it was updated with a code 'C' in the Org Specific tab.

(0022509) psarobe (manager) 2009-12-07 20:18	Renate, are you sure that that product is not a problem in your environment?

(0022653) rafaroda (developer) 2009-12-10 17:43	Reminder sent to: RenateNieuwkoop Renate, Please comment on this issue otherwise it will be closed as properly resolved. Thanks.

(0022657) RenateNieuwkoop (reporter) 2009-12-10 17:51	Sorry Rafa, I will retest it tomorrow! thanks for your patience Renate

(0022715) RenateNieuwkoop (reporter) 2009-12-11 17:00	Hi Rafa, I reran the report and now it was updated correctly. I'll keep an eye on it, but for now you can close this issue thanks Renate

(0023920) RenateNieuwkoop (reporter) 2010-02-03 10:29	Hi As I am testing MP11 this bug is reappearing. Please fix this again. See snapshots. thanks Renate

(0023921) RenateNieuwkoop (reporter) 2010-02-03 10:43	The ABC report states that the code for Boots is A (which is correct, since we always use the same data in Training). However, when committing the data, the ABC code in the product setup for boots appears as C. Also when creating the Physical Inventory count list based on the ABC code, the boots are considered C instead of A.

(0023928) RenateNieuwkoop (reporter) 2010-02-03 15:57	Actually I had MP10. Now testing in MP11 it appears to be correct again.