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