Partial (inventory related) include files cleanup.
[fa-stable.git] / inventory / manage / sales_kits.php
index 4368bb7a12fee3323dfb42ce725443f192e81fed..ab4831f0914dba831479355b4c4a5375cefc638c 100644 (file)
@@ -19,8 +19,6 @@ include_once($path_to_root . "/includes/date_functions.inc");
 include_once($path_to_root . "/includes/ui.inc");
 include_once($path_to_root . "/includes/data_checks.inc");
 
-include_once($path_to_root . "/includes/manufacturing.inc");
-
 check_db_has_stock_items(_("There are no items defined in the system."));
 
 simple_page_mode(true);
@@ -36,7 +34,7 @@ function display_kit_items($selected_kit)
 {
        $result = get_item_kit($selected_kit);
        div_start('bom');
-       start_table(TABLESTYLE, "width=60%");
+       start_table(TABLESTYLE, "width='60%'");
        $th = array(_("Stock Item"), _("Description"), _("Quantity"), _("Units"),
                '','');
        table_header($th);
@@ -73,7 +71,7 @@ function update_component($kit_code, $selected_item)
                set_focus('quantity');
                return;
        }
-       elseif ($_POST['description'] == '')
+       elseif (get_post('description') == '')
        {
        display_error( _("Item code description cannot be empty."));
                set_focus('description');
@@ -82,14 +80,14 @@ function update_component($kit_code, $selected_item)
        elseif ($selected_item == -1)   // adding new item or new alias/kit
        {
                if (get_post('item_code') == '') { // New kit/alias definition
-                       $kit = get_item_kit($_POST['kit_code']);
+                       $kit = get_item_kit($kit_code);
                if (db_num_rows($kit)) {
                                $input_error = 1;
                        display_error( _("This item code is already assigned to stock item or sale kit."));
                                set_focus('kit_code');
                                return;
                        }
-                       if (get_post('kit_code') == '') {
+                       if ($kit_code == '') {
                        display_error( _("Kit/alias code cannot be empty."));
                                set_focus('kit_code');
                                return;
@@ -97,21 +95,20 @@ function update_component($kit_code, $selected_item)
                }
        }
 
-       if (check_item_in_kit($selected_item, $kit_code, $_POST['component'], true)) {
+       if (check_item_in_kit($selected_item, $kit_code, get_post('component'), true)) {
                display_error(_("The selected component contains directly or on any lower level the kit under edition. Recursive kits are not allowed."));
                set_focus('component');
                return;
        }
 
                /*Now check to see that the component is not already in the kit */
-       if (check_item_in_kit($selected_item, $kit_code, $_POST['component'])) {
+       if (check_item_in_kit($selected_item, $kit_code, get_post('component'))) {
                display_error(_("The selected component is already in this kit. You can modify it's quantity but it cannot appear more than once in the same kit."));
                set_focus('component');
                return;
        }
        if ($selected_item == -1) { // new item alias/kit
-               if ($_POST['item_code']=='') {
-                       $kit_code = $_POST['kit_code'];
+               if (get_post('item_code') == '') {
                        $selected_kit = $_POST['item_code'] = $kit_code;
                        $msg = _("New alias code has been created.");
                } 
@@ -123,7 +120,7 @@ function update_component($kit_code, $selected_item)
                display_notification($msg);
 
        } else {
-               $props = get_kit_props($_POST['item_code']);
+               $props = get_kit_props(get_post('item_code'));
                update_item_code($selected_item, $kit_code, get_post('component'),
                        $props['description'], $props['category_id'], input_num('quantity'), 0);
                display_notification(_("Component of selected kit has been updated."));
@@ -141,7 +138,7 @@ if (get_post('update_name')) {
 }
 
 if ($Mode=='ADD_ITEM' || $Mode=='UPDATE_ITEM')
-       update_component($_POST['item_code'], $selected_id, $selected_kit);
+       update_component($_POST['item_code'], $selected_id);
 
 if ($Mode == 'Delete')
 {