Author: jmorliaguet
Date: Thu Jun 15 14:34:20 2006
New Revision: 3415
Modified:
cpsskins/branches/paris-sprint-2006/browser/rendering/context.py
cpsskins/branches/paris-sprint-2006/configuration/presentations/presentation.py
cpsskins/branches/paris-sprint-2006/elements/format.py
cpsskins/branches/paris-sprint-2006/standard/formats/widget.py
cpsskins/branches/paris-sprint-2006/storage/relations.py
Log:
- re-added default values when needed
Modified: cpsskins/branches/paris-sprint-2006/browser/rendering/context.py
==============================================================================
--- cpsskins/branches/paris-sprint-2006/browser/rendering/context.py
(original)
+++ cpsskins/branches/paris-sprint-2006/browser/rendering/context.py Thu Jun
15 14:34:20 2006
@@ -63,10 +63,11 @@
"""
implements(IContextInfo)
- def __init__(self, contexts={}):
+ def __init__(self, contexts=None):
# to avoid creating a reference, 'contexts' is not set directly.
self.contexts = {}
- self.update(contexts)
+ if contexts is not None:
+ self.update(contexts)
def __call__(self):
return self
Modified:
cpsskins/branches/paris-sprint-2006/configuration/presentations/presentation.py
==============================================================================
---
cpsskins/branches/paris-sprint-2006/configuration/presentations/presentation.py
(original)
+++
cpsskins/branches/paris-sprint-2006/configuration/presentations/presentation.py
Thu Jun 15 14:34:20 2006
@@ -28,5 +28,5 @@
def __init__(self, display, formats):
self.display = display
- self.formats = formats
+ self.formats = formats or []
Modified: cpsskins/branches/paris-sprint-2006/elements/format.py
==============================================================================
--- cpsskins/branches/paris-sprint-2006/elements/format.py (original)
+++ cpsskins/branches/paris-sprint-2006/elements/format.py Thu Jun 15
14:34:20 2006
@@ -37,8 +37,8 @@
types = []
- def __init__(self, types):
- self.types = types
+ def __init__(self, types=None):
+ self.types = types or []
def findDisplays(self):
"""Return the list of display elements associated to this
Modified: cpsskins/branches/paris-sprint-2006/standard/formats/widget.py
==============================================================================
--- cpsskins/branches/paris-sprint-2006/standard/formats/widget.py
(original)
+++ cpsskins/branches/paris-sprint-2006/standard/formats/widget.py Thu Jun
15 14:34:20 2006
@@ -53,8 +53,8 @@
"""
implements(IWidget)
- def __init__(self, types):
- self.types = types
+ def __init__(self, types=None):
+ self.types = types or []
def __repr__(self):
return "<Widget: %s>" % ', '.join(self.types)
Modified: cpsskins/branches/paris-sprint-2006/storage/relations.py
==============================================================================
--- cpsskins/branches/paris-sprint-2006/storage/relations.py (original)
+++ cpsskins/branches/paris-sprint-2006/storage/relations.py Thu Jun 15
14:34:20 2006
@@ -278,12 +278,12 @@
"""Remove relations for the storage.
"""
if ids is None:
- ids = []
+ raise ValueError("Must specify a list of relation ids to remove")
if isinstance(ids, basestring):
ids = [ids]
for id in ids:
if id not in self:
- raise KeyError, "No such relation (%s) in the storage." % id
+ raise KeyError("No such relation (%s) in the storage." % id)
del self[id]
logger.debug("Removed the relations with ids: '%s'", ids)
--
http://lists.nuxeo.com/mailman/listinfo/z3lab-checkins
