X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=gl%2Fmanage%2Fgl_account_types.php;h=04f80c62305784dd4b954488eef35fb5792ee112;hb=6ae59a0b8741ce89f30b64c2d0f376835eb0a778;hp=3cef7a07540acdc4e051ae30c8c8665eae737ae3;hpb=46c5f7a65a7659a44ae8254c63152074363d3987;p=fa-stable.git

diff --git a/gl/manage/gl_account_types.php b/gl/manage/gl_account_types.php
index 3cef7a07..04f80c62 100644
--- a/gl/manage/gl_account_types.php
+++ b/gl/manage/gl_account_types.php
@@ -22,7 +22,7 @@ include($path_to_root . "/includes/ui.inc");
 simple_page_mode(false);
 //-----------------------------------------------------------------------------------
 
-function can_process() 
+function can_process($selected_id) 
 {
 	if (strlen(trim($_POST['id'])) == 0) 
 	{
@@ -36,6 +36,13 @@ function can_process()
 		set_focus('name');
 		return false;
 	}
+	$type = get_account_type(trim($_POST['id']));
+	if ($type && ($type['id'] != $selected_id)) 
+	{
+		display_error( _("This account group id is already in use."));
+		set_focus('id');
+		return false;
+	}
 
 	//if (strcmp($_POST['id'], $_POST['parent']) == 0) 
 	if ($_POST['id'] === $_POST['parent']) 
@@ -52,7 +59,7 @@ function can_process()
 if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM') 
 {
 
-	if (can_process()) 
+	if (can_process($selected_id)) 
 	{
 
     	if ($selected_id != "") 
@@ -72,19 +79,18 @@ if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM')
 
 //-----------------------------------------------------------------------------------
 
-function can_delete($selected_id)
+function can_delete($type)
 {
-	if ($selected_id == "")
+	if ($type == "")
 		return false;
-	$type = db_escape($selected_id);
 
-	if (key_in_foreign_table($type, 'chart_master', 'account_type', true))
+	if (key_in_foreign_table($type, 'chart_master', 'account_type'))
 	{
 		display_error(_("Cannot delete this account group because GL accounts have been created referring to it."));
 		return false;
 	}
 
-	if (key_in_foreign_table($type, 'chart_types', 'parent', true))
+	if (key_in_foreign_table($type, 'chart_types', 'parent'))
 	{
 		display_error(_("Cannot delete this account group because GL account groups have been created referring to it."));
 		return false;
@@ -119,7 +125,7 @@ $result = get_account_types(check_value('show_inactive'));
 
 start_form();
 start_table(TABLESTYLE);
-$th = array(_("Group ID"), _("Group Name"), _("Subgroup Of"), _("Class Type"), "", "");
+$th = array(_("Group ID"), _("Group Name"), _("Subgroup Of"), _("Class"), "", "");
 inactive_control_column($th);
 table_header($th);
 
@@ -183,7 +189,7 @@ text_row_ex(_("Name:"), 'name', 50);
 
 gl_account_types_list_row(_("Subgroup Of:"), 'parent', null, _("None"), true);
 
-class_list_row(_("Class Type:"), 'class_id', null);
+class_list_row(_("Class:"), 'class_id', null);
 
 end_table(1);
 
@@ -195,4 +201,3 @@ end_form();
 
 end_page();
 
-?>