Thread Rating:
  • 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
stock transfer session incorrectly built?
04-29-2020, 02:45 AM,
#1
stock transfer session incorrectly built?
weberp version 4.14.1

Hi

When looking at StockTransfers.php, line 97:

Code:
$_SESSION['Transfer']= new StockTransfer(0,
$_POST['StockLocationFrom'],
'',
$_POST['StockLocationTo'],
'',
Date($_SESSION['DefaultDateFormat'])
);

there is a discrepancy with the number of parameters expected by DefineStockTransfers.php :

Code:
function __construct($TrfID,
$StockLocationFrom,
$StockLocationFromName,
$StockLocationFromAccount,
$StockLocationTo,
$StockLocationToName,
$StockLocationToAccount,
$TranDate )

This causes the $_SESSION['Transfer'] values being incorrect because the $StockLocationFromAccount and StockLocationToAccount are not included as parameters during the new StockTransfer creation.

Code:
StockTransfer Object
(
    [TrfID] => 0
    [StockLocationFrom] => ENG
    [StockLocationFromName] =>
    [StockLocationFromAccount] => WRET
    [StockLocationTo] => WRET
    [StockLocationToName] => 04/28/2020
    [StockLocationToAccount] =>
    [TranDate] =>
    [TransferItem] => Array

Hope this might help.


Reply
04-29-2020, 07:23 AM,
#2
RE: stock transfer session incorrectly built?
You are quite right. I will try and fix it up tomorrow unless Paul gets to it first.

Tim
Reply
04-29-2020, 11:39 AM, (This post was last modified: 04-29-2020, 10:59 PM by TurboPT.)
#3
RE: stock transfer session incorrectly built?
I believe this may have been fixed by this commit back in October?
(definitely after HDeriauFF's 4.14.1 version, but is likely a minor change that could easily be applied nonetheless)
Reply
04-29-2020, 05:28 PM,
#4
RE: stock transfer session incorrectly built?
Ah, you are right. It's this particular commit that does it:

https://github.com/timschofield/webERP-s...965cd9c994

Thanks
Tim
Reply


Forum Jump:


Users browsing this thread: 1 Guest(s)