blob: b5334b73da1c8c5a62666e212dbe9aaa74f780f8 (
plain)
Thanks for coming up with a patch.. Let me make sure I understand its
rationalle.
The meta plugin already allows modifying the page creation time,
which is what the calendar plugin uses.
So, it seems to me that the use of this patch is for recording events in
the future. You'd not want a page for a future event to claim it was
created in the future. I suppose you could also use it for events in the
past, if you didn't want to change the creation time for some reason.
(Perhaps you're doing a calendar of historical events, for example.)
Accurate? --[[Joey]]
Thanks for the feedback. Thinking about what you said ... I suspect my patch
doesn't belong in the calendar plugin, which does a very specific thing
(create a calendar to show when blog posts were created). I'm really angling
toward an event calendar (as mentioned on [[todo/plugin]]). I'd like to preserve
the page creation time - which is useful and important information in its own right
- and be able to generate a calendar with links to particular posts that will show
up on the calendar based on an arbitrary date. Perhaps this should be re-considered
as a separate plugin? --[[Jamie]]
I think it makes sense to have only one calendar, if possible.
I think your event stuff is fine, the only thing we might want to add
is a config option for the calendar, to control whether it looks at the
event date, or the creation date. --[[Joey]]
Ok - I can work on that. One question - the existing calendar module has it's own
functions for building an html display of a calendar. HTML::CalendarMonth seems to
provide that functionality. My instincts are to rip out the code in the calendar plugin
and use the existing module. On the other hand, that creates added dependencies.
Suggestions anyone? --[[Jamie]]
I'm all for ripping code out of ikiwiki where CPAN can be used, as
long as the resulting code and html are good. --[[Joey]]
Sounds good. I'll work on HTML::CalendarMonth for the next version. In the current version I
did the event date vs. creation date option as a parameter to the calendar
preprocessor rather than as a config variable so you could do it differently on
different calendars in the same wiki. I also opted for values of auto vs. event
rather than creation time vs. event since if you want to use creation time you
can simply not include the event preprocessor directive. auto seems to give you that
plus more flexibility. Feedback welcome :). --[[Jamie]]
|