X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=sales%2Fcustomer_payments.php;h=e1b672c9ae37cd69c5b2e563307faf5296daeff0;hb=3f3cfa578adbc2b71f4fd9c8d0f5536d26af43d8;hp=565d9f95538b285a45bdd58b4453c798b83e6258;hpb=5f06887dedd4d14701864fb72994d7e20352086d;p=fa-stable.git

diff --git a/sales/customer_payments.php b/sales/customer_payments.php
index 565d9f95..e1b672c9 100644
--- a/sales/customer_payments.php
+++ b/sales/customer_payments.php
@@ -39,6 +39,35 @@ check_db_has_customers(_("There are no customers defined in the system."));
 check_db_has_bank_accounts(_("There are no bank accounts defined in the system."));
 
 //----------------------------------------------------------------------------------------
+if (isset($_GET['customer_id']))
+{
+	$_POST['customer_id'] = $_GET['customer_id'];
+}
+
+if (!isset($_POST['bank_account']))
+{ // first page call
+	$_SESSION['alloc'] = new allocation(ST_CUSTPAYMENT,0);
+
+	if (isset($_GET['SInvoice'])) {
+		//  get date and supplier
+		$inv = get_customer_trans($_GET['SInvoice'], ST_SALESINVOICE);
+		if($inv) {
+			$_POST['customer_id'] = $inv['debtor_no'];
+			$_POST['DateBanked'] = sql2date($inv['tran_date']);
+			foreach($_SESSION['alloc']->allocs as $line => $trans) {
+				if ($trans->type == ST_SALESINVOICE && $trans->type_no == $_GET['SInvoice']) {
+					$_POST['amount'] =
+						$_SESSION['alloc']->amount = price_format($_SESSION['alloc']->allocs[$line]->amount);
+					$_SESSION['alloc']->allocs[$line]->current_allocated =
+						$_SESSION['alloc']->allocs[$line]->amount;
+					break;
+				}
+			}
+			unset($inv);
+		} else
+			display_error(_("Invalid sales invoice number."));
+	}
+}
 
 if (list_updated('BranchID')) {
 	// when branch is selected via external editor also customer can change
@@ -56,6 +85,7 @@ if (!isset($_POST['DateBanked'])) {
 	}
 }
 
+
 if (isset($_GET['AddedID'])) {
 	$payment_no = $_GET['AddedID'];
 
@@ -65,6 +95,21 @@ if (isset($_GET['AddedID'])) {
 
 	display_note(get_gl_view_str(ST_CUSTPAYMENT, $payment_no, _("&View the GL Journal Entries for this Customer Payment")));
 
+//	hyperlink_params($path_to_root . "/sales/allocations/customer_allocate.php", _("&Allocate this Customer Payment"), "trans_no=$payment_no&trans_type=12");
+
+	hyperlink_no_params($path_to_root . "/sales/customer_payments.php", _("Enter Another &Customer Payment"));
+	
+	display_footer_exit();
+}
+elseif (isset($_GET['UpdatedID'])) {
+	$payment_no = $_GET['UpdatedID'];
+
+	display_notification_centered(_("The customer payment has been successfully updated."));
+
+	submenu_print(_("&Print This Receipt"), ST_CUSTPAYMENT, $payment_no."-".ST_CUSTPAYMENT, 'prtopt');
+
+	display_note(get_gl_view_str(ST_CUSTPAYMENT, $payment_no, _("&View the GL Journal Entries for this Customer Payment")));
+
 //	hyperlink_params($path_to_root . "/sales/allocations/customer_allocate.php", _("&Allocate this Customer Payment"), "trans_no=$payment_no&trans_type=12");
 
 	hyperlink_no_params($path_to_root . "/sales/customer_payments.php", _("Enter Another &Customer Payment"));
@@ -78,12 +123,26 @@ function can_process()
 {
 	global $Refs;
 
+	if (!get_post('customer_id'))
+	{
+		display_error(_("There is no customer selected."));
+		set_focus('customer_id');
+		return false;
+	} 
+	
+	if (!get_post('BranchID')) 
+	{
+		display_error(_("This customer has no branch defined."));
+		set_focus('BranchID');
+		return false;
+	} 
+	
 	if (!isset($_POST['DateBanked']) || !is_date($_POST['DateBanked'])) {
 		display_error(_("The entered date is invalid. Please enter a valid date for the payment."));
 		set_focus('DateBanked');
 		return false;
 	} elseif (!is_date_in_fiscalyear($_POST['DateBanked'])) {
-		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('DateBanked');
 		return false;
 	}
@@ -94,11 +153,19 @@ function can_process()
 		return false;
 	}
 
-	if (!is_new_reference($_POST['ref'], ST_CUSTPAYMENT)) {
+	//Chaitanya : 13-OCT-2011 - To support Edit feature
+	if (isset($_POST['trans_no']) && $_POST['trans_no'] == 0 && (!is_new_reference($_POST['ref'], ST_CUSTPAYMENT))) {
 		display_error(_("The entered reference is already in use."));
 		set_focus('ref');
 		return false;
 	}
+	//Avoid duplicate reference while modifying
+	elseif ($_POST['ref'] != $_POST['old_ref'] && !is_new_reference($_POST['ref'], ST_CUSTPAYMENT))
+	{
+		display_error( _("The entered reference is already in use."));
+		set_focus('ref');
+		return false;
+	}
 
 	if (!check_num('amount', 0)) {
 		display_error(_("The entered amount is invalid or negative and cannot be processed."));
@@ -112,7 +179,7 @@ function can_process()
 		return false;
 	}
 	if (isset($_POST['charge']) && input_num('charge') > 0) {
-		$charge_acct = get_company_pref('bank_charge_act');
+		$charge_acct = get_bank_charge_account($_POST['bank_account']);
 		if (get_gl_account($charge_acct) == false) {
 			display_error(_("The Bank Charge Account has not been set in System and General GL Setup."));
 			set_focus('charge');
@@ -138,11 +205,14 @@ function can_process()
 		return false;
 	}
 
-	if ((input_num('amount') - input_num('discount') <= 0)) {
+	//if ((input_num('amount') - input_num('discount') <= 0)) {
+	if (input_num('amount') <= 0) {
 		display_error(_("The balance of the amount and discout is zero or negative. Please enter valid amounts."));
 		set_focus('discount');
 		return false;
 	}
+	if (!db_has_currency_rates(get_customer_currency($_POST['customer_id']), $_POST['DateBanked'], true))
+		return false;
 
 	$_SESSION['alloc']->amount = input_num('amount');
 
@@ -168,10 +238,19 @@ if (isset($_POST['_customer_id_button'])) {
 if (isset($_POST['_DateBanked_changed'])) {
   $Ajax->activate('_ex_rate');
 }
-if (list_updated('customer_id') || list_updated('bank_account')) {
-  $_SESSION['alloc']->read();
-  $Ajax->activate('alloc_tbl');
+
+//Chaitanya : 13-OCT-2011 - To support Edit feature
+if (isset($_POST['ref']) && $_SESSION['alloc']->trans_no == 0) // added by Joe to fix the browser back button
+{
+	$tno = get_customer_trans_from_ref(ST_CUSTPAYMENT, $_POST['ref']);
+	if ($tno != false)
+	{
+		display_error( _("The entered reference is already in use."));
+		display_footer_exit();
+	}
 }
+$new = $_SESSION['alloc']->trans_no == 0;
+
 //----------------------------------------------------------------------------------------------
 
 if (isset($_POST['AddPaymentItem'])) {
@@ -186,40 +265,100 @@ if (isset($_POST['AddPaymentItem'])) {
 
 	new_doc_date($_POST['DateBanked']);
 
-	$payment_no = write_customer_payment(0, $_POST['customer_id'], $_POST['BranchID'],
+	//Chaitanya : 13-OCT-2011 - To support Edit feature
+	$payment_no = write_customer_payment($_SESSION['alloc']->trans_no, $_POST['customer_id'], $_POST['BranchID'],
 		$_POST['bank_account'], $_POST['DateBanked'], $_POST['ref'],
 		input_num('amount'), input_num('discount'), $_POST['memo_'], $rate, input_num('charge'));
 
 	$_SESSION['alloc']->trans_no = $payment_no;
 	$_SESSION['alloc']->write();
+	
+	unset($_POST);
+	unset($_SESSION);
 
-	meta_forward($_SERVER['PHP_SELF'], "AddedID=$payment_no");
+	//Chaitanya : 13-OCT-2011 - To support Edit feature
+	//meta_forward($_SERVER['PHP_SELF'], "AddedID=$payment_no");
+	meta_forward($_SERVER['PHP_SELF'], $new ? "AddedID=$payment_no" : "UpdatedID=$payment_no");
 }
 
 //----------------------------------------------------------------------------------------------
 
 function read_customer_data()
 {
-	global $Refs;
+	global $Refs, $new;
 
 	$myrow = get_customer_habit($_POST['customer_id']);
 
 	$_POST['HoldAccount'] = $myrow["dissallow_invoices"];
 	$_POST['pymt_discount'] = $myrow["pymt_discount"];
-	$_POST['ref'] = $Refs->get_next(ST_CUSTPAYMENT);
+	//Chaitanya : 13-OCT-2011 - To support Edit feature
+	//If page is called first time and New entry fetch the nex reference number
+	if ($new && !isset($_POST['charge'])) 
+		$_POST['ref'] = $Refs->get_next(ST_CUSTPAYMENT);
+}
+
+//----------------------------------------------------------------------------------------------
+$new = 1;
+$old_ref = 0;
+
+//Chaitanya : 13-OCT-2011 - To support Edit feature
+if (isset($_GET['trans_no']) && $_GET['trans_no'] > 0 )
+	$_POST['trans_no'] = $_GET['trans_no'];
+//Read data
+if (isset($_POST['trans_no']) && $_POST['trans_no'] > 0 )
+{	
+	$new = 0;
+	$myrow = get_customer_trans($_POST['trans_no'], ST_CUSTPAYMENT);
+	$_POST['customer_id'] = $myrow["debtor_no"];
+	$_POST['customer_name'] = $myrow["DebtorName"];
+	$_POST['BranchID'] = $myrow["branch_code"];
+	$_POST['bank_account'] = $myrow["bank_act"];
+	$_POST['ref'] =  $myrow["reference"];
+	$old_ref = $myrow["reference"];
+	//$_POST['charge'] =  $myrow[""];
+	$_POST['DateBanked'] =  sql2date($myrow['tran_date']);
+	$_POST["amount"] = price_format($myrow['Total'] - $myrow['ov_discount']);
+	$_POST["discount"] = price_format($myrow['ov_discount']);
+	$_POST["memo_"] = get_comments_string(ST_CUSTPAYMENT,$_POST['trans_no']);
 }
+else
+	$_POST['trans_no'] = 0;
 
 //----------------------------------------------------------------------------------------------
 
 start_form();
 
-	start_outer_table("$table_style2 width=60%", 5);
+	hidden('trans_no', $_POST['trans_no']);
+	hidden('old_ref', $old_ref);
+
+	start_outer_table(TABLESTYLE2, "width=60%", 5);
 	table_section(1);
 
-	customer_list_row(_("From Customer:"), 'customer_id', null, false, true);
-	if (!isset($_POST['bank_account'])) // first page call
-		  $_SESSION['alloc'] = new allocation(ST_CUSTPAYMENT,0);
+	if ($new)
+		customer_list_row(_("From Customer:"), 'customer_id', null, false, true);
+	else {
+		label_cells(_("From Customer:"), $_POST['customer_name'], "class='label'");
+		hidden('customer_id', $_POST['customer_id']);
+	}
+
+	if (list_updated('customer_id') || ($new && list_updated('bank_account'))) {
+		$_SESSION['alloc']->read();
+		$_POST['memo_'] = $_POST['amount'] = $_POST['discount'] = '';
+		$Ajax->activate('alloc_tbl');
+	}
 
+	if (!isset($_POST['charge'])) // first page call
+	{
+		//Prepare allocation cart 
+		if (isset($_POST['trans_no']) && $_POST['trans_no'] > 0 )
+			$_SESSION['alloc'] = new allocation(ST_CUSTPAYMENT,$_POST['trans_no']);
+		else
+		{
+			$_SESSION['alloc'] = new allocation(ST_CUSTPAYMENT,0);
+			$Ajax->activate('alloc_tbl');
+		}
+	}
+	
 	if (db_customer_has_branches($_POST['customer_id'])) {
 		customer_branches_list_row(_("Branch:"), $_POST['customer_id'], 'BranchID', null, false, true, true);
 	} else {
@@ -229,60 +368,60 @@ start_form();
 	read_customer_data();
 
 	set_global_customer($_POST['customer_id']);
-	if (isset($_POST['HoldAccount']) && $_POST['HoldAccount'] != 0)	{
-		end_outer_table();
-		display_error(_("This customer account is on hold."));
-	} else {
-		$display_discount_percent = percent_format($_POST['pymt_discount']*100) . "%";
-
-		table_section(2);
-
-		bank_accounts_list_row(_("Into Bank Account:"), 'bank_account', null, true);
-
+	if (isset($_POST['HoldAccount']) && $_POST['HoldAccount'] != 0)	
+		display_warning(_("This customer account is on hold."));
+	$display_discount_percent = percent_format($_POST['pymt_discount']*100) . "%";
 
-		text_row(_("Reference:"), 'ref', null, 20, 40);
+	table_section(2);
+	if (!list_updated('bank_account'))
+		$_POST['bank_account'] = get_default_customer_bank_account($_POST['customer_id']);	
 
-		table_section(3);
+	//Chaitanya : 13-OCT-2011 - Is AJAX call really needed ???
+	//bank_accounts_list_row(_("Into Bank Account:"), 'bank_account', null, true);
+	bank_accounts_list_row(_("Into Bank Account:"), 'bank_account', null, false);
+	text_row(_("Reference:"), 'ref', null, 20, 40);
 
-		date_row(_("Date of Deposit:"), 'DateBanked', '', true, 0, 0, 0, null, true);
+	table_section(3);
 
+	date_row(_("Date of Deposit:"), 'DateBanked', '', true, 0, 0, 0, null, true);
 
+	$comp_currency = get_company_currency();
+	$cust_currency = get_customer_currency($_POST['customer_id']);
+	$bank_currency = get_bank_account_currency($_POST['bank_account']);
 
-		$comp_currency = get_company_currency();
-		$cust_currency = get_customer_currency($_POST['customer_id']);
-		$bank_currency = get_bank_account_currency($_POST['bank_account']);
-
-		if ($cust_currency != $bank_currency) {
-			exchange_rate_display($bank_currency, $cust_currency, $_POST['DateBanked'], ($bank_currency == $comp_currency));
-		}
+	if ($cust_currency != $bank_currency) {
+		exchange_rate_display($bank_currency, $cust_currency, $_POST['DateBanked'], ($bank_currency == $comp_currency));
+	}
 
-		amount_row(_("Bank Charge:"), 'charge');
+	amount_row(_("Bank Charge:"), 'charge');
 
-		end_outer_table(1);
+	end_outer_table(1);
 
-		if ($cust_currency == $bank_currency) {
-	  		div_start('alloc_tbl');
-			show_allocatable(false);
-			div_end();
-		}
+	if ($cust_currency == $bank_currency) {
+		div_start('alloc_tbl');
+		show_allocatable(false);
+		div_end();
+	}
 
-		start_table("$table_style width=60%");
+	start_table(TABLESTYLE, "width=60%");
 
-		label_row(_("Customer prompt payment discount :"), $display_discount_percent);
-		amount_row(_("Amount of Discount:"), 'discount');
+	label_row(_("Customer prompt payment discount :"), $display_discount_percent);
+	amount_row(_("Amount of Discount:"), 'discount');
 
-		amount_row(_("Amount:"), 'amount');
+	amount_row(_("Amount:"), 'amount');
 
-		textarea_row(_("Memo:"), 'memo_', null, 22, 4);
-		end_table(1);
+	textarea_row(_("Memo:"), 'memo_', null, 22, 4);
+	end_table(1);
 
-		if ($cust_currency != $bank_currency)
-			display_note(_("Amount and discount are in customer's currency."));
+	if ($cust_currency != $bank_currency)
+		display_note(_("Amount and discount are in customer's currency."));
 
-		br();
+	br();
 
+	if (isset($_POST['trans_no']) && $_POST['trans_no'] > 0 )
+		submit_center('AddPaymentItem', _("Update Payment"), true, '', 'default');
+	else
 		submit_center('AddPaymentItem', _("Add Payment"), true, '', 'default');
-	}
 
 	br();