X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=gl%2Fmanage%2Fgl_accounts.php;h=718be9bd872f692f7b257154933e348a60c8fe65;hb=56631aa65f39f4f2471f1e31350f5b6c912cabbf;hp=9ff0fa7a73c906d28f88edbf0e81cf70221c0d6f;hpb=9e8f7958e937c24a1b0ca859bbca8455e0b41404;p=fa-stable.git

diff --git a/gl/manage/gl_accounts.php b/gl/manage/gl_accounts.php
index 9ff0fa7a..718be9bd 100644
--- a/gl/manage/gl_accounts.php
+++ b/gl/manage/gl_accounts.php
@@ -1,7 +1,16 @@
 <?php
-
-$page_security = 10;
-$path_to_root="../..";
+/**********************************************************************
+    Copyright (C) FrontAccounting, LLC.
+	Released under the terms of the GNU General Public License, GPL, 
+	as published by the Free Software Foundation, either version 3 
+	of the License, or (at your option) any later version.
+    This program is distributed in the hope that it will be useful,
+    but WITHOUT ANY WARRANTY; without even the implied warranty of
+    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  
+    See the License here <http://www.gnu.org/licenses/gpl-3.0.html>.
+***********************************************************************/
+$page_security = 'SA_GLACCOUNT';
+$path_to_root = "../..";
 include($path_to_root . "/includes/session.inc");
 
 page(_("Chart of Accounts"));
@@ -14,9 +23,10 @@ check_db_has_gl_account_groups(_("There are no account groups defined. Please de
 
 //-------------------------------------------------------------------------------------
 
-if (isset($_POST['Select'])) 
+if (isset($_POST['_AccountList_update'])) 
 {
 	$_POST['selected_account'] = $_POST['AccountList'];
+	unset($_POST['account_code']);
 }
 
 if (isset($_POST['selected_account']))
@@ -41,25 +51,45 @@ if (isset($_POST['add']) || isset($_POST['update']))
 	{
 		$input_error = 1;
 		display_error( _("The account code must be entered."));
+		set_focus('account_code');
 	} 
 	elseif (strlen($_POST['account_name']) == 0) 
 	{
 		$input_error = 1;
 		display_error( _("The account name cannot be empty."));
+		set_focus('account_name');
 	} 
-	elseif (!is_numeric($_POST['account_code'])) 
+	elseif (!$accounts_alpha && !is_numeric($_POST['account_code'])) 
 	{
-		$input_error = 1;
-		display_error( _("The account code must be numeric."));
+	    $input_error = 1;
+	    display_error( _("The account code must be numeric."));
+		set_focus('account_code');
 	}
 
 	if ($input_error != 1)
 	{
-    	if ($selected_account)
-    		update_gl_account($_POST['account_code'], $_POST['account_name'], $_POST['account_type'], $_POST['account_code2'], $_POST['tax_code']);    		
-    	else
-    		add_gl_account($_POST['account_code'], $_POST['account_name'], $_POST['account_type'], $_POST['account_code2'], $_POST['tax_code']);
-		meta_forward($_SERVER['PHP_SELF']);    	
+		if ($accounts_alpha == 2)
+			$_POST['account_code'] = strtoupper($_POST['account_code']);
+    	if ($selected_account) 
+		{
+    		if (update_gl_account($_POST['account_code'], $_POST['account_name'], 
+				$_POST['account_type'], $_POST['account_code2'])) {
+				update_record_status($_POST['account_code'], $_POST['inactive'],
+					'chart_master', 'account_code');
+				$Ajax->activate('account_code'); // in case of status change
+				display_notification(_("Account data has been updated."));
+			}
+		}
+    	else 
+		{
+    		if (add_gl_account($_POST['account_code'], $_POST['account_name'], 
+				$_POST['account_type'], $_POST['account_code2']))
+				{
+					display_notification(_("New account has been added."));
+					$selected_account = $_POST['AccountList'] = $_POST['account_code'];
+				}
+		}
+		$Ajax->activate('_page_body');
 	}
 } 
 
@@ -69,7 +99,7 @@ function can_delete($selected_account)
 {
 	if ($selected_account == "")
 		return false;
-	$sql= "SELECT COUNT(*) FROM ".TB_PREF."gl_trans WHERE account=$selected_account";
+	$sql= "SELECT COUNT(*) FROM ".TB_PREF."gl_trans WHERE account='$selected_account'";
 	$result = db_query($sql,"Couldn't test for existing transactions");
 
 	$myrow = db_fetch_row($result);
@@ -79,23 +109,18 @@ function can_delete($selected_account)
 		return false;
 	}
 
-	$sql= "SELECT COUNT(*) FROM ".TB_PREF."company WHERE debtors_act=$selected_account 
-		OR pyt_discount_act=$selected_account 
-		OR creditors_act=$selected_account 
-		OR grn_act=$selected_account 
-		OR exchange_diff_act=$selected_account 
-		OR purch_exchange_diff_act=$selected_account 
-		OR retained_earnings_act=$selected_account
-		OR freight_act=$selected_account
-		OR default_sales_act=$selected_account 
-		OR default_sales_discount_act=$selected_account
-		OR default_prompt_payment_act=$selected_account
-		OR default_inventory_act=$selected_account
-		OR default_cogs_act=$selected_account
-		OR default_adj_act=$selected_account
-		OR default_inv_sales_act=$selected_account
-		OR default_assembly_act=$selected_account
-		OR payroll_act=$selected_account";
+	$sql= "SELECT COUNT(*) FROM ".TB_PREF."company WHERE debtors_act='$selected_account' 
+		OR pyt_discount_act='$selected_account' 
+		OR creditors_act='$selected_account' 
+		OR freight_act='$selected_account'
+		OR default_sales_act='$selected_account' 
+		OR default_sales_discount_act='$selected_account'
+		OR default_prompt_payment_act='$selected_account'
+		OR default_inventory_act='$selected_account'
+		OR default_cogs_act='$selected_account'
+		OR default_adj_act='$selected_account'
+		OR default_inv_sales_act='$selected_account'
+		OR default_assembly_act='$selected_account'";
 	$result = db_query($sql,"Couldn't test for default company GL codes");
 
 	$myrow = db_fetch_row($result);
@@ -105,7 +130,7 @@ function can_delete($selected_account)
 		return false;
 	}
 	
-	$sql= "SELECT COUNT(*) FROM ".TB_PREF."bank_accounts WHERE account_code=$selected_account";
+	$sql= "SELECT COUNT(*) FROM ".TB_PREF."bank_accounts WHERE account_code='$selected_account'";
 	$result = db_query($sql,"Couldn't test for bank accounts");
 
 	$myrow = db_fetch_row($result);
@@ -116,10 +141,10 @@ function can_delete($selected_account)
 	}	
 
 	$sql= "SELECT COUNT(*) FROM ".TB_PREF."stock_master WHERE 
-		inventory_account=$selected_account 
-		OR cogs_account=$selected_account
-		OR adjustment_account=$selected_account 
-		OR sales_account=$selected_account";
+		inventory_account='$selected_account' 
+		OR cogs_account='$selected_account'
+		OR adjustment_account='$selected_account' 
+		OR sales_account='$selected_account'";
 	$result = db_query($sql,"Couldn't test for existing stock GL codes");
 
 	$myrow = db_fetch_row($result);
@@ -129,7 +154,7 @@ function can_delete($selected_account)
 		return false;
 	}	
 	
-	$sql= "SELECT COUNT(*) FROM ".TB_PREF."tax_types WHERE sales_gl_code=$selected_account OR purchasing_gl_code=$selected_account";
+	$sql= "SELECT COUNT(*) FROM ".TB_PREF."tax_types WHERE sales_gl_code='$selected_account' OR purchasing_gl_code='$selected_account'";
 	$result = db_query($sql,"Couldn't test for existing tax GL codes");
 
 	$myrow = db_fetch_row($result);
@@ -140,10 +165,10 @@ function can_delete($selected_account)
 	}	
 	
 	$sql= "SELECT COUNT(*) FROM ".TB_PREF."cust_branch WHERE 
-		sales_account=$selected_account 
-		OR sales_discount_account=$selected_account
-		OR receivables_account=$selected_account
-		OR payment_discount_account=$selected_account";
+		sales_account='$selected_account' 
+		OR sales_discount_account='$selected_account'
+		OR receivables_account='$selected_account'
+		OR payment_discount_account='$selected_account'";
 	$result = db_query($sql,"Couldn't test for existing cust branch GL codes");
 
 	$myrow = db_fetch_row($result);
@@ -154,9 +179,9 @@ function can_delete($selected_account)
 	}		
 	
 	$sql= "SELECT COUNT(*) FROM ".TB_PREF."suppliers WHERE 
-		purchase_account=$selected_account 
-		OR payment_discount_account=$selected_account 
-		OR payable_account=$selected_account";
+		purchase_account='$selected_account' 
+		OR payment_discount_account='$selected_account' 
+		OR payable_account='$selected_account'";
 	$result = db_query($sql,"Couldn't test for existing suppliers GL codes");
 
 	$myrow = db_fetch_row($result);
@@ -166,6 +191,17 @@ function can_delete($selected_account)
 		return false;
 	}									
 	
+	$sql= "SELECT COUNT(*) FROM ".TB_PREF."quick_entry_lines WHERE 
+		dest_id='$selected_account' AND UPPER(LEFT(action, 1)) <> 'T'";
+	$result = db_query($sql,"Couldn't test for existing suppliers GL codes");
+
+	$myrow = db_fetch_row($result);
+	if ($myrow[0] > 0) 
+	{
+		display_error(_("Cannot delete this account because it is used by one or more Quick Entry Lines."));
+		return false;
+	}									
+
 	return true;
 }
 
@@ -177,7 +213,10 @@ if (isset($_POST['delete']))
 	if (can_delete($selected_account))
 	{
 		delete_gl_account($selected_account);
-		meta_forward($_SERVER['PHP_SELF']);		
+		$selected_account = $_POST['AccountList'] = '';
+		display_notification(_("Selected account has been deleted"));
+		unset($_POST['account_code']);
+		$Ajax->activate('_page_body');
 	}
 } 
 
@@ -187,17 +226,20 @@ start_form();
 
 if (db_has_gl_accounts()) 
 {
-	echo "<center>";
-    echo _("Select an Account:") . "&nbsp;";
-    gl_all_accounts_list('AccountList', null);
-    echo "&nbsp;";
-    submit('Select', _("Edit Account"));
-    echo "</center>";
-} 
+	start_table("class = 'tablestyle_noborder'");
+	start_row();
+    gl_all_accounts_list_cells(null, 'AccountList', null, false, false,
+		_('New account'), true, check_value('show_inactive'));
+	check_cells(_("Show inactive:"), 'show_inactive', null, true);
+	end_row();
+	end_table();
+	if (get_post('_show_inactive_update')) {
+		$Ajax->activate('AccountList');
+		set_focus('AccountList');
+	}
+}
 	
-hyperlink_no_params($_SERVER['PHP_SELF'], _("New Account"));
 br(1);
-
 start_table($table_style2);
 
 if ($selected_account != "") 
@@ -209,15 +251,20 @@ if ($selected_account != "")
 	$_POST['account_code2'] = $myrow["account_code2"];
 	$_POST['account_name']	= $myrow["account_name"];
 	$_POST['account_type'] = $myrow["account_type"];
-	$_POST['tax_code'] = $myrow["tax_code"];
+ 	$_POST['inactive'] = $myrow["inactive"];
 
 	hidden('account_code', $_POST['account_code']);
-	hidden('selected_account', $_POST['selected_account']);
+	hidden('selected_account', $selected_account);
 		
 	label_row(_("Account Code:"), $_POST['account_code']);
 } 
-else 
+else
 {
+	if (!isset($_POST['account_code'])) {
+		$_POST['account_code'] = $_POST['account_code2'] = '';
+		$_POST['account_name']	= $_POST['account_type'] = '';
+ 		$_POST['inactive'] = 0;
+	}
 	text_row_ex(_("Account Code:"), 'account_code', 11);
 }
 
@@ -227,20 +274,18 @@ text_row_ex(_("Account Name:"), 'account_name', 60);
 
 gl_account_types_list_row(_("Account Group:"), 'account_type', null);
 
-tax_types_list_row(_("Tax Type:"), 'tax_code', null, true, _('No Tax'));
-
+record_status_list_row(_("Account status:"), 'inactive');
 end_table(1);
 
 if ($selected_account == "") 
 {
-	submit_center('add', _("Add Account"));
+	submit_center('add', _("Add Account"), true, '', 'default');
 } 
 else 
 {
-    submit_center_first('update', _("Update Account"));
-    submit_center_last('delete', _("Delete account"));
+    submit_center_first('update', _("Update Account"), '', 'default');
+    submit_center_last('delete', _("Delete account"), '',true);
 }
-
 end_form();
 
 end_page();