MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
See the License here <http://www.gnu.org/licenses/gpl-3.0.html>.
***********************************************************************/
-function get_qoh_on_date($stock_id, $location=null, $date_=null, $exclude=0)
+function get_qoh_on_date($stock_id, $location=null, $date_=null)
{
if ($date_ == null)
- {
- $sql = "SELECT SUM(qty) FROM ".TB_PREF."stock_moves
- WHERE stock_id=".db_escape($stock_id);
$date_ = Today();
- $date = date2sql($date_);
- }
- else
- {
- $date = date2sql($date_);
- $sql = "SELECT SUM(qty) FROM ".TB_PREF."stock_moves
+
+ $date = date2sql($date_);
+
+ $sql = "SELECT SUM(qty) FROM ".TB_PREF."stock_moves
WHERE stock_id=".db_escape($stock_id)."
AND tran_date <= '$date'";
- }
-
+
if ($location != null)
$sql .= " AND loc_code = ".db_escape($location);
- $result = db_query($sql, "QOH calulcation failed");
+ $result = db_query($sql, "QOH calculation failed");
$myrow = db_fetch_row($result);
- if ($exclude > 0)
- {
- $sql = "SELECT SUM(qty) FROM ".TB_PREF."stock_moves
- WHERE stock_id=".db_escape($stock_id)
- ." AND type=".db_escape($exclude)
- ." AND tran_date = '$date'";
-
- $result = db_query($sql, "QOH calulcation failed");
- $myrow2 = db_fetch_row($result);
- if ($myrow2 !== false)
- $myrow[0] -= $myrow2[0];
- }
- return $myrow[0];
+ $qoh = $myrow[0];
+ return $qoh ? $qoh : 0;
+}
+
+/**
+* Check whether change in stock on date would not cause negative qoh in stock history.
+* Returns null on success or max. available quantity with respective date otherwise.
+* Running balance is checked on daily basis only, as we do not control time of transaction.
+*
+* $delta_qty - tested change in stock qty at $date.
+* $date - check date; when set to null checks all the stock history.
+**/
+
+function check_negative_stock($stock_id, $delta_qty, $location=null, $date=null)
+{
+
+ if ($delta_qty >= 0)
+ return null; // qty increese is always safe
+
+ if (!isset($date))
+ $date = Today();
+
+ $date = date2sql($date);
+
+ // check stock status on date
+ $sql = "SELECT SUM(qty) qty, '$date' tran_date FROM ".TB_PREF."stock_moves
+ WHERE stock_id=".db_escape($stock_id)."
+ AND tran_date <= '$date'";
+
+ if ($location)
+ $sql .= " AND loc_code = ".db_escape($location);
+
+ $result = db_query($sql, "QOH calculation failed");
+ $qos = db_fetch_assoc($result);
+
+ // check also all stock changes after the date to avoid negative stock in future
+ $sql = TB_PREF."stock_moves WHERE stock_id=".db_escape($stock_id) . " AND tran_date > '$date'";
+
+ if ($location)
+ $sql .= " AND loc_code=".db_escape($location);
+
+ $rt = running_total_sql($sql, 'qty', 'tran_date');
+
+ $sql = "SELECT {$qos['qty']}+total qty, tran_date FROM ($rt) stock_status ORDER by total, tran_date";
+ $history = db_query($sql, 'cannot check stock history');
+ $min_qos = db_fetch($history);
+
+ if ($min_qos && ($min_qos['qty'] < $qos['qty']))
+ $qos = $min_qos;
+
+ return -$delta_qty > $qos['qty'] ? $qos : null;
}
//--------------------------------------------------------------------------------------
$qty = 0;
$flag = 0;
- $negative_trans_id = 1;
+ $negative_trans_id = -1;
while ($myrow = db_fetch($result))
{
$flag = 0;
}
- return $negative_trans_id;
+ if ($flag == 1)
+ return $negative_trans_id;
+ else
+ return false;
}
//-------------------------------------------------------------------
function get_deliveries_from_trans($stock_id, $from)
{
// -ve qty is delivery either by ST_CUSTDELIVERY or inventory adjustment
- $sql = "SELECT SUM(-qty), SUM(-qty*standard_cost) FROM ".TB_PREF."stock_moves
- WHERE stock_id=".db_escape($stock_id)." AND qty < 0 AND
- trans_id>='$from' GROUP BY stock_id";
+ //Price for GRN and SUPPCREDIT and std_cost for other trans_types
+ $sql = "SELECT SUM(-qty), SUM(-qty*IF(type=".ST_SUPPRECEIVE." OR type=".ST_SUPPCREDIT.", price, standard_cost))
+ FROM ".TB_PREF."stock_moves
+ WHERE stock_id=".db_escape($stock_id)." AND qty < 0 AND
+ trans_id>='$from' GROUP BY stock_id";
$result = db_query($sql, "The deliveries could not be updated");
$row = db_fetch_row($result);
- $sql = "SELECT standard_cost FROM ".TB_PREF."stock_moves
- WHERE stock_id=".db_escape($stock_id)
- ." AND trans_id ='$from'";
- $result = db_query($sql, "The deliveries could not be updated");
- $cost = db_fetch_row($result);
+ $sql = "SELECT IF(type=".ST_SUPPRECEIVE." OR type=".ST_SUPPCREDIT.", price, standard_cost)
+ FROM ".TB_PREF."stock_moves
+ WHERE stock_id=".db_escape($stock_id)
+ ." AND trans_id ='$from'";
+ $result = db_query($sql, "The deliveries could not be updated");
+ $cost = db_fetch_row($result);
// Adjusting QOH valuation
$sql = "SELECT SUM(qty) FROM ".TB_PREF."stock_moves
$result = db_query($sql, "The deliveries could not be updated");
$qoh = db_fetch_row($result);
- $qty = $row[0] - $qoh[0]; //QOH prior to -ve stock is subtracted\r
+ $qty = $row[0] - $qoh[0]; //QOH prior to -ve stock is subtracted
$final_cost = $row[1] - $qoh[0]*$cost[0];
return array($qty,$final_cost);
return;
$from = last_negative_stock_trans_id($stock_id, $to);
- if ($from == false || $from == "")
+ if ($from == false || $from == -1)
return;
$row = get_deliveries_from_trans($stock_id, $from);
$dec = user_price_dec();
$old_cost = -round2($old_sales_cost-$old_purchase_cost,$dec);
$new_cost = -round2($new_sales_cost-$new_purchase_cost,$dec);
-
- $memo_ = _("Cost was ") . $old_cost. _(" changed to ") . $new_cost . _(" for item ")."'$stock_id'";
+
+ $memo_ = sprintf(_("Cost was %s changed to %s x quantity on hand for item '%s'"),
+ number_format2($old_cost, 2), number_format2($new_cost, 2), $stock_id);
add_gl_trans_std_cost(ST_COSTUPDATE, $update_no, $to, $stock_gl_code["cogs_account"],
$stock_gl_code["dimension_id"], $stock_gl_code["dimension2_id"], $memo_, $diff);
}
//-----------------------------------------------------------------------------------------
-\r
+
function handle_negative_inventory($stock_id, $quantity, $standard_cost, $date_)
{
- //If negative adjustment result in negative or zero inventory \r
+ //If negative adjustment result in negative or zero inventory
//then difference should be adjusted
$qoh = get_qoh_on_date($stock_id);
$id = get_next_trans_no(ST_JOURNAL);
$ref = $Refs->get_next(ST_JOURNAL);
- $diff = get_standard_cost($stock_id) - $standard_cost;
-
- if ($diff !=0)
+ $diff = round($qoh*get_standard_cost($stock_id) + $quantity*$standard_cost, user_price_dec());
+
+ if ($diff != 0)
{
$stock_gl_code = get_stock_gl_code($stock_id);
$memo = _("Zero/negative inventory handling");
add_gl_trans_std_cost(ST_JOURNAL, $id, $date_,
$stock_gl_code["inventory_account"],
$stock_gl_code['dimension_id'], $stock_gl_code['dimension2_id'], $memo,
- -$qoh * $diff);
+ -$diff);
//GL Posting to inventory adjustment account
add_gl_trans_std_cost(ST_JOURNAL, $id, $date_,
$stock_gl_code["adjustment_account"],
$stock_gl_code['dimension_id'], $stock_gl_code['dimension2_id'], $memo,
- $qoh * $diff);
-
+ $diff);
+
add_audit_trail(ST_JOURNAL, $id, $date_);
add_comments(ST_JOURNAL, $id, $date_, $memo);
$Refs->save(ST_JOURNAL, $id, $ref);
function void_stock_move($type, $type_no)
{
- \r
$sql = "SELECT * from ".TB_PREF."stock_moves WHERE type=".db_escape($type)." AND trans_no=".db_escape($type_no);
$result = db_query($sql, "Could not void stock moves");
while ($row = db_fetch($result))
{
//Skip cost averaging of service items
if (is_inventory_item($row["stock_id"]))
- {
+ {
// The cost has to be adjusted.
- // Transaction rates are stored either as price or standard_cost depending on types\r
- $types = array(ST_SUPPCREDIT);
- if (in_array($type,$types))
- $trans_rate = $row["price"];
+ // Transaction rates are stored either as price or standard_cost depending on types
+ $types = array(ST_SUPPCREDIT, ST_SUPPRECEIVE);
+ if (in_array($type, $types))
+ $unit_cost = $row["price"];
else
- $trans_rate = $row["standard_cost"];
+ $unit_cost = $row["standard_cost"];
- update_average_material_cost(0, $row["stock_id"],
- $trans_rate, -$row["qty"], sql2date($row["tran_date"]));
+ update_average_material_cost($row["person_id"], $row["stock_id"],
+ $unit_cost, -$row["qty"], sql2date($row["tran_date"]));
}
-
- //Post stock move for service items also
- add_stock_move($type, $row["stock_id"], $type_no, $row["loc_code"],
- sql2date($row["tran_date"]), $row["reference"], -$row["qty"]
- , $row["standard_cost"], $row["person_id"], $row["visible"],
- $row["price"], $row["discount_percent"]);
+
}
+ $sql = "DELETE FROM ".TB_PREF."stock_moves WHERE type=".db_escape($type)
+ ." AND trans_no=".db_escape($type_no);
+ db_query($sql, "The stock movement cannot be delated");
}
//--------------------------------------------------------------------------------------------------
//--------------------------------------------------------------------------------------------------
-?>
\ No newline at end of file
+?>