Removed unneeded double forms in scripts with db_pager. Missing $_POST variables.

diff --git a/CHANGELOG.txt b/CHANGELOG.txt
index 59635d771998144d35345c52daf642ed2099ae51..c4f9cdb1e686b65f0a9831ccbac9275cbd785837 100644 (file)
--- a/CHANGELOG.txt
+++ b/CHANGELOG.txt
@@ -19,6 +19,20 @@ Legend:
 ! -> Note
 $ -> Affected files
 
+27-Oct-2009 Joe Hunt
+# Removed unneeded double forms in scripts with db_pager. Missing $_POST variables.
+$ /dimensions/inquiry/search_dimensions.php
+  /manufacturing/search_work_orders.php
+  /purchasing/allocations/supplier_allocation_main.php
+  /purchasing/inquiry/po_search_completed.php
+  /purchasing/inquiry/po_search.php
+  /purchasing/inquiry/supplier_inquiry.php
+  /purchasing/inquiry/supplier_allocation_inquiry.php
+  /sales/inquiry/customer_allocation_inquiry.php
+  /sales/inquiry/sales_deliveries_view.php
+  /sales/inquiry/sales_orders_view.php
+  /sales/inquiry/customer_inquiry.php
+  
 26-Oct-2009 Janusz Dobrowolski
 # [0000177] Fixed error during gl class update.
 $ /gl/includes/db/gl_db_account_types.inc

diff --git a/dimensions/inquiry/search_dimensions.php b/dimensions/inquiry/search_dimensions.php
index 752b5507ab2574ab164425d76aa1efd8583a5d90..1c63c2fc89171f7019a3accdedcf2db29378ec06 100644 (file)
--- a/dimensions/inquiry/search_dimensions.php
+++ b/dimensions/inquiry/search_dimensions.php
@@ -90,8 +90,6 @@ submit_cells('SearchOrders', _("Search"), '', '', 'default');
 end_row();
 end_table();
 
-end_form();
-
 $dim = get_company_pref('use_dimension');
 
 function view_link($row) 
@@ -188,7 +186,6 @@ $table =& new_db_pager('dim_tbl', $sql, $cols);
 $table->set_marker('is_overdue', _("Marked dimensions are overdue."));
 
 $table->width = "80%";
-start_form();
 
 display_db_pager($table);
 

diff --git a/manufacturing/search_work_orders.php b/manufacturing/search_work_orders.php
index cf93441cfaa7332ef74cc7f5e020667a52e9e1a7..a088786cf897eacb2703b68b3035d317637cfbc4 100644 (file)
--- a/manufacturing/search_work_orders.php
+++ b/manufacturing/search_work_orders.php
@@ -79,8 +79,6 @@ submit_cells('SearchOrders', _("Search"),'',_('Select documents'),  'default');
 end_row();
 end_table();
 
-end_form();
-
 //-----------------------------------------------------------------------------
 function check_overdue($row)
 {
@@ -226,7 +224,6 @@ $table =& new_db_pager('orders_tbl', $sql, $cols);
 $table->set_marker('check_overdue', _("Marked orders are overdue."));
 
 $table->width = "90%";
-start_form();
 
 display_db_pager($table);
 

diff --git a/purchasing/allocations/supplier_allocation_main.php b/purchasing/allocations/supplier_allocation_main.php
index 47360922f77a5f1cbb46cf9a1bc7d59a0df25138..a3b04a532bc269663d29d31630bddb3e63c51615 100644 (file)
--- a/purchasing/allocations/supplier_allocation_main.php
+++ b/purchasing/allocations/supplier_allocation_main.php
@@ -38,7 +38,6 @@ start_form();
     echo "<br>";
     check(_("Show Settled Items:"), 'ShowSettled', null, true);
        echo "</center><br><br>";
-       end_form();
        set_global_supplier($_POST['supplier_id']);
 
        if (isset($_POST['supplier_id']) && ($_POST['supplier_id'] == ALL_TEXT)) 
@@ -112,7 +111,6 @@ $table =& new_db_pager('alloc_tbl', $sql, $cols);
 $table->set_marker('check_settled', _("Marked items are settled."), 'settledbg', 'settledfg');
 
 $table->width = "80%";
-start_form();
 
 display_db_pager($table);
 

diff --git a/purchasing/inquiry/po_search.php b/purchasing/inquiry/po_search.php
index a82a37435ff8c68eef47e7b7a179b3db9f06e8f3..a455e56c326acbc4d7ab91bd88c6f0265c18c3bf 100644 (file)
--- a/purchasing/inquiry/po_search.php
+++ b/purchasing/inquiry/po_search.php
@@ -71,7 +71,6 @@ stock_items_list_cells(_("Item:"), 'SelectStockFromList', null, true);
 submit_cells('SearchOrders', _("Search"),'',_('Select documents'), 'default');
 end_row();
 end_table();
-end_form();
 //---------------------------------------------------------------------------------------------
 function trans_view($trans)
 {
@@ -188,7 +187,6 @@ $table =& new_db_pager('orders_tbl', $sql, $cols);
 $table->set_marker('check_overdue', _("Marked orders have overdue items."));
 
 $table->width = "80%";
-start_form();
 
 display_db_pager($table);
 

diff --git a/purchasing/inquiry/po_search_completed.php b/purchasing/inquiry/po_search_completed.php
index 23053bae4cc850e4dbeedea3b564d7fdbeba1461..a7672f9931361580ca798328f1585c0238463052 100644 (file)
--- a/purchasing/inquiry/po_search_completed.php
+++ b/purchasing/inquiry/po_search_completed.php
@@ -69,7 +69,6 @@ stock_items_list_cells(_("for item:"), 'SelectStockFromList', null, true);
 submit_cells('SearchOrders', _("Search"),'',_('Select documents'), 'default');
 end_row();
 end_table();
-end_form();
 //---------------------------------------------------------------------------------------------
 if (isset($_POST['order_number']))
 {
@@ -171,7 +170,6 @@ if (get_post('StockLocation') != $all_items) {
 $table =& new_db_pager('orders_tbl', $sql, $cols);
 
 $table->width = "80%";
-start_form();
 
 display_db_pager($table);
 

diff --git a/purchasing/inquiry/supplier_allocation_inquiry.php b/purchasing/inquiry/supplier_allocation_inquiry.php
index 0a5c65d8f0f5e88ac12985571d7cf543374f728a..2f80b49c88da0a2ba477c4b1f9bf9020068796c9 100644 (file)
--- a/purchasing/inquiry/supplier_allocation_inquiry.php
+++ b/purchasing/inquiry/supplier_allocation_inquiry.php
@@ -60,7 +60,6 @@ set_global_supplier($_POST['supplier_id']);
 
 end_row();
 end_table();
-end_form();
 //------------------------------------------------------------------------------------------------
 function check_overdue($row)
 {
@@ -198,7 +197,6 @@ $table =& new_db_pager('doc_tbl', $sql, $cols);
 $table->set_marker('check_overdue', _("Marked items are overdue."));
 
 $table->width = "90%";
-start_form();
 
 display_db_pager($table);
 

diff --git a/purchasing/inquiry/supplier_inquiry.php b/purchasing/inquiry/supplier_inquiry.php
index 0ff509844042ce390dfb55ada9206bc491bdc0d1..3a78565948efb2f6f1e10943b99082a8352b9a34 100644 (file)
--- a/purchasing/inquiry/supplier_inquiry.php
+++ b/purchasing/inquiry/supplier_inquiry.php
@@ -54,7 +54,6 @@ submit_cells('RefreshInquiry', _("Search"),'',_('Refresh Inquiry'), 'default');
 
 end_row();
 end_table();
-end_form();
 set_global_supplier($_POST['supplier_id']);
 
 //------------------------------------------------------------------------------------------------
@@ -234,7 +233,6 @@ $table =& new_db_pager('trans_tbl', $sql, $cols);
 $table->set_marker('check_overdue', _("Marked items are overdue."));
 
 $table->width = "85%";
-start_form();
 
 display_db_pager($table);
 

diff --git a/sales/inquiry/customer_allocation_inquiry.php b/sales/inquiry/customer_allocation_inquiry.php
index fae2ad0ff7c0c840558e3b3f1bef71d75e9e22e9..badd4b1899b3137f67c43eb4fd63a8fd60ba0e55 100644 (file)
--- a/sales/inquiry/customer_allocation_inquiry.php
+++ b/sales/inquiry/customer_allocation_inquiry.php
@@ -54,7 +54,6 @@ set_global_customer($_POST['customer_id']);
 
 end_row();
 end_table();
-end_form();
 //------------------------------------------------------------------------------------------------
 function check_overdue($row)
 {
@@ -225,7 +224,6 @@ $table =& new_db_pager('doc_tbl', $sql, $cols);
 $table->set_marker('check_overdue', _("Marked items are overdue."));
 
 $table->width = "80%";
-start_form();
 
 display_db_pager($table);
 

diff --git a/sales/inquiry/customer_inquiry.php b/sales/inquiry/customer_inquiry.php
index b076faadc6a29c688e8e73f10657a929b43cb24f..9d397b946c8ecef2abd5ddabbcd866a73a3f671a 100644 (file)
--- a/sales/inquiry/customer_inquiry.php
+++ b/sales/inquiry/customer_inquiry.php
@@ -54,7 +54,7 @@ cust_allocations_list_cells(null, 'filterType', $_POST['filterType'], true);
 submit_cells('RefreshInquiry', _("Search"),'',_('Refresh Inquiry'), 'default');
 end_row();
 end_table();
-end_form();
+
 set_global_customer($_POST['customer_id']);
 
 //------------------------------------------------------------------------------------------------
@@ -300,7 +300,6 @@ $table->set_marker('check_overdue', _("Marked items are overdue."));
 
 $table->width = "85%";
 
-start_form();
 display_db_pager($table);
 
 end_form();

diff --git a/sales/inquiry/sales_deliveries_view.php b/sales/inquiry/sales_deliveries_view.php
index 72e6d132da71930d9f5e84a1a0bdb40aa0d3d9c2..aaa3421ffee598e41aeeae5d4b59946bf5e1243c 100644 (file)
--- a/sales/inquiry/sales_deliveries_view.php
+++ b/sales/inquiry/sales_deliveries_view.php
@@ -115,7 +115,6 @@ hidden('OutstandingOnly', $_POST['OutstandingOnly']);
 end_row();
 
 end_table();
-end_form();
 //---------------------------------------------------------------------------------------------
 
 if (isset($_POST['SelectStockFromList']) && ($_POST['SelectStockFromList'] != "") &&
@@ -258,8 +257,6 @@ $table->set_marker('check_overdue', _("Marked items are overdue."));
 
 //$table->width = "92%";
 
-start_form();
-
 display_db_pager($table);
 
 end_form();

diff --git a/sales/inquiry/sales_orders_view.php b/sales/inquiry/sales_orders_view.php
index dac158eaf7c133aa31679c9a2611f13ba8f25982..50a232764fdd686ed39e51399be78dafbc608691 100644 (file)
--- a/sales/inquiry/sales_orders_view.php
+++ b/sales/inquiry/sales_orders_view.php
@@ -242,7 +242,6 @@ hidden('type', $trans_type);
 end_row();
 
 end_table(1);
-end_form();
 //---------------------------------------------------------------------------------------------
 //     Orders inquiry table
 //
@@ -373,7 +372,6 @@ $table =& new_db_pager('orders_tbl', $sql, $cols);
 $table->set_marker('check_overdue', _("Marked items are overdue."));
 
 $table->width = "80%";
-start_form();
 
 display_db_pager($table);
 submit_center('Update', _("Update"), true, '', null);