The IRC channel is #sakai on the freenode network, you can join using a web client at http://webchat.freenode.net/?channels=sakai and you can view the complete logs at http://sakai.iriscouch.com/irc/_design/viewer/index.html
08:43 <SakaiGithub> [3akai-ux/1.5.0] SAKIII-5778 added screen reader text for tag lists - jsloane 08:53 <SakaiGithub> [3akai-ux] bp323 pushed 1 new commit to 1.5.0: https://github.com/sakaiproject/3akai-ux/commit/ca5d3000d2d383d7ff2f3930d4d58e30190053c5 08:53 <SakaiGithub> [3akai-ux/1.5.0] SAKIII-5851 fixed keyboard focus for widget settings in content authoring - jsloane 09:20 <SakaiGithub> [3akai-ux] bp323 pushed 1 new commit to 1.5.0: https://github.com/sakaiproject/3akai-ux/commit/3f7d710211d6504c7bb5d37f042a40c7a69fee02 09:20 <SakaiGithub> [3akai-ux/1.5.0] SAKIII-5395 Most Sakai pages have no WAI-ARIA landmarks - jsloane 09:51 <SakaiGithub> [3akai-ux/1.5.0] SAKIII-5573 - Uniquely identify the Sakai2Tools widget from the BasicLTI widget - Christian Vuerings 13:39 <PhysX> stuartf ping 13:39 <PhysX> stuartf: I'm in the middle of fixing styling issues for SAKIII-6062, I noticed you took over. What have you done so far? 13:42 <stuartf> PhysX: I just pushed the branch with what I've done to my github 13:43 <stuartf> https://github.com/stuartf/3akai-ux/tree/SAKIII-6062 13:43 <PhysX> stuartf: oh cool, I just replied your email. We're not working on the same thing so that's good. 13:43 <PhysX> But I'll probably just push up what I have for you to fetch and continue on 13:43 <PhysX> to avoid conflicts 13:44 <stuartf> I was worried for a second there :) 13:44 <PhysX> hah yeah me too 13:44 <PhysX> Can you fetch my https://github.com/bp323/3akai-ux/commits/SAKIII-6062 branch and see if it conflicts at all? 13:45 <stuartf> the thing is, there may be conflicts, because stuff like that quotes fix touch tons of lines 13:45 <PhysX> stuartf: I've only touched dev/javascript/ and some of the dev/lib/sakai files for quotes 13:46 <stuartf> 95 conflicts 13:46 <PhysX> stuartf: is your double -> single quotes fix complete? 13:46 <PhysX> I can revert mine if it is 13:46 <PhysX> should help with the conflicts 13:47 <stuartf> mine should be complete 13:47 <PhysX> I'll revert it 13:47 <PhysX> stuartf: try again 13:48 <stuartf> that brought it down to 93 13:48 <PhysX> hah seriously 13:50 <PhysX> stuartf: The fixes that are in there are all easy to do by a global replace 13:50 <stuartf> yeah, that's how I've been doing a lot of my fixes 13:51 <PhysX> stuartf: function ( -> function(, if( -> if (, ){ -> ) { 13:52 <stuartf> so no space between function and ( but yes space between if and ( 13:52 <PhysX> correct 13:52 <stuartf> ok 14:00 <stuartf> PhysX: take a look at the latest commit on my SAKIII-6062 branch 14:01 <PhysX> stuartf: looks good 15:45 <ctweney> ping zacht 15:45 <ctweney> zacht: I'm trying to use the wait_for_indexer() method from nakamura/test.rb in one of my cuke tests 15:46 <ctweney> what I get is "undefined method `assert' for #<Object:0x00000101429530> (NoMethodError)" 15:47 <zacht> ctweney require 'test/unit.rb' 15:47 <ctweney> yeah I already added that 15:48 <ctweney> as well as include SlingTest 15:53 <ctweney> is there a way to get debug info on the environment? 15:58 <zacht> ctweney We need someone with better ruby chops than mine. 16:02 <zacht> simong Hi. 16:27 <zacht> ctweney Ugh. Here's something I didn't want to be dealing with: when I try to use nakamura gem to POST to /system/userManager/user.create.html Sling mysteriously resolves to the SlingPostServlet. If I do the same thing from curl, it resolves (as I expect) to LiteCreateSakaiUserServlet. 16:28 <ctweney> whoa, spooky 16:29 <simong> Is there a new prefered way to unit testing code that deals with ServletHttpRequests? I'm thinking of mocking it up with Mockito. (I only need to test some string manipulation of a couple of parameters) 16:29 <ctweney> although I haven't seen anyone make it official 16:34 <thecarlhall> simong, yay for mockito. boo the other stuff :) 16:35 <zacht> ctweney holy moly! 16:36 <thecarlhall> zacht, :vars are special in ruby so I guess ^^ works on that? 16:37 <raydavis> Ruby Net::HTTP is an effin mess. // simong look around at the more recent unit tests to catch up. Also, we'd really love to stop relying on HTTP requests so much -- if you're doing new code, you'll find much friendliness to moving servlet-embedded logic into a testable shareable service. 16:39 <ctweney> I second what raydavis says, simong … where "testable shareable service" means one that has no http or sparsemap dependencies 16:40 <zacht> raydavis @s.execute_post(@s.url_for("/system/userManager/user.create.html"), {":create-auth" => "reCAPTCHA.net"}) 16:40 <zacht> raydavis When I do it with curl, it does what I want. 16:44 <zacht> raydavis s'ok. 16:46 <raydavis> thecarlhall you saw my Solr-dependency trim pull request, right? No hurry, just wanted to make sure it didn't get lost. 16:47 <thecarlhall> raydavis, I check pull requests religiously and see all the emails about them too :) 16:48 <raydavis> simong, well, hey, that seems pretty HTTP request specific all right. :) 16:52 <raydavis> I'd go Mockito, as you said. 16:53 <simong> Don't want to kick the wrong leg on my first days ^^ 17:03 <MrVisser> thecarlhall: except for one package import "StorageClientListener" 17:03 <thecarlhall> mrvisser, and they're for different branches 17:04 <MrVisser> thecarlhall: correct.. 17:04 <GitHub164> [sparsemapcontent] thecarlhall pushed 2 new commits to 1.3: https://github.com/sakaiproject/sparsemapcontent/compare/32ec1c4de4e7...b168b5571a55 17:04 <GitHub164> [sparsemapcontent/1.3] KERN-3083 Rollback a transaction during an insert when an uncaught error happens during insert/update - Branden Visser 17:21 <MrVisser> thecarlhall: how should that be handled? I thought we normally just did different PR's for multiple-branch merges. 17:22 <thecarlhall> mrvisser, this is where branch managers come in 17:23 <thecarlhall> mrvisser, branch manager would normally fix that 17:23 <MrVisser> in the PR, at least. 17:27 <thecarlhall> mrvisser, an example of what happens shows up here: https://github.com/sakaiproject/sparsemapcontent/pull/66 19:15 <thecarlhall> ctweney, I have a question on PR 954. I see the new content in the explore categories list but I don't see it when I click on the category where it was listed 19:17 <thecarlhall> raydavis, didn't we merge in your fixes for escaping solr queries? 19:18 <thecarlhall> raydavis, I think so. Testing a bit first. 19:19 <thecarlhall> raydavis, looks like bytag.json spits up a bit 19:20 <raydavis> Thanks for finding that one, thecarlhall. I'll be glad to take it if you want, I'm just finishing up another 2 pull requests now. 19:22 <thecarlhall> raydavis, ByTagSearchPropertyProvider is the culprit. The strings are pretty obvious. I'll put in a PR against your JIRA. 19:23 <raydavis> I'm glad Lucene doesn't do this kind of thing often. 19:28 <thecarlhall> raydavis, seems like they would've used some common syntax to note regex in an otherwise harmless string 19:30 <thecarlhall> ctweney, would you mind adding a commit to your PR 954? I can give you the patch. 19:58 <ctweney> sure thing, thecarlhall, what's the commit? 20:08 <GitHub100> [sparsemapcontent] thecarlhall pushed 2 new commits to 1.3: https://github.com/sakaiproject/sparsemapcontent/compare/b168b5571a55...dfb0f7836ea7 20:08 <GitHub100> [sparsemapcontent/1.3] Merge pull request #75 from mrvisser/kern-3083-1.3-2 - Carl Hall 20:55 <SakaiGithub> [3akai-ux] christianv pushed 2 new commits to master: https://github.com/sakaiproject/3akai-ux/compare/fbe12e7634b6...fefd68e156c0 20:55 <SakaiGithub> [3akai-ux/master] SAKIII-5557 Improve internationalization in searchgroups - Bert Pareyn 20:55 <SakaiGithub> [3akai-ux/master] Merge pull request #2393 from bp323/SAKIII-5557 - Christian Vuerings 21:05 <MrVisser> Anyone know of any cases of OSGi bundles or containers simply restarting under high load? 21:12 <GitHub100> [nakamura] thecarlhall pushed 2 new commits to master: https://github.com/sakaiproject/nakamura/compare/0a6740fec021...fed91229b623 21:12 <GitHub100> [nakamura/master] KERN-3075 Avoid triggering Tika text-parsing error from this integration test - Ray Davis 21:12 <GitHub100> [nakamura/master] Merge pull request #956 from raydavis/KERN-3075 - Carl Hall 21:44 <SakaiGithub> [nakamura-ruby/master] KERN-3070 Add support for setting/removing editors to manage_members(), bump nakamura ruby version to 0.14 - Chris Tweney 21:49 <GitHub48> [nakamura] thecarlhall pushed 2 new commits to master: https://github.com/sakaiproject/nakamura/compare/fed91229b623...2d58b1c464d9 21:49 <GitHub48> [nakamura/master] KERN-3034 Do not default search results to full content tree traversal - Ray Davis 21:49 <GitHub48> [nakamura/master] Merge pull request #952 from raydavis/KERN-3034 - Carl Hall 22:07 <GitHub136> [nakamura] thecarlhall pushed 3 new commits to master: https://github.com/sakaiproject/nakamura/compare/2d58b1c464d9...af73a700bb79 22:07 <GitHub136> [nakamura/master] KERN-3085 Use annotations to instantiate LiteFileSearchBatchResultProcessor so that all its dependencies will work - Chris Tweney 22:07 <GitHub136> [nakamura/master] Merge pull request #954 from ctweney/kern-3085 - Carl Hall 22:07 <GitHub136> [nakamura/master] KERN-3085 Applying patch submitted by Carl to fix query syntax - Chris Tweney 22:37 <GitHub46> [nakamura] thecarlhall pushed 2 new commits to master: https://github.com/sakaiproject/nakamura/compare/af73a700bb79...8d85b082793b 22:37 <zacht> ctweney I'm getting one failing scenario in your kern-3070-squash branch. Is that expected? 22:38 <zacht> ctweney http://cl.ly/IV4t 22:38 <raydavis> Thanks, thecarlhall, followup pull request a-comin' as fast as I can build & mvn -P integration! 22:38 <thecarlhall> raydavis, woot! 22:38 <thecarlhall> raydavis++ 22:38 <ctweney> that's the failure that kern-3085 addressed, zacht 22:38 <thecarlhall> raydavis, thanks for entertaining my requests 22:38 <raydavis> thecarlhall, just wanted to check -- you did a mvn clean deploy , right? 22:38 <ctweney> so in master (now) that feature will pass, zacht 22:38 <raydavis> NP, thecarlhall, it was very reasonable. 22:39 <thecarlhall> raydavis, I did mvn clean install. deploy wouldn't help from me as I don't have access to the upstream repos. 22:39 <thecarlhall> raydavis, zacht and arwhyte1 have that magic 22:39 <raydavis> Ah, we need someone to deploy before my next pull request will be safe. 22:39 <raydavis> zacht you are our only hope (as usual). 22:39 <zacht> raydavis can do. 22:40 <raydavis> Thank you, & sorry for the interruption, zacht++ 22:40 <zacht> Oooh, I feel I should have a cake. 22:43 <raydavis> zacht, ya gotta do it from maven/solr-example-resources -- only needs to be done once, every time we update our Solr/Lucene dependency. 22:43 <raydavis> It's the minified version of that ginormous Solr binary deployment I asked you to do. 22:44 <zacht> raydavis sign of the times: [ERROR] /bin/sh: svn: command not found 22:44 <raydavis> Well, hey. 22:44 <zacht> raydavis remedying now. 23:05 <zacht> raydavis Having to shave a few yaks over here. Should be done soon. 23:05 <raydavis> zacht, understand. Sorry about the glitch. 23:06 <zacht> raydavis it's really a Mountain Lion glitch. 23:06 <raydavis> The integration tests are still running, no hurry. :) 23:13 <zacht> raydavis http://source.sakaiproject.org/maven2/org/sakaiproject/nakamura/solr-example-resources/ 23:13 <raydavis> Thanks, zacht! 23:14 <GitHub77> [nakamura] zathomas pushed 1 new commit to master: https://github.com/sakaiproject/nakamura/commit/bb2580e8ff3eec8ad6aca2f66e0870fa91beb04a 23:14 <GitHub77> [nakamura/master] SOLR-75 tweak the pom file so deployment works. - Zach A. Thomas 23:15 <raydavis> Oh bummer, zacht, I apologize for messing that up. 23:15 <zacht> raydavis no big deal. You wouldn't have any way to test it. 23:16 <zacht> gotta run. 23:16 <raydavis> Maven: There's More than One Way to Break It. _______________________________________________ oae-dev mailing list oae-dev@collab.sakaiproject.org http://collab.sakaiproject.org/mailman/listinfo/oae-dev
