X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=gl%2Fmanage%2Fgl_account_classes.php;h=1ec62b88431965f4ab37d643fbcac3af19dfb1cf;hb=02dd4aefa3e3babb64d61140bb7f7de14e37f755;hp=95eee3701ffbcd29cbb003f89d27322737ebbcb2;hpb=0b253e5e0d23400838d3bfb4f27fb3fb2637b3ab;p=fa-stable.git

diff --git a/gl/manage/gl_account_classes.php b/gl/manage/gl_account_classes.php
index 95eee370..1ec62b88 100644
--- a/gl/manage/gl_account_classes.php
+++ b/gl/manage/gl_account_classes.php
@@ -9,11 +9,11 @@
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  
     See the License here <http://www.gnu.org/licenses/gpl-3.0.html>.
 ***********************************************************************/
-$page_security = 3;
-$path_to_root="../..";
+$page_security = 'SA_GLACCOUNTCLASS';
+$path_to_root = "../..";
 include($path_to_root . "/includes/session.inc");
 
-page(_("GL Account Classes"));
+page(_($help_context = "GL Account Classes"));
 
 include($path_to_root . "/gl/includes/gl_db.inc");
 
@@ -24,19 +24,21 @@ simple_page_mode(true);
 
 function can_process() 
 {
-	if (!is_numeric($_POST['id'])) 
+	global $use_oldstyle_convert;
+	/*if (!is_numeric($_POST['id'])) 
 	{
 		display_error( _("The account class ID must be numeric."));
 		set_focus('id');
 		return false;
-	}
+	}*/
 	if (strlen($_POST['name']) == 0) 
 	{
 		display_error( _("The account class name cannot be empty."));
 		set_focus('name');
 		return false;
 	}
-
+	if (isset($use_oldstyle_convert) && $use_oldstyle_convert == 1)
+		$_POST['Balance'] = check_value('Balance');
 	return true;
 }
 
@@ -50,15 +52,16 @@ if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM')
 
     	if ($selected_id != -1) 
     	{
-    		update_account_class($selected_id, $_POST['name'], $_POST['Balance'], $_POST['convert']);
-			display_notification(_('Selected account class settings has been updated'));
+    		if(update_account_class($selected_id, $_POST['name'], $_POST['ctype']))
+				display_notification(_('Selected account class settings has been updated'));
     	} 
     	else 
     	{
-    		add_account_class($_POST['id'], $_POST['name'], $_POST['Balance'], $_POST['convert']);
-			display_notification(_('New account class has been added'));
+    		if(add_account_class($_POST['id'], $_POST['name'], $_POST['ctype'])) {
+				display_notification(_('New account class has been added'));
+				$Mode = 'RESET';
+			}
     	}
-		$Mode = 'RESET';
 	}
 }
 
@@ -68,11 +71,7 @@ function can_delete($selected_id)
 {
 	if ($selected_id == -1)
 		return false;
-	$sql= "SELECT COUNT(*) FROM ".TB_PREF."chart_types
-		WHERE class_id=$selected_id";
-	$result = db_query($sql, "could not query chart master");
-	$myrow = db_fetch_row($result);
-	if ($myrow[0] > 0) 
+	if (key_in_foreign_table($selected_id, 'chart_types', 'class_id'))	
 	{
 		display_error(_("Cannot delete this account class because GL account types have been created referring to it."));
 		return false;
@@ -99,7 +98,7 @@ if ($Mode == 'Delete')
 if ($Mode == 'RESET')
 {
 	$selected_id = -1;
-	$_POST['id']  = $_POST['name']  = $_POST['Balance'] = $_POST['sign_convert'] = '';
+	$_POST['id']  = $_POST['name']  = $_POST['ctype'] =  '';
 }
 //-----------------------------------------------------------------------------------
 
@@ -107,7 +106,9 @@ $result = get_account_classes(check_value('show_inactive'));
 
 start_form();
 start_table($table_style);
-$th = array(_("Class ID"), _("Class Name"), _("Balance Sheet"), _("Sign Convert"), "", "");
+$th = array(_("Class ID"), _("Class Name"), _("Class Type"), "", "");
+if (isset($use_oldstyle_convert) && $use_oldstyle_convert == 1)
+	$th[2] = _("Balance Sheet");
 inactive_control_column($th);
 table_header($th);
 
@@ -117,26 +118,15 @@ while ($myrow = db_fetch($result))
 
 	alt_table_row_color($k);
 
-	if ($myrow["balance_sheet"] == 0) 
-	{
-		$bs_text = _("No");
-	} 
-	else 
-	{
-		$bs_text = _("Yes");
-	}
-	if ($myrow["sign_convert"] == 0) 
-	{
-		$sc_text = _("No");
-	} 
-	else 
-	{
-		$sc_text = _("Yes");
-	}
 	label_cell($myrow["cid"]);
 	label_cell($myrow['class_name']);
-	label_cell($bs_text);
-	label_cell($sc_text);
+	if (isset($use_oldstyle_convert) && $use_oldstyle_convert == 1)
+	{
+		$myrow['ctype'] = ($myrow["ctype"] >= CL_ASSETS && $myrow["ctype"] < CL_INCOME ? 1 : 0);
+		label_cell(($myrow['ctype'] == 1 ? _("Yes") : _("No")));
+	}	
+	else	
+		label_cell($class_types[$myrow["ctype"]]);
 	inactive_control_cell($myrow["cid"], $myrow["inactive"], 'chart_class', 'cid');
 	edit_button_cell("Edit".$myrow["cid"], _("Edit"));
 	delete_button_cell("Delete".$myrow["cid"], _("Delete"));
@@ -150,16 +140,18 @@ start_table($table_style2);
 
 if ($selected_id != -1) 
 {
- if ($Mode == 'Edit') {
-	//editing an existing status code
-	$myrow = get_account_class($selected_id);
-
-	$_POST['id']  = $myrow["cid"];
-	$_POST['name']  = $myrow["class_name"];
-	$_POST['Balance']  = $myrow["balance_sheet"];
-	$_POST['convert']  = $myrow["sign_convert"];
-	hidden('selected_id', $selected_id);
- }
+ 	if ($Mode == 'Edit') {
+		//editing an existing status code
+		$myrow = get_account_class($selected_id);
+	
+		$_POST['id']  = $myrow["cid"];
+		$_POST['name']  = $myrow["class_name"];
+		if (isset($use_oldstyle_convert) && $use_oldstyle_convert == 1)
+			$_POST['ctype'] = ($myrow["ctype"] >= CL_ASSETS && $myrow["ctype"] < CL_INCOME ? 1 : 0);
+		else
+			$_POST['ctype']  = $myrow["ctype"];
+		hidden('selected_id', $selected_id);
+ 	}
 	hidden('id');
 	label_row(_("Class ID:"), $_POST['id']);
 
@@ -172,9 +164,10 @@ else
 
 text_row_ex(_("Class Name:"), 'name', 50, 60);
 
-yesno_list_row(_("Balance Sheet:"), 'Balance', null, "", "", false);
-
-yesno_list_row(_("Sign Convert (Balance Sheet/PL statement):"), 'convert', null, "", "", false);
+if (isset($use_oldstyle_convert) && $use_oldstyle_convert == 1)
+	check_row(_("Balance Sheet"), 'ctype', null);
+else
+	class_types_list_row(_("Class Type:"), 'ctype', null);
 
 end_table(1);