From 38c2bf2ba44e48d5297c6bdf052c1cdfc50bcb9b Mon Sep 17 00:00:00 2001 From: "Jeroen van Meeuwen (Kolab Systems)" Date: Fri, 14 Dec 2012 12:29:59 +0000 Subject: Fix issuing len() on a possible None --- pykolab/cli/sieve/cmd_refresh.py | 4 ++-- pykolab/plugins/sievemgmt/__init__.py | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/pykolab/cli/sieve/cmd_refresh.py b/pykolab/cli/sieve/cmd_refresh.py index 77eecc5..226fb93 100644 --- a/pykolab/cli/sieve/cmd_refresh.py +++ b/pykolab/cli/sieve/cmd_refresh.py @@ -258,7 +258,7 @@ def execute(*args, **kw): mgmt_script.require(required_extension) if vacation_active: - if len(vacation_react_domains) > 0: + if not vacation_react_domains == None and len(vacation_react_domains) > 0: mgmt_script.addfilter( 'vacation', [('envelope', ':domain', ":is", "from", vacation_react_domains)], @@ -275,7 +275,7 @@ def execute(*args, **kw): ] ) - elif len(vacation_noreact_domains) > 0: + elif not vacation_noreact_domains == None and len(vacation_noreact_domains) > 0: mgmt_script.addfilter( 'vacation', [('not', ('envelope', ':domain', ":is", "from", vacation_noreact_domains))], diff --git a/pykolab/plugins/sievemgmt/__init__.py b/pykolab/plugins/sievemgmt/__init__.py index f341eae..1e22a47 100644 --- a/pykolab/plugins/sievemgmt/__init__.py +++ b/pykolab/plugins/sievemgmt/__init__.py @@ -271,7 +271,7 @@ class KolabSievemgmt(object): mgmt_script.require(required_extension) if vacation_active: - if len(vacation_react_domains) > 0: + if not vacation_react_domains == None and len(vacation_react_domains) > 0: mgmt_script.addfilter( 'vacation', [('envelope', ':domain', ":is", "from", vacation_react_domains)], @@ -288,7 +288,7 @@ class KolabSievemgmt(object): ] ) - elif len(vacation_noreact_domains) > 0: + elif not vacation_noreact_domains == None and len(vacation_noreact_domains) > 0: mgmt_script.addfilter( 'vacation', [('not', ('envelope', ':domain', ":is", "from", vacation_noreact_domains))], -- cgit v1.1