Redundant sql sentences in Report Filters.
[fa-stable.git] / reporting / includes / reports_classes.inc
index fd1fae781bc19f721652aa26b2bc5e3cd60d4a0c..1b9694d3890783ddd6acd5bb3a4625744b34627e 100644 (file)
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  
     See the License here <http://www.gnu.org/licenses/gpl-3.0.html>.
 ***********************************************************************/
+
+// Standard report classess
+define('RC_CUSTOMER', 0);
+define('RC_SUPPLIER', 1);
+define('RC_INVENTORY', 2);
+define('RC_MANUFACTURE', 3);
+define('RC_DIMENSIONS', 4);
+define('RC_BANKING', 5);
+define('RC_GL', 6);
+define('RC_FIXEDASSETS', 7);
+
 class BoxReports
 {
-       var $ar_classes;
+       var $ar_classes; // report class names
+       var $ar_reports;
+       var $ctrl_handlers = array();
 
-       function ReportClasses()
+       function __construct()
        {
                $this->ar_classes = array();
        }
 
-       function addReportClass($class_name)
+       function addReportClass($class_name, $class_id=null)
        {
-               $this->ar_classes[$class_name] = array();
+               if (!$class_id)
+                       $class_id = count($this->ar_classes);
+               $this->ar_classes[$class_id] = $class_name;
+
+               return $class_id;
        }
 
-       function addReport($class_name, $id, $rep_name, $params=null)
+       function addReport($class, $id, $rep_name, $params=null)
        {
-               $this->ar_classes[$class_name][] = new Report($id,$rep_name,$params);
+               unset($this->ar_reports[$class][$id]); // unset std report if any
+               $this->ar_reports[$class][$id] = new Report($id, $rep_name, $params);
        }
 
        function getDisplay($class=null)
        {
-               global $table_style2, $comp_path, $path_to_root, $pdf_debug, $Ajax;
+               global $path_to_root, $SysPrefs, $Ajax;
+
+               if (find_submit('Rep') != -1) {
+                       include($path_to_root.'/reporting/prn_redirect.php');
+                       return;
+               }
 
                $temp = array_values($this->ar_classes);
                $display_class = $class==null ? $temp[0] : $this->ar_classes[$class];
-               $class_counter = 0;
-               $rep_counter = 0;
                $st_reports = "";
                $st_params = "";
                $st_classes = "<b>" . _("Report Classes:") . "</b><br>";
-               foreach($this->ar_classes as $key=>$value)
+
+               if (isset($_GET['Class']))
+                       set_focus('class'.$_GET['Class']);
+
+               $cur_class = $_REQUEST['Class'];
+
+               foreach($this->ar_classes as $class_id => $name)
                {
-                       $style = $class_counter==$_REQUEST['Class'] ? '' : "style='display:none'";
-                       $acc = access_string($key);
+                       if (!isset($this->ar_reports[$class_id]))
+                               continue; // e.g. no dimensions
+
+                       $acc = access_string($name);
                        $st_classes .= "<a href='"
-                               .$_SERVER['PHP_SELF']."?Class=$class_counter'"
-                               ." class='menu_option' id='".default_focus()."'"
-                               ." onclick='return showClass($class_counter);'$acc[1]>$acc[0]</a> <br>";
-                       $st_reports .= "<table id='TAB_" . $class_counter ."' $style cellpadding=0 cellspacing=0 border=0 width='100%'><tr><td><b>" . _("Reports For Class: ") . "&nbsp;$key</b></td></tr>\n";
-                       foreach($value as $report)
+                               .$_SERVER['PHP_SELF']."?Class=$class_id'"
+                               ." style='font-weight:". ($cur_class == $class_id ? 'bold' : 'normal')."'"
+                               ." class='repclass_link' id='".'class'.$class_id."'"
+                               ."$acc[1]>$acc[0]</a> <br>";
+
+                       $style = $class_id==$cur_class ? '' : "style='display:none'";
+                       $st_reports .= "<table class='repclass' id='TAB_" . $class_id ."' $style cellpadding='0' cellspacing='0' border='0' width='100%'><tr><td><b>" . _("Reports For Class: ") . "&nbsp;$name</b></td></tr>\n";
+                       foreach($this->ar_reports[$class_id] as $rep_id => $report)
                        {
                                $acc = access_string($report->name);
-                               $st_reports .= "<tr><td><a class='printlink' href='"
-                                       .$_SERVER['PHP_SELF']."?Class=$class_counter&rep_id=$report->id'"
-                                       ." id='".default_focus()."'"
+                               $st_reports .= "<tr><td><a class='repopts_link'"
+                                       ." href='".$_SERVER['PHP_SELF']."?Class=$class_id&REP_ID=$report->id'"
+                                       ." style='font-weight:". (@$_REQUEST['REP_ID'] == $report->id ? 'bold' : 'normal')."'"
+                                       ." id='".$id = default_focus()."'"
                                        ."$acc[1]>$acc[0]</a><tr><td>\n";
-                               if (isset($_REQUEST['rep_id']) && $_REQUEST['rep_id']==$report->id) {
 
-                                       $action = $path_to_root.'/reporting/prn_redirect.php';
-       
-                                       $st_params = "<table border=0><tr><td>\n"
+                               if (@$_REQUEST['REP_ID'] == $report->id) {
+                                       $Ajax->activate($id);
+                               }
+
+                               if (isset($_REQUEST['REP_ID']) && $_REQUEST['REP_ID']==$report->id) {
+                                       // parameters form
+                                       $action = $_SERVER['PHP_SELF'];
+                                       $st_params = "<table><tr><td>\n"
                                                . "<form method='POST' action='$action' target='_blank'>\n";
-                                       $st_params .= hidden('REP_ID', $report->id, false);
                                        $st_params .= submit('Rep'.$report->id,  
                                                _("Display: ") . access_string($report->name, true),
-                                               false, '', $pdf_debug ? false : 'default') . '<br><br>';
-                                       $st_params .= $report->getDisplay() 
-                                               . "\n</form></td></tr></table>\n";
+                                               false, '', $SysPrefs->pdf_debug ? false : 'default process') . hidden('REP_ID', $report->id, false).'<br><br>';
+                                       $st_params .= $this->getOptions($report->get_controls(), $report->id);
+                                       $st_params .= "\n<input type=hidden name='Class' value=".$cur_class.">"
+                                               ."\n</form></td></tr></table>\n";
                                        set_focus('Rep'.$report->id);
                                        $Ajax->addUpdate(true, 'rep_form', $st_params);
                                }
                        }
                        $st_reports .= "</table>";
-                       $class_counter++;
                }
 
                $st_params = "<div id='rep_form'>".
                        "$st_params</div>";
                
-
-               $st =   "<script language='javascript'>
-                                       function showClass(pClass) {
-                                               for(i=0; i<$class_counter; i++) {
-                                                       document.getElementById(\"TAB_\" + i).style.display=
-                                                       i==pClass ? \"block\" : \"none\";
-                                               }
-                                               document.getElementById('rep_form').innerHTML = '';
-//                                             document.getElementById('rep_form').style.display = 'none';
-                                               return false;
-                                       }
-                                       function checkDate(pObj) {
-                                               var re = /^(3[01]|0[1-9]|[12]\d)\/(0[1-9]|1[012])\/\d{4}/;
-                                               if (re.test(pObj.value)==false) {
-                                                       alert('" . _("Invalid date format") . "')
-                                               }
-                                       }
-                               </script>
-                               ";
-               $st .= "<table align='center' width='80%' $table_style2><tr valign='top'>";
+               $st = "<table align='center' width='80%' style='border:1px solid #cccccc;'><tr valign='top'>";
                $st .= "<td width='30%'>$st_classes</td>";
-               $st .= "<td width='35%'>$st_reports</td>";
+               $st .= "<td width='35%' style='border-left:1px solid #cccccc;border-right:1px solid #cccccc;padding-left:3px;'>$st_reports</td>";
                $st .= "<td width='35%'>$st_params</td>";
                $st .= "</tr></table><br>";
 
                return $st;
        }
-}
-
-class Report
-{
-       var $id;
-       var $name;
-       var $ar_params;
 
-       function Report($id, $name, $ar_params)
+       function getOptions($controls, $id = 0)
        {
-               $this->id                       = $id;
-               $this->name             = $name;
-               $this->ar_params        = $ar_params;
+               $st = '';
+               if ($controls == null)
+                       return "";
+
+               $cnt = 0;
+               foreach($controls as $title => $type)
+               {
+                       $ctrl = '';
+                       foreach($this->ctrl_handlers as $fun) { // first check for non-standard controls
+                               $ctrl = $fun('PARAM_'.$cnt, $type);
+                               if ($ctrl) break;
+                       }
+                       if ($ctrl == '') {
+                               if (($id == 102 || $id == 202) && $cnt == 3) // set default Show Also Allocated to Yes in aged reports 2014-09-19 Joe Hunt
+                                       $_POST['PARAM_'.$cnt] = 1;
+                               
+                               if (isset($_COOKIE['select'][$id][$cnt])) // saved parameters 2010-10-06 Joe Hunt
+                                       $_POST['PARAM_'.$cnt] = $_COOKIE['select'][$id][$cnt];
+                               $ctrl = $this->get_ctrl('PARAM_'.$cnt, $type);
+                       }
+                       if ($ctrl != '')  {
+                               $st .= $title . ':<br>';
+                               $st .= $ctrl;
+                               $st .= "<br><br>";
+                       } else {
+                               display_error(_('Unknown report parameter type:').$type);
+                       }
+                       $cnt++;
+               }
+               return $st;
        }
-       
-       function getDisplay()
+
+       //
+       //      Register additional control handler
+       // $handle - name of global function f($name, $type) returning html code for control
+       function register_controls($handler) {
+               $this->ctrl_handlers[] = $handler;
+       }
+       //
+       //      Returns html code for input control $name of type $type 
+       //
+       function get_ctrl($name, $type)
        {
-               global $path_to_root, $use_date_picker, $pdf_debug, $print_invoice_no;
+               global $path_to_root, $SysPrefs,
+                       $type_shortcuts;
 
                $st = '';
-               $dummy = "";
-               if ($this->ar_params==null)
-                       return "";
-               foreach($this->ar_params as $index=>$param)
-               {
-                       $st .= $param->param_name . ':<br>';
-                       switch ($param->param_type)
+                       switch ($type)
                        {
                                case 'CURRENCY':
-                                       $sql = "SELECT curr_abrev, concat(curr_abrev,' - ', currency) FROM ".TB_PREF."currencies";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Currency Filter"));
-                                       break;
+                                       $sql = "SELECT curr_abrev, concat(curr_abrev,' - ', currency) AS name FROM ".TB_PREF."currencies";
+                                       return combo_input($name, '', $sql, 'curr_abrev', 'name',array('spec_option'=>_("No Currency Filter"),'spec_id'=>ALL_TEXT,'order'=>false));
+
                                case 'DATE':
                                case 'DATEBEGIN':
                                case 'DATEEND':
@@ -144,317 +185,340 @@ class Report
                                case 'DATEENDM':
                                case 'DATEBEGINTAX':
                                case 'DATEENDTAX':
-                                       if ($param->param_type == 'DATEBEGIN')
-                                               $date = begin_fiscalyear();
-                                       elseif ($param->param_type == 'DATEEND')
-                                               $date = end_fiscalyear();
-                                       else
-                                               $date = Today();
-                                       if ($param->param_type == 'DATEBEGINM')
-                                               $date = begin_month($date);
-                                       elseif ($param->param_type == 'DATEENDM')
-                                               $date = end_month($date);
-                                       elseif ($param->param_type == 'DATEBEGINTAX' || $param->param_type == 'DATEENDTAX')
+                                       if (!isset($_POST[$name]))
                                        {
-                                               $row = get_company_prefs();
-                                               $edate = add_months($date, -$row['tax_last']);
-                                               $edate = end_month($edate);
-                                               if ($param->param_type == 'DATEENDTAX')
-                                                       $date = $edate;
+                                               if ($type == 'DATEBEGIN')
+                                                       $date = begin_fiscalyear();
+                                               elseif ($type == 'DATEEND')
+                                                       $date = end_fiscalyear();
                                                else
+                                                       $date = Today();
+                                               if ($type == 'DATEBEGINM')
+                                                       $date = begin_month($date);
+                                               elseif ($type == 'DATEENDM')
+                                                       $date = end_month($date);
+                                               elseif ($type == 'DATEBEGINTAX' || $type == 'DATEENDTAX')
                                                {
-                                                       $bdate = begin_month($edate);
-                                                       $bdate = add_months($bdate, -$row['tax_prd'] + 1);
-                                                       $date = $bdate;
+                                                       $row = get_company_prefs();
+                                                       $edate = add_months($date, -$row['tax_last']);
+                                                       $edate = end_month($edate);
+                                                       if ($type == 'DATEENDTAX')
+                                                               $date = $edate;
+                                                       else
+                                                       {
+                                                               $bdate = begin_month($edate);
+                                                               $bdate = add_months($bdate, -$row['tax_prd'] + 1);
+                                                               $date = $bdate;
+                                                       }
                                                }
                                        }
-                                       $name = "PARAM_$index";
-
-                                       $st .= "<input type='text' name='$name' value='$date'>";
-                                       if ($use_date_picker)
+                                       else // saved parameters
+                                               $date = $_POST[$name];
+                                       $st = "<input type='text' name='$name' value='$date'>";
+                                       if (user_use_date_picker())
+                                       {
+                                               $calc_image = (file_exists("$path_to_root/themes/".user_theme()."/images/cal.gif")) ? 
+                                                       "$path_to_root/themes/".user_theme()."/images/cal.gif" : "$path_to_root/themes/default/images/cal.gif";
                                                $st .= "<a href=\"javascript:date_picker(document.forms[0].$name);\">"
-                                               . "     <img src='$path_to_root/themes/default/images/cal.gif' width='16' height='16' border='0' alt='"._('Click Here to Pick up the date')."'></a>\n";
-
+                                               . "     <img src='$calc_image' style='vertical-align:middle;padding-bottom:4px;width:16px;height:16px;border:0;' alt='"._('Click Here to Pick up the date')."'></a>\n";
+                                       }       
+                                       return $st;
                                        break;
+
                                case 'YES_NO':
-                                       $sel = array(_('No'), _("Yes"));
-                                       $st .= dup_simple_name_list("PARAM_$index", $sel);
-                                       break;
+                                       return yesno_list($name);
+
                                case 'PAYMENT_LINK':
-                                       $sel = array(_("No Payment Link"), "PayPal");
-                                       $st .= dup_simple_name_list("PARAM_$index", $sel);
-                                       break;
+                                       return payment_services($name, null);
+
                                case 'DESTINATION':
                                        $sel = array(_("PDF/Printer"), "Excel");
-                                       $st .= dup_simple_name_list("PARAM_$index", $sel);
-                                       break;
+                                       $def = 0;
+                                       if (user_def_print_destination() == 1)
+                                               $def = 1;
+                                       return array_selector($name, $def, $sel);
+
+                               case 'ORIENTATION':
+                                       $sel = array(_("Portrait"), _("Landscape"));
+                                       $def = 0;
+                                       if (user_def_print_orientation() == 1)
+                                               $def = 1;
+                                       return array_selector($name, $def, $sel);
+
                                case 'COMPARE':
                                        $sel = array(_("Accumulated"), _("Period Y-1"), _("Budget"));
-                                       $st .= dup_simple_name_list("PARAM_$index", $sel);
-                                       break;
+                                       return array_selector($name, null, $sel);
+
                                case 'GRAPHIC':
-                                       $sel = array(_("No Graphics"), _("Vertical bars"), _("Horizontal bars"), _("Dots"), _("Lines"), _("Pie"), _("Donut"));
-                                       $st .= dup_simple_name_list("PARAM_$index", $sel);
-                                       break;
+                                       $sel = array(_("No Graphics"), _("Vertical bars"), _("Horizontal bars"), 
+                                                               _("Dots"), _("Lines"), _("Pie"), _("Donut"));
+                                       return array_selector($name, null, $sel);
+
                                case 'SYS_TYPES':
+                                       return gl_systypes_list($name, null, _("No Type Filter"));
+
                                case 'SYS_TYPES_ALL':
-                                       $st .= dup_systypes_list("PARAM_$index", $dummy, true, _("No Type Filter"), ($param->param_type=='SYS_TYPES_ALL'));
-                                       break;
+                                       return systypes_list($name, null, _("No Type Filter"));
+
                                case 'TEXT':
-                                       $st .= "<input type='text' name='PARAM_$index'>";
-                                       break;
+                                       return "<input type='text' name='$name'>";
+
                                case 'TEXTBOX':
-                                       $st .= "<textarea rows=4 cols=30 name='PARAM_$index'></textarea>";
-                                       break;
-                               case 'ACCOUNTS':
-                                       $sql = "SELECT id, name FROM ".TB_PREF."chart_types ORDER BY name";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Account Group Filter"), true);
-                                       break;
+                                       $value = (isset($_POST[$name]) ? $_POST[$name] : "");
+                                       return "<textarea rows=4 cols=30 maxlength=130 name='$name'>$value</textarea>";
+
+                               case 'ACCOUNTS': // not used
+                                       return gl_account_types_list($name, null, _("No Account Group Filter"), true);
+
+                               case 'ACCOUNTS_NO_FILTER': // not used
+                                       return gl_account_types_list($name);
+
                                case 'GL_ACCOUNTS':
-                                       $sql = "SELECT account_code, concat(account_code, ' - ', account_name) as account_name FROM ".TB_PREF."chart_master ORDER BY account_code";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       return gl_all_accounts_list($name);
+
                                case 'BANK_ACCOUNTS':
-                                       $sql = "SELECT ".TB_PREF."bank_accounts.id, concat(bank_account_name, if (bank_curr_code=curr_default,'', concat(' - ', bank_curr_code))) FROM ".TB_PREF."bank_accounts, ".TB_PREF."company";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       return bank_accounts_list($name);
+                                       
                                case 'DIMENSION':
-                                       $sql = "SELECT id, concat(reference, ' - ', name) AS DimName FROM ".TB_PREF."dimensions ORDER BY reference";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       return dimensions_list($name, null, false, ' ', false, true, 0);
+                                       
                                case 'DIMENSIONS':
-                                       $sql = "SELECT id, concat(reference, ' - ', name) as DimName FROM ".TB_PREF."dimensions ORDER BY reference";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Dimension Filter"), true);
-                                       break;
+                                       return dimensions_list($name, null, true, _("No Dimension Filter"), false, true, 0);
+
                                case 'DIMENSION1':
-                                       $sql = "SELECT id, concat(reference, ' - ', name) AS DimName FROM ".TB_PREF."dimensions WHERE type_=1 ORDER BY reference";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index". "_$index", $dummy);
-                                       break;
+                                       return dimensions_list($name, null, false, ' ', false, true, 1);
+
                                case 'DIMENSIONS1':
-                                       $sql = "SELECT id, concat(reference, ' - ', name) as DimName FROM ".TB_PREF."dimensions WHERE type_=1 ORDER BY reference";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Dimension Filter"), true);
-                                       break;
+                                       return dimensions_list($name, null, true, _("No Dimension Filter"), false, true, 1);
+
                                case 'DIMENSION2':
-                                       $sql = "SELECT id, concat(reference, ' - ', name) AS DimName FROM ".TB_PREF."dimensions WHERE type_=2 ORDER BY reference";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       return dimensions_list($name, null, false, ' ', false, true, 2);
+
                                case 'DIMENSIONS2':
-                                       $sql = "SELECT id, concat(reference, ' - ', name) as DimName FROM ".TB_PREF."dimensions WHERE type_=2 ORDER BY reference";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Dimension Filter"), true);
-                                       break;
+                                       return dimensions_list($name, null, true, _("No Dimension Filter"), false, true, 2);
+
                                case 'CUSTOMERS_NO_FILTER':
                                case 'CUSTOMERS':
-                                       $sql = "SELECT debtor_no, name FROM ".TB_PREF."debtors_master ORDER BY name";
-                                       if ($param->param_type == 'CUSTOMERS_NO_FILTER')
-                                               $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Customer Filter"), true);
+                                       if ($type == 'CUSTOMERS_NO_FILTER')
+                                               return customer_list($name, null, _("No Customer Filter"));
                                        else
-                                               $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                               return customer_list($name);
+
                                case 'SUPPLIERS_NO_FILTER':
                                case 'SUPPLIERS':
-                                       $sql = "SELECT supplier_id, supp_name FROM ".TB_PREF."suppliers ORDER BY supp_name";
-                                       if ($param->param_type == 'SUPPLIERS_NO_FILTER')
-                                               $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Supplier Filter"), true);
+                                       if ($type == 'SUPPLIERS_NO_FILTER')
+                                               return supplier_list($name, null, _("No Supplier Filter"));
                                        else
-                                               $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                               return supplier_list($name);
+
                                case 'INVOICE':
-                                       $IV = _("IV");
-                                       $CN = _("CN");
-                                       if ($print_invoice_no == 1)
-                                               $sql = "SELECT concat(".TB_PREF."debtor_trans.trans_no, '-',
-                                                       ".TB_PREF."debtor_trans.type) AS TNO, concat(".TB_PREF."debtor_trans.trans_no, if (type=".ST_SALESINVOICE.", ' $IV ', ' $CN '), ".TB_PREF."debtors_master.name) as IName
-                                                       FROM ".TB_PREF."debtors_master, ".TB_PREF."debtor_trans WHERE (type=".ST_SALESINVOICE." OR type=".ST_CUSTCREDIT.") AND ".TB_PREF."debtors_master.debtor_no=".TB_PREF."debtor_trans.debtor_no ORDER BY ".TB_PREF."debtor_trans.trans_no DESC";
-                                       else            
-                                               $sql = "SELECT concat(".TB_PREF."debtor_trans.trans_no, '-',
-                                                       ".TB_PREF."debtor_trans.type) AS TNO, concat(".TB_PREF."debtor_trans.reference, if (type=".ST_SALESINVOICE.", ' $IV ', ' $CN '), ".TB_PREF."debtors_master.name) as IName
-                                                       FROM ".TB_PREF."debtors_master, ".TB_PREF."debtor_trans WHERE (type=".ST_SALESINVOICE." OR type=".ST_CUSTCREDIT.") AND ".TB_PREF."debtors_master.debtor_no=".TB_PREF."debtor_trans.debtor_no ORDER BY ".TB_PREF."debtor_trans.trans_no DESC";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       $IV = $type_shortcuts[ST_SALESINVOICE];
+                                       $ref = ($SysPrefs->print_invoice_no() == 1 ? "trans_no" : "reference");
+                                       $sql = "SELECT concat(debtor_trans.trans_no, '-', debtor_trans.type) AS TNO,
+                                                               concat('$IV ', debtor_trans.$ref,' ', debtor.name) as IName
+                                               FROM ".TB_PREF."debtors_master debtor,"
+                                                       .TB_PREF."debtor_trans debtor_trans
+                                               WHERE type=".ST_SALESINVOICE." AND debtor.debtor_no=debtor_trans.debtor_no
+                                               ORDER BY debtor_trans.trans_no DESC";
+                                       return combo_input($name, '', $sql, 'TNO', 'IName',array('order'=>false));
+
+                               case 'CREDIT':
+                                       $CN = $type_shortcuts[ST_CUSTCREDIT];
+                                       $ref = ($SysPrefs->print_invoice_no() == 1 ? "trans_no" : "reference");
+                                       $sql = "SELECT concat(debtor_trans.trans_no, '-', debtor_trans.type) AS TNO,
+                                                               concat('$CN ', debtor_trans.$ref,' ', debtor.name) as IName
+                                               FROM ".TB_PREF."debtors_master debtor,"
+                                                       .TB_PREF."debtor_trans debtor_trans
+                                               WHERE type=".ST_CUSTCREDIT." AND debtor.debtor_no=debtor_trans.debtor_no
+                                               ORDER BY debtor_trans.trans_no DESC";
+                                       return combo_input($name, '', $sql, 'TNO', 'IName',array('order'=>false));
+
                                case 'DELIVERY':
-                                       $DN = _("DN");
-                                       $sql = "SELECT
-                                       concat(".TB_PREF."debtor_trans.trans_no, '-', ".TB_PREF."debtor_trans.type) AS TNO, concat(".TB_PREF."debtor_trans.trans_no, ' $DN ',
-                                        ".TB_PREF."debtors_master.name) as IName
-                                               FROM ".TB_PREF."debtors_master, ".TB_PREF."debtor_trans
-                                               WHERE type=".ST_CUSTDELIVERY." AND ".TB_PREF."debtors_master.debtor_no=".
-                                               TB_PREF."debtor_trans.debtor_no ORDER BY ".TB_PREF."debtor_trans.trans_no DESC";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       $DN = $type_shortcuts[ST_CUSTDELIVERY];
+                                       $sql = "SELECT  concat(debtor_trans.trans_no, '-', debtor_trans.type) AS TNO,
+                                                               concat(debtor_trans.trans_no, ' $DN ', debtor.name) as IName
+                                               FROM ".TB_PREF."debtors_master debtor,"
+                                                       .TB_PREF."debtor_trans debtor_trans
+                                               WHERE type=".ST_CUSTDELIVERY." AND debtor.debtor_no=debtor_trans.debtor_no
+                                               ORDER BY debtor_trans.trans_no DESC";
+                                       return combo_input($name, '', $sql, 'TNO', 'IName',array('order'=>false));
+
                                case 'ORDERS':
-                                       $sql = "SELECT ".TB_PREF."sales_orders.order_no, concat(".TB_PREF."sales_orders.order_no, '-',
-                                               ".TB_PREF."debtors_master.name) as IName
-                                               FROM ".TB_PREF."debtors_master, ".TB_PREF."sales_orders WHERE ".TB_PREF."debtors_master.debtor_no=".TB_PREF."sales_orders.debtor_no 
-                                               AND ".TB_PREF."sales_orders.trans_type=".ST_SALESORDER." ORDER BY ".TB_PREF."sales_orders.order_no DESC";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       $ref = ($SysPrefs->print_invoice_no() == 1 ? "order_no" : "reference");
+                                       $sql = "SELECT sorder.order_no, concat(sorder.$ref, '-', debtor.name) as IName
+                                               FROM ".TB_PREF."debtors_master debtor,"
+                                                       .TB_PREF."sales_orders sorder
+                                               WHERE debtor.debtor_no=sorder.debtor_no AND sorder.trans_type=".ST_SALESORDER
+                                               ." ORDER BY sorder.order_no DESC";
+                                       return combo_input($name, '', $sql, 'order_no', 'IName',array('order'=>false));
+
                                case 'QUOTATIONS':
-                                       $sql = "SELECT ".TB_PREF."sales_orders.order_no, concat(".TB_PREF."sales_orders.order_no, '-',
-                                               ".TB_PREF."debtors_master.name) as IName
-                                               FROM ".TB_PREF."debtors_master, ".TB_PREF."sales_orders WHERE ".TB_PREF."debtors_master.debtor_no=".TB_PREF."sales_orders.debtor_no 
-                                               AND ".TB_PREF."sales_orders.trans_type=".ST_SALESQUOTE." ORDER BY ".TB_PREF."sales_orders.order_no DESC";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       $ref = ($SysPrefs->print_invoice_no() == 1 ? "order_no" : "reference");
+                                       $sql = "SELECT sorder.order_no, concat(sorder.$ref, '-', debtor.name) as IName
+                                               FROM ".TB_PREF."debtors_master debtor,"
+                                                       .TB_PREF."sales_orders sorder
+                                               WHERE debtor.debtor_no=sorder.debtor_no AND sorder.trans_type=".ST_SALESQUOTE
+                                                       ." ORDER BY sorder.order_no DESC";
+                                       return combo_input($name, '', $sql, 'order_no', 'IName',array('order'=>false));
+
                                case 'PO':
-                                       $sql = "SELECT ".TB_PREF."purch_orders.order_no, concat(".TB_PREF."purch_orders.order_no, '-',
-                                               ".TB_PREF."suppliers.supp_name) as IName
-                                               FROM ".TB_PREF."suppliers, ".TB_PREF."purch_orders WHERE ".TB_PREF."suppliers.supplier_id=".TB_PREF."purch_orders.supplier_id ORDER BY ".TB_PREF."purch_orders.order_no DESC";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       $ref = ($SysPrefs->print_invoice_no() == 1 ? "order_no" : "reference");
+                                       $sql = "SELECT po.order_no, concat(po.$ref, '-', supplier.supp_name) as IName
+                                               FROM ".TB_PREF."suppliers supplier,"
+                                                       .TB_PREF."purch_orders po
+                                               WHERE supplier.supplier_id=po.supplier_id
+                                               ORDER BY po.order_no DESC";
+                                       return combo_input($name, '', $sql, 'order_no', 'IName',array('order'=>false));
+
+                               case 'REMITTANCE':
+                                       $BP = $type_shortcuts[ST_BANKPAYMENT];
+                                       $SP = $type_shortcuts[ST_SUPPAYMENT];
+                                       $CN = $type_shortcuts[ST_SUPPCREDIT];
+                                       $ref = ($SysPrefs->print_invoice_no() == 1 ? "trans_no" : "reference");
+                                       $sql = "SELECT concat(trans.trans_no, '-',trans.type) AS TNO,
+                                                               concat(trans.$ref, IF(type=".ST_BANKPAYMENT.", ' $BP ', IF(type=".ST_SUPPAYMENT.", ' $SP ', ' $CN ')), supplier.supp_name) as IName
+                                                       FROM ".TB_PREF."suppliers supplier, "
+                                                               .TB_PREF."supp_trans trans
+                                                       WHERE type IN(".ST_BANKPAYMENT.",".ST_SUPPAYMENT.",".ST_SUPPCREDIT.")
+                                                               AND supplier.supplier_id=trans.supplier_id
+                                                       ORDER BY trans.trans_no DESC";
+                                       return combo_input($name, '', $sql, 'TNO', 'IName',array('order'=>false));
+
+                               case 'RECEIPT':
+                                       $BD = $type_shortcuts[ST_BANKDEPOSIT];
+                                       $CP = $type_shortcuts[ST_CUSTPAYMENT];
+                                       $ref = ($SysPrefs->print_invoice_no() == 1 ? "trans_no" : "reference");
+                                       $sql = "SELECT concat(trans.trans_no, '-', trans.type) AS TNO,
+                                                               concat(trans.$ref, IF(type=".ST_BANKDEPOSIT.", ' $BD ', ' $CP '), debtor.name) as IName
+                                               FROM ".TB_PREF."debtors_master debtor,"
+                                                       .TB_PREF."debtor_trans trans
+                                               WHERE type IN(".ST_BANKDEPOSIT.",".ST_CUSTPAYMENT.",".ST_CUSTCREDIT.") AND debtor.debtor_no=trans.debtor_no
+                                               ORDER BY trans.trans_no DESC";
+                                       return combo_input($name, '', $sql, 'TNO', 'IName',array('order'=>false));
+
                                case 'ITEMS':
-                                       $sql = "SELECT stock_id, concat(stock_id, '-', description) as name FROM ".TB_PREF."stock_master WHERE (mb_flag='A' OR mb_flag='M') ORDER BY stock_id";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       return stock_manufactured_items_list($name);
+
+                               case 'ITEMS_ALL':
+                                       return stock_manufactured_items_list($name, null, true);
+
+                               case 'ITEMS_P':
+                                       return stock_purchasable_items_list($name, null, true);
+                               
                                case 'WORKORDER':
-                                       $sql = "SELECT ".TB_PREF."workorders.id, concat(".TB_PREF."workorders.id, '-',
-                                               ".TB_PREF."stock_master.description) as IName
-                                               FROM ".TB_PREF."stock_master, ".TB_PREF."workorders WHERE ".TB_PREF."stock_master.stock_id=".TB_PREF."workorders.stock_id ORDER BY ".TB_PREF."workorders.id DESC";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       $sql = "SELECT wo.id, concat(wo.id, '-', item.description) as IName
+                                               FROM ".TB_PREF."stock_master item,"
+                                                       .TB_PREF."workorders wo
+                                               WHERE item.stock_id=wo.stock_id
+                                               ORDER BY wo.id DESC";
+                                       return combo_input($name, '', $sql, 'id', 'IName',array('order'=>false));
+
                                case 'LOCATIONS':
-                                       $sql = "SELECT loc_code, location_name FROM ".TB_PREF."locations ORDER BY location_name";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Location Filter"), false);
-                                       break;
+                                       return  locations_list($name, null, _("No Location Filter"));
+                               case 'FLOCATIONS':
+                                       return  locations_list($name, null, _("No Location Filter"), false, true);
+
                                case 'CATEGORIES':
-                                       $sql = "SELECT category_id, description FROM ".TB_PREF."stock_category ORDER BY description";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Category Filter"), true);
-                                       break;
+                                       return stock_categories_list($name, null, _("No Category Filter"));
+                               case 'FCATEGORIES':
+                                       return stock_categories_list($name, null, _("No Category Filter"), false, true);
+                               case 'FCLASS':
+                                       return fixed_asset_classes_list($name, null, _("No Class Filter"), false);
                                case 'SALESTYPES':
-                                       $sql = "SELECT id, sales_type FROM ".TB_PREF."sales_types ORDER BY sales_type";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       return sales_types_list($name);
+
                                case 'AREAS':
-                                       $sql = "SELECT area_code, description FROM ".TB_PREF."areas ORDER BY description";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Area Filter"), true);
-                                       break;
+                                       return sales_areas_list($name, null, _("No Areas Filter"));
+
                                case 'SALESMEN':
-                                       $sql = "SELECT salesman_code, salesman_name FROM ".TB_PREF."salesman ORDER BY salesman_name";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Sales Folk Filter"), true);
-                                       break;
+                                       return sales_persons_list($name, null, _("No Sales Folk Filter"));
+                                       
                                case 'TRANS_YEARS':
-                                       $sql = "SELECT DISTINCT YEAR(tran_date), YEAR(tran_date) FROM ".TB_PREF."gl_trans";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       return fiscalyears_list($name);
+
                                case 'USERS':
-                                       $sql = "SELECT id, user_id FROM ".TB_PREF."users ORDER BY user_id";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy, true, _("No Users Filter"), true);
-                                       break;
-                               case 'ACCOUNTS_NO_FILTER':
-                                       $sql = "SELECT id, name FROM ".TB_PREF."chart_types ORDER BY name";
-                                       $st .= dup_simple_codeandname_list($sql, "PARAM_$index", $dummy);
-                                       break;
+                                       $sql = "SELECT id, user_id FROM ".TB_PREF."users";
+                                       return combo_input($name, '', $sql, 'id', 'user_id',array('spec_option'=>_("No Users Filter"),'spec_id'=>ALL_NUMERIC));
+
+                               case 'ACCOUNTTAGS':
+                               case 'DIMENSIONTAGS':
+                                       if ($type == 'ACCOUNTTAGS')
+                                               $tag_type = TAG_ACCOUNT;
+                                       else
+                                               $tag_type = TAG_DIMENSION;
+                                       return tag_list($name, 5, $tag_type, true);
 
                        }
-                       $st .= "<br><br>";
-               }
-               return $st;
+               return '';
        }
 }
 
-class ReportParam
+class Report
 {
-       var $param_name;
-       var $param_type;
-
-       function ReportParam($param_name, $param_type)
+       var $id;
+       var $name;
+       var $ar_params;
+       var $controls;
+       
+       function __construct($id, $name, $ar_params = null)
        {
-               $this->param_name = $param_name;
-               $this->param_type = $param_type;
+               $this->id = $id;
+               $this->name = $name;
+               if ($ar_params) $this->set_controls($ar_params);
        }
-}
-
-function dup_simple_codeandname_list($sql, $name, &$selected_id,
-       $all_option=false, $all_option_name=null, $all_option_numeric=false,
-       $submit_on_change=false)
-{
-       if ($submit_on_change == true)
-               $st = "<select name='$name' onchange='this.form.submit();'>";
-       else
-               $st = "<select name='$name'>";
-
-       if ($all_option == true)
-       {
-               if ($all_option_numeric)
-                       $reserved_word = ALL_NUMERIC;
-               else
-                       $reserved_word = ALL_TEXT;
-
-       if ($reserved_word == $selected_id)
-       {
-            $st .= "<option selected value='" . $reserved_word . "'>" . $all_option_name . "</option>\n";
-       }
-       else
-       {
-            $st .= "<option value='" . $reserved_word . "'>" . $all_option_name . "</option>\n";
-       }
-               if ($selected_id == "")
-               {
-                       $selected_id =  $reserved_word;
-               }
+       
+       function set_controls($ar_params) {
+               $this->controls = $ar_params;
        }
-
-       $result = db_query($sql);
-
-       while ($row = db_fetch_row($result))
-       {
-               if ($selected_id == $row[0])
-               {
-                       $st .= "<option selected value='" . $row[0] . "'>";
-               }
-               else
-               {
-                       $st .= "<option value='" . $row[0] . "'>";
-               }
-               $st .= $row[1] . "</option>\n";
-
-               if ($selected_id == "")
-               {
-                       $selected_id = $row[0];
-               }
+       
+       function get_controls() {
+               return $this->controls;
        }
-
-       $st .= "</select>";
-       db_free_result($result);
-
-       return $st;
-}
-
-function dup_simple_name_list($name, $selections)
-{
-       $st = "<select name='$name'>";
-       for ($i = 0; $i < count($selections); $i++)
-               $st .= "<option value='" . $i . "'>" . $selections[$i] . "</option>\n";
-       $st .= "</select>";
-       return $st;
+       
 }
 
 //------------------------------------------------------------------------------------------------
-
-function dup_systypes_list($name, &$selected_id, $all_option=false, $all_option_name=null, $all_types=false)
+function gl_systypes_list($name, $value=null, $spec_opt=false)
 {
        global $systypes_array;
-
-       $st = "<select name='$name'>";
-       if ($all_option == true)
-       {
-               $reserved_word = ALL_NUMERIC;
-       if ($reserved_word == $selected_id)
-            $st .= "<option selected value='" . $reserved_word . "'>" . $all_option_name . "</option>\n";
-       else
-            $st .= "<option value='" . $reserved_word . "'>" . $all_option_name . "</option>\n";
-               if ($selected_id == "")
-                       $selected_id = $reserved_word;
-       }
-
-       foreach ($systypes_array as $i => $value)
+       
+       $types = $systypes_array;
+       
+       foreach(array(ST_LOCTRANSFER, ST_PURCHORDER, ST_SUPPRECEIVE, ST_MANUISSUE,
+                               ST_MANURECEIVE, ST_SALESORDER, ST_SALESQUOTE, ST_DIMENSION) as $type)
+               unset($types[$type]);
+
+       return array_selector($name, $value, $types, 
+               array( 
+                       'spec_option'=> $spec_opt,
+                       'spec_id' => ALL_NUMERIC,
+                       'async' => false,
+                       )
+       );
+}
+/*
+       Add extension/custom reports to report manager.
+*/
+function add_custom_reports(&$reports)
+{
+       global $installed_extensions, $path_to_root;
+       
+       // include reports installed inside extension modules
+       if (count($installed_extensions) > 0)
        {
-               if (!$all_types && ($i == 16 || $i == 18 || $i == 25 || ($i >= 28 && $i <= 30) || $i >= 40))
-                       continue;
-       $st .= "<option" . ($selected_id == $i ? " selected" : "") . " value='$i'>" . $value['name'] . "</option>\n";
+               $extensions = $installed_extensions;
+               foreach ($extensions as $ext)
+                       if (($ext['active'] && $ext['type'] == 'extension')) {
+                               $file = $path_to_root.'/'.$ext['path']."/reporting/reports_custom.php";
+                               if (file_exists($file)) {
+                                       set_ext_domain($ext['path']);
+                                       include_once($file);
+                                       set_ext_domain();
+                               }
+                       }
        }
-    $st .= "</select>";
-       return $st;
+       $file = company_path()."/reporting/reports_custom.php";
+       if (file_exists($file))
+               include_once($file);
 }
 
-
-?>
\ No newline at end of file