Properly handle multi-valued attributes when using setattr/addattr.

The problem was that the normalizer was returning each value as a tuple
which we were then appending to a list, so it looked like
[(u'value1',), (u'value2',),...]. If there was a single value we could
end up adding a tuple to a list which would fail. Additionally python-ldap
doesn't like lists of lists so it was failing later in the process as well.

I've added some simple tests for setattr and addattr.

ticket 565

1 files changed, 8 insertions, 2 deletions

diff --git a/ipalib/frontend.py b/ipalib/frontend.py
index ac1f67ee1..6be50ba52 100644
--- a/ipalib/frontend.py
+++ b/ipalib/frontend.py
@@ -538,9 +538,15 @@ class Command(HasParam):
             if attr in self.params:
                 value = self.params[attr](value)
             if append and attr in newdict:
-                newdict[attr].append(value)
+                if type(value) in (tuple,):
+                    newdict[attr] += list(value)
+                else:
+                    newdict[attr].append(value)
             else:
-                newdict[attr] = [value]
+                if type(value) in (tuple,):
+                    newdict[attr] = list(value)
+                else:
+                    newdict[attr] = [value]
         return newdict
 
     def __attributes_2_entry(self, kw):