0016105: Wrong quantity in Suggested Purchase Order when running Purchasing Plan

Openbravo Issue Tracking System - Openbravo ERP

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0016105

Openbravo ERP

06. Material requirement planning (MRP)

public

2011-03-02 13:37

2011-04-04 12:19

Reporter

maite

Assigned To

adrianromero

Priority

urgent

Severity

major

Reproducibility

always

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

2.50MP26

Target Version

2.50MP28

Fixed in Version

Merge Request Status

Review Assigned To

OBNetwork customer

OBPS

Web browser

Modules

Core

Support ticket

Regression level

Regression date

Regression introduced in release

Regression introduced by commit

Triggers an Emergency Pack

Summary

0016105: Wrong quantity in Suggested Purchase Order when running Purchasing Plan

Description

Wrong quantity in Suggested Purchase Order when running Purchasing Plan: Quantity of suggested purchase order should be sum of quantities of pending sales order

Steps To Reproduce

1. Master Data Management || Product || Product : search for Hammer
2. Master Data Management || Product || Product >> Purchasing : add record for Mcgiver with "current vendor " checked
3. Master Data Management || Product || Product >> Org Specific : add record for Main and Capacity=10000 and Safety Stock=0
4. Warehouse Management || Transactions || Physical Inventory || Header : set stock for hammers to 0 units
5. Material Requirement (MRP) || Transactions || Purchasing Plan || Header : create record for
  - Main
  - 02/03/2011
  - time horizon=10
  - safety = 0
  - vendor= mcgiver
  - product=hammer
Material Requirement (MRP) || Transactions || Purchasing Plan || Header >> Lines : add lines
  - line1: hammer + quantities= 0 + dates=02/03/2011 + type= Stock + fixed=Y
  - line2: hammer + quantities= 0 + dates=02/03/2011 + type= MinStock + fixed=Y
  - line3: hammer + quantities= -5 + dates=02/03/2011 + type= PendingSalesOrder + fixed=Y
  - line4: hammer + quantities= -3 + dates=02/03/2011 + type= PendingSalesOrder + fixed=Y
  - line5: hammer + quantities= -2 + dates=02/03/2011 + type= PendingSalesOrder + fixed=Y
6. Come back to Purchasing Plan header and press"Process Purchase Plan". Relize that line of type "Suggested Purchase Order" has Quantity different from 10 (which is sum of 5+3+2)

Proposed Solution

in line 139 of function mrp_purchaseplan should be added
v_neededqty_old := v_neededqty_new;
because actually this variable only is being updated when executing "IF" clause but not "ELSE" clause.

As same block:
v_neededqty_old := v_neededqty_new;
v_qty_old := v_qty_new;
v_stock := v_stock + v_qty_new;
will be executed in IF and ELSE they can be put outside

Additional Information