===================================================================
RCS file: /cvs/RT/Invoicing/rt_invoices.pl,v
retrieving revision 1.13
retrieving revision 1.25
diff -u -r1.13 -r1.25
--- RT/Invoicing/rt_invoices.pl	2011/03/22 02:16:53	1.13
+++ RT/Invoicing/rt_invoices.pl	2011/04/19 04:18:13	1.25
@@ -1,5 +1,5 @@
 #!/usr/bin/perl
-# $AFresh1$
+# $AFresh1: rt_invoices.pl,v 1.24 2011/04/12 19:32:15 andrew Exp $
 ########################################################################
 # Copyright (c) 2011 Andrew Fresh <andrew@afresh1.com>
 #
@@ -20,12 +20,12 @@
 
 use 5.010;
 
-use YAML::Any;
 use Template;
 use RT::Client::REST;
 use RT::Client::REST::Ticket;
 use RT::Client::REST::User;
 
+use File::Path;
 use DateTime;
 
 my $config = RTI::Config->new();
@@ -40,89 +40,13 @@
 
 $rt->login( username => $rt_conf->{user}, password => $rt_conf->{pass} );
 
+#use YAML;
 #print Dump $config, $state; exit;
 
 my $startdate;
 
 my $customers = $config->get('customers');
-CUSTOMER: while ( my ( $custid, $cust ) = each %{$customers} ) {
 
-    my $invoice = make_invoice();
-
-    if ( $cust->{base_rate} ) {
-        my $day  = $cust->{day}       || 1;
-        my $freq = $cust->{frequency} || 1;
-
-        my $per;
-        my $day_method;
-        given ( $cust->{per} ) {
-            when ('week')  { $per = 'weeks';  $day_method = 'dow' }
-            when ('month') { $per = 'months'; $day_method = 'day' }
-            default { die "Unknown per [$cust->{per}]\n" }
-        }
-
-        my $billends = $invoice->{end}->clone;
-        $billends->subtract( days => 1 ) while $billends->$day_method != $day;
-
-        my $date = $billends->clone->subtract( $per => $freq );
-
-        my $lastinvoice = $state->last_invoice($custid);
-        if ( $lastinvoice->{date} ) {
-            my $last_invoice_date = ymd_to_DateTime( $lastinvoice->{date} );
-            $date = $last_invoice_date->clone->add( days => 1 );
-        }
-
-        next CUSTOMER if $billends <= $date;
-
-        my $title
-            = $freq == 1
-            ? ucfirst( $cust->{per} . 'ly' )
-            : $freq . ' ' . ucfirst( $cust->{per} );
-        $title .= ' Retainer';
-
-        my %project = ( title => $title, fees => [], );
-
-        while ( $date < $billends ) {
-            my $start = $date->clone;
-
-            $date->add( $per => $freq );
-            $date = $billends->clone if $date > $billends;
-
-            # XXX This is helpful, but monthly and billday > 28 == !!!
-            $date->subtract( days => 1 ) while $date->$day_method != $day;
-
-            my $end = $date->clone->subtract( seconds => 1 );
-
-            next if $end < $start;
-
-            $startdate = $start->clone if !$startdate || $startdate > $start;
-            $invoice->{start} ||= $start->clone;
-            $invoice->{end} = $end->clone;
-            my %hours = (
-                start => $start->clone,
-                end   => $end->clone,
-                hours => { %{ $cust->{hours} } },
-            );
-
-            push @{ $invoice->{hours} }, \%hours;
-            push @{ $project{fees} },
-                {
-                count    => 1,
-                rate     => $cust->{base_rate},
-                contents => $start->ymd . ' to ' . $end->ymd,
-                };
-
-        }
-
-        if ( @{ $project{fees} } ) {
-            push @{ $invoice->{projects} }, \%project;
-        }
-    }
-
-    $cust->{address} ||= get_user($custid);
-    $cust->{invoice} = $invoice;
-}
-
 my @limits = map +{
     attribute  => 'status',
     operator   => '=',
@@ -152,23 +76,25 @@
 
 my $iterator = $results->get_iterator;
 while ( my $ticket = &$iterator ) {
-    my $cust = find_customer_for_ticket( $customers, $ticket );
+    my $cust = find_customer_for_ticket( $ticket );
     if ( !$cust ) {
         warn "No customer found for ticket " . $ticket->id;
         next;
     }
-    if ( !$cust->{invoice} ) {
 
-        #say "Customer has no open invoices for ticket " . $ticket->id;
+    say 'Ticket ' . $ticket->id . " belongs to $cust->{id}";
+
+    $cust->{invoice} ||= make_invoice($cust);
+    if ( !$cust->{invoice}->{end} ) {
+        say "  $cust->{id} has no open invoices [" . $ticket->id . ']';
         next;
     }
-    my $invoice = $cust->{invoice};
 
     my $project = make_project( $ticket, $cust );
     next unless @{ $project->{fees} } || @{ $project->{expenses} };
 
     foreach my $fee ( @{ $project->{fees} } ) {
-        my $hours = hours_for_date( $invoice, $fee->{date} );
+        my $hours = hours_for_date( $cust->{invoice}, $fee->{date} );
 
         my $h_type
             = exists $hours->{ $fee->{type} }
@@ -188,22 +114,23 @@
         }
 
         if ($discount_time) {
-            $invoice->{discount}{amount} += $discount_time * $fee->{rate};
-            $invoice->{discount}{hours}{$h_type} += $discount_time;
+            $cust->{invoice}->{discount}->{amount}
+                += round( $discount_time * $fee->{rate} );
+            $cust->{invoice}->{discount}->{hours}{$h_type} += $discount_time;
 
             $h_type = '' if $h_type eq 'default';
             $fee->{detail} = "$discount_time $h_type Hours Discounted";
         }
     }
 
-    push @{ $invoice->{projects} }, $project;
+    push @{ $cust->{invoice}->{projects} }, $project;
 }
 
-while ( my ( $custid, $cust ) = each %{$customers} ) {
+foreach my $cust ( @{$customers} ) {
     my $invoice = $cust->{invoice};
-    next unless $invoice->{projects} && @{ $invoice->{projects} };
+    next unless $invoice && $invoice->{projects} && @{ $invoice->{projects} };
 
-    my %li = ( custid => $custid, invdate => DateTime->now->ymd, );
+    my %li = ( custid => $cust->{id}, invdate => DateTime->now->ymd, );
 
     foreach my $project ( @{ $invoice->{projects} } ) {
         if ( $project->{transactions} ) {
@@ -240,14 +167,18 @@
 
     next unless $invoice->{total} > 0 || $invoice->{total_due};
 
-    # XXX Here we need to "make_address"
     $invoice->{info} = $config->get('info');
-    $invoice->{from} = make_address( $config->get('from') );
-    $invoice->{to}   = make_address( $cust->{address} );
+    my $from = $config->get('from');
+    $from = get_user($from) if !ref $from;
 
+    $invoice->{organization} = $from->{organization} || $from->{name};
+    $invoice->{from}         = make_address($from);
+    $invoice->{to}           = make_address( $cust->{address} || $cust->{id} );
+
     $state->{lastinvoice}++;
-    $invoice->{id}   = $state->{lastinvoice};
-    $invoice->{file} = 'invoice_' . $state->{lastinvoice} . '.pdf';
+    $invoice->{id} = $state->{lastinvoice};
+    $invoice->{file} = sprintf 'invoice_%06d.pdf', $state->{lastinvoice};
+    $invoice->{logo} = $config->get('logo');
 
     foreach my $k (qw/ file transactions start end total past_due total_due /)
     {
@@ -271,11 +202,18 @@
             $invoice->{$key} = $invoice->{$key}->strftime('%B %d, %Y');
         }
     }
-    my $tt = Template->new;
-    $tt->process( 'invoice.tex.tt', $invoice, $invoice->{file} )
+
+    my $invoice_dir = $config->get('invoice_dir');
+    File::Path::make_path($invoice_dir);
+    my $file = join '/', $invoice_dir, $invoice->{file};
+
+    my $tt = Template->new( INCLUDE_PATH => $config->get('template_dir'), )
+        || die $Template::ERROR, "\n";
+
+    $tt->process( $config->get('invoice_template'), $invoice, $file )
         or die $tt->error . "\n";
 
-    printf "Generated %s for %s: \$%.02f\n", $invoice->{file}, $custid,
+    printf "Generated %s for %s: \$%.02f\n", $invoice->{file}, $cust->{id},
         $invoice->{total};
 }
 
@@ -290,60 +228,36 @@
 }
 
 sub find_customer_for_ticket {
-    my ( $customers, $ticket ) = @_;
+    my ($ticket) = @_;
 
-INVOICE: foreach my $cust ( values %{$customers} ) {
-        next INVOICE unless $cust->{match};
+    foreach my $cust ( @{$customers} ) {
+        next unless $cust->{match};
         foreach my $m ( @{ $cust->{match} } ) {
             my $type = $m->{type};
-            my $thing = join ' ', $ticket->$type;
+            my $match = exists $m->{$type} ? lc($m->{$type}) : qr/\Q$m->{regex}\E/;
+            my $thing = [ map { lc } $ticket->$type ];
 
-            if ( $m->{$type} ) {
-                my $match = lc $m->{$type};
-                next INVOICE unless lc($thing) ~~ $match;
-            }
-            elsif ( $m->{regex} ) {
-                next INVOICE unless $thing ~~ /\Q$m->{regex}\E/;
-            }
-            else {
+            if (! $match) {
                 warn "Invalid match!";
-                next INVOICE;
+                next;
             }
+            return $cust if ($match ~~ $thing);
         }
-        return $cust;
     }
 
-    return fake_customer( $customers, $ticket );
-}
-
-sub fake_customer {
-    my ( $customers, $ticket ) = @_;
-
-    # make the custid the first requestor
-    my ($custid) = $ticket->requestors;
-    return unless $custid;
-
+    # Fake customer if we didn't find one
     my $cust = $config->get('default') || {};
-    $cust->{address} = get_user($custid);
 
-    $cust->{match} = [{
-            type => 'requestors',
-            regex => $custid,
-    }];
+    ( $cust->{id} ) = $ticket->requestors;
+    return unless $cust->{id};
+    push @{$customers}, $cust;
 
-    my $invoice = make_invoice();
+    $cust->{match} = [
+        {   type  => 'requestors',
+            regex => $cust->{id},
+        }
+    ];
 
-    my $lastinvoice = $state->last_invoice($custid);
-    if ( $lastinvoice->{date} ) {
-        my $last_invoice_date = ymd_to_DateTime( $lastinvoice->{date} );
-            $invoice->{start} = $last_invoice_date->clone->add( days => 1 );
-    }
-
-    if ( !( $invoice->{start} && $invoice->{start} < $invoice->{end} ) ) {
-        $cust->{invoice} = $invoice;
-    }
-
-    $customers->{$custid} = $cust;
     return $cust;
 }
 
@@ -381,18 +295,136 @@
 }
 
 sub make_invoice {
+    my ($cust) = @_;
 
-    my $billends
-            = DateTime->now->set( hour => 0, minute => 0, second => 0 );
-    my %invoice = ( end => $billends->clone->subtract( days => 1, seconds => 1 ) );
+    my $day  = $cust->{day}       ||= 0;
+    my $per  = $cust->{per}       ||= 'week';
+    my $freq = $cust->{frequency} ||= 1;
 
+    my $day_method;
+    given ($per) {
+        when ('week')  { $per = 'weeks';  $day_method = 'dow' }
+        when ('month') { $per = 'months'; $day_method = 'day' }
+        default { die "Unknown per [$per]\n" }
+    }
+
+    my $billends = DateTime->now
+        ->subtract( days => 1 )
+        ->set( hour => 23, minute => 59, second => 59 );
+
+    # XXX This is helpful, but monthly and billday > 28 == !!!
+    $billends->subtract( days => 1 )
+        while $day && $billends->$day_method != $day;
+
+    my $date = $billends->clone->subtract( $per => $freq )
+        ->set( hour => 0, minute => 0, second => 0 );
+
+    my %invoice = ( end => $billends->clone->subtract( seconds => 1 ) );
+
+    my $lastinvoice = $state->last_invoice( $cust->{id} );
+    if ( $lastinvoice->{date} ) {
+        my $last_invoice_date = ymd_to_DateTime( $lastinvoice->{date} );
+        $date = $last_invoice_date->clone->add( days => 1 );
+
+        $invoice{start} = $date->clone;
+
+        $startdate = $date->clone->subtract( $per => $freq )
+            if !$startdate || $startdate > $date;
+    }
+
+    # Is the start date more than $freq $per before the end date?
+    my $diff = $billends - $date;
+    return {} if $diff->in_units($per) < 1;
+
+    if ( $cust->{base_rate} ) {
+        my ( $project, $hours ) = make_base_project(
+            $cust,
+            {   startdate  => $date,
+                enddate    => $billends,
+                per        => $per,
+                freq       => $freq,
+                day        => $day,
+                day_method => $day_method,
+            }
+        );
+
+        if ( @{ $project->{fees} } ) {
+            $invoice{end}   = $project->{end};
+            $invoice{hours} = $hours;
+            push @{ $invoice{projects} }, $project;
+        }
+    }
+    elsif ( $cust->{hours} ) {
+        $invoice{hours} = [
+            {   end   => $invoice{end}->clone,
+                hours => $cust->{hours},
+            }
+        ];
+    }
+
+    return {} if $invoice{start} && $invoice{end} < $invoice{start};
+
     return \%invoice;
 }
 
+sub make_base_project {
+    my ( $cust, $args ) = @_;
+
+    my $date       = $args->{startdate};
+    my $enddate    = $args->{enddate};
+    my $per        = $args->{per};
+    my $freq       = $args->{freq};
+    my $day        = $args->{day};
+    my $day_method = $args->{day_method};
+
+    my $title
+        = $freq == 1
+        ? ucfirst( $cust->{per} . 'ly' )
+        : $freq . ' ' . ucfirst( $cust->{per} );
+    $title .= ' Retainer';
+
+    my %project = ( title => $title, start => $date->clone, fees => [], );
+    my @hours;
+
+    while ( $date < $enddate ) {
+        my $start = $date->clone;
+
+        $date->add( $per => $freq );
+        $date = $enddate->clone if $date > $enddate;
+
+        # XXX This is helpful, but monthly and billday > 28 == !!!
+        $date->subtract( days => 1 ) while $day && $date->$day_method != $day;
+
+        my $end = $date->clone->subtract( seconds => 1 );
+
+        last if $end < $start;
+
+        $project{end} = $end->clone;
+
+        push @{ $project{fees} },
+            {
+            count    => 1,
+            rate     => $cust->{base_rate},
+            contents => $start->ymd . ' to ' . $end->ymd,
+            };
+
+        push @hours,
+            {
+            start => $start->clone,
+            end   => $end->clone,
+            hours => { %{ $cust->{hours} } },
+            };
+    }
+
+    return \%project, \@hours;
+}
+
 sub make_address {
     my ($addr) = @_;
     my @adr;
 
+    $addr = get_user($addr) unless ref $addr;
+
     if ( $addr->{organization} ) {
         push @adr, $addr->{organization};
     }
@@ -428,8 +460,8 @@
         title  => $ticket->subject,
         detail => 'Ticket: '
             . $ticket->id
-            . ' Queue: '
-            . $ticket->queue
+            . ' Status: '
+            . $ticket->status
             . ' Requestors: '
             . join( ', ', $ticket->requestors ),
         fees     => [],
@@ -527,7 +559,7 @@
 use 5.010;
 
 use YAML::Any qw/ LoadFile Dump Load /;
-use File::Basename;
+use File::Spec;
 
 sub new {
     my ( $class, $args ) = @_;
@@ -562,9 +594,9 @@
 
     my $c = LoadFile($file) or die "Unable to load $file\n";
 
-    $c->{customers} ||= {};
+    $c->{customers} ||= [];
     if ( $c->{default} ) {
-        foreach my $cust ( values %{ $c->{customers} } ) {
+        foreach my $cust ( @{ $c->{customers} } ) {
             foreach my $k ( keys %{ $c->{default} } ) {
                 $cust->{$k} //= Load( Dump( $c->{default}->{$k} ) );
             }
@@ -578,13 +610,30 @@
 sub get {
     my ( $self, $key ) = @_;
     my $value = Load( Dump( $self->{_config}->{$key} ) );
-    if ( !$value ) {
-        given ($key) {
-            when ('state') {
-                $value = dirname( $self->{file} ) . '/rt_invoice.state'
-            }
+
+    return $value if $value;
+    my ($volume,$directories,$file) =File::Spec->splitpath(
+         File::Spec->rel2abs( $self->{file} ));
+
+    given ($key) {
+        when ('state') {
+            $value = $self->{file};
+            $value =~ s/(?:\.[^.]+)?$/\.state/;
         }
+        when ('invoice_dir') {
+            $value = File::Spec->catdir($volume, $directories, 'invoices' );
+        }
+        when ('template_dir') {
+            $value = File::Spec->catdir( $volume, $directories );
+        }
+        when ('invoice_template') {
+            $value = 'invoice.tex.tt';
+        }
+        when ('logo') {
+            $value = File::Spec->catfile($volume, $directories, 'Logo.pdf' );
+        }
     }
+
     return $value;
 }