[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Libreplanet-dev] Calendar extension error

From: Peter
Subject: Re: [Libreplanet-dev] Calendar extension error
Date: Sat, 20 Jun 2009 10:47:40 +0000
User-agent: KMail/1.8.2

On Friday 19 June 2009 20:13, you wrote:
> Peter <address@hidden> writes:
> > A much better approach would be to add an <event title="Event name"
> > icon="event/icon.png"/> tag, which users can put anywhere on the event
> > page. Calendar can search for it and properly parse it.
> >
> > Better yet, allow a single page to have several event tags and either
> > combine them into one calendar event, or display them separately.
> >
> > By using a tag, Calendar will be able to identify the event information
> > and validate it properly. Users will be able to edit the event page as
> > they like without compromising calendar, or the website. Pages without
> > event tags can be flagged in calendar as 'invalid event', so users can
> > move or fix the page.
> Sure, that all sounds nice. I don't see much of a problem with the
> current approach though, after fixing this bug.

Reserving the first line for calendar is a poor design choice, imho. Imagine 
twenty extensions all doing the same thing. Now imagine them looking for 
their own tag, instead.

> >> I fixed this particular one, but any fix to the extension itself would
> >> be most welcome.
> >
> > By 'fixed' do you mean you changed the event page, or calendar code?
> I mean I changed the event page so it wouldn't trigger the error
> anymore.
> > I _have_ _already_ offered my patch, has it been applied? It was
> > submitted about 5 days ago. If this is the wrong place to submit patches,
> > please point me to the right place.
> We have a new intern on board who hopefully will be helping us with
> Mediawiki issues. I'll ask him to look at the patch.


> But please do also submit it upstream (the calendar extension home page
> is linked to from below the calendar).




reply via email to

[Prev in Thread] Current Thread [Next in Thread]