fix: use context for formatted date in event_view #127

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

Member
Gagaro commented Jan 22, 2014

Using context allows the localization of the date.

It's already the case in event_listing: https://github.com/plone/plone.app.event/blob/master/plone/app/event/browser/event_listing.py#L200

Owner
thet commented Feb 11, 2014

in that case, the occurrences are already IOccurrence and IEvent objects, as returned by IRecurrenceSupport. so no need to access the context of an IEventAccessor as done in the example you referenced.

I agree this is confusing, so i changed the code (plone.app.event.base.dates_for_display and plone.app.event.browser.event_view.get_location) to be able to take IEventAccessor as well as IEvent and IOcurrence based objects in this commit:
6fdd64a

if you still have some specific issues with that, let me know.

@thet thet closed this Feb 11, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment