FEFO picking considers item that uses a blocked lot number from a blocked bin incorrectly when you create a warehouse pick for the item in Microsoft Dynamics NAV 2009

View products that this article applies to.

This article applies to Microsoft Dynamics NAV for all countries and all language locales.

↑ Back to the top

Symptoms

Consider the following scenario in Microsoft Dynamics NAV 2009:

You enable the Pick According to FEFO and the Always Create Pick Line features for a location.
You put an item that uses a lot number into a bin in the location.
The lot of the item has the earliest expiration date.
You block the lot and the bin.
You create a warehouse pick for the item.

In this scenario, the lot of the item from the blocked bin is suggested. This problem occurs in the following products:

Microsoft Dynamics NAV 2009 R2
Microsoft Dynamics NAV 2009 Service Pack 1

↑ Back to the top

Resolution

Hotfix information

A supported hotfix is now available from Microsoft. However, it is only intended to correct the problem that is described in this article. Apply it only to systems that are experiencing this specific problem. This hotfix may receive additional testing. Therefore, if you are not severely affected by this problem, we recommend that you wait for the next Microsoft Dynamics NAV 2009 service pack or the next Microsoft Dynamics NAV version that contains this hotfix.

Note In special cases, charges that are ordinarily incurred for support calls may be canceled if a Technical Support Professional for Microsoft Dynamics and related products determines that a specific update will resolve your problem. The usual support costs will apply to additional support questions and issues that do not qualify for the specific update in question.

↑ Back to the top

Installation information

Microsoft provides programming examples for illustration only, without warranty either expressed or implied. This includes, but is not limited to, the implied warranties of merchantability or fitness for a particular purpose. This article assumes that you are familiar with the programming language that is being demonstrated and with the tools that are used to create and to debug procedures. Microsoft support engineers can help explain the functionality of a particular procedure, but they will not modify these examples to provide added functionality or construct procedures to meet your specific requirements.

Note Before you install this hotfix, verify that all Microsoft Navision client users are logged off the system. This includes Microsoft Navision Application Services (NAS) client users. You should be the only client user who is logged on when you implement this hotfix.

To implement this hotfix, you must have a developer license.

We recommend that the user account in the Windows Logins window or in the Database Logins window be assigned the "SUPER" role ID. If the user account cannot be assigned the "SUPER" role ID, you must verify that the user account has the following permissions:

The Modify permission for the object that you will be changing.
The Execute permission for the System Object ID 5210 object and for the System Object ID 9015 object.

Note You do not have to have rights to the data stores unless you have to perform data repair.

Code changes

Note Always test code fixes in a controlled environment before you apply the fixes to your production computers.
To resolve this problem, follow these steps:

Change the code in the CalcQtyAvailToPick function in the Bin Content table (7302) as follows:
Existing code

...
ERROR(Text008,CaptionField,TABLECAPTION);
END;
PROCEDURE CalcQtyAvailToPick@2(ExcludeQtyBase@1000 : Decimal) : Decimal;
BEGIN

// Delete the following lines.
CALCFIELDS("Quantity (Base)","Negative Adjmt. Qty. (Base)","Pick Quantity (Base)");
EXIT("Quantity (Base)" - ("Pick Quantity (Base)" - ExcludeQtyBase + "Negative Adjmt. Qty. (Base)"));
// End of the lines.

END;
PROCEDURE CalcQtyAvailToPutAway@3(ExcludeQtyBase@1000 : Decimal) : Decimal;
BEGIN
CALCFIELDS("Quantity (Base)","Positive Adjmt. Qty. (Base)","Put-away Quantity (Base)");
...

Replacement code

...
ERROR(Text008,CaptionField,TABLECAPTION);
END;
PROCEDURE CalcQtyAvailToPick@2(ExcludeQtyBase@1000 : Decimal) : Decimal;
BEGIN

// Add the following lines.
IF "Block Movement" IN ["Block Movement"::All,"Block Movement"::Outbound] THEN
EXIT(0);
CALCFIELDS("Quantity (Base)","Negative Adjmt. Qty. (Base)","Pick Quantity (Base)");
EXIT("Quantity (Base)" - ("Pick Quantity (Base)" - ExcludeQtyBase + "Negative Adjmt. Qty. (Base)") -
CalcQtyWithBlockedItemTracking);
END;
PROCEDURE CalcQtyWithBlockedItemTracking@19() : Decimal;
VAR
SerialNoInfo@1005 : Record 6504;
LotNoInfo@1004 : Record 6505;
XBinContent@1003 : Record 7302;
QtySNBlocked@1002 : Decimal;
QtyLNBlocked@1001 : Decimal;
QtySNAndLNBlocked@1000 : Decimal;
SNGiven@1006 : Boolean;
LNGiven@1007 : Boolean;
NoITGiven@1008 : Boolean;
BEGIN
SerialNoInfo.SETRANGE("Item No.","Item No.");
SerialNoInfo.SETRANGE("Variant Code","Variant Code");
COPYFILTER("Serial No. Filter",SerialNoInfo."Serial No.");
SerialNoInfo.SETRANGE(Blocked,TRUE);
LotNoInfo.SETRANGE("Item No.","Item No.");
LotNoInfo.SETRANGE("Variant Code","Variant Code");
COPYFILTER("Lot No. Filter",LotNoInfo."Lot No.");
LotNoInfo.SETRANGE(Blocked,TRUE);
IF SerialNoInfo.ISEMPTY AND LotNoInfo.ISEMPTY THEN
EXIT;
SNGiven := NOT (GETFILTER("Serial No. Filter") = '');
LNGiven := NOT (GETFILTER("Lot No. Filter") = '');
XBinContent.COPY(Rec);
SETRANGE("Serial No. Filter");
SETRANGE("Lot No. Filter");
NoITGiven := NOT SNGiven AND NOT LNGiven;
IF SNGiven OR NoITGiven THEN
IF SerialNoInfo.FINDSET THEN
REPEAT
SETRANGE("Serial No. Filter",SerialNoInfo."Serial No.");
CALCFIELDS("Quantity (Base)");
QtySNBlocked += "Quantity (Base)";
SETRANGE("Serial No. Filter");
UNTIL SerialNoInfo.NEXT = 0;
IF LNGiven OR NoITGiven THEN
IF LotNoInfo.FINDSET THEN
REPEAT
SETRANGE("Lot No. Filter",LotNoInfo."Lot No.");
CALCFIELDS("Quantity (Base)");
QtyLNBlocked += "Quantity (Base)";
SETRANGE("Lot No. Filter");
UNTIL LotNoInfo.NEXT = 0;
IF (SNGiven AND LNGiven) OR NoITGiven THEN
IF SerialNoInfo.FINDSET THEN
REPEAT
IF LotNoInfo.FINDSET THEN
REPEAT
SETRANGE("Serial No. Filter",SerialNoInfo."Serial No.");
SETRANGE("Lot No. Filter",LotNoInfo."Lot No.");
CALCFIELDS("Quantity (Base)");
QtySNAndLNBlocked += "Quantity (Base)";
UNTIL LotNoInfo.NEXT = 0;
UNTIL SerialNoInfo.NEXT = 0;
COPY(XBinContent);
EXIT(QtySNBlocked + QtyLNBlocked - QtySNAndLNBlocked);
// End of the lines.

END;
PROCEDURE CalcQtyAvailToPutAway@3(ExcludeQtyBase@1000 : Decimal) : Decimal;
BEGIN
CALCFIELDS("Quantity (Base)","Positive Adjmt. Qty. (Base)","Put-away Quantity (Base)");
...

Change the code in the CalcQtyBase function in the Bin Content table (7302) as follows:
Existing code 1

...
EXIT(WhseEntry."Qty. (Base)");
END;
PROCEDURE CalcQtyBase@49() : Decimal;
VAR

// Delete the following lines.
WhseEntry@1003 : Record 7312;
WhseActivLine@1004 : Record 5767;
WhseJnlLine@1005 : Record 7311;
BEGIN
WhseEntry.SETCURRENTKEY(
"Item No.","Bin Code","Location Code","Variant Code",
"Unit of Measure Code","Lot No.","Serial No.");
WhseEntry.SETRANGE("Item No.","Item No.");
WhseEntry.SETRANGE("Bin Code","Bin Code");
WhseEntry.SETRANGE("Location Code","Location Code");
WhseEntry.SETRANGE("Variant Code","Variant Code");
WhseEntry.SETRANGE("Unit of Measure Code","Unit of Measure Code");
COPYFILTER("Lot No. Filter",WhseEntry."Lot No.");
COPYFILTER("Serial No. Filter",WhseEntry."Serial No.");
WhseEntry.CALCSUMS("Qty. (Base)");
// End of the lines.

WhseActivLine.SETCURRENTKEY(
"Item No.","Bin Code","Location Code",
"Action Type","Variant Code","Unit of Measure Code",
"Breakbulk No.","Activity Type","Lot No.","Serial No.");
WhseActivLine.SETRANGE("Item No.","Item No." );
...

Replacement code 1

...
EXIT(WhseEntry."Qty. (Base)");
END;
PROCEDURE CalcQtyBase@49() : Decimal;
VAR

// Add the following lines.
WhseActivLine@1004 : Record 5767;
WhseJnlLine@1005 : Record 7311;
BEGIN
// End of the lines.

WhseActivLine.SETCURRENTKEY(
"Item No.","Bin Code","Location Code",
"Action Type","Variant Code","Unit of Measure Code",
"Breakbulk No.","Activity Type","Lot No.","Serial No.");
WhseActivLine.SETRANGE("Item No.","Item No." );
...

Existing code 2

...
WhseJnlLine.SETRANGE("Unit of Measure Code","Unit of Measure Code");
COPYFILTER("Lot No. Filter",WhseJnlLine."Lot No.");
COPYFILTER("Serial No. Filter",WhseJnlLine."Serial No.");
WhseJnlLine.CALCSUMS("Qty. (Absolute, Base)");

// Delete the following lines.
EXIT(
WhseEntry."Qty. (Base)" +
// End of the lines.

WhseActivLine."Qty. Outstanding (Base)" +
WhseJnlLine."Qty. (Absolute, Base)");
END;
PROCEDURE CalcQtyUOM@13() : Decimal;
...

Replacement code 2

...
WhseJnlLine.SETRANGE("Unit of Measure Code","Unit of Measure Code");
COPYFILTER("Lot No. Filter",WhseJnlLine."Lot No.");
COPYFILTER("Serial No. Filter",WhseJnlLine."Serial No.");
WhseJnlLine.CALCSUMS("Qty. (Absolute, Base)");

// Add the following lines.
CALCFIELDS("Quantity (Base)");
EXIT(
"Quantity (Base)" +
// End of the lines.

WhseActivLine."Qty. Outstanding (Base)" +
WhseJnlLine."Qty. (Absolute, Base)");
END;
PROCEDURE CalcQtyUOM@13() : Decimal;
...

Change the code in the CalcQtyOnBlkdITOrOutbndBins function in the WMS Management codeunit (7302) as follows:
Existing code

...
EXIT("Qty. Shipped (Base)");
END;
END;
PROCEDURE GetCaption@44(DestType@1004 : ' ,Customer,Vendor,Location,Item,Family,Sales Order';SourceDoc@1009 : ' ,Sales Order,,,Sales Return Order,Purchase Order,,,Purchase Return Order,Inbound Transfer,Outbound Transfer,Prod. Consumption,Prod. Output';Selection@1000 : Integer) : Text[50];
VAR
PurchHeader@1003 : Record 38;
Vendor@1001 : Record 23;
Customer@1002 : Record 18;
...

Replacement code

...
EXIT("Qty. Shipped (Base)");
END;
END;

// Add the following lines.
PROCEDURE CalcQtyOnBlkdITOrOutbndBins@1000(LocationCode@1170000000 : Code[10];ItemNo@1002 : Code[20];VariantCode@1003 : Code[10];LotNo@1004 : Code[20];SerialNo@1005 : Code[20];LNRequired@1006 : Boolean;SNRequired@1007 : Boolean) QtyBlocked : Decimal;
VAR
BinContent@1008 : Record 7302;
BEGIN
WITH BinContent DO BEGIN
SETCURRENTKEY("Location Code","Item No.","Variant Code");
SETRANGE("Location Code",LocationCode);
SETRANGE("Item No.",ItemNo);
SETRANGE("Variant Code",VariantCode);
IF LotNo <> '' THEN
IF LNRequired THEN
SETRANGE("Lot No. Filter",LotNo);
IF SerialNo <> '' THEN
IF SNRequired THEN
SETRANGE("Serial No. Filter",SerialNo);
IF FINDSET THEN
REPEAT
IF "Block Movement" IN ["Block Movement"::All,"Block Movement"::Outbound] THEN BEGIN
CALCFIELDS("Quantity (Base)");
QtyBlocked += "Quantity (Base)";
END ELSE
QtyBlocked += CalcQtyWithBlockedItemTracking;
UNTIL NEXT = 0; END;
END;
// End of the lines.

PROCEDURE GetCaption@44(DestType@1004 : ' ,Customer,Vendor,Location,Item,Family,Sales Order';SourceDoc@1009 : ' ,Sales Order,,,Sales Return Order,Purchase Order,,,Purchase Return Order,Inbound Transfer,Outbound Transfer,Prod. Consumption,Prod. Output';Selection@1000 : Integer) : Text[50];
VAR
PurchHeader@1003 : Record 38;
Vendor@1001 : Record 23;
Customer@1002 : Record 18;
...

Change the code in the CalcTotalAvailQtyToPick function in the Create Pick codeunit (7312) as follows:
Existing code 1

...
QtyInWhse@1008 : Decimal;
QtyOnPickBins@1007 : Decimal;
QtyOnPutAwayBins@1025 : Decimal;
QtyOnOutboundBins@1009 : Decimal;
QtyOnReceiveBins@1000 : Decimal;
SubTotal@1010 : Decimal;
QtyReservedOnPickShip@1011 : Decimal;
LineReservedQty@1005 : Decimal;
QtyAssignedPick@1012 : Decimal;
QtyAssignedToPick@1021 : Decimal;
...

Replacement code 1

...
QtyInWhse@1008 : Decimal;
QtyOnPickBins@1007 : Decimal;
QtyOnPutAwayBins@1025 : Decimal;
QtyOnOutboundBins@1009 : Decimal;
QtyOnReceiveBins@1000 : Decimal;

// Add the following line.
QtyBlocked@1026 : Decimal;

SubTotal@1010 : Decimal;
QtyReservedOnPickShip@1011 : Decimal;
LineReservedQty@1005 : Decimal;
QtyAssignedPick@1012 : Decimal;
QtyAssignedToPick@1021 : Decimal;
...

Existing code 2

...
QtyOnOutboundBins :=
CalcQtyOnOutboundBins(
LocationCode,ItemNo,VariantCode,LotNo,SerialNo);
END;
TempWhseItemTrkgLine2.COPY(TempWhseItemTrkgLine);
IF ReqFEFOPick AND CalledFromWksh THEN BEGIN
TempWhseItemTrkgLine2."Entry No." := TempWhseItemTrkgLine2."Entry No." + 1;
TempWhseItemTrkgLine2."Lot No." := LotNo;
TempWhseItemTrkgLine2."Serial No." := SerialNo;
...

Replacement code 2

...
QtyOnOutboundBins :=
CalcQtyOnOutboundBins(
LocationCode,ItemNo,VariantCode,LotNo,SerialNo);
END;

// Add the following lines.
QtyBlocked :=
WMSMgt.CalcQtyOnBlkdITOrOutbndBins(
LocationCode,ItemNo,VariantCode,LotNo,SerialNo,LNRequired,SNRequired);
// End of the lines.

TempWhseItemTrkgLine2.COPY(TempWhseItemTrkgLine);
IF ReqFEFOPick AND CalledFromWksh THEN BEGIN
TempWhseItemTrkgLine2."Entry No." := TempWhseItemTrkgLine2."Entry No." + 1;
TempWhseItemTrkgLine2."Lot No." := LotNo;
TempWhseItemTrkgLine2."Serial No." := SerialNo;
...

Existing code 3

...
WhseActivLine.CALCSUMS("Qty. Outstanding (Base)");
QtyAssignedPick := QtyAssignedPick - WhseActivLine."Qty. Outstanding (Base)";
END;
SubTotal :=

// Delete the following line.
QtyInWhse - QtyOnPickBins - QtyOnPutAwayBins - QtyOnOutboundBins - QtyOnReceiveBins - ABS(Item."Reserved Qty. on Inventory");

IF (SubTotal < 0) OR CalledFromPickWksh OR CalledFromMoveWksh THEN BEGIN
QtyReservedOnPickShip :=
CalcReservQtyOnPicksShips(
LocationCode,ItemNo,VariantCode);
...

Replacement code 3

...
WhseActivLine.CALCSUMS("Qty. Outstanding (Base)");
QtyAssignedPick := QtyAssignedPick - WhseActivLine."Qty. Outstanding (Base)";
END;
SubTotal :=

// Add the following lines.
QtyInWhse - QtyOnPickBins - QtyOnPutAwayBins - QtyOnOutboundBins - QtyOnReceiveBins - ABS(Item."Reserved Qty. on Inventory") - 
QtyBlocked;
// End of the lines.

IF (SubTotal < 0) OR CalledFromPickWksh OR CalledFromMoveWksh THEN BEGIN
QtyReservedOnPickShip :=
CalcReservQtyOnPicksShips(
LocationCode,ItemNo,VariantCode);
...

Change the code in the CalcTotalQtyOnBinType function in the Create Pick codeunit (7312) as follows:
Existing code

...
CALCSUMS("Qty. Outstanding (Base)");
EXIT("Qty. Outstanding (Base)");
END;
END;

// Delete the following lines.
LOCAL PROCEDURE CalcTotalQtyOnBinType@104(BinTypeFilter@1004 : Text[1024];LocationCode@1003 : Code[10];ItemNo@1002 : Code[20];VariantCode@1001 : Code[10]) : Decimal;
VAR
WhseEntry@1000 : Record 7312;
// End of the lines.

BEGIN
WITH WhseEntry DO BEGIN
SETCURRENTKEY("Item No.","Location Code","Variant Code","Bin Type Code");
SETRANGE("Item No.",ItemNo);
SETRANGE("Location Code",LocationCode);
...

Replacement code

...
CALCSUMS("Qty. Outstanding (Base)");
EXIT("Qty. Outstanding (Base)");
END;
END;

// Add the following lines.
LOCAL PROCEDURE CalcTotalQtyOnBinType@67(BinTypeFilter@1004 : Text[1024];LocationCode@1003 : Code[10];ItemNo@1002 : Code[20];VariantCode@1001 : Code[10]) : Decimal;
VAR
WhseEntry@1010 : Record 7312;
// End of the lines.

BEGIN
WITH WhseEntry DO BEGIN
SETCURRENTKEY("Item No.","Location Code","Variant Code","Bin Type Code");
SETRANGE("Item No.",ItemNo);
SETRANGE("Location Code",LocationCode);
...

Change the code in the InsertPickBinWhseActivLinefunction in the Create Inventory Pick codeunit (7322) as follows:
Existing code

...
LineCreated := TRUE;
NextLineNo := NextLineNo + 10000;
RemQtyToPickBase := RemQtyToPickBase - QtyToPickBase;
QtyAvailToPickBase := QtyAvailToPickBase - QtyToPickBase;

// Delete the following line.
UNTIL QtyAvailToPickBase = 0;

END;
UNTIL (NEXT = 0) OR (RemQtyToPickBase = 0);
END;
END;
...

Replacement code

...
LineCreated := TRUE;
NextLineNo := NextLineNo + 10000;
RemQtyToPickBase := RemQtyToPickBase - QtyToPickBase;
QtyAvailToPickBase := QtyAvailToPickBase - QtyToPickBase;

// Add the following line.
UNTIL QtyAvailToPickBase <= 0;

END;
UNTIL (NEXT = 0) OR (RemQtyToPickBase = 0);
END;
END;
...

Change the code in the CalcInvtAvailability function in the Create Inventory Pick codeunit (7322) as follows:
Existing code 1

...
VAR
Item2@1002 : Record 27;
QtyAssgndtoPick@1003 : Decimal;
LineReservedQty@1001 : Decimal;
QtyReservedOnPickShip@1005 : Decimal;
BEGIN
WITH WhseActivLine DO BEGIN
GetItem("Item No.");
Item2 := Item;
Item2.SETRANGE("Location Filter","Location Code");
...

Replacement code 1

...
VAR
Item2@1002 : Record 27;
QtyAssgndtoPick@1003 : Decimal;
LineReservedQty@1001 : Decimal;
QtyReservedOnPickShip@1005 : Decimal;

// Add the following line.
QtyBlocked@1008 : Decimal;

BEGIN
WITH WhseActivLine DO BEGIN
GetItem("Item No.");
Item2 := Item;
Item2.SETRANGE("Location Filter","Location Code");
...

Existing code 2

...
Item2 := Item;
Item2.SETRANGE("Location Filter","Location Code");
Item2.SETRANGE("Variant Filter","Variant Code");
Item2.CALCFIELDS(Inventory,"Reserved Qty. on Inventory");
QtyAssgndtoPick := WMSMgt.CalcQtyAssgndtoPick(Location,"Item No.","Variant Code",'','');
LineReservedQty :=
WMSMgt.CalcLineReservedQtyonInvt(
"Source Type","Source Subtype","Source No.","Source Line No.","Source Subline No.",'','');
QtyReservedOnPickShip :=
CalcReservQtyOnPicksShips("Location Code","Item No.","Variant Code");
...

Replacement code 2

...
Item2 := Item;
Item2.SETRANGE("Location Filter","Location Code");
Item2.SETRANGE("Variant Filter","Variant Code");
Item2.CALCFIELDS(Inventory,"Reserved Qty. on Inventory");
QtyAssgndtoPick := WMSMgt.CalcQtyAssgndtoPick(Location,"Item No.","Variant Code",'','');

// Add the following lines.
QtyBlocked :=
WMSMgt.CalcQtyOnBlkdITOrOutbndBins("Location Code","Item No.","Variant Code",'','',FALSE,FALSE);
// End of the lines.

LineReservedQty :=
WMSMgt.CalcLineReservedQtyonInvt(
"Source Type","Source Subtype","Source No.","Source Line No.","Source Subline No.",'','');
QtyReservedOnPickShip :=
CalcReservQtyOnPicksShips("Location Code","Item No.","Variant Code");
...

Existing code 3

...
QtyReservedOnPickShip :=
CalcReservQtyOnPicksShips("Location Code","Item No.","Variant Code");
END;
EXIT(

// Delete the following line.
Item2.Inventory - ABS(Item2."Reserved Qty. on Inventory") - QtyAssgndtoPick
+ LineReservedQty + QtyReservedOnPickShip);

END;
PROCEDURE CalcReservQtyOnPicksShips@53(LocationCode@1001 : Code[10];ItemNo@1002 : Code[20];VariantCode@1003 : Code[10]) ReservedForThisLine : Decimal;
VAR
...

Replacement code 3

...
QtyReservedOnPickShip :=
CalcReservQtyOnPicksShips("Location Code","Item No.","Variant Code");
END;
EXIT(

// Add the following line.
Item2.Inventory - ABS(Item2."Reserved Qty. on Inventory") - QtyAssgndtoPick - QtyBlocked +
+ LineReservedQty + QtyReservedOnPickShip);

END;
PROCEDURE CalcReservQtyOnPicksShips@53(LocationCode@1001 : Code[10];ItemNo@1002 : Code[20];VariantCode@1003 : Code[10]) ReservedForThisLine : Decimal;
VAR
...

Prerequisites

You must have one of the following products installed to apply this hotfix:

Microsoft Dynamics NAV 2009 R2
Microsoft Dynamics NAV 2009 Service Pack 1

Removal information

You cannot remove this hotfix.

↑ Back to the top

Status

Microsoft has confirmed that this is a problem in the Microsoft products that are listed in the "Applies to" section.

↑ Back to the top

Note This is a "FAST PUBLISH" article created directly from within the Microsoft support organization. The information contained herein is provided as-is in response to emerging issues. As a result of the speed in making it available, the materials may include typographical errors and may be revised at any time without notice. See Terms of Use for other considerations.

↑ Back to the top

Applies to:

Dynamics NAV 2009

↑ Back to the top

Keywords: kbmbscodefix, kbqfe, kbmbsquickpub, kbmbspartner, kbmbsmigrate, kbnoloc, kbsurveynew, kb

↑ Back to the top

Article Info

Article ID	:	2791785
Revision	:	1
Created on	:	1/7/2017
Published on	:	1/25/2013
Exists online	:	False
Views	:	266

Microsoft KB Archive Search

FEFO picking considers item that uses a blocked lot number from a blocked bin incorrectly when you create a warehouse pick for the item in Microsoft Dynamics NAV 2009

Symptoms

Resolution

Hotfix information

Installation information

Code changes

Prerequisites

Removal information

Status

Applies to: