[freeside-commits] branch master updated. f3145308f69d7d3d42af6132439cff26b4fc9008

[Carl J. Adams-Collier]

The branch, master has been updated
       via  f3145308f69d7d3d42af6132439cff26b4fc9008 (commit)
      from  42fd3cde3a01c5ae6e70c50d4487f36141a2adc3 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit f3145308f69d7d3d42af6132439cff26b4fc9008
Author: C.J. Adams-Collier <cjac at colliertech.org>
Date:   Fri Sep 5 10:17:16 2014 -0700

    FS RT #884; correcting date comparison logic for signup by date report

diff --git a/httemplate/graph/signupdate.cgi b/httemplate/graph/signupdate.cgi
index 23e4596..011817c 100644
--- a/httemplate/graph/signupdate.cgi
+++ b/httemplate/graph/signupdate.cgi
@@ -40,13 +40,13 @@ my $sdate = DateTime->new(
 
 my $edate = DateTime->new(
     year       => $cgi->param('end_year'),
-    month      => ($cgi->param('end_month') % 12 + 1) # first day of the next month
-)->epoch();
+    month      => $cgi->param('end_month')
+)->add( months => 1 )->epoch();
 
 my $where .= " AND signupdate >= $sdate ".
-             " AND signupdate <= $edate ";
+             " AND signupdate < $edate ";
 
-foreach my $cust (qsearch({ table   => 'cust_main', 
+foreach my $cust (qsearch({ table   => 'cust_main',
                             hashref => \%where,
                             extra_sql => $where } )) {
   next if !$cust->signupdate;

-----------------------------------------------------------------------

Summary of changes:
 httemplate/graph/signupdate.cgi |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)