A bug in Customer Allocations fixed.

[fa-stable.git] / sales / allocations / customer_allocation_main.php

diff --git a/sales/allocations/customer_allocation_main.php b/sales/allocations/customer_allocation_main.php
index ee373036ac4be6b18e9a4589063ae6b31954582a..56d4312f5fb1c2fa0a40f4679ade924faf5d8770 100644 (file)
--- a/sales/allocations/customer_allocation_main.php
+++ b/sales/allocations/customer_allocation_main.php
@@ -9,69 +9,53 @@
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  
     See the License here <http://www.gnu.org/licenses/gpl-3.0.html>.
 ***********************************************************************/
-$path_to_root="../..";
-$page_security = 3;
+$page_security = 'SA_SALESALLOC';
+$path_to_root = "../..";
 include($path_to_root . "/includes/db_pager.inc");
 include_once($path_to_root . "/includes/session.inc");
 
 include_once($path_to_root . "/sales/includes/sales_ui.inc");
 include_once($path_to_root . "/sales/includes/sales_db.inc");
 $js = "";
-if ($use_popup_windows)
+if ($SysPrefs->use_popup_windows)
        $js .= get_js_open_window(900, 500);
-page(_("Customer Allocations"), false, false, "", $js);
+page(_($help_context = "Customer Allocations"), false, false, "", $js);
 
 //--------------------------------------------------------------------------------
-if ($ret = context_restore()) {
-       if(isset($ret['customer_id']))
-               $_POST['customer_id'] = $ret['customer_id'];
-}
-if (isset($_POST['_customer_id_editor'])) {
-       context_call($path_to_root.'/sales/manage/customers.php?debtor_no='.$_POST['customer_id'] );
-}
 
 start_form();
-       /* show all outstanding receipts and credits to be allocated */
-       /*Clear any previous allocation records */
-       if (isset($_SESSION['alloc']))
-       {
-               unset($_SESSION['alloc']->allocs);
-               unset($_SESSION['alloc']);
-       }
-    if (!isset($_POST['customer_id']))
-       $_POST['customer_id'] = get_global_customer();
-
-    echo "<center>" . _("Select a customer: ") . "&nbsp;&nbsp;";
-       customer_list('customer_id', $_POST['customer_id'], true, true);
-    echo "<br>";
-    check(_("Show Settled Items:"), 'ShowSettled', null, true);
-       echo "</center><br><br>";
-
-       set_global_customer($_POST['customer_id']);
-
-       if (isset($_POST['customer_id']) && ($_POST['customer_id'] == reserved_words::get_all()))
-       {
-               unset($_POST['customer_id']);
-       }
-
-       /*if (isset($_POST['customer_id'])) {
-               $custCurr = get_customer_currency($_POST['customer_id']);
-               if (!is_company_currency($custCurr))
-                       echo _("Customer Currency:") . $custCurr;
-       }*/
-
-       $settled = false;
-       if (check_value('ShowSettled'))
-               $settled = true;
-
-       $customer_id = null;
-       if (isset($_POST['customer_id']))
-               $customer_id = $_POST['customer_id'];
+/* show all outstanding receipts and credits to be allocated */
+
+if (!isset($_POST['customer_id']))
+       $_POST['customer_id'] = get_global_customer();
+
+echo "<center>" . _("Select a customer: ") . "&nbsp;&nbsp;";
+echo customer_list('customer_id', $_POST['customer_id'], true, true);
+echo "<br>";
+check(_("Show Settled Items:"), 'ShowSettled', null, true);
+echo "</center><br><br>";
+
+set_global_customer($_POST['customer_id']);
+
+if (isset($_POST['customer_id']) && ($_POST['customer_id'] == ALL_TEXT))
+{
+       unset($_POST['customer_id']);
+}
+
+$settled = false;
+if (check_value('ShowSettled'))
+       $settled = true;
+
+$customer_id = null;
+if (isset($_POST['customer_id']))
+       $customer_id = $_POST['customer_id'];
 
 //--------------------------------------------------------------------------------
 function systype_name($dummy, $type)
 {
-       return systypes::name($type);
+       global $systypes_array;
+
+       return $systypes_array[$type];
 }
 
 function trans_view($trans)
@@ -83,12 +67,17 @@ function alloc_link($row)
 {
        return pager_link(_("Allocate"),
                "/sales/allocations/customer_allocate.php?trans_no="
-                       .$row["trans_no"] . "&trans_type=" . $row["type"], ICON_MONEY);
+                       .$row["trans_no"] . "&trans_type=" . $row["type"]. "&debtor_no=" . $row["debtor_no"], ICON_ALLOC);
+}
+
+function amount_total($row)
+{
+       return price_format($row['type'] == ST_JOURNAL && $row["Total"] < 0 ? -$row["Total"] : $row["Total"]);
 }
 
 function amount_left($row)
 {
-       return price_format($row["Total"]-$row["alloc"]);
+       return price_format(($row['type'] == ST_JOURNAL && $row["Total"] < 0 ? -$row["Total"] : $row["Total"])-$row["alloc"]);
 }
 
 function check_settled($row)
@@ -101,12 +90,12 @@ $sql = get_allocatable_from_cust_sql($customer_id, $settled);
 
 $cols = array(
        _("Transaction Type") => array('fun'=>'systype_name'),
-       _("#") => array('fun'=>'trans_view'),
+       _("#") => array('fun'=>'trans_view', 'align'=>'right'),
        _("Reference"), 
        _("Date") => array('name'=>'tran_date', 'type'=>'date', 'ord'=>'asc'),
        _("Customer") => array('ord'=>''),
        _("Currency") => array('align'=>'center'),
-       _("Total") => 'amount', 
+       _("Total") => array('align'=>'right','fun'=>'amount_total'), 
        _("Left to Allocate") => array('align'=>'right','insert'=>true, 'fun'=>'amount_left'), 
        array('insert'=>true, 'fun'=>'alloc_link')
        );
@@ -119,14 +108,9 @@ if (isset($_POST['customer_id'])) {
 $table =& new_db_pager('alloc_tbl', $sql, $cols);
 $table->set_marker('check_settled', _("Marked items are settled."), 'settledbg', 'settledfg');
 
-if (get_post('_ShowSettled_update') || get_post('_customer_id_update')) {
-       $table->set_sql($sql);
-       $table->set_columns($cols);
-}
 $table->width = "75%";
 
 display_db_pager($table);
 end_form();
 
 end_page();
-?>
\ No newline at end of file