11 May
2013
11 May
'13
4:48 p.m.
Hi devs,
I'd like to copy all Document*Event in
xwiki-platform-model/src/main/java/org/xwiki/model/event/* and deprecate (move to legacy)
all the ones in the bridge module.
It seems they don't need XWikiContext, XWikiDocument or the like so it should be
pretty easy.
WDYT?
Thanks
-Vincent
12 May
12 May
8:37 a.m.
New subject: [xwiki-devs] [Proposal] Move Document Events from bridge to model module (REMOVED)
Actually, after starting the move on my computer and more thinking, I don't think we
can do this now.
Technically there's no issue, we could move the events to the model module.
The problem is a problem of what will happen in the future. Right now in the locations
where we send the events, we pass XWikiDocument and XWikiContext as parameters to notify()
and the receivers are *expecting* to receive them. In the future we'll want to pass a
new model object for the document. When we do this, even though the event classes
won't need to change a bit, all listeners will be broken if the senders continue to
send the same events with different values. Thus we'll need new events when we do this
to not break backward compatibility…
And thus it's better to keep the model namespace free for when we'll have the new
model…
I'm thus removing this proposal.
Thanks
-Vincent
On May 11, 2013, at 4:48 PM, Vincent Massol <vincent(a)massol.net> wrote:
Hi devs,
I'd like to copy all Document*Event in
xwiki-platform-model/src/main/java/org/xwiki/model/event/* and deprecate (move to legacy)
all the ones in the bridge module.
It seems they don't need XWikiContext, XWikiDocument or the like so it should be
pretty easy.
WDYT?
Thanks
-Vincent
13 May
13 May
8:41 a.m.
New subject: [xwiki-devs] [Proposal] Move Document Events from bridge to model module (REMOVED)
I would have voted -1 anyway, I insisted for this event to not be in
the model module the first time for a reason...
On Sun, May 12, 2013 at 8:37 AM, Vincent Massol <vincent(a)massol.net> wrote:
Actually, after starting the move on my computer and
more thinking, I don't think we can do this now.
Technically there's no issue, we could move the events to the model module.
The problem is a problem of what will happen in the future. Right now in the locations
where we send the events, we pass XWikiDocument and XWikiContext as parameters to notify()
and the receivers are *expecting* to receive them. In the future we'll want to pass a
new model object for the document. When we do this, even though the event classes
won't need to change a bit, all listeners will be broken if the senders continue to
send the same events with different values. Thus we'll need new events when we do this
to not break backward compatibility…
And thus it's better to keep the model namespace free for when we'll have the new
model…
I'm thus removing this proposal.
Thanks
-Vincent
On May 11, 2013, at 4:48 PM, Vincent Massol <vincent(a)massol.net> wrote:
--
Thomas Mortagne
Hi devs,
I'd like to copy all Document*Event in
xwiki-platform-model/src/main/java/org/xwiki/model/event/* and deprecate (move to legacy)
all the ones in the bridge module.
It seems they don't need XWikiContext, XWikiDocument or the like so it should be
pretty easy.
WDYT?
Thanks
-Vincent
_______________________________________________
devs mailing list
devs(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/devs
9:20 a.m.
New subject: [xwiki-devs] [Proposal] Move Document Events from bridge to model module (REMOVED)
On May 13, 2013, at 8:41 AM, Thomas Mortagne <thomas.mortagne(a)xwiki.com> wrote:
I would have voted -1 anyway, I insisted for this
event to not be in
the model module the first time for a reason…
Sure and I read that email thread about 10 times slowly but since you didn't explain
why precisely, I didn't understand… Which is why I took the time to explain it below
(which would have saved me several hours if you had in your first email ;)).
Precisely you said "we can't move them to model module since theses events still
send XWikiContext and XWikiDocument" which is obviously not correct. These events do
not use XWikiContext nor XWikiDocument… It's the code that sends those events that use
them and we can't move them *because* we don't want to reserve the namespace so
that it's available the future… Not that obvious to figure out for my stupid brain… :)
Sorry not to be more intelligent...
I've now created http://jira.xwiki.org/browse/XWIKI-9137 to remember it.
Thanks
-Vincent
On Sun, May 12, 2013 at 8:37 AM, Vincent Massol
<vincent(a)massol.net> wrote:
> Actually, after starting the move on my computer and more thinking, I don't think
we can do this now.
>
> Technically there's no issue, we could move the events to the model module.
>
> The problem is a problem of what will happen in the future. Right now in the
locations where we send the events, we pass XWikiDocument and XWikiContext as parameters
to notify() and the receivers are *expecting* to receive them. In the future we'll
want to pass a new model object for the document. When we do this, even though the event
classes won't need to change a bit, all listeners will be broken if the senders
continue to send the same events with different values. Thus we'll need new events
when we do this to not break backward compatibility…
>
> And thus it's better to keep the model namespace free for when we'll have the
new model…
>
> I'm thus removing this proposal.
>
> Thanks
> -Vincent
>
>
> On May 11, 2013, at 4:48 PM, Vincent Massol <vincent(a)massol.net> wrote:
>
>> Hi devs,
>>
>> I'd like to copy all Document*Event in
xwiki-platform-model/src/main/java/org/xwiki/model/event/* and deprecate (move to legacy)
all the ones in the bridge module.
>>
>> It seems they don't need XWikiContext, XWikiDocument or the like so it should
be pretty easy.
>>
>> WDYT?
>>
>> Thanks
>> -Vincent
4432
days inactive
4434
days old
3 comments
2 participants
participants (2)
-
Thomas Mortagne -
Vincent Massol