Fixed typo in sql query
[fa-stable.git] / taxes / tax_types.php
index 101144d2da378140722637d3ca0cc4cc37374371..55e40950d43ff4534c5a5272aae4d9c21087507b 100644 (file)
@@ -3,7 +3,6 @@ $page_security = 3;
 $path_to_root="..";
 
 include($path_to_root . "/includes/session.inc");
-
 page(_("Tax Types"));
 
 include_once($path_to_root . "/includes/ui.inc");
@@ -12,28 +11,25 @@ include_once($path_to_root . "/taxes/db/tax_types_db.inc");
 if (isset($_GET['selected_id']))
 {
        $selected_id = $_GET['selected_id'];
-} 
+}
 elseif(isset($_POST['selected_id']))
 {
        $selected_id = $_POST['selected_id'];
 }
 //-----------------------------------------------------------------------------------
 
-function can_process() 
+function can_process()
 {
-       if (strlen($_POST['name']) == 0) 
+       if (strlen($_POST['name']) == 0)
        {
                display_error(_("The tax type name cannot be empty."));
+               set_focus('name');
                return false;
-       } 
-       elseif (!is_numeric($_POST['rate'])) 
-       {
-               display_error( _("The default tax rate must be numeric."));
-               return false;
-       } 
-       elseif ($_POST['rate'] < 0) 
+       }
+       elseif (!check_num('rate', 0))
        {
-               display_error( _("The default tax rate cannot be less than zero."));
+               display_error( _("The default tax rate must be numeric and not less than zero."));
+               set_focus('rate');
                return false;
        }
 
@@ -42,21 +38,21 @@ function can_process()
 
 //-----------------------------------------------------------------------------------
 
-if (isset($_POST['ADD_ITEM']) && can_process()) 
+if (isset($_POST['ADD_ITEM']) && can_process())
 {
 
        add_tax_type($_POST['name'], $_POST['sales_gl_code'],
-               $_POST['purchasing_gl_code'], $_POST['rate'], check_value('out'));
+               $_POST['purchasing_gl_code'], input_num('rate'));
        meta_forward($_SERVER['PHP_SELF']);
 }
 
 //-----------------------------------------------------------------------------------
 
-if (isset($_POST['UPDATE_ITEM']) && can_process()) 
+if (isset($_POST['UPDATE_ITEM']) && can_process())
 {
 
        update_tax_type($selected_id, $_POST['name'],
-       $_POST['sales_gl_code'], $_POST['purchasing_gl_code'], $_POST['rate'], check_value('out'));
+       $_POST['sales_gl_code'], $_POST['purchasing_gl_code'], input_num('rate'));
        meta_forward($_SERVER['PHP_SELF']);
 }
 
@@ -67,9 +63,10 @@ function can_delete($selected_id)
        $sql= "SELECT COUNT(*) FROM ".TB_PREF."tax_group_items  WHERE tax_type_id=$selected_id";
        $result = db_query($sql, "could not query tax groups");
        $myrow = db_fetch_row($result);
-       if ($myrow[0] > 0) 
+       if ($myrow[0] > 0)
        {
                display_error(_("Cannot delete this tax type because tax groups been created referring to it."));
+
                return false;
        }
 
@@ -79,7 +76,7 @@ function can_delete($selected_id)
 
 //-----------------------------------------------------------------------------------
 
-if (isset($_GET['delete'])) 
+if (isset($_GET['delete']))
 {
 
        if (can_delete($selected_id))
@@ -96,27 +93,23 @@ $result = get_all_tax_types();
 start_table($table_style);
 
 $th = array(_("Description"), _("Default Rate (%)"),
-       _("Sales GL Account"), _("Purchasing GL Account"), _("Outstanding"), "", "");
-table_header($th);     
+       _("Sales GL Account"), _("Purchasing GL Account"), "", "");
+table_header($th);
 
 $k = 0;
-while ($myrow = db_fetch($result)) 
+while ($myrow = db_fetch($result))
 {
 
-       if ($myrow['out'] == 1)
-               $out = _("Yes");
-       else
-               $out = _("No");
        alt_table_row_color($k);
 
        label_cell($myrow["name"]);
-       label_cell(number_format2($myrow["rate"],user_percent_dec()), "align=right");
-       label_cell($myrow["sales_gl_code"] . "&nbsp" . $myrow["SalesAccountName"]);
-       label_cell($myrow["purchasing_gl_code"] . "&nbsp" . $myrow["PurchasingAccountName"]);
-       label_cell($out);
+       label_cell(percent_format($myrow["rate"]), "align=right");
+       label_cell($myrow["sales_gl_code"] . "&nbsp;" . $myrow["SalesAccountName"]);
+       label_cell($myrow["purchasing_gl_code"] . "&nbsp;" . $myrow["PurchasingAccountName"]);
 
        edit_link_cell("selected_id=".$myrow["id"]);
        delete_link_cell("selected_id=".$myrow["id"]."&delete=1");
+
        end_row();
 }
 
@@ -132,28 +125,25 @@ start_form();
 
 start_table($table_style2);
 
-if (isset($selected_id)) 
+if (isset($selected_id))
 {
        //editing an existing status code
 
        $myrow = get_tax_type($selected_id);
 
        $_POST['name']  = $myrow["name"];
-       $_POST['rate']  = $myrow["rate"];
+       $_POST['rate']  = percent_format($myrow["rate"]);
        $_POST['sales_gl_code']  = $myrow["sales_gl_code"];
        $_POST['purchasing_gl_code']  = $myrow["purchasing_gl_code"];
-       $_POST['out']  = $myrow["out"];
 
        hidden('selected_id', $selected_id);
 }
 text_row_ex(_("Description:"), 'name', 50);
-text_row_ex(_("Default Rate:"), 'rate', 10, 10, "", "%");
+small_amount_row(_("Default Rate:"), 'rate', '', "", "%", user_percent_dec());
 
 gl_all_accounts_list_row(_("Sales GL Account:"), 'sales_gl_code', null);
 gl_all_accounts_list_row(_("Purchasing GL Account:"), 'purchasing_gl_code', null);
 
-check_row(_("Outstanding:"), 'out', null);
-
 end_table(1);
 
 submit_add_or_update_center(!isset($selected_id));