[Rt-commit] r13355 - in rt/3.8/branches/3.8.0-releng: .

jesse at bestpractical.com jesse at bestpractical.com
Tue Jun 17 13:17:55 EDT 2008


Author: jesse
Date: Tue Jun 17 13:17:54 2008
New Revision: 13355

Modified:
   rt/3.8/branches/3.8.0-releng/   (props changed)
   rt/3.8/branches/3.8.0-releng/lib/RT/Graph/Tickets.pm

Log:
 r32996 at 31b (orig r13344):  ruz | 2008-06-16 19:15:17 -0400
 * add error handling when there is no property callback defined
   for graphs and FillUsing argument


Modified: rt/3.8/branches/3.8.0-releng/lib/RT/Graph/Tickets.pm
==============================================================================
--- rt/3.8/branches/3.8.0-releng/lib/RT/Graph/Tickets.pm	(original)
+++ rt/3.8/branches/3.8.0-releng/lib/RT/Graph/Tickets.pm	Tue Jun 17 13:17:54 2008
@@ -239,7 +239,12 @@
     
     if ( $args{'FillUsing'} ) {
         my ($key, @subkeys) = $self->_SplitProperty( $args{'FillUsing'} );
-        my $value = $property_cb{ $key }->( $args{'Ticket'}, @subkeys );
+        my $value;
+        if ( $property_cb{ $key } ) {
+            $value = $property_cb{ $key }->( $args{'Ticket'}, @subkeys );
+        } else {
+            $RT::Logger->error("Couldn't find property callback for '$key'");
+        }
         if ( defined $value && length $value && $value =~ /\S/ ) {
             my $fill = $fill_cache{ $value };
             $fill = $fill_cache{ $value } = shift @available_colors


More information about the Rt-commit mailing list