# HG changeset patch
# User Sylvain Thénault <sylvain.thenault@logilab.fr>
# Date 1279273168 -7200
# Node ID 6dc04e75c8e12543b89bd08b3e290736221bafe7
# Parent  561b6c2eb288ff4e2f797a9bb04def75fc6111bf
[entity] fix setdefault implementation: should return actual value

diff -r 561b6c2eb288 -r 6dc04e75c8e1 entity.py
--- a/entity.py	Fri Jul 16 11:00:42 2010 +0200
+++ b/entity.py	Fri Jul 16 11:39:28 2010 +0200
@@ -382,13 +382,14 @@
 
     def setdefault(self, attr, default):
         """override setdefault to update self.edited_attributes"""
-        self.cw_attr_cache.setdefault(attr, default)
+        value = self.cw_attr_cache.setdefault(attr, default)
         # don't add attribute into skip_security if already in edited
         # attributes, else we may accidentaly skip a desired security check
         if hasattr(self, 'edited_attributes') and \
                attr not in self.edited_attributes:
             self.edited_attributes.add(attr)
             self._cw_skip_security_attributes.add(attr)
+        return value
 
     def pop(self, attr, default=_marker):
         """override pop to update self.edited_attributes on cleanup of