fluid-work IRC Logs-2010-12-15
[07:51:21 CST(-0600)] <Justin_o> anastasiac: can FLUID-3881 be closed now... has everyone checked in about the unsupported features?
[07:52:23 CST(-0600)] <anastasiac> Justin_o, yes, I think FLUID-3881 can be closed. It's quite likely that some functions that have been in the framework, etc. for quite a while might actually be "unsupported," but I think we need to take care of that as we go; there's too much to hold up the release
[07:52:30 CST(-0600)] <anastasiac> anything new has been considered
[07:52:42 CST(-0600)] <Justin_o> anastasiac: thanks...
[07:53:07 CST(-0600)] <Justin_o> did bosmon happen to comment at all about the reorderer functions that were made public?
[07:56:37 CST(-0600)] <Justin_o> anastasiac: what's happening with FLUID-3799?
[07:57:35 CST(-0600)] <anastasiac> Justin_o, two things: a) I posted an email to the list about the question of whether or not our demos should be written as components. Until we actually have a bit of a discussion about that, I think we should wait, because it affects at least three demos
[07:58:02 CST(-0600)] <anastasiac> b) I posted a patch on the JIRA that addresses Bosmon's other issue, with some hard-coded selectors. I've assigned the JIRA to Bosmon for review
[07:58:08 CST(-0600)] <anastasiac> so it's in his hands
[07:58:11 CST(-0600)] <anastasiac> or for someone else to review
[07:58:38 CST(-0600)] <Justin_o> anastasiac: thank you
[08:01:02 CST(-0600)] <Justin_o> anastasiac: the demos that would have to be rewritten.. are they all shown through the demo portal, or is that including the standalone and integration demos?
[08:01:21 CST(-0600)] <anastasiac> no, portal demos: keyboard-a11y plugin, progress and inline edit
[08:01:51 CST(-0600)] <jessm> anastasiac: inline-edit isn't a component in the portal?
[08:02:08 CST(-0600)] <anastasiac> the demo application is not, itself, implemented as a fluid component
[08:02:11 CST(-0600)] * anastasiac double-checks this
[08:02:31 CST(-0600)] <anastasiac> the demo app uses the inline edit component, but it itself isn't a component
[08:03:23 CST(-0600)] <jessm> ok, my sense is the following: let's focus on getting 1.3 out the door. And this can be a planning point for 1.4 since we're slated to do a portal facelift for that release
[08:03:44 CST(-0600)] <jessm> i just don't think we have the time to take this on for 1.3 and it might just be a distraction to us
[08:03:51 CST(-0600)] <jessm> does that seem reasonable?
[08:04:11 CST(-0600)] <Justin_o> jessm: i would agree with that
[08:04:52 CST(-0600)] <jessm> Justin_o: i'm feeling urgency for 1.3 – i'm assuming that's appropriate since it's Wednesday – correct me if i'm wrong!
[08:05:02 CST(-0600)] <jessm> anastasiac: does that seem reasonable?
[08:05:14 CST(-0600)] <Justin_o> jessm: i think you're right... we're already 3 days overdue on testing
[08:05:40 CST(-0600)] <anastasiac> yes, jessm, that's what I was thinking. If we decide to change the demos, we can do it for 1.4
[08:05:59 CST(-0600)] <jessm> cool, all eyes on 1.3 then
[08:25:43 CST(-0600)] <heidi_> hey anastasiac is there more linting to do
[08:26:25 CST(-0600)] <anastasiac> heidi_, that depends on how far you and jhung got on the tests last night
[08:26:36 CST(-0600)] <anastasiac> I didn't get a chance to do anything on the tests, just framework
[08:26:41 CST(-0600)] <heidi_> k jhung how are we doin on /tests linting
[08:26:52 CST(-0600)] <anastasiac> your patch has been attached to the JIRA with all the others, awaiting review
[08:27:07 CST(-0600)] <heidi_> k
[08:27:14 CST(-0600)] <anastasiac> heidi_, did you get through all of the component test files, or only some?
[08:27:20 CST(-0600)] <heidi_> all
[08:27:36 CST(-0600)] <anastasiac> ok, good. and jhung was working on the other test files?
[08:28:04 CST(-0600)] <heidi_> anastasiac do you know the jira # off hand
[08:28:17 CST(-0600)] <heidi_> yeah, but not sure if jhung was able to work late to finish them - there were a lot
[08:30:09 CST(-0600)] <anastasiac> heidi_, the jira is 3757; jhung said he didn't get to the framework test files, so I'm going to have a look at them now
[08:30:37 CST(-0600)] <jhung> heidi_, anastasiac: I was working on the Escalated test but discovered that the tabs test is missing a dependency. I'm planning on fixing that today.
[08:31:02 CST(-0600)] <heidi_> jhung anastasiac okay i'll do /test-core
[08:31:04 CST(-0600)] <jhung> justin_o: Patch-i of FLUID-3487 looks good in IE6 and IE7.
[08:31:15 CST(-0600)] <heidi_> or sorry
[08:31:26 CST(-0600)] <heidi_> i meant /manual-tests !
[08:31:55 CST(-0600)] <heidi_> do we need to lint /test-core ? are those our js or external
[08:32:08 CST(-0600)] <anastasiac> jhung, when you're done that, maybe you can help with the framework tests - there are a lot!
[08:32:24 CST(-0600)] <jhung> Sure.
[08:32:59 CST(-0600)] <heidi_> justin_o ^
[08:33:32 CST(-0600)] <anastasiac> jhung, heidi_, just a reminder to use the JSLint settings described on the wiki: http://wiki.fluidproject.org/display/fluid/Coding+and+Commit+Standards
[08:33:41 CST(-0600)] <heidi_> yep
[08:33:47 CST(-0600)] <heidi_> and detab
[09:25:02 CST(-0600)] <colinclark> michelled: hey fish
[09:25:24 CST(-0600)] <colinclark> I'm reading over Justin_o's extremely helpful update email
[09:25:33 CST(-0600)] <colinclark> I see that we've got two frameworky reviews on our plate before QA testing can start
[09:25:46 CST(-0600)] <colinclark> I've had some coffee and I'm raring to go
[09:25:55 CST(-0600)] <colinclark> How do you want to split up FLUID-3819 and FLUID-3778?
[09:26:44 CST(-0600)] <colinclark> Justin_o: I've closed http://issues.fluidproject.org/browse/FLUID-3487
[09:26:52 CST(-0600)] <Justin_o> colinclark: thank you
[09:26:54 CST(-0600)] <colinclark> It's good to go
[09:27:59 CST(-0600)] <colinclark> Justin_o: I've also reviewed and closed http://issues.fluidproject.org/browse/FLUID-3730
[09:28:15 CST(-0600)] <Justin_o> colinclark: thank you for that one too
[09:28:29 CST(-0600)] <michelled> colinclark: I've looked at 3819
[09:28:35 CST(-0600)] <colinclark> michelled: excellent
[09:28:38 CST(-0600)] <colinclark> Shall I grab 3778?
[09:28:43 CST(-0600)] <michelled> sure
[09:28:52 CST(-0600)] <colinclark> ok
[09:31:09 CST(-0600)] <colinclark> michelled: DOS line ending changes by nature are difficult to review
[09:31:23 CST(-0600)] <colinclark> so I'll just do a scan to ensure that the files that were changed are generally sensible
[09:31:34 CST(-0600)] <colinclark> seem reasonable?
[09:31:43 CST(-0600)] <michelled> yes, seems ok
[09:32:35 CST(-0600)] <heidi_> for http://build.fluidproject.org/infusion/tests/manual-tests/html/dynamic-reorderer.html , is the 'new div' button supposed to do something?
[09:32:49 CST(-0600)] <colinclark> And then run all the unit tests
[09:35:04 CST(-0600)] <colinclark> heidi_: Maybe a good place to start is to look at the implementation and see what it is intended to do?
[09:36:39 CST(-0600)] <heidi_> k. this one is also lookin rough http://build.fluidproject.org/infusion/tests/manual-tests/html/renderer-component-types.html
[09:41:59 CST(-0600)] <colinclark> Justin_o: I've just closed http://issues.fluidproject.org/browse/FLUID-3778
[09:42:14 CST(-0600)] <colinclark> Now seems like a good opportunity to hop on a subway, if you agree, Justin_o
[09:43:15 CST(-0600)] <Justin_o> colinclark: sounds good
[09:43:28 CST(-0600)] <colinclark> see y'all in the office shortly
[09:46:29 CST(-0600)] <anastasiac> Justin_o, jhung, heidi_: I've finished delinting the framework tests, just attaching a patch file now.
[09:46:36 CST(-0600)] <anastasiac> Justin_o, what's the plan for reviewing all these patches?
[09:47:05 CST(-0600)] <anastasiac> mlam, did you ever finish your delinting of framework files? can I help?
[09:47:24 CST(-0600)] <mlam> anastasiac: just finished. fixing one test
[09:48:32 CST(-0600)] <Justin_o> anastasiac: the framework ones will be reviewed and committed post release.. for the linting that has been committed we'll have to work out who will review the framework, reorderer, and pager ones...
[09:48:42 CST(-0600)] <Justin_o> for the rest we can review and commit as we go along
[09:49:16 CST(-0600)] <anastasiac> Justin_o - all the framework ones? not just the renderer?
[09:49:50 CST(-0600)] <anastasiac> i.e. committed post release
[09:51:00 CST(-0600)] <Justin_o> anastasiac: yes... i think that's what we're going to say for this release...
[09:51:29 CST(-0600)] <anastasiac> ok, well, the patches for the test files are on the JIRA
[09:52:47 CST(-0600)] <jhung> justin_o: I have fixed the broken escalated test. How should this be handled? Should I create a JIRA and attach a patch?
[09:53:53 CST(-0600)] <anastasiac> jhung, that sounds like the right approach
[09:54:11 CST(-0600)] <jhung> anastasiac: okay.
[10:05:04 CST(-0600)] <jhung> anastasiac: I've created a JIRA and uploaded a patch. http://issues.fluidproject.org/browse/FLUID-3903
[10:05:28 CST(-0600)] <anastasiac> jhung, thanks
[10:07:04 CST(-0600)] <michelled> Justin_o: I just reviewed the linting for the component files and the linting on Fluid.js and DataBinding.js
[10:07:19 CST(-0600)] <michelled> there was a tiny change to UIEnhancer which I've committed - otherwise it was all fine
[10:07:32 CST(-0600)] <Justin_o> michelled: thanks...
[10:11:32 CST(-0600)] <michelled> fluid-everyone: we are almost ready to start QA. anastasiac is going to review heidi_ linting patches and commit them. Justin_o is going to review anastasiac's linting patches and commit them
[10:12:04 CST(-0600)] <heidi_> i'm just finishing up linting for tests/test-core ... patch up soon
[10:12:11 CST(-0600)] <michelled> cool
[10:12:36 CST(-0600)] <Justin_o> heidi_, anastasiac is that all that's left that needs to be linted then?
[10:13:07 CST(-0600)] <heidi_> jhung did you get escalated tests done?
[10:13:24 CST(-0600)] <anastasiac> Justin_o, I think so - assuming the actual components, etc. are ok
[10:59:08 CST(-0600)] <Justin_o> anastasiac: I just committed your framework test linting... is there any other patch there that I can take a look at?
[11:00:00 CST(-0600)] <anastasiac> Justin_o, I haven't looked at the escalated tests patch
[11:00:42 CST(-0600)] <anastasiac> Justin_o, what's the status of the manual test? some don't seem to work properly... should I try to fix them, or leave it to 1.4 to either fix or remove them?
[11:03:20 CST(-0600)] <Justin_o> anastasiac: yah... i think maybe just filing a blocker for 1.4 to review the manual and escalated tests
[11:03:55 CST(-0600)] <Justin_o> heidi_: did you lint the test core files?
[11:04:06 CST(-0600)] <heidi_> justin_o yep just putting up patch now!
[11:04:10 CST(-0600)] <Justin_o> heidi_: thanks
[11:04:18 CST(-0600)] <Justin_o> i think that will be the last one we look at
[11:05:32 CST(-0600)] <anastasiac> Justin_o, FYI I filed that issue: http://issues.fluidproject.org/browse/FLUID-3904
[11:05:44 CST(-0600)] <heidi_> justin_o its on the issue now
[11:05:52 CST(-0600)] <Justin_o> anastasiac, heidi_ thanks
[11:06:16 CST(-0600)] <heidi_> so lint stuff finished?
[11:06:56 CST(-0600)] <Justin_o> heidi_: just need to get your last patch in... i think that will be ie
[11:06:57 CST(-0600)] <Justin_o> it
[11:07:03 CST(-0600)] <Justin_o> so we can start testing right after that
[11:07:36 CST(-0600)] <Justin_o> anastasiac: would you be able to review/commit heidi's latest patch?
[11:07:51 CST(-0600)] <anastasiac> Justin_o, will do
[11:07:57 CST(-0600)] <Justin_o> thanks
[11:18:55 CST(-0600)] * jhung enjoying his steeped gnat tea.
[11:29:02 CST(-0600)] <michelled> just don't drink the tea anastasiac has been feeding them!
[11:30:16 CST(-0600)] <jhung> michelled: Suddenly I feel like going outside in the extreme cold to drink my buggy tea.
[11:30:38 CST(-0600)] <michelled>
[11:31:10 CST(-0600)] <jhung> justin_o: oh king. What should I do now?
[11:31:58 CST(-0600)] <Justin_o> jhung: we are almost ready to test... as soon as anastasiac gets the last linting patch in... we will be ready to go
[11:32:16 CST(-0600)] <anastasiac> getting there...
[11:32:19 CST(-0600)] <Justin_o> thanks
[11:32:40 CST(-0600)] <jhung> k
[11:36:49 CST(-0600)] <anastasiac> Justin_o, I've reviewed and committed all relevant de-linting patches
[11:37:34 CST(-0600)] <Justin_o> anastasiac: thank you very much...
[11:37:56 CST(-0600)] <Justin_o> anastasiac: could you please file another jira that indicates which files we still have left to commit and make that a 1.4 blocker as well.
[11:38:05 CST(-0600)] <anastasiac> will do
[11:38:08 CST(-0600)] <Justin_o> anastasiac: thanks
[11:39:51 CST(-0600)] <Justin_o> fluid-everyone: we can now start testing Infusion 1.3.. i've just sent an e-mail to the list about it.. please make sure to read the notes at the top and bottom
[11:41:01 CST(-0600)] <heidi_> justin_o build servr has been rebuilt with newly delinted files?
[11:44:33 CST(-0600)] <Justin_o> heidi_: oops not yet
[11:44:38 CST(-0600)] <Justin_o> thanks.. i forgot about that
[11:44:43 CST(-0600)] <heidi_> np!
[11:45:07 CST(-0600)] <heidi_> i wish i was playing soccer with ya'll today! i could use some running around to warm up. so c-c-c-old here.
[11:45:52 CST(-0600)] <Justin_o> yah.. it's been pretty frigid the last few days... looking forward to running around
[11:46:10 CST(-0600)] <anastasiac> Justin_o, has the build site been re-deployed since my last commit?
[11:46:22 CST(-0600)] <anastasiac> ah, heidi_ just mentioned that
[11:46:24 CST(-0600)] <Justin_o> anastasiac: just triggered
[11:46:46 CST(-0600)] <Justin_o> anastasiac, heidi_ : it should be ready now
[11:46:54 CST(-0600)] <heidi_> cool
[11:48:59 CST(-0600)] <Justin_o> heidi_: which tests do you think you'll start with?
[11:50:01 CST(-0600)] <Justin_o> I think we can do InlineEdit ones, and progress at the moment
[11:50:12 CST(-0600)] <heidi_> justin_o i can start with those!
[11:50:52 CST(-0600)] <Justin_o> heidi_: okay.. just move the ones you will be doing over to your task list...
[11:50:57 CST(-0600)] <Justin_o> ah...do i need to make you one?
[11:53:45 CST(-0600)] <Justin_o> heidi_: i just added a task list for you...so you can fill it up as you go
[11:53:51 CST(-0600)] <heidi_> okay thanks justin
[12:02:02 CST(-0600)] <heidi_> anastasiac do i move a task from left to my name, or copy
[12:03:52 CST(-0600)] <heidi_> oh n/m, it's different now
[13:34:13 CST(-0600)] <Justin_o> jhung, jameswy are you able to help with testing?
[13:35:20 CST(-0600)] <jameswy> justin_o: Yep, but I do have a small problem: I'm running 10.5. Thoughts?
[13:35:45 CST(-0600)] <Justin_o> jameswy: you can do some winxp tests on the other machine
[13:35:58 CST(-0600)] <Justin_o> or you can use one of mine for mac 10.6 testing
[13:36:38 CST(-0600)] <michelled> fluid-everyone: let's skip dev meeting today in favour of release
[13:36:42 CST(-0600)] <michelled> any complaints?
[13:36:55 CST(-0600)] <colinclark_> +1
[13:36:56 CST(-0600)] <Justin_o> michelled: i'm fine with that
[13:36:59 CST(-0600)] <Justin_o> +1
[13:37:00 CST(-0600)] <anastasiac> +1
[13:37:49 CST(-0600)] <michelled> fluid-everyone: can you give us all a quick update on what you are working on to the moment?
[13:38:08 CST(-0600)] <heidi_> testing progress component on win xp
[13:38:08 CST(-0600)] <anastasiac> testing, of course
[13:38:16 CST(-0600)] <anastasiac> rich-text inline edit
[13:38:33 CST(-0600)] <jameswy> Fluid style guide/suggest, but starting testing within the half-hour.
[13:38:53 CST(-0600)] <michelled> I've got 4 reviews on my plate. I'll probably starte with 3825 and switch to uploader when colinclark_ and mlam are ready to do that review
[13:39:03 CST(-0600)] <colinclark_> Reviewing FLUID-3711, then other reviews
[13:39:25 CST(-0600)] <Justin_o> testing
[13:39:55 CST(-0600)] <colinclark_> mlam, jhung: ^
[13:40:22 CST(-0600)] <colinclark_> jessm: ^
[13:40:29 CST(-0600)] <colinclark_> ping ping ping
[13:41:11 CST(-0600)] <mlam> I'm ready anytime
[13:41:16 CST(-0600)] <jessm> i am working on wrapping up planning from the infrastructure mtg. then i'm onto planning
[13:41:24 CST(-0600)] <jessm> as soon as the king tells me where to jump
[13:41:40 CST(-0600)] <mlam> I'm going back to helping Cindy with integrating the uploader into Atutor
[13:41:48 CST(-0600)] <jhung> justin_o: I can do any NVDA, Win7, IE8, and FF3.6 testing.
[13:43:23 CST(-0600)] <Justin_o> jhung: could you start by doing the Win7 tasks for the inline edits and progress components
[13:46:18 CST(-0600)] <Bosmon> Hi all - I am wondering whether FLUID-3899 has fallen through the cracks so far?
[13:47:38 CST(-0600)] <mlam> Justin_o: cindyli is trying to implement the uploader from scratch again and wont' need my assistance for a little while. anything i can help with? if not, i can go back to finish up my linting task
[13:48:02 CST(-0600)] <Justin_o> mlam: sure... could you do some testing work
[13:48:13 CST(-0600)] <Justin_o> do you still have ie6 and ie7?
[13:48:17 CST(-0600)] <mlam> i have 6
[13:48:33 CST(-0600)] <Justin_o> do you mind starting in on the progress and inline edit tests with that
[13:48:52 CST(-0600)] <mlam> yup, np
[13:49:19 CST(-0600)] <jhung> fluid-everyone: make sure to keep an eye on this page while testing: http://wiki.fluidproject.org/display/fluid/Release+Testing+Tasks
[13:49:28 CST(-0600)] <jhung> So we don't duplicate.
[13:49:45 CST(-0600)] <jhung> Assign tasks to yourself if you're planning on testing.
[13:51:19 CST(-0600)] <Justin_o> fluid-everyone: thanks jhung.. so to add tasks to your name you have to edit the wiki markup... cut and paste the task from the general tasklist to your own.
[13:52:39 CST(-0600)] <jessm> Bosmon: i don't see FLUID-3899 in the bug parade, perhaps Justin_o can take a look after he gets testing rolling
[13:52:54 CST(-0600)] <Bosmon> Ok
[13:53:02 CST(-0600)] <Bosmon> I did add the issue itself to bug parade
[13:53:31 CST(-0600)] <jessm> Bosmon: oh, did you add it yesterday?
[13:53:36 CST(-0600)] <Bosmon> But I think it really is a blocker, we either have to apply the fix or withdraw the functionality completely
[13:53:38 CST(-0600)] <Bosmon> On Monday
[13:53:54 CST(-0600)] <Justin_o> Bosmon: I think we had to talk about that patch
[13:56:45 CST(-0600)] <colinclark_> Justin_o, Bosmon: Let's talk about it now
[13:56:49 CST(-0600)] <colinclark_> I see a patch here
[13:56:59 CST(-0600)] <Justin_o> colinclark_: now's probably a good time
[13:57:07 CST(-0600)] <colinclark_> Is this the issue that yura_ was struggling with yesterday?
[13:57:27 CST(-0600)] <Bosmon> It is one of them
[13:57:49 CST(-0600)] <Bosmon> It will actually cause the primeCacheFromResources call to fail for "almost everyone"
[13:58:06 CST(-0600)] <Bosmon> It turns out the test cases were passing only because they do not include the FluidIoC.js file
[13:58:23 CST(-0600)] <Bosmon> Anyone who does include it, which we imagine will be most people, will get an exception
[13:59:08 CST(-0600)] <colinclark_> Bosmon: This is pretty precarious
[13:59:11 CST(-0600)] <colinclark_> We've started QA testing
[13:59:13 CST(-0600)] <Bosmon> yura_: You seem to have falledn off IM
[13:59:23 CST(-0600)] <colinclark_> if we modify the framework now, we're going to have to restart QA again
[14:00:11 CST(-0600)] <colinclark_> I've been a little behind on email, but I didn't see anything about this issue on the list this morning
[14:00:30 CST(-0600)] <colinclark_> So my understanding is that we went ahead with testing under the assumption that this issue couldn't be patched in time.
[14:00:36 CST(-0600)] <Bosmon> Ah
[14:00:40 CST(-0600)] <colinclark_> Looks like you got the fix up at around 1 am last night
[14:00:42 CST(-0600)] <Bosmon> I brought up the issue a few times yesterday
[14:00:47 CST(-0600)] <Bosmon> No, the fix was the previous day
[14:00:56 CST(-0600)] <colinclark_> Ok
[14:01:03 CST(-0600)] <Bosmon> Various people assured me they would get time to look at it yesterday, but it just fell through the cracks I geuss
[14:01:43 CST(-0600)] <colinclark_> Was I one of them?
[14:02:04 CST(-0600)] <Justin_o> Bosmon: will the change be isolated to ioc stuff?
[14:02:44 CST(-0600)] <colinclark_> The only reference I see to FLUID-3899 in the channel yesterday was you asking a question about it, Bosmon, with no response
[14:02:45 CST(-0600)] <Bosmon> Justin_o: Yes, the change is just related to the code path for that one function
[14:03:12 CST(-0600)] <Bosmon> Well anyway, I doubt the exact history of what happened with this issue will decide what we do about it
[14:03:25 CST(-0600)] <Bosmon> The question is, what are we inclined to do NOW, now we see what situation we are in
[14:03:38 CST(-0600)] <colinclark_> I'm going to take a look at the patch closely now, Bosmon
[14:03:46 CST(-0600)] <Bosmon> Thanks, colinclark_
[14:04:05 CST(-0600)] <colinclark_> Can you please do an Infusion-wide search for any references to primeCacheFromResources()?
[14:04:12 CST(-0600)] <Bosmon> There are none
[14:04:13 CST(-0600)] <colinclark_> My theory is that there are no dependencies here
[14:04:15 CST(-0600)] <colinclark_> ok
[14:04:17 CST(-0600)] <Bosmon> Except in its definition and its test cases
[14:04:26 CST(-0600)] <Justin_o> colinclark_, Bosmon: if the code change is in fact isolated to IoC and won't affect other parts of the framework or other code then we should be okay because we haven't started testing the uploader yet.
[14:04:34 CST(-0600)] <Bosmon> The Reorderer also depends on IoC now
[14:04:35 CST(-0600)] <colinclark_> But you've modified withComponent(), which is central to every use of IoC
[14:04:40 CST(-0600)] <Justin_o> Bosmon: really
[14:04:40 CST(-0600)] <colinclark_> ok
[14:04:43 CST(-0600)] <Justin_o> didn't know that
[14:04:46 CST(-0600)] <Bosmon> As does the ARIA labelling component
[14:05:03 CST(-0600)] <Justin_o> okay.. we haven't started testing Reorderer yet
[14:05:07 CST(-0600)] <Bosmon> Although this is "sneak peek" as a public API, in implementation, large parts of the framework have started to depend on it
[14:05:30 CST(-0600)] <Bosmon> Which is, ironically, why the issue is so serious... most people will include IoC to have the framework work, and then primeCacheFromResources will fail
[14:06:09 CST(-0600)] <colinclark_> okay, here we go...
[14:06:22 CST(-0600)] <colinclark_> So, the patch touches two public functions:
[14:06:29 CST(-0600)] <colinclark_> fluid.fetchResources.primeCacheFromResources()
[14:06:42 CST(-0600)] <colinclark_> and fluid.withNewComponent() (which, incidentally, is "unsupported")
[14:06:59 CST(-0600)] <colinclark_> These functions are only used by components that are IoC-ified
[14:07:13 CST(-0600)] <colinclark_> In Infusion 1.3, that includes Uploader, Reorderer, and the ariaLabeller
[14:07:20 CST(-0600)] <colinclark_> Nothing depends on Uploader
[14:07:24 CST(-0600)] <colinclark_> nothing depends on Reorderer
[14:07:31 CST(-0600)] <colinclark_> only Uploader and Reorderer depend on the ariaLabller
[14:07:39 CST(-0600)] <Bosmon> Well, that is nice and tidy
[14:07:43 CST(-0600)] <colinclark_> So, I'll apply this patch and test Uploader and Reorderer
[14:07:51 CST(-0600)] <colinclark_> if they all continue to work, we're in business
[14:07:58 CST(-0600)] <colinclark_> and I will vote +1 to sneak this one in
[14:08:01 CST(-0600)] <Justin_o> colinclark_: please test in IE6 as well
[14:08:08 CST(-0600)] <colinclark_> Justin_o: For sure, King
[14:08:18 CST(-0600)] <colinclark_> I will not make that mistake again any time soon, after yesterday
[14:08:19 CST(-0600)] <colinclark_>
[14:08:22 CST(-0600)] <Justin_o>
[14:08:35 CST(-0600)] <colinclark_> Justin_o: What do you think of this plan?
[14:08:35 CST(-0600)] <Bosmon> "please don't shell us, King"
[14:08:56 CST(-0600)] <Justin_o> the plan sounds good
[14:09:12 CST(-0600)] <colinclark_> Then I'll get back to reviewing FLUID-3711
[14:09:53 CST(-0600)] <Justin_o> Bosmon: now that you are here.. i just want to double check that you went through the reorderer and the labeller and marked anything that needed to be as unsupported
[14:09:59 CST(-0600)] <colinclark_> Bosmon: One other thing
[14:10:13 CST(-0600)] <colinclark_> I assume this patch is unlikely to apply, post DOS and lint changes?
[14:10:21 CST(-0600)] <Bosmon> I think that's right, yes
[14:10:25 CST(-0600)] <Bosmon> Shall I regenerate it now?
[14:10:34 CST(-0600)] <colinclark_> If you would, it would help me a lot
[14:16:14 CST(-0600)] <Bosmon> colinclark_: Version "b" of the patch is attached to FLUID-3899 which now applied to current trunk
[14:31:13 CST(-0600)] <colinclark_> Bosmon: Thank you
[14:36:08 CST(-0600)] <colinclark_> Bosmon: Your new patch...
[14:36:17 CST(-0600)] <colinclark_> it doesn't quite seem right
[14:36:29 CST(-0600)] <colinclark_> For example, in CachingTests.js, you seem to be removing this line:
[14:36:30 CST(-0600)] <colinclark_> -/global jQuery, fluid, jqUnit, window, start, stop/
[14:36:51 CST(-0600)] <colinclark_> And replacing it with three separate global lines that aren't quite as comprehensive as these are
[14:41:10 CST(-0600)] <Bosmon> I'm sorry.... it's possible that I resolved some of the linting conflicts incorrectly
[14:41:36 CST(-0600)] <Bosmon> I did this yesterday when I was in a bit of a hurry and the file was quite a mess
[14:41:46 CST(-0600)] <Bosmon> That file might need relinting again
[14:42:03 CST(-0600)] <colinclark_> Bosmon: I can say from experience that it was a very difficult file to lint
[14:42:13 CST(-0600)] <colinclark_> and due to a mistake on my part, I've already linted it fully twice
[14:42:20 CST(-0600)] <Bosmon> Oh dear
[14:42:31 CST(-0600)] <colinclark_> Is there any way around it?
[14:42:37 CST(-0600)] <colinclark_> I guess I can apply your changes by hand
[14:42:50 CST(-0600)] <Bosmon> I don't think so, there were a quite a number of changes in that file to fix the test cases
[14:43:02 CST(-0600)] <Bosmon> I'm not really sure which would be easier out of relinting and applying changes by hand
[14:43:04 CST(-0600)] <colinclark_> ok
[14:43:15 CST(-0600)] <colinclark_> I'm going to try applying by hand
[14:43:26 CST(-0600)] <colinclark_> I just don't think I can bare the task of relinting
[14:43:52 CST(-0600)] <mlam> Justin_o: in the QA test plan of progress, tests 3-5 are no longer applicable, right?
[14:45:37 CST(-0600)] <Justin_o> those are for a different demo
[14:45:43 CST(-0600)] <Justin_o> they are for the manual test
[14:45:49 CST(-0600)] <Justin_o> mlam: ^
[14:46:06 CST(-0600)] <mlam> ohh ok
[14:47:39 CST(-0600)] <heidi_> justin_o im doing the wcag test for progress - is that right?
[14:50:05 CST(-0600)] <Justin_o> heidi_: sure... i'm not sure if those all got updated though
[14:50:12 CST(-0600)] <heidi_> k
[14:50:22 CST(-0600)] <heidi_> running through achecker is prob a good idea for all components
[14:50:26 CST(-0600)] <heidi_> at least
[14:51:00 CST(-0600)] <Justin_o> heidi_: yes.. i'm wondering if once everything is in.. you can take on the check all shipped exmples tasks in a browser or two and run them through achecker and with a screen reader
[14:51:37 CST(-0600)] <heidi_> justin_o sure
[14:51:42 CST(-0600)] <jhung> Simple Inline Edit failing a unit test in IE8 win7. Filed here: http://issues.fluidproject.org/browse/FLUID-3908
[14:51:51 CST(-0600)] <heidi_> justin_o let me know when
[14:53:08 CST(-0600)] <Justin_o> heidi_: colin and michelled are just finishing up the reviews now.. i think it will be sometime tomorrow
[14:54:05 CST(-0600)] <heidi_> cool ill be in the office tomorra
[14:54:27 CST(-0600)] <Justin_o> heidi_: yes... it's celebration day tomorrow...
[14:54:43 CST(-0600)] <heidi_> yay
[14:56:42 CST(-0600)] <Bosmon> jhung: I believe this issue is caused by a jQuery bug... which I guess is still not resolved
[14:57:01 CST(-0600)] <Bosmon> There has been a report that "visibility calculations" done by the ":visible" selector are not correct on IE8
[14:57:07 CST(-0600)] <Bosmon> But I guess we should look into it again
[14:57:20 CST(-0600)] <Bosmon> You may well find an earlier report of this issue, since it comes round pretty regularly every release cycle
[14:58:16 CST(-0600)] <Bosmon> So please unify this JIRA with any of the probably large number of duplicates of it that there should be
[14:58:43 CST(-0600)] <michelled> heidi_: I recommend wearing stretchy pants tomorrow
[14:58:50 CST(-0600)] <heidi_> lol
[15:04:45 CST(-0600)] <jhung> bosmon: yes I saw the other Jira. I have added Win7 to the list and closed the one I filed.
[15:06:58 CST(-0600)] <heidi_> mlam did you test the progress setting to less than 0 value?
[15:09:11 CST(-0600)] <Justin_o> heidi_: that's a known issue
[15:09:14 CST(-0600)] <Justin_o> http://issues.fluidproject.org/secure/IssueNavigator.jspa?reset=true&jqlQuery=project+%3D+FLUID+AND+component+%3D+Progress+AND+resolution+%3D+Unresolved+ORDER+BY+priority+DESC%2C+key+DESC&mode=hide
[15:09:22 CST(-0600)] <Justin_o> that's all the progress issues
[15:09:23 CST(-0600)] <heidi_> okay cool, was just going to check if it was reported
[15:09:27 CST(-0600)] <heidi_> thanks
[15:09:32 CST(-0600)] <Justin_o> no problem
[15:25:56 CST(-0600)] <anastasiac> cindyli, should I expect the Builder php unit tests to all pass?
[15:26:18 CST(-0600)] <cindyli> anastasiac: yes. why? not passing?
[15:26:27 CST(-0600)] <anastasiac> not all, not in my browser...
[15:26:38 CST(-0600)] <anastasiac> is this the right URL, cindyli? http://forge.fluidproject.org/infusionBuilder/tests/php/InfusionBuilderTests.php
[15:26:56 CST(-0600)] <cindyli> anastasiac: checking..
[15:27:38 CST(-0600)] <jhung> Does anyone know of a Rich Text Inline Edit bug where pressing cancel renders the Undo Edit link? I tried searching and didn't see anything.
[15:28:05 CST(-0600)] <cindyli> anastasiac: yes, right url, is there any change on module or lib dependencies recently?
[15:28:53 CST(-0600)] <anastasiac> ah, cindyli, I think I might know what it is: we removed the dependencies on FSS sub-modules - now there's just the one dependency on FSS - the other modules (e.g. fss-layout) are gone
[15:29:06 CST(-0600)] <cindyli> anastasiac: alrite. u get it
[15:29:19 CST(-0600)] <cindyli> anastasiac: need to change unit test accordingly
[15:29:51 CST(-0600)] <anastasiac> cindyli, is that something you could do? (I don't know PHP Justin_o: should cindyli commit changes to the PHP unit tests now?
[15:33:05 CST(-0600)] <mlam> jhung: it wasn't happening in IE6 for me, but it's now happening in FF3.6 right now
[15:34:48 CST(-0600)] <jhung> mlam: It's happening for me in IE8 and FF3.6. I'll make note of that. Filed a Jira for it.
[15:35:12 CST(-0600)] <anastasiac> cindyli: http://issues.fluidproject.org/browse/FLUID-3910
[15:35:21 CST(-0600)] <cindyli> anastasiac: thx
[15:35:30 CST(-0600)] <mlam> anastasiac: were you getting the same thing for rich text inline edit in FF3.6 and MacOS?
[15:35:39 CST(-0600)] * anastasiac catches up
[15:36:17 CST(-0600)] <jhung> mlam: which OS you running?
[15:36:27 CST(-0600)] <mlam> i'm testing on a vbox , XP
[15:37:12 CST(-0600)] <jhung> mlam, anastasiac: http://issues.fluidproject.org/browse/FLUID-3909
[15:37:26 CST(-0600)] <anastasiac> jhung, mlam: yes, this is happening in FF3.6 on Mac OS 10.6, but only for the CKEditor, not TinyMCE
[15:38:16 CST(-0600)] <anastasiac> sometimes
[15:41:26 CST(-0600)] <Justin_o> cindyli, anastasiac: this patch isn't committed to trunk right?
[15:41:44 CST(-0600)] <cindyli> Justin_o: anastasiac: no
[15:42:13 CST(-0600)] <cindyli> Justin_o: anastasiac: it goes https://source.fluidproject.org/svn/fluid/infrastructure/infusionBuilder/trunk
[15:42:33 CST(-0600)] <Justin_o> cindyli: okay.. i think we should commit that one
[15:42:47 CST(-0600)] <cindyli> Justin_o: which one?
[15:42:49 CST(-0600)] <Justin_o> anastasiac: can you review and commit cindy's patch
[15:43:02 CST(-0600)] <Justin_o> cindyli: the php unit test one
[15:43:12 CST(-0600)] <cindyli> Justin_o: i haven't come up with a patch yet
[15:43:33 CST(-0600)] <Justin_o> cindyli: okay... whenever it's ready
[15:43:37 CST(-0600)] <anastasiac> Justin_o, I can review, but I don't know how well I'll be able to test - it's changes to the PHP code
[15:43:53 CST(-0600)] <Justin_o> cindyli: do you still have a local commit?
[15:44:01 CST(-0600)] <Justin_o> not comit.. sorry
[15:44:02 CST(-0600)] <Justin_o> version
[15:44:31 CST(-0600)] <cindyli> Justin_o: local version of infusion builder? no, but setting up one
[15:45:17 CST(-0600)] <Justin_o> cindyli: okay... thanks
[15:45:29 CST(-0600)] <cindyli> np
[15:47:16 CST(-0600)] <mlam> jhung: are you the only one with Win 7?
[15:50:17 CST(-0600)] <jhung> mlam: seems like it.
[15:51:02 CST(-0600)] <mlam> ok, i can test some of the win 7 stuff when i get home
[15:52:00 CST(-0600)] <jhung> fluid-everyone: Since some of us are testing the same component but on different configurations, I have started a child page to compile testing notes specific to a component. Example: http://wiki.fluidproject.org/display/fluid/Infusion+1.3+Testing+Notes+-+Inline+Edit
[15:52:30 CST(-0600)] <jhung> The idea is to compile notes in a central place as we test, then file Jiras at the end after testing for that component is complete.
[15:57:28 CST(-0600)] <Bosmon> fluid-everyone: Are we planning to commit anythings for FLUID-3881?
[15:57:39 CST(-0600)] <Bosmon> I have made a patch for commenting unsupported reorderer functions
[15:58:31 CST(-0600)] <anastasiac> Bosmon, if you could attach the patch to the JIRA, I could review it, and if Justin_o agrees, commit it (I already commented the framework based on the notes you gave me)
[15:59:05 CST(-0600)] <anastasiac> but Justin_o might say the commit has to wait till after the release; he's in a meeting right now with colinclark
[16:02:43 CST(-0600)] <Bosmon> Ok, I have attached a patch
[16:11:02 CST(-0600)] <mlam> Justin_o: I'm heading home to pick up my new phone from the post office before it closes I'll knock off some of the component testing in Win 7 tonight since not many of us have that OS on their machines
[17:07:57 CST(-0600)] <colinclark> Bosmon: I've got a reworked version of your FLUID-3899 patch
[17:08:03 CST(-0600)] <colinclark> would you be willing to look it over very carefully?
[17:08:07 CST(-0600)] <colinclark> All the unit tests run
[17:08:13 CST(-0600)] <colinclark> but is there anything else I need to test with?
[17:08:21 CST(-0600)] <colinclark> I'll check over Uploader and Reorderer carefully as well
[17:08:37 CST(-0600)] <colinclark> I am fairly worried that I missed a detail here or there, Bosmon
[17:08:46 CST(-0600)] <colinclark> So a "reverse review" is in order
[17:11:52 CST(-0600)] <Bosmon> Ok, thanks
[17:11:55 CST(-0600)] <Bosmon> I will look it over
[17:12:15 CST(-0600)] <Bosmon> Is it on the issue?
[17:15:29 CST(-0600)] <colinclark> Bosmon: I'm just attaching it now
[17:16:22 CST(-0600)] <colinclark> Bosmon: