[lmi-commits] [lmi] master 09b367e 1/5: Further restrict "weird" 0% columns

[Greg Chicares]

branch: master
commit 09b367efdb6420c9e2021983bde12f0ef59f5650
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>

    Further restrict "weird" 0% columns
    
    An old input file used a general-account product, but selected some
    zero-percent columns that should be available only for separate-account
    products. These weird columns were available because inforce separate
    account value exceeded zero, which shouldn't be possible (and indeed
    that field is grayed in lmi today, but presumably wasn't a decade ago
    when this file was created). XSL-FO allowed an illustration to be
    printed, although columns of nonexistent separate-account values were
    all blank. However, the newer wxPdfDoc replacement threw an exception,
    which is a reasonable behavior, yet one that end users would not
    welcome. With this commit, the impermissible columns are removed from
    the supplemental report, and unavailable in the GUI, for products that
    don't offer a separate as well as a general account.
---
 input_harmonization.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/input_harmonization.cpp b/input_harmonization.cpp
index 62661e3..3aefcf2 100644
--- a/input_harmonization.cpp
+++ b/input_harmonization.cpp
@@ -845,7 +845,9 @@ false // Silly workaround for now.
     weird_report_columns.push_back(mce_guaranteed_0_cash_surrender_value       
   );
 
     bool enable_weirdness =
-           0.0 < InforceGeneralAccountValue .value()
+           allow_gen_acct
+        && allow_sep_acct
+        && 0.0 < InforceGeneralAccountValue .value()
         && 0.0 < InforceSeparateAccountValue.value()
         ;