Optimizing the F2, F3 and F4 keys in sales/purchase forms.

[fa-stable.git] / gl / manage / gl_account_classes.php

diff --git a/gl/manage/gl_account_classes.php b/gl/manage/gl_account_classes.php
index 9015c5835b4af0a378de734cae698c3a30534f15..a08a17e4652e40bdc22a97819daec91f6afaec68 100644 (file)
--- a/gl/manage/gl_account_classes.php
+++ b/gl/manage/gl_account_classes.php
@@ -9,34 +9,37 @@
     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");
 
 include($path_to_root . "/includes/ui.inc");
 
-simple_page_mode(true);
+simple_page_mode(false);
 //-----------------------------------------------------------------------------------
 
 function can_process() 
 {
-       if (!is_numeric($_POST['id'])) 
+       global $SysPrefs;
+
+       if (strlen(trim($_POST['id'])) == 0) 
        {
-               display_error( _("The account class ID must be numeric."));
+               display_error( _("The account class ID cannot be empty."));
                set_focus('id');
                return false;
        }
-       if (strlen($_POST['name']) == 0) 
+       if (strlen(trim($_POST['name'])) == 0) 
        {
                display_error( _("The account class name cannot be empty."));
                set_focus('name');
                return false;
        }
-
+       if (isset($SysPrefs->use_oldstyle_convert) && $SysPrefs->use_oldstyle_convert == 1)
+               $_POST['Balance'] = check_value('Balance');
        return true;
 }
 
@@ -48,17 +51,18 @@ if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM')
        if (can_process()) 
        {
 
-       if ($selected_id != -1) 
+       if ($selected_id != "") 
        {
-               update_account_class($selected_id, $_POST['name'], $_POST['Balance']);
-                       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']);
-                       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';
        }
 }
 
@@ -66,13 +70,9 @@ if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM')
 
 function can_delete($selected_id)
 {
-       if ($selected_id == -1)
+       if ($selected_id == "")
                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;
@@ -98,51 +98,60 @@ if ($Mode == 'Delete')
 //-----------------------------------------------------------------------------------
 if ($Mode == 'RESET')
 {
-       $selected_id = -1;
-       $_POST['id']  = $_POST['name']  = $_POST['Balance'] = '';
+       $selected_id = "";
+       $_POST['id']  = $_POST['name']  = $_POST['ctype'] =  '';
 }
 //-----------------------------------------------------------------------------------
 
-$result = get_account_classes();
+$result = get_account_classes(check_value('show_inactive'));
+
 start_form();
-start_table($table_style);
+start_table(TABLESTYLE);
 $th = array(_("Class ID"), _("Class Name"), _("Class Type"), "", "");
+if (isset($SysPrefs->use_oldstyle_convert) && $SysPrefs->use_oldstyle_convert == 1)
+       $th[2] = _("Balance Sheet");
+inactive_control_column($th);
 table_header($th);
 
 $k = 0;
 while ($myrow = db_fetch($result)) 
 {
-
        alt_table_row_color($k);
 
        label_cell($myrow["cid"]);
-       label_cell($myrow['class_name']);
-       label_cell($class_types[$myrow["balance_sheet"]]);
+       label_cell('<a href="./gl_account_types.php?cid='.$myrow["cid"].'">'.$myrow['class_name'].'</a>');
+       if (isset($SysPrefs->use_oldstyle_convert) && $SysPrefs->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"));
        end_row();
 }
-
-end_table();
-end_form();
-echo '<br>';
+inactive_control_row($th);
+end_table(1);
 //-----------------------------------------------------------------------------------
 
-start_form();
-
-start_table($table_style2);
+start_table(TABLESTYLE2);
 
-if ($selected_id != -1) 
+if ($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"];
-       $_POST['Balance']  = $myrow["balance_sheet"];
-       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($SysPrefs->use_oldstyle_convert) && $SysPrefs->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']);
 
@@ -155,11 +164,14 @@ else
 
 text_row_ex(_("Class Name:"), 'name', 50, 60);
 
-class_types_list_row(_("Class Type:"), 'Balance', null);
+if (isset($SysPrefs->use_oldstyle_convert) && $SysPrefs->use_oldstyle_convert == 1)
+       check_row(_("Balance Sheet"), 'ctype', null);
+else
+       class_types_list_row(_("Class Type:"), 'ctype', null);
 
 end_table(1);
 
-submit_add_or_update_center($selected_id == -1, '', true);
+submit_add_or_update_center($selected_id == "", '', 'both');
 
 end_form();
 
@@ -167,4 +179,3 @@ end_form();
 
 end_page();
 
-?>