X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=gl%2Fmanage%2Fgl_account_types.php;h=57ab4a65730ca161b8a1462ed5d20962dcc84d4c;hb=9478a0cdeaf9201d2f3947e11e58d90354050025;hp=8485eb7633dd8f77cb4afa2774e4811049a8b484;hpb=6183e9450dcd7e4c747889c10903f6ff1b0add33;p=fa-stable.git

diff --git a/gl/manage/gl_account_types.php b/gl/manage/gl_account_types.php
index 8485eb76..57ab4a65 100644
--- a/gl/manage/gl_account_types.php
+++ b/gl/manage/gl_account_types.php
@@ -1,31 +1,37 @@
 <?php
-
-$page_security = 3;
-$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_GLACCOUNTGROUP';
+$path_to_root = "../..";
 include($path_to_root . "/includes/session.inc");
 
-page(_("GL Account Groups"));
+page(_($help_context = "GL Account Groups"));
 
 include($path_to_root . "/gl/includes/gl_db.inc");
 
 include($path_to_root . "/includes/ui.inc");
 
-if (isset($_GET['selected_id']))
-{
-	$selected_id = $_GET['selected_id'];
-} 
-elseif(isset($_POST['selected_id']))
-{
-	$selected_id = $_POST['selected_id'];
-}
-else
-	$selected_id = "";
+simple_page_mode(true);
 //-----------------------------------------------------------------------------------
 
 function can_process() 
 {
 	global $selected_id;
 
+	if (!input_num('id'))
+	{
+	    display_error( _("The account id must be an integer and cannot be empty."));
+	    set_focus('id');
+	    return false;
+	}
 	if (strlen($_POST['name']) == 0) 
 	{
 		display_error( _("The account group name cannot be empty."));
@@ -44,24 +50,24 @@ function can_process()
 
 //-----------------------------------------------------------------------------------
 
-if (isset($_POST['ADD_ITEM']) || isset($_POST['UPDATE_ITEM'])) 
+if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM') 
 {
 
 	if (can_process()) 
 	{
 
-    	if ($selected_id != "") 
+    	if ($selected_id != -1) 
     	{
-
-    		update_account_type($selected_id, $_POST['name'], $_POST['class_id'], $_POST['parent']);
-
+    		if (update_account_type($selected_id, $_POST['name'], $_POST['class_id'], $_POST['parent']))
+				display_notification(_('Selected account type has been updated'));
     	} 
     	else 
     	{
-
-    		add_account_type($_POST['name'], $_POST['class_id'], $_POST['parent']);
+    		if (add_account_type($_POST['id'], $_POST['name'], $_POST['class_id'], $_POST['parent'])) {
+				display_notification(_('New account type has been added'));
+				$Mode = 'RESET';
+			}
     	}
-		meta_forward($_SERVER['PHP_SELF']);
 	}
 }
 
@@ -69,23 +75,17 @@ if (isset($_POST['ADD_ITEM']) || isset($_POST['UPDATE_ITEM']))
 
 function can_delete($selected_id)
 {
-	if ($selected_id == "")
+	if ($selected_id == -1)
 		return false;
-	$sql= "SELECT COUNT(*) FROM ".TB_PREF."chart_master
-		WHERE account_type=$selected_id";
-	$result = db_query($sql, "could not query chart master");
-	$myrow = db_fetch_row($result);
-	if ($myrow[0] > 0) 
+	$type = db_escape($selected_id);
+
+	if (key_in_foreign_table($type, 'chart_master', 'account_type', true))
 	{
 		display_error(_("Cannot delete this account group because GL accounts have been created referring to it."));
 		return false;
 	}
 
-	$sql= "SELECT COUNT(*) FROM ".TB_PREF."chart_types
-		WHERE parent=$selected_id";
-	$result = db_query($sql, "could not query chart types");
-	$myrow = db_fetch_row($result);
-	if ($myrow[0] > 0) 
+	if (key_in_foreign_table($type, 'chart_types', 'parent', true))
 	{
 		display_error(_("Cannot delete this account group because GL account groups have been created referring to it."));
 		return false;
@@ -97,22 +97,31 @@ function can_delete($selected_id)
 
 //-----------------------------------------------------------------------------------
 
-if (isset($_GET['delete'])) 
+if ($Mode == 'Delete')
 {
 
 	if (can_delete($selected_id))
 	{
 		delete_account_type($selected_id);
-		meta_forward($_SERVER['PHP_SELF']);
+		display_notification(_('Selected account group has been deleted'));
 	}
+	$Mode = 'RESET';
+}
+if ($Mode == 'RESET')
+{
+ 	$selected_id = -1;
+	$_POST['id']  = $_POST['name']  = '';
+	unset($_POST['parent']);
+	unset($_POST['class_id']);
 }
-
 //-----------------------------------------------------------------------------------
 
-$result = get_account_types();
+$result = get_account_types(check_value('show_inactive'));
 
+start_form();
 start_table($table_style);
-$th = array(_("Name"), _("Subgroup Of"), _("Class Type"), "", "");
+$th = array(_("ID"), _("Name"), _("Subgroup Of"), _("Class Type"), "", "");
+inactive_control_column($th);
 table_header($th);
 
 $k = 0;
@@ -123,7 +132,7 @@ while ($myrow = db_fetch($result))
 
 	$bs_text = get_account_class_name($myrow["class_id"]);
 
-	if ($myrow["parent"] == reserved_words::get_any_numeric()) 
+	if ($myrow["parent"] == ANY_NUMERIC) 
 	{
 		$parent_text = "";
 	} 
@@ -132,46 +141,49 @@ while ($myrow = db_fetch($result))
 		$parent_text = get_account_type_name($myrow["parent"]);
 	}
 
+	label_cell($myrow["id"]);
 	label_cell($myrow["name"]);
 	label_cell($parent_text);
 	label_cell($bs_text);
-	edit_link_cell("selected_id=" . $myrow["id"]);
-	delete_link_cell("selected_id=" . $myrow["id"]. "&delete=1");
+	inactive_control_cell($myrow["id"], $myrow["inactive"], 'chart_types', 'id');
+	edit_button_cell("Edit".$myrow["id"], _("Edit"));
+	delete_button_cell("Delete".$myrow["id"], _("Delete"));
 	end_row();
 }
 
-end_table();
-
+inactive_control_row($th);
+end_table(1);
 //-----------------------------------------------------------------------------------
 
-hyperlink_no_params($_SERVER['PHP_SELF'], _("New Account Group"));
-
-start_form();
-
 start_table($table_style2);
 
-if ($selected_id != "") 
+if ($selected_id != -1)
 {
-	//editing an existing status code
-
-	$myrow = get_account_type($selected_id);
-
-	$_POST['name']  = $myrow["name"];
-	$_POST['parent']  = $myrow["parent"];
-	$_POST['class_id']  = $myrow["class_id"];
-
-	hidden('selected_id', $selected_id);
+	if ($Mode == 'Edit') 
+	{
+		//editing an existing status code
+		$myrow = get_account_type($selected_id);
+	
+		$_POST['id']  = $myrow["id"];
+		$_POST['name']  = $myrow["name"];
+		$_POST['parent']  = $myrow["parent"];
+		$_POST['class_id']  = $myrow["class_id"];
+		hidden('selected_id', $selected_id);
+ 	}
+ 	hidden('id');
+	label_row(_("ID:"), $_POST['id']);
 }
-
+else
+	text_row_ex(_("ID:"), 'id', 10);
 text_row_ex(_("Name:"), 'name', 50);
 
-gl_account_types_list_row(_("Subgroup Of:"), 'parent', null, true, _("None"), true);
+gl_account_types_list_row(_("Subgroup Of:"), 'parent', null, _("None"), true);
 
 class_list_row(_("Class Type:"), 'class_id', null);
 
 end_table(1);
 
-submit_add_or_update_center($selected_id == "");
+submit_add_or_update_center($selected_id == -1, '', 'both');
 
 end_form();