[freeside-commits] branch FREESIDE_3_BRANCH updated. 4fbac69c76d393adc7221c036a996a2ab1b860f1

[Mark Wells]

The branch, FREESIDE_3_BRANCH has been updated
       via  4fbac69c76d393adc7221c036a996a2ab1b860f1 (commit)
      from  87829bfd7466f835e5d4bd37e6c8b0d03a0daed9 (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 4fbac69c76d393adc7221c036a996a2ab1b860f1
Author: Mark Wells <mark at freeside.biz>
Date:   Thu Mar 20 11:52:14 2014 -0700

    when enumerating the subsets of a set, include the null set, #27973

diff --git a/FS/FS/part_pkg_report_option.pm b/FS/FS/part_pkg_report_option.pm
index 372b119..e3c353a 100644
--- a/FS/FS/part_pkg_report_option.pm
+++ b/FS/FS/part_pkg_report_option.pm
@@ -146,9 +146,13 @@ sub subsets {
     or die $dbh->errstr; # seriously, this should never happen
   $sth->execute
     or die $sth->errstr;
-  # return the first (only) column
-  map { [ split(',',$_->[0]) ],
-        [ split(',',$_->[1]) ] } @{ $sth->fetchall_arrayref };
+  # return a pair of entries for the null set (conventionally we use zero
+  # for that)
+  ( [ 0 ], [ '(empty class)' ],
+  # followed by the first two columns: report class numbers and names
+    map { [ split(',',$_->[0]) ],
+          [ split(',',$_->[1]) ] } @{ $sth->fetchall_arrayref }
+  );
 }
 
 =back

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

Summary of changes:
 FS/FS/part_pkg_report_option.pm |   10 +++++++---
 1 files changed, 7 insertions(+), 3 deletions(-)