Skip to content
This repository has been archived by the owner. It is now read-only.

isPseudoCalendarCollection() and isCalendarCollection() used on possibly non CalDAVResource resources #54

Closed
macosforgebot opened this issue Sep 26, 2006 · 10 comments

Comments

@macosforgebot
Copy link

@macosforgebot macosforgebot commented Sep 26, 2006

@cyrusdaboo originally submitted this as ticket:63


There are some uses of isPseudoCalendarCollection() and isCalendarCollection() that may operate on a resource that is not a CalDAVResource and hence does not implement that method (resulting in an unhandled exception): report_common.applyToCalendarCollections is one example. These need to be changed to use the generic (and resource type safe) isPseudoCalendarCollectionResource() or isCalendarCollectionResource().

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Sep 26, 2006

@cyrusdaboo originally submitted this as comment:1:⁠ticket:63


PS This was seen when a client attempted a calendar report on the principal hierarchy.

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Sep 26, 2006

@wsanchez originally submitted this as comment:2:⁠ticket:63

  • Milestone changed from CalDAV-access to Preview 1
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Apr 19, 2007

@wsanchez originally submitted this as comment:3:⁠ticket:63

  • Milestone changed from Preview 1 to Version One
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Jul 30, 2007

@wsanchez originally submitted this as comment:4:⁠ticket:63

  • Milestone changed from 1.0 to 2.0
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Oct 26, 2009

@wsanchez originally submitted this as comment:5:⁠ticket:63

  • Milestone changed from CalendarServer-2.x to CalendarServer-3.x
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Aug 1, 2011

@wsanchez originally submitted this as comment:6:⁠ticket:63

  • Radar deleted

Let's get rid of these methods entirely in 4.0, please.

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Aug 1, 2011

@wsanchez originally submitted this as comment:7:⁠ticket:63

  • Milestone changed from CalendarServer-3.x to CalendarServer-4.x
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Jan 24, 2013

@wsanchez originally submitted this as comment:11:⁠ticket:63

  • Milestone changed from CalendarServer-4.x to CalendarServer-6.x
@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Jan 7, 2014

@wsanchez originally submitted this as comment:12:⁠ticket:63

  • Milestone changed from CalendarServer-6.x to Later

I think the real plan is going to have to be to rewrite all of the resource hierarchy in terms of twisted.web so we can toss out txweb2. So… later.

@macosforgebot
Copy link
Author

@macosforgebot macosforgebot commented Mar 26, 2015

@wsanchez originally submitted this as comment:13:⁠ticket:63

  • Status changed from new to closed
  • Resolution set to Not to be fixed

Expiring old bugs with unknown state and impact.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.