Hia sorry for jumping in so late, when Michael asked me yesterday, I had
a vague memory of something along those lines that when I coded the line
it was definitely name in the specs afair, have in mind this is one of
the older code parts of the javascripts coded probably around March-May
2009 or so, there were a load of Ajax changes between May and Agust
afair, and I had to change a lot to be compliant with the latest specs.
So I came to the conclusion after being asked that it was one of those
changes I simply have missed.
Anyway, if we are not within the spec in this part then it is best to
simply fix it and be done with it with further discussions.
Werner
Matthias Wessendorf schrieb:
Found it on the archives of the "open" mailing list ([email protected])
Thread is called "data payload for ajax calls", started by David Geary.
(some "fighting" in there, but I don't see a line that indicates a ticket on
their issue tracker)
+1 from me on the change...
-M
On Tue, Jan 12, 2010 at 12:11 PM, Matthias Wessendorf <[email protected]> wrote:
hrm,
there was some renaming done; I guess there is no change list on the
spec, right ?
On Tue, Jan 12, 2010 at 12:07 PM, Michael Kurz <[email protected]> wrote:
I had the chance to talk to Werner about this issue and he thinks that the
name "name" was used in an older version of the spec. Created MYFACES-2481
and provided a patch for this.
- Michael
Michael Kurz schrieb:
Hi,
I found a minor issue with client side error and event handling in the
ajax api. According to the spec (14.4.1) the event data payload for the
callback functions passed to addOnEvent and addOnError should contain the
property status with the name of the event. In MyFaces this property seems
to be named "name". Is this on purpose or a bug?
Renaming "name" to "status" in
myfaces._impl.core._jsfImpl.prototype.sendEvent
makes my example work (like in Mojarra).
regards
Michael
--
Matthias Wessendorf
blog: http://matthiaswessendorf.wordpress.com/
sessions: http://www.slideshare.net/mwessendorf
twitter: http://twitter.com/mwessendorf
