[freeside-commits] freeside/httemplate/browse part_pkg.cgi, 1.51, 1.52

[Ivan,,,]

Update of /home/cvs/cvsroot/freeside/httemplate/browse
In directory wavetail.420.am:/tmp/cvs-serv31035/browse

Modified Files:
	part_pkg.cgi 
Log Message:
package definition browse/search, filter by package class, RT#5255

Index: part_pkg.cgi
===================================================================
RCS file: /home/cvs/cvsroot/freeside/httemplate/browse/part_pkg.cgi,v
retrieving revision 1.51
retrieving revision 1.52
diff -u -d -r1.51 -r1.52
--- part_pkg.cgi	2 Apr 2009 00:14:30 -0000	1.51
+++ part_pkg.cgi	31 May 2009 06:59:37 -0000	1.52
@@ -1,6 +1,7 @@
 <% include( 'elements/browse.html',
                  'title'                 => 'Package Definitions',
                  'html_init'             => $html_init,
+                 'html_posttotal'        => $html_posttotal,
                  'name'                  => 'package definitions',
                  'disableable'           => 1,
                  'disabled_statuspos'    => 3,
@@ -48,10 +49,10 @@
   $orderby = 'num_active DESC';
 }
 
-my $extra_sql = '';
+my @where = ();
 
 #if ( $cgi->param('activeONLY') ) {
-#  $extra_sql = ' WHERE num_active > 0 '; #XXX doesn't affect count...
+#  push @where, ' WHERE num_active > 0 '; #XXX doesn't affect count...
 #}
 
 if ( $cgi->param('recurring') ) {
@@ -59,21 +60,30 @@
   $extra_count = ' freq != 0 ';
 }
 
-if ( $cgi->param('missing_recur_fee') ) {
-  my $missing = "0 = ( SELECT COUNT(*) FROM part_pkg_option
-                         WHERE optionname = 'recur_fee'
-                           AND part_pkg_option.pkgpart = part_pkg.pkgpart
-                           AND CAST ( optionvalue AS NUMERIC ) > 0
-                     )";
-  $extra_sql .= ( ( scalar(keys %hash) || $extra_sql ) ? ' AND ' : ' WHERE ' ).
-                $missing;
+my $classnum = '';
+if ( $cgi->param('classnum') =~ /^(\d+)$/ ) {
+  $classnum = $1;
+  push @where, $classnum ? "classnum =  $classnum"
+                         : "classnum IS NULL";
 }
+$cgi->delete('classnum');
 
-unless ( $acl_edit_global ) {
-  $extra_sql .= ( ( scalar(keys %hash) || $extra_sql ) ? ' AND ' : ' WHERE ' ).
-                 FS::part_pkg->curuser_pkgs_sql;
+if ( $cgi->param('missing_recur_fee') ) {
+  push @where, "0 = ( SELECT COUNT(*) FROM part_pkg_option
+                        WHERE optionname = 'recur_fee'
+                          AND part_pkg_option.pkgpart = part_pkg.pkgpart
+                          AND CAST ( optionvalue AS NUMERIC ) > 0
+                    )";
 }
 
+push @where, FS::part_pkg->curuser_pkgs_sql
+  unless $acl_edit_global;
+
+my $extra_sql = scalar(@where)
+                ? ( scalar(keys %hash) ? ' AND ' : ' WHERE ' ).
+                  join( 'AND ', @where)
+                : '';
+
 my $agentnums = join(',', $curuser->agentnums);
 my $count_cust_pkg = "
   SELECT COUNT(*) FROM cust_pkg LEFT JOIN cust_main USING ( custnum )
@@ -117,6 +127,31 @@
   !;
 #}
 
+$cgi->param('dummy', 1);
+
+my $filter_change =
+  qq(\n<SCRIPT TYPE="text/javascript">\n).
+  "function filter_change() {".
+  "  window.location = '". $cgi->self_url.
+       ";classnum=' + document.getElementById('classnum').options[document.getElementById('classnum').selectedIndex].value".
+  "}".
+  "\n</SCRIPT>\n";
+
+#restore this so pagination works
+$cgi->param('classnum', $classnum) if length($classnum);
+
+my $html_posttotal =
+  "$filter_change\n<BR>( show class: ".
+  include('/elements/select-pkg_class.html',
+            #'curr_value'    => $classnum,
+            'value'         => $classnum, #insist on 0 :/
+            'onchange'      => 'filter_change()',
+            'pre_options'   => [ '-1' => 'all',
+                                 '0'  => '(none)', ],
+            'disable_empty' => 1,
+         ).
+  ' )';
+
 # ------
 
 my $link = [ $p.'edit/part_pkg.cgi?', 'pkgpart' ];