diff options
author | Ivan Kohler <ivan@freeside.biz> | 2017-01-31 18:45:44 -0800 |
---|---|---|
committer | Ivan Kohler <ivan@freeside.biz> | 2017-01-31 18:45:44 -0800 |
commit | 0549def83bcba56e53482ea0f33472e18b215649 (patch) | |
tree | 46f89b3b2eb91667cf9a931a5921fffce67df5ad | |
parent | ed33058882b17e252dba3bd27051daeb0773f02b (diff) |
add condition_sql optimization to "Customer does not have uncancelled package of specified definitions, RT#74449
-rw-r--r-- | FS/FS/part_event/Condition/hasnt_pkgpart.pm | 21 |
1 files changed, 16 insertions, 5 deletions
diff --git a/FS/FS/part_event/Condition/hasnt_pkgpart.pm b/FS/FS/part_event/Condition/hasnt_pkgpart.pm index 421d023..79104eb 100644 --- a/FS/FS/part_event/Condition/hasnt_pkgpart.pm +++ b/FS/FS/part_event/Condition/hasnt_pkgpart.pm @@ -22,19 +22,30 @@ sub option_fields { ); } +#false laziness w/has_pkgpart.pm + sub condition { my( $self, $object ) = @_; my $cust_main = $self->cust_main($object); - #XXX test my $unless_pkgpart = $self->option('unless_pkgpart') || {}; ! grep $unless_pkgpart->{ $_->pkgpart }, $cust_main->ncancelled_pkgs; } -#XXX -#sub condition_sql { -# -#} +sub condition_sql { + my( $self, $table ) = @_; + + 'NOT '. + 'ARRAY'. $self->condition_sql_option_option_integer('unless_pkgpart'). + ' && '. #overlap (have elements in common) + 'ARRAY( SELECT pkgpart FROM cust_pkg AS has_pkgpart_cust_pkg + WHERE has_pkgpart_cust_pkg.custnum = cust_main.custnum + AND ( has_pkgpart_cust_pkg.cancel IS NULL + OR has_pkgpart_cust_pkg.cancel = 0 + ) + ) + '; +} 1; |