X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=gl%2Fgl_bank.php;h=2d220c4fd25fbca9bfa4a2b19eb580ba54cd1bec;hb=25379bdc9f10bda4aae4b37f04d16eb1bbe7e4f6;hp=0458b15213402f25fe6d3d400290075a8cf6ea03;hpb=af927d3e6f6fb02c6c3082e7cb9efea9c198d147;p=fa-stable.git diff --git a/gl/gl_bank.php b/gl/gl_bank.php index 0458b152..2d220c4f 100644 --- a/gl/gl_bank.php +++ b/gl/gl_bank.php @@ -22,26 +22,35 @@ include_once($path_to_root . "/includes/data_checks.inc"); include_once($path_to_root . "/gl/includes/ui/gl_bank_ui.inc"); include_once($path_to_root . "/gl/includes/gl_db.inc"); include_once($path_to_root . "/gl/includes/gl_ui.inc"); +include_once($path_to_root . "/admin/db/attachments_db.inc"); $js = ''; -if ($use_popup_windows) +if ($SysPrefs->use_popup_windows) $js .= get_js_open_window(800, 500); -if ($use_date_picker) +if (user_use_date_picker()) $js .= get_js_date_picker(); if (isset($_GET['NewPayment'])) { - $_SESSION['page_title'] = _("Bank Account Payment Entry"); - handle_new_order(ST_BANKPAYMENT); - + $_SESSION['page_title'] = _($help_context = "Bank Account Payment Entry"); + create_cart(ST_BANKPAYMENT, 0); } else if(isset($_GET['NewDeposit'])) { - $_SESSION['page_title'] = _("Bank Account Deposit Entry"); - handle_new_order(ST_BANKDEPOSIT); + $_SESSION['page_title'] = _($help_context = "Bank Account Deposit Entry"); + create_cart(ST_BANKDEPOSIT, 0); +} else if(isset($_GET['ModifyPayment'])) { + $_SESSION['page_title'] = _($help_context = "Modify Bank Account Entry")." #".$_GET['trans_no']; + create_cart(ST_BANKPAYMENT, $_GET['trans_no']); +} else if(isset($_GET['ModifyDeposit'])) { + $_SESSION['page_title'] = _($help_context = "Modify Bank Deposit Entry")." #".$_GET['trans_no']; + create_cart(ST_BANKDEPOSIT, $_GET['trans_no']); } page($_SESSION['page_title'], false, false, '', $js); //----------------------------------------------------------------------------------------------- check_db_has_bank_accounts(_("There are no bank accounts defined in the system.")); +if (isset($_GET['ModifyDeposit']) || isset($_GET['ModifyPayment'])) + check_is_editable($_SESSION['pay_items']->trans_type, $_SESSION['pay_items']->order_id); + //---------------------------------------------------------------------------------------- if (list_updated('PersonDetailID')) { $br = get_branch(get_post('PersonDetailID')); @@ -51,10 +60,15 @@ if (list_updated('PersonDetailID')) { //-------------------------------------------------------------------------------------------------- function line_start_focus() { - global $Ajax; - - $Ajax->activate('items_table'); - set_focus('_code_id_edit'); + global $Ajax; + + unset($_POST['amount']); + unset($_POST['dimension_id']); + unset($_POST['dimension2_id']); + unset($_POST['LineMemo']); + $Ajax->activate('items_table'); + $Ajax->activate('footer'); + set_focus('_code_id_edit'); } //----------------------------------------------------------------------------------------------- @@ -64,7 +78,25 @@ if (isset($_GET['AddedID'])) $trans_no = $_GET['AddedID']; $trans_type = ST_BANKPAYMENT; - display_notification_centered(_("Payment has been entered")); + display_notification_centered(sprintf(_("Payment %d has been entered"), $trans_no)); + + display_note(get_gl_view_str($trans_type, $trans_no, _("&View the GL Postings for this Payment"))); + + hyperlink_params($_SERVER['PHP_SELF'], _("Enter Another &Payment"), "NewPayment=yes"); + + hyperlink_params($_SERVER['PHP_SELF'], _("Enter A &Deposit"), "NewDeposit=yes"); + + hyperlink_params("$path_to_root/admin/attachments.php", _("Add an Attachment"), "filterType=$trans_type&trans_no=$trans_no"); + + display_footer_exit(); +} + +if (isset($_GET['UpdatedID'])) +{ + $trans_no = $_GET['UpdatedID']; + $trans_type = ST_BANKPAYMENT; + + display_notification_centered(sprintf(_("Payment %d has been modified"), $trans_no)); display_note(get_gl_view_str($trans_type, $trans_no, _("&View the GL Postings for this Payment"))); @@ -80,7 +112,7 @@ if (isset($_GET['AddedDep'])) $trans_no = $_GET['AddedDep']; $trans_type = ST_BANKDEPOSIT; - display_notification_centered(_("Deposit has been entered")); + display_notification_centered(sprintf(_("Deposit %d has been entered"), $trans_no)); display_note(get_gl_view_str($trans_type, $trans_no, _("View the GL Postings for this Deposit"))); @@ -90,32 +122,101 @@ if (isset($_GET['AddedDep'])) display_footer_exit(); } -if (isset($_POST['_date__changed'])) { - $Ajax->activate('_ex_rate'); +if (isset($_GET['UpdatedDep'])) +{ + $trans_no = $_GET['UpdatedDep']; + $trans_type = ST_BANKDEPOSIT; + + display_notification_centered(sprintf(_("Deposit %d has been modified"), $trans_no)); + + display_note(get_gl_view_str($trans_type, $trans_no, _("&View the GL Postings for this Deposit"))); + + hyperlink_params($_SERVER['PHP_SELF'], _("Enter Another &Deposit"), "NewDeposit=yes"); + + hyperlink_params($_SERVER['PHP_SELF'], _("Enter A &Payment"), "NewPayment=yes"); + + display_footer_exit(); } + //-------------------------------------------------------------------------------------------------- -function handle_new_order($type) +function create_cart($type, $trans_no) { + global $Refs; + if (isset($_SESSION['pay_items'])) { unset ($_SESSION['pay_items']); } - session_register("pay_items"); + $cart = new items_cart($type); + $cart->order_id = $trans_no; + + if ($trans_no) { + + $bank_trans = db_fetch(get_bank_trans($type, $trans_no)); + $_POST['bank_account'] = $bank_trans["bank_act"]; + $_POST['PayType'] = $bank_trans["person_type_id"]; + $cart->reference = $bank_trans["ref"]; + + if ($bank_trans["person_type_id"] == PT_CUSTOMER) + { + $trans = get_customer_trans($trans_no, $type); + $_POST['person_id'] = $trans["debtor_no"]; + $_POST['PersonDetailID'] = $trans["branch_code"]; + } + elseif ($bank_trans["person_type_id"] == PT_SUPPLIER) + { + $trans = get_supp_trans($trans_no, $type); + $_POST['person_id'] = $trans["supplier_id"]; + } + elseif ($bank_trans["person_type_id"] == PT_MISC) + $_POST['person_id'] = $bank_trans["person_id"]; + elseif ($bank_trans["person_type_id"] == PT_QUICKENTRY) + $_POST['person_id'] = $bank_trans["person_id"]; + else + $_POST['person_id'] = $bank_trans["person_id"]; + + $cart->memo_ = get_comments_string($type, $trans_no); + $cart->tran_date = sql2date($bank_trans['trans_date']); + + $cart->original_amount = $bank_trans['amount']; + $result = get_gl_trans($type, $trans_no); + if ($result) { + while ($row = db_fetch($result)) { + if (is_bank_account($row['account'])) { + // date exchange rate is currenly not stored in bank transaction, + // so we have to restore it from original gl amounts + $ex_rate = $bank_trans['amount']/$row['amount']; + } else { + $cart->add_gl_item( $row['account'], $row['dimension_id'], + $row['dimension2_id'], $row['amount'], $row['memo_']); + } + } + } + + // apply exchange rate + foreach($cart->gl_items as $line_no => $line) + $cart->gl_items[$line_no]->amount *= $ex_rate; + + } else { + $cart->reference = $Refs->get_next($cart->trans_type, null, $cart->tran_date); + $cart->tran_date = new_doc_date(); + if (!is_date_in_fiscalyear($cart->tran_date)) + $cart->tran_date = end_fiscalyear(); + } - $_SESSION['pay_items'] = new items_cart($type); + $_POST['memo_'] = $cart->memo_; + $_POST['ref'] = $cart->reference; + $_POST['date_'] = $cart->tran_date; - $_POST['date_'] = new_doc_date(); - if (!is_date_in_fiscalyear($_POST['date_'])) - $_POST['date_'] = end_fiscalyear(); - $_SESSION['pay_items']->tran_date = $_POST['date_']; + $_SESSION['pay_items'] = &$cart; } - //----------------------------------------------------------------------------------------------- -if (isset($_POST['Process'])) +function check_trans() { + global $Refs, $systypes_array; $input_error = 0; @@ -125,15 +226,33 @@ if (isset($_POST['Process'])) $input_error = 1; } - if (!$Refs->is_valid($_POST['ref'])) + if ($_SESSION['pay_items']->gl_items_total() == 0.0) { + display_error(_("The total bank amount cannot be 0.")); + set_focus('code_id'); + $input_error = 1; + } + + $limit = get_bank_account_limit($_POST['bank_account'], $_POST['date_']); + + $amnt_chg = -$_SESSION['pay_items']->gl_items_total()-$_SESSION['pay_items']->original_amount; + + if ($limit !== null && floatcmp($limit, -$amnt_chg) < 0) { - display_error( _("You must enter a reference.")); - set_focus('ref'); + display_error(sprintf(_("The total bank amount exceeds allowed limit (%s)."), price_format($limit-$_SESSION['pay_items']->original_amount))); + set_focus('code_id'); $input_error = 1; } - elseif (!is_new_reference($_POST['ref'], $_SESSION['pay_items']->trans_type)) + if ($trans = check_bank_account_history($amnt_chg, $_POST['bank_account'], $_POST['date_'])) { + + if (isset($trans['trans_no'])) { + display_error(sprintf(_("The bank transaction would result in exceed of authorized overdraft limit for transaction: %s #%s on %s."), + $systypes_array[$trans['type']], $trans['trans_no'], sql2date($trans['trans_date']))); + set_focus('amount'); + $input_error = 1; + } + } + if (!check_reference($_POST['ref'], $_SESSION['pay_items']->trans_type, $_SESSION['pay_items']->order_id)) { - display_error( _("The entered reference is already in use.")); set_focus('ref'); $input_error = 1; } @@ -145,23 +264,45 @@ if (isset($_POST['Process'])) } elseif (!is_date_in_fiscalyear($_POST['date_'])) { - display_error(_("The entered date is not in fiscal year.")); + display_error(_("The entered date is out of fiscal year or is closed for further data entry.")); set_focus('date_'); $input_error = 1; + } + + if (get_post('PayType')==PT_CUSTOMER && (!get_post('person_id') || !get_post('PersonDetailID'))) { + display_error(_("You have to select customer and customer branch.")); + set_focus('person_id'); + $input_error = 1; + } elseif (get_post('PayType')==PT_SUPPLIER && (!get_post('person_id'))) { + display_error(_("You have to select supplier.")); + set_focus('person_id'); + $input_error = 1; } + if (!db_has_currency_rates(get_bank_account_currency($_POST['bank_account']), $_POST['date_'], true)) + $input_error = 1; - if ($input_error == 1) - unset($_POST['Process']); + if (isset($_POST['settled_amount']) && in_array(get_post('PayType'), array(PT_SUPPLIER, PT_CUSTOMER)) && (input_num('settled_amount') <= 0)) { + display_error(_("Settled amount have to be positive number.")); + set_focus('person_id'); + $input_error = 1; + } + return $input_error; } -if (isset($_POST['Process'])) +if (isset($_POST['Process']) && !check_trans()) { + begin_transaction(); + + $_SESSION['pay_items'] = &$_SESSION['pay_items']; + $new = $_SESSION['pay_items']->order_id == 0; + + add_new_exchange_rate(get_bank_account_currency(get_post('bank_account')), get_post('date_'), input_num('_ex_rate')); - $trans = add_bank_transaction( - $_SESSION['pay_items']->trans_type, $_POST['bank_account'], + $trans = write_bank_transaction( + $_SESSION['pay_items']->trans_type, $_SESSION['pay_items']->order_id, $_POST['bank_account'], $_SESSION['pay_items'], $_POST['date_'], $_POST['PayType'], $_POST['person_id'], get_post('PersonDetailID'), - $_POST['ref'], $_POST['memo_']); + $_POST['ref'], $_POST['memo_'], true, input_num('settled_amount', null)); $trans_type = $trans[0]; $trans_no = $trans[1]; @@ -170,40 +311,35 @@ if (isset($_POST['Process'])) $_SESSION['pay_items']->clear_items(); unset($_SESSION['pay_items']); - meta_forward($_SERVER['PHP_SELF'], $trans_type==ST_BANKPAYMENT ? - "AddedID=$trans_no" : "AddedDep=$trans_no"); + commit_transaction(); -} /*end of process credit note */ + if ($new) + meta_forward($_SERVER['PHP_SELF'], $trans_type==ST_BANKPAYMENT ? + "AddedID=$trans_no" : "AddedDep=$trans_no"); + else + meta_forward($_SERVER['PHP_SELF'], $trans_type==ST_BANKPAYMENT ? + "UpdatedID=$trans_no" : "UpdatedDep=$trans_no"); + +} //----------------------------------------------------------------------------------------------- function check_item_data() { - //if (!check_num('amount', 0)) - //{ - // display_error( _("The amount entered is not a valid number or is less than zero.")); - // set_focus('amount'); - // return false; - //} - - if ($_POST['code_id'] == $_POST['bank_account']) + if (!check_num('amount', 0)) { - display_error( _("The source and destination accouts cannot be the same.")); - set_focus('code_id'); + display_error( _("The amount entered is not a valid number or is less than zero.")); + set_focus('amount'); + return false; + } + if (isset($_POST['_ex_rate']) && input_num('_ex_rate') <= 0) + { + display_error( _("The exchange rate cannot be zero or a negative number.")); + set_focus('_ex_rate'); return false; } - //if (is_bank_account($_POST['code_id'])) - //{ - // if ($_SESSION['pay_items']->trans_type == ST_BANKPAYMENT) - // display_error( _("You cannot make a payment to a bank account. Please use the transfer funds facility for this.")); - // else - // display_error( _("You cannot make a deposit from a bank account. Please use the transfer funds facility for this.")); - // set_focus('code_id'); - // return false; - //} - - return true; + return true; } //----------------------------------------------------------------------------------------------- @@ -213,8 +349,8 @@ function handle_update_item() $amount = ($_SESSION['pay_items']->trans_type==ST_BANKPAYMENT ? 1:-1) * input_num('amount'); if($_POST['UpdateItem'] != "" && check_item_data()) { - $_SESSION['pay_items']->update_gl_item($_POST['Index'], $_POST['dimension_id'], - $_POST['dimension2_id'], $amount , $_POST['LineMemo']); + $_SESSION['pay_items']->update_gl_item($_POST['Index'], $_POST['code_id'], + $_POST['dimension_id'], $_POST['dimension2_id'], $amount , $_POST['LineMemo']); } line_start_focus(); } @@ -250,7 +386,7 @@ if (isset($_POST['AddItem'])) if (isset($_POST['UpdateItem'])) handle_update_item(); -if (isset($_POST['CancelItemChanges'])) +if (isset($_POST['CancelItemChanges']) || isset($_POST['Index'])) line_start_focus(); if (isset($_POST['go'])) @@ -266,12 +402,12 @@ start_form(); display_bank_header($_SESSION['pay_items']); -start_table("$table_style2 width=90%", 10); +start_table(TABLESTYLE2, "width='90%'", 10); start_row(); echo ""; display_gl_items($_SESSION['pay_items']->trans_type==ST_BANKPAYMENT ? _("Payment Items"):_("Deposit Items"), $_SESSION['pay_items']); -gl_options_controls(); +gl_options_controls($_SESSION['pay_items']); echo ""; end_row(); end_table(1); @@ -286,4 +422,3 @@ end_form(); end_page(); -?>