summaryrefslogtreecommitdiff
path: root/FS/FS/cust_pkg
diff options
context:
space:
mode:
authorIvan Kohler <ivan@freeside.biz>2014-11-20 12:02:08 -0800
committerIvan Kohler <ivan@freeside.biz>2014-11-20 12:02:08 -0800
commit03f11df0ddb098c448c237c6640855f55abb3755 (patch)
tree2875354626f469c80efaed17641e69c6918b5bdf /FS/FS/cust_pkg
parent2d7110ecd22194c00aa81c798f9ca9056d9189dd (diff)
optimize
Diffstat (limited to 'FS/FS/cust_pkg')
-rw-r--r--FS/FS/cust_pkg/Search.pm16
1 files changed, 8 insertions, 8 deletions
diff --git a/FS/FS/cust_pkg/Search.pm b/FS/FS/cust_pkg/Search.pm
index 9cd1ff0..aacd387 100644
--- a/FS/FS/cust_pkg/Search.pm
+++ b/FS/FS/cust_pkg/Search.pm
@@ -312,10 +312,10 @@ sub search {
if (@report_option) {
# this will result in the empty set for the dangling comma case as it should
push @where,
- map{ "0 < ( SELECT count(*) FROM part_pkg_option
- WHERE part_pkg_option.pkgpart = part_pkg.pkgpart
- AND optionname = 'report_option_$_'
- AND optionvalue = '1' )"
+ map{ "EXISTS ( SELECT 1 FROM part_pkg_option
+ WHERE part_pkg_option.pkgpart = part_pkg.pkgpart
+ AND optionname = 'report_option_$_'
+ AND optionvalue = '1' )"
} @report_option;
}
@@ -331,10 +331,10 @@ sub search {
if (@report_option_any) {
# this will result in the empty set for the dangling comma case as it should
push @where, ' ( '. join(' OR ',
- map{ "0 < ( SELECT count(*) FROM part_pkg_option
- WHERE part_pkg_option.pkgpart = part_pkg.pkgpart
- AND optionname = 'report_option_$_'
- AND optionvalue = '1' )"
+ map{ "EXISTS ( SELECT 1 FROM part_pkg_option
+ WHERE part_pkg_option.pkgpart = part_pkg.pkgpart
+ AND optionname = 'report_option_$_'
+ AND optionvalue = '1' )"
} @report_option_any
). ' ) ';
}