[Freeipa-devel] [PATCH] ignore empty values in multi-valued UI attribute

[Rob Crittenden <rcritten redhat com>]

When converting from a multi-valued UI attribute back to a list drop any 
blank values. This will avoid errors in the UniqueList() validator.

rob

>From 1b2a4476a0db6594b8d5f985a747357b4ebd75bc Mon Sep 17 00:00:00 2001
From: Rob Crittenden <rcritten redhat com>
Date: Tue, 3 Jun 2008 14:55:24 -0400
Subject: [PATCH] Ignore empty values when converting a multi-valued form field back to a list.
 This will prevent validation errors when validate.UniqueList() is executed.

449785
---
 ipa-server/ipa-gui/ipagui/helpers/ipahelper.py |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/ipa-server/ipa-gui/ipagui/helpers/ipahelper.py b/ipa-server/ipa-gui/ipagui/helpers/ipahelper.py
index 511e28d..4eb7644 100644
--- a/ipa-server/ipa-gui/ipagui/helpers/ipahelper.py
+++ b/ipa-server/ipa-gui/ipagui/helpers/ipahelper.py
@@ -53,7 +53,8 @@ def fix_incoming_fields(fields, fieldname, multifieldname):
     fields[fieldname] = []
     try:
         for i in range(len(fields[multifieldname])):
-            fields[fieldname].append(fields[multifieldname][i][fieldname])
+            if fields[multifieldname][i][fieldname] is not None and len(fields[multifieldname][i][fieldname]) > 0:
+                fields[fieldname].append(fields[multifieldname][i][fieldname])
         del(fields[multifieldname])
     except Exception, e:
         logging.warn("fix_incoming_fields error: " + str(e))
-- 
1.5.4

Attachment: smime.p7s
Description: S/MIME Cryptographic Signature