[Calypso] [PATCH] Use item when possible, rather than collection.
Guido Günther
agx at sigxcpu.org
Thu Apr 21 23:39:40 PDT 2016
On Thu, Apr 14, 2016 at 01:03:18AM +0000, Jelmer Vernooij wrote:
> From: Jelmer Vernooij <jelmer at jelmer.uk>
>
> This matters, as item can represent a subcollection.
LGTM but I guess chrysn and yourself know more about this portion of
code.
Cheers,
-- Guido
> ---
> calypso/xmlutils.py | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/calypso/xmlutils.py b/calypso/xmlutils.py
> index 0715764..6d92658 100644
> --- a/calypso/xmlutils.py
> +++ b/calypso/xmlutils.py
> @@ -159,10 +159,10 @@ def propfind(path, xml_request, collection, depth, context):
> for tag in props:
> element = ET.Element(tag)
> if tag == _tag("D", "resourcetype") and is_collection:
> - if collection.is_calendar:
> + if item.is_calendar:
> tag = ET.Element(_tag("C", "calendar"))
> element.append(tag)
> - if collection.is_addressbook:
> + if item.is_addressbook:
> tag = ET.Element(_tag("A", "addressbook"))
> element.append(tag)
> tag = ET.Element(_tag("D", "collection"))
> @@ -179,9 +179,9 @@ def propfind(path, xml_request, collection, depth, context):
> elif tag == _tag("D", "getetag"):
> element.text = item.etag
> elif tag == _tag("D", "displayname") and is_collection:
> - element.text = collection.name
> + element.text = item.name
> elif tag == _tag("E", "calendar-color") and is_collection:
> - element.text = collection.color
> + element.text = item.color
> elif tag == _tag("D", "principal-URL"):
> # TODO: use a real principal URL, read rfc3744-4.2 for info
> tag = ET.Element(_tag("D", "href"))
> @@ -223,7 +223,7 @@ def propfind(path, xml_request, collection, depth, context):
> element.append(tag)
> elif tag in (_tag("A", "addressbook-description"),
> _tag("C", "calendar-description")) and is_collection:
> - element.text = collection.get_description()
> + element.text = item.get_description()
> prop.append(element)
>
> status = ET.Element(_tag("D", "status"))
> --
> 2.8.0.rc3
>
> _______________________________________________
> Calypso mailing list
> Calypso at keithp.com
> http://keithp.com/mailman/listinfo/calypso
More information about the Calypso
mailing list