X-Git-Url: https://delta.frontaccounting.com/gitweb/?a=blobdiff_plain;f=admin%2Fpayment_terms.php;h=e58a9fded3bf811a622c8a11887524673229e7a4;hb=55c51a9db64c9fe402d615e204824f8e4cd6a310;hp=cf46b90665ba70f531c40565e0dd518eedaf3f91;hpb=da8311619dd73feae101d246a1957b972e00cbd2;p=fa-stable.git diff --git a/admin/payment_terms.php b/admin/payment_terms.php index cf46b906..e58a9fde 100644 --- a/admin/payment_terms.php +++ b/admin/payment_terms.php @@ -1,28 +1,26 @@ . +***********************************************************************/ +$page_security = 'SA_PAYTERMS'; $path_to_root=".."; include($path_to_root . "/includes/session.inc"); -page(_("Payment Terms")); +page(_($help_context = "Payment Terms")); 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']; -} - +simple_page_mode(true); //------------------------------------------------------------------------------------------- -if (isset($_POST['ADD_ITEM']) OR isset($_POST['UPDATE_ITEM'])) +if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM') { $inpug_error = 0; @@ -31,21 +29,25 @@ if (isset($_POST['ADD_ITEM']) OR isset($_POST['UPDATE_ITEM'])) { $inpug_error = 1; display_error( _("The number of days or the day in the following month must be numeric.")); + set_focus('DayNumber'); } elseif (strlen($_POST['terms']) == 0) { $inpug_error = 1; display_error( _("The Terms description must be entered.")); - } - elseif ($_POST['DayNumber'] > 30 && !check_value('DaysOrFoll')) - { - $inpug_error = 1; - display_error( _("When the check box to indicate a day in the following month is the due date, the due date cannot be a day after the 30th. A number between 1 and 30 is expected.")); - } + set_focus('terms'); + } // there should be no limits by 30 here if they want longer payment terms. Joe Hunt 2010-05-31 + //elseif ($_POST['DayNumber'] > 30 && !check_value('DaysOrFoll')) + //{ + // $inpug_error = 1; + // display_error( _("When the check box to indicate a day in the following month is the due date, the due date cannot be a day after the 30th. A number between 1 and 30 is expected.")); + // set_focus('DayNumber'); + //} elseif ($_POST['DayNumber'] > 500 && check_value('DaysOrFoll')) { $inpug_error = 1; display_error( _("When the check box is not checked to indicate that the term expects a number of days after which accounts are due, the number entered should be less than 500 days.")); + set_focus('DayNumber'); } if ($_POST['DayNumber'] == '') @@ -53,90 +55,61 @@ if (isset($_POST['ADD_ITEM']) OR isset($_POST['UPDATE_ITEM'])) if ($inpug_error != 1) { - if (isset($selected_id)) + if ($selected_id != -1) { - if (check_value('DaysOrFoll')) - { - $sql = "UPDATE ".TB_PREF."payment_terms SET terms='" . $_POST['terms'] . "', - day_in_following_month=0, - days_before_due=" . $_POST['DayNumber'] . " - WHERE terms_indicator = '" . $selected_id . "'"; - } - else - { - $sql = "UPDATE ".TB_PREF."payment_terms SET terms='" . $_POST['terms'] . "', - day_in_following_month=" . $_POST['DayNumber'] . ", - days_before_due=0 - WHERE terms_indicator = '" . $selected_id . "'"; - } - + update_payment_terms($selected_id, check_value('DaysOrFoll'), $_POST['terms'], $_POST['DayNumber']); + $note = _('Selected payment terms have been updated'); } else { - - if (check_value('DaysOrFoll')) - { - $sql = "INSERT INTO ".TB_PREF."payment_terms (terms, - days_before_due, day_in_following_month) - VALUES ('" . - $_POST['terms'] . "', " . $_POST['DayNumber'] . ", 0)"; - } - else - { - $sql = "INSERT INTO ".TB_PREF."payment_terms (terms, - days_before_due, day_in_following_month) - VALUES ('" . $_POST['terms'] . "', - 0, " . $_POST['DayNumber'] . ")"; - } - + add_payment_terms(check_value('DaysOrFoll'), $_POST['terms'], $_POST['DayNumber']); + $note = _('New payment terms have been added'); } //run the sql from either of the above possibilites - db_query($sql,"The payment term could not be added or updated"); - - meta_forward($_SERVER['PHP_SELF']); + display_notification($note); + $Mode = 'RESET'; } } -if (isset($_GET['delete'])) +if ($Mode == 'Delete') { // PREVENT DELETES IF DEPENDENT RECORDS IN debtors_master - - $sql= "SELECT COUNT(*) FROM ".TB_PREF."debtors_master WHERE payment_terms = '$selected_id'"; - $result = db_query($sql,"check failed"); - $myrow = db_fetch_row($result); - if ($myrow[0] > 0) + if (key_in_foreign_table($selected_id, 'debtors_master', 'payment_terms')) { display_error(_("Cannot delete this payment term, because customer accounts have been created referring to this term.")); } else { - $sql= "SELECT COUNT(*) FROM ".TB_PREF."suppliers WHERE payment_terms = '$selected_id'"; - $result = db_query($sql,"check failed"); - $myrow = db_fetch_row($result); - if ($myrow[0] > 0) + if (key_in_foreign_table($selected_id, 'suppliers', 'payment_terms')) { display_error(_("Cannot delete this payment term, because supplier accounts have been created referring to this term")); } else { //only delete if used in neither customer or supplier accounts - - $sql="DELETE FROM ".TB_PREF."payment_terms WHERE terms_indicator='$selected_id'"; - db_query($sql,"could not delete a payment terms"); - - meta_forward($_SERVER['PHP_SELF']); + delete_payment_terms($selected_id); + display_notification(_('Selected payment terms have been deleted')); } } //end if payment terms used in customer or supplier accounts + $Mode = 'RESET'; } +if ($Mode == 'RESET') +{ + $selected_id = -1; + $sav = get_post('show_inactive'); + unset($_POST); + $_POST['show_inactive'] = $sav; +} //------------------------------------------------------------------------------------------------- -$sql = "SELECT * FROM ".TB_PREF."payment_terms"; -$result = db_query($sql,"could not get payment terms"); +$result = get_payment_terms_all(check_value('show_inactive')); -start_table($table_style); +start_form(); +start_table(TABLESTYLE); $th = array(_("Description"), _("Following Month On"), _("Due After (Days)"), "", ""); +inactive_control_column($th); table_header($th); $k = 0; //row colour counter @@ -165,37 +138,33 @@ while ($myrow = db_fetch($result)) label_cell($myrow["terms"]); label_cell($full_text); label_cell($after_text); - edit_link_cell("selected_id=".$myrow["terms_indicator"]); - delete_link_cell("selected_id=".$myrow["terms_indicator"]."&delete=1"); + inactive_control_cell($myrow["terms_indicator"], $myrow["inactive"], 'payment_terms', "terms_indicator"); + edit_button_cell("Edit".$myrow["terms_indicator"], _("Edit")); + delete_button_cell("Delete".$myrow["terms_indicator"], _("Delete")); end_row(); } //END WHILE LIST LOOP -end_table(); - -hyperlink_no_params($_SERVER['PHP_SELF'], _("New Payment Term")); +inactive_control_row($th); +end_table(1); //------------------------------------------------------------------------------------------------- -start_form(); - -start_table($table_style2); +start_table(TABLESTYLE2); $day_in_following_month = $days_before_due = 0; -if (isset($selected_id)) +if ($selected_id != -1) { - //editing an existing payment terms - $sql = "SELECT * FROM ".TB_PREF."payment_terms - WHERE terms_indicator='$selected_id'"; - - $result = db_query($sql,"could not get payment term"); - $myrow = db_fetch($result); - - $_POST['terms'] = $myrow["terms"]; - $days_before_due = $myrow["days_before_due"]; - $day_in_following_month = $myrow["day_in_following_month"]; - + if ($Mode == 'Edit') { + //editing an existing payment terms + $myrow = get_payment_terms($selected_id); + + $_POST['terms'] = $myrow["terms"]; + $days_before_due = $myrow["days_before_due"]; + $day_in_following_month = $myrow["day_in_following_month"]; + unset($_POST['DayNumber']); + } hidden('selected_id', $selected_id); } text_row(_("Terms Description:"), 'terms', null, 40, 40); @@ -214,7 +183,7 @@ text_row_ex(_("Days (Or Day In Following Month):"), 'DayNumber', 3); end_table(1); -submit_add_or_update_center(!isset($selected_id)); +submit_add_or_update_center($selected_id == -1, '', 'both'); end_form();