Fixed tabbed dialogs. Removed redundant 'popup' variables.

[fa-stable.git] / inventory / manage / items.php

diff --git a/inventory/manage/items.php b/inventory/manage/items.php
index e3d710ee054db4c40dae837b02b5082556f2cc9f..fc7f1a7a8f3ff4b634ecc7d2eeb6337614a185d4 100644 (file)
--- a/inventory/manage/items.php
+++ b/inventory/manage/items.php
@@ -19,7 +19,7 @@ if ($SysPrefs->use_popup_windows)
 if (user_use_date_picker())
        $js .= get_js_date_picker();
 
-page(_($help_context = "Items"), @$_REQUEST['popup'], false, "", $js);
+page(_($help_context = "Items"), false, false, "", $js);
 
 include_once($path_to_root . "/includes/date_functions.inc");
 include_once($path_to_root . "/includes/ui.inc");
@@ -79,7 +79,7 @@ if (isset($_FILES['pic']) && $_FILES['pic']['name'] != '')
                $imagetype = $type;
        else
                $imagetype = false;
-       //$imagetype = exif_imagetype($_FILES['pic']['tmp_name']);
+
        if ($imagetype != IMAGETYPE_GIF && $imagetype != IMAGETYPE_JPEG && $imagetype != IMAGETYPE_PNG)
        {       //File type Check
                display_warning( _('Only graphics files can be uploaded'));
@@ -98,7 +98,7 @@ if (isset($_FILES['pic']) && $_FILES['pic']['name'] != '')
        elseif ( $_FILES['pic']['type'] == "text/plain" ) 
        {  //File type Check
                display_warning( _('Only graphics files can be uploaded'));
-               $upload_file ='No';
+        $upload_file ='No';
        } 
        elseif (file_exists($filename))
        {
@@ -265,13 +265,13 @@ if (isset($_POST['delete']) && strlen($_POST['delete']) > 1)
 
 function item_settings(&$stock_id, $new_item) 
 {
-       global $SysPrefs, $path_to_root;
+       global $SysPrefs, $path_to_root, $page_nested;
 
        start_outer_table(TABLESTYLE2);
 
        table_section(1);
 
-       table_section_title(_("Item"));
+       table_section_title(_("General Settings"));
 
        //------------------------------------------------------------------------------------
        if ($new_item) 
@@ -402,7 +402,7 @@ function item_settings(&$stock_id, $new_item)
        if (isset($_POST['NewStockID']) && file_exists(company_path().'/images/'
                .item_img_name($_POST['NewStockID']).".jpg")) 
        {
-        // 31/08/08 - rand() call is necessary here to avoid caching problems. Thanks to Peter D.
+        // 31/08/08 - rand() call is necessary here to avoid caching problems.
                $stock_img_link .= "<img id='item_img' alt = '[".$_POST['NewStockID'].".jpg".
                        "]' src='".company_path().'/images/'.item_img_name($_POST['NewStockID']).
                        ".jpg?nocache=".rand()."'"." height='".$SysPrefs->pic_height."' border='0'>";
@@ -428,7 +428,7 @@ function item_settings(&$stock_id, $new_item)
        else 
        {
                submit_center_first('addupdate', _("Update Item"), '', 
-                       @$_REQUEST['popup'] ? true : 'default');
+                       $page_nested ? true : 'default');
                submit_return('select', get_post('stock_id'), 
                        _("Select this items and return to document entry."), 'default');
                submit('clone', _("Clone This Item"), true, '', true);
@@ -487,17 +487,17 @@ tabbed_content_start('tabs', array(
                        break;
                case 'sales_pricing':
                        $_GET['stock_id'] = $stock_id;
-                       $_GET['popup'] = 1;
+                       $_GET['page_level'] = 1;
                        include_once($path_to_root."/inventory/prices.php");
                        break;
                case 'purchase_pricing':
                        $_GET['stock_id'] = $stock_id;
-                       $_GET['popup'] = 1;
+                       $_GET['page_level'] = 1;
                        include_once($path_to_root."/inventory/purchasing_data.php");
                        break;
                case 'standard_cost':
                        $_GET['stock_id'] = $stock_id;
-                       $_GET['popup'] = 1;
+                       $_GET['page_level'] = 1;
                        include_once($path_to_root."/inventory/cost_update.php");
                        break;
                case 'reorder_level':
@@ -505,18 +505,16 @@ tabbed_content_start('tabs', array(
                        {
                                break;
                        }       
+                       $_GET['page_level'] = 1;
                        $_GET['stock_id'] = $stock_id;
-                       $_GET['popup'] = 1;
                        include_once($path_to_root."/inventory/reorder_level.php");
                        break;
                case 'movement':
                        $_GET['stock_id'] = $stock_id;
-                       $_GET['popup'] = 1;
                        include_once($path_to_root."/inventory/inquiry/stock_movements.php");
                        break;
                case 'status':
                        $_GET['stock_id'] = $stock_id;
-                       $_GET['popup'] = 1;
                        include_once($path_to_root."/inventory/inquiry/stock_status.php");
                        break;
        };
@@ -525,9 +523,8 @@ tabbed_content_end();
 
 div_end();
 
-hidden('popup', @$_REQUEST['popup']);
 end_form();
 
 //------------------------------------------------------------------------------------
 
-end_page(@$_REQUEST['popup']);
+end_page();