[freeside-commits] freeside/httemplate/elements tr-select-reason.html, 1.3.2.1, 1.3.2.2

[Ivan,,,]

Update of /home/cvs/cvsroot/freeside/httemplate/elements
In directory wavetail:/tmp/cvs-serv20811

Modified Files:
      Tag: FREESIDE_1_7_BRANCH
	tr-select-reason.html 
Log Message:
show types in reason selection, order by type and reason

Index: tr-select-reason.html
===================================================================
RCS file: /home/cvs/cvsroot/freeside/httemplate/elements/tr-select-reason.html,v
retrieving revision 1.3.2.1
retrieving revision 1.3.2.2
diff -u -d -r1.3.2.1 -r1.3.2.2
--- tr-select-reason.html	4 Dec 2007 18:19:08 -0000	1.3.2.1
+++ tr-select-reason.html	5 Dec 2007 20:47:51 -0000	1.3.2.2
@@ -37,14 +37,9 @@
   <TD ALIGN="right">Reason</TD>
   <TD>
     <SELECT id="<% $name %>" name="<% $name %>" onFocus="sh_add<% $name %>()" onChange="sh_add<% $name %>()">
-%    my @reasons = qsearch( { table =>'reason', 
-%                             hashref => {},
-%                             extra_sql => $extra_sql,
-%                             addl_from => 'LEFT JOIN reason_type ON reason_type.typenum = reason.reason_type',
-%			     });
       <OPTION VALUE="" <% ($init_reason eq "") ? 'SELECTED' : '' %>>Select Reason...</OPTION>
 %    foreach my $reason (@reasons) {
-      <OPTION VALUE="<% $reason->reasonnum %>" <% ($init_reason == $reason->reasonnum) ? 'SELECTED' : '' %>><% $reason->reason %></OPTION>
+      <OPTION VALUE="<% $reason->reasonnum %>" <% ($init_reason == $reason->reasonnum) ? 'SELECTED' : '' %>><% $reason->reasontype->type %> : <% $reason->reason %></OPTION>
 %    }
 %    if ($curuser->access_right($access_right)) {
       <OPTION VALUE="-1" <% ($init_reason == -1) ? 'SELECTED' : '' %>>Add new reason</OPTION>
@@ -80,7 +75,7 @@
   </TD>
   <TD>
     <SELECT id="new<% $name %>T" name="new<% $name %>T" disabled="<% $disabled %>" style="display:<% $display %>">
-%     for my $type (@types){
+%     for my $type (@types) {
         <OPTION VALUE="<% $type->typenum %>" <% ($init_type == $type->typenum) ? 'SELECTED' : '' %>><% $type->type %></OPTION>
 %     }
     </SELECT>
@@ -97,7 +92,7 @@
 
 <%init>
 my($name, $class, $init_reason, $init_type, $init_newreason, $controlledbutton) = @_;
-my($extra_sql, $curuser, $access_right, $display, $disabled); 
+my($access_right, $display, $disabled); 
 
 if ($class eq 'C') {
   $access_right='Add on-the-fly cancel reason';
@@ -117,9 +112,18 @@
   $disabled = 'true';
 }
 
-$extra_sql = "WHERE class = '$class' and (disabled = '' OR disabled is NULL) ".
-             "ORDER BY reason_type";
-$curuser = $FS::CurrentUser::CurrentUser;
+my $extra_sql =
+  "WHERE class = '$class' and (disabled = '' OR disabled is NULL)";
 
-</%init>
+my @reasons = qsearch({
+  table     => 'reason', 
+  hashref   => {},
+  extra_sql => $extra_sql,
+  addl_from => 'LEFT JOIN reason_type '.
+               ' ON reason_type.typenum = reason.reason_type',
+  order_by  => 'reason_type.type ASC, reason.reason ASC',
+});
 
+my $curuser = $FS::CurrentUser::CurrentUser;
+
+</%init>