[Freeipa-devel] [PATCH] 054 Fix checking for arguments in DNS plugins

2011-02-12 Thread Jakub Hrozek
I couldn't reproduce the traceback, but the code shows where the error
most probably is.

http://fedorahosted.org/freeipa/ticket/956
>From c9a9bc8c316c0c921a2865d600aaedc8135c8552 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek 
Date: Sat, 12 Feb 2011 15:36:19 -0500
Subject: [PATCH] Fix checking for arguments in DNS plugins

https://fedorahosted.org/freeipa/ticket/956
---
 ipalib/plugins/dns.py |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/ipalib/plugins/dns.py b/ipalib/plugins/dns.py
index fea48db..1437011 100644
--- a/ipalib/plugins/dns.py
+++ b/ipalib/plugins/dns.py
@@ -146,7 +146,7 @@ _record_validators = {
 def has_cli_options(entry, no_option_msg):
 entry = dict((t, entry.get(t, [])) for t in _record_attributes)
 numattr = reduce(lambda x,y: x+y,
- map(lambda x: len(x), entry.values()))
+ map(lambda x: len(x), [ v for v in entry.values() if v is 
not None ]))
 if numattr == 0:
 raise errors.OptionError(no_option_msg)
 return entry
-- 
1.7.4

___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 054 Fix checking for arguments in DNS plugins

2011-02-14 Thread Adam Young

On 02/12/2011 03:45 PM, Jakub Hrozek wrote:

I couldn't reproduce the traceback, but the code shows where the error
most probably is.

http://fedorahosted.org/freeipa/ticket/956


___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

 ACK, pushed to master
___
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel