Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.
Comment: Migration of unmigrated content due to installation of a new plugin

...

[14:09:32 CDT(-0500)] <Justin_o> Bosmon: thanks

[14:14:08 CDT(-0500)] <cindyli> Bosmon: in IoC testing system, can I pass an argument list to an event listener? for example,

[14:14:10 CDT(-0500)] <cindyli> sequence: [{

[14:14:10 CDT(-0500)] <cindyli> listener: "fluid.tests.checkTopComponents",

[14:14:10 CDT(-0500)]

Wiki Markup
 &lt;cindyli&gt;                     event: &#034;{fluid.tests.videoPlayerMediaPanels}.events.onReady&#034;

[14:14:10 CDT(-0500)] <cindyli> }]

[14:14:52 CDT(-0500)] <cindyli> i tried to pass an "args" option but doesn't work

[14:15:38 CDT(-0500)] <Bosmon> cindyli - "doesn't work" is the classic terrible problem report (smile)

[14:15:48 CDT(-0500)] <cindyli> haha

[14:15:53 CDT(-0500)] <Bosmon> You can see some examples of "args" in the "TestingTest.js" test cases

[14:16:03 CDT(-0500)] <Bosmon> What kind of "doesn't work" did you experience?

[14:16:31 CDT(-0500)] <Bosmon> I guess there isn't an example of "args" with "listener" ....

[14:16:38 CDT(-0500)] <cindyli> i did, the test cases with "args" are all for modelChanged type of listener, not any for event listeners

[14:17:12 CDT(-0500)] <cindyli> well, modelChanged is an event too i guess, but perhaps being processed differently

[14:17:31 CDT(-0500)] <Bosmon> I see

[14:17:36 CDT(-0500)] <Bosmon> Actually, what would you want "args" to do in this case?

[14:17:43 CDT(-0500)] <Bosmon> I guess some boiling

[14:17:48 CDT(-0500)] <cindyli> exactly

[14:18:01 CDT(-0500)] <Bosmon> What in particular do you want to boil into the signature?

[14:18:06 CDT(-0500)] <cindyli> and "expectedResult", "testType"

[14:18:20 CDT(-0500)] <Bosmon> aha

[14:18:29 CDT(-0500)] <Bosmon> Yes, it seems this isn't implemented

[14:18:33 CDT(-0500)] <Bosmon> Would you like to implement it? : P

[14:18:54 CDT(-0500)] <cindyli> sure

[14:19:35 CDT(-0500)] <Bosmon> I'll be happy to review your pull (smile)

[14:19:35 CDT(-0500)] <cindyli> a bit scary to modify framework code but you will review it anyway

[14:19:54 CDT(-0500)] <Bosmon> Well, I think it's time for people to stop being scared (smile)

[14:20:04 CDT(-0500)] <cindyli> will give a try

[14:20:30 CDT(-0500)] <Bosmon> It should only take an "if" statement and a call to fluid.expandOptions...

[14:20:41 CDT(-0500)] <Bosmon> Actually my latest pull request with declarative change binding contains a good example of how to do this

[14:20:50 CDT(-0500)] <Bosmon> The current framework code for event boiling is very convoluted

[14:20:50 CDT(-0500)] <cindyli> ah ha

[14:20:57 CDT(-0500)] <Bosmon> But the change binding pull takes a much more direct approach

[16:01:44 CDT(-0500)] <yzen_> Bosmon: hi