option to override requestor in new tickets, #7777
authormark <mark>
Fri, 4 Mar 2011 02:26:15 +0000 (02:26 +0000)
committermark <mark>
Fri, 4 Mar 2011 02:26:15 +0000 (02:26 +0000)
FS/FS/Conf.pm
FS/FS/TicketSystem/RT_External.pm
httemplate/elements/dashboard-toplist.html
httemplate/view/cust_main/tickets.html

index d42956b..00b0cec 100644 (file)
@@ -2570,6 +2570,13 @@ and customer address. Include units.',
   },
 
   {
+    'key'         => 'ticket_system-requestor',
+    'section'     => '',
+    'description' => 'Email address to use as the requestor for new tickets.  If blank, the customer\'s invoicing address(es) will be used.',
+    'type'        => 'text',
+  },
+
+  {
     'key'         => 'ticket_system-priority_reverse',
     'section'     => '',
     'description' => 'Enable this to consider lower numbered priorities more important.  A bad habit we picked up somewhere.  You probably want to avoid it and use the default.',
index 2995b88..8a8c3ff 100644 (file)
@@ -285,6 +285,12 @@ sub href_params_new_ticket {
     $cust_main = qsearchs('cust_main', { 'custnum' => $custnum } );
   }
 
+  # explicit $requestors > config option > invoicing_list
+  $requestors = $conf->config('ticket_system-requestor')
+      if !$requestors;
+  $requestors = $cust_main->invoicing_list_emailonly_scalar
+      if (!$requestors) and defined($cust_main);
+
   my %param = (
     'Queue'       => ($cust_main->agent->ticketing_queueid || $default_queueid),
     'new-MemberOf'=> "freeside://freeside/cust_main/$custnum",
index d8cd7f3..6e250b6 100644 (file)
@@ -27,7 +27,9 @@
             <% include('/elements/mcp_lint.html', 'cust_main'=>$cust_main) %>
           </TD>
          <TD CLASS="grid" BGCOLOR="<% $bgcolor %>" ALIGN="right">
-           <FONT SIZE="-1"><A HREF="<% FS::TicketSystem->href_new_ticket($cust_main, join(', ', grep { $_ !~ /^(POST|FAX)$/ } $cust_main->invoicing_list ) ) %>">(new ticket)</A></FONT>
+           <FONT SIZE="-1">
+            <A HREF="<% FS::TicketSystem->href_new_ticket($cust_main) %>">(new ticket)</A>
+            </FONT>
          </TD>
 
 %         foreach my $priority ( @custom_priorities, '' ) {
index eea5a7f..064f511 100644 (file)
@@ -106,13 +106,9 @@ my $res_link  = FS::TicketSystem->href_customer_tickets(
                   { 'statuses' => [ 'resolved' ] }
                 );
 
-my( $new_base, %new_param ) = FS::TicketSystem->href_params_new_ticket(
-                 $cust_main,
-                 join(', ', $cust_main->invoicing_list_emailonly )               );
+my( $new_base, %new_param ) = 
+  FS::TicketSystem->href_params_new_ticket( $cust_main );
 
-my $new_link = FS::TicketSystem->href_new_ticket(
-                 $cust_main,
-                 join(', ', $cust_main->invoicing_list_emailonly )
-               );
+my $new_link = FS::TicketSystem->href_new_ticket( $cust_main );
 
 </%init>