fluid-work IRC Logs-2013-06-28
[08:10:18 CDT(-0500)] <anastasiac> hi, ramil, how's it going?
[08:11:19 CDT(-0500)] <ramil> good, the firefox thing seems to be fixed now, i just restarted my computer and it now looks fine
[08:11:29 CDT(-0500)] <anastasiac> that's good to hear
[08:11:50 CDT(-0500)] <ramil> i found out that it doesnt work in ubuntu though
[08:13:49 CDT(-0500)] <anastasiac> I don't have an Ubuntu VM
[08:13:58 CDT(-0500)] <anastasiac> what error do you get?
[08:14:04 CDT(-0500)] <ramil> for some reason the library api seems to not be recognizing the library in ubuntu
[08:14:29 CDT(-0500)] <ramil> it's something about the libraries api module not recognizing the library and therefore not loading MyInfusion.js
[08:14:48 CDT(-0500)] <anastasiac> sounds like a problem with the libraries api or module
[08:15:34 CDT(-0500)] <ramil> i will need to check that out again
[08:17:32 CDT(-0500)] <ramil> anastasiac, it seems to also not work in Opera in windows
[08:18:13 CDT(-0500)] <anastasiac> ramil, is the opera error related to the libraries module as well?
[08:19:17 CDT(-0500)] <ramil> there's a lot of errors
[08:20:07 CDT(-0500)] <ramil> a lot of css errors
[08:21:50 CDT(-0500)] <anastasiac> ramil, We haven't really tested in Opera: http://wiki.fluidproject.org/display/Infusion14/Browser+Support+Chart
[08:22:09 CDT(-0500)] <anastasiac> I seem to get the same types of errors when testing our demo page, so the errors likely aren't specific to your implementation
[08:22:56 CDT(-0500)] <ramil> it works in safari though
[08:23:17 CDT(-0500)] <anastasiac> Justin_o, I'm looking at our current browser support chart http://wiki.fluidproject.org/display/docs/Browser+Support+Chart and it's not quite up-to-date. It says we've tested in FF 6
[08:23:29 CDT(-0500)] <anastasiac> ramil, yes, Safari is one of the supported browsers
[08:38:16 CDT(-0500)] <Justin_o> anastasiac: yes.. not really up-to-date sorry.. we haven't set a time for the next release yet so i haven't updated it yet.. i guess we should put something there that is more current though
[08:40:39 CDT(-0500)] <anastasiac> Justin_o, we could at least update it to indicate that we test against "current" releases of FF, as we do with Chrome
[09:00:37 CDT(-0500)] <jessm> anastasiac: ping
[09:00:53 CDT(-0500)] <anastasiac> jessm: polo
[09:03:05 CDT(-0500)] <Justin_o> anastasiac: yes.. it's funny i feel like i drafted something for this already, but don't know what happened to it
[09:03:09 CDT(-0500)] <Justin_o> maybe it was for video player
[09:03:59 CDT(-0500)] <Justin_o> yes⦠i guess it was for video player http://wiki.fluidproject.org/display/docs/Video+Player+Browser+Support+Chart
[09:05:47 CDT(-0500)] <jessm> anastasiac: where's the geoffrey shea page url?
[09:07:06 CDT(-0500)] <anastasiac> jessm, I'm deployed geoff's info to the dev site. Once it's ready, I'll ask him to review, then we'll push to production. He's already approved the text, through email
[09:08:36 CDT(-0500)] <anastasiac> jessm, only the image is missing. I'm just waiting for that to refresh (which is I think hourly)
[09:46:49 CDT(-0500)] <anastasiac> cindyli, with the new IoC testing framework, do you know if there's an easy way to make tests conditional on available functionality, for example different tests in IE8 because there's no native video support and caption support isn't instantiated?
[09:51:20 CDT(-0500)] <anastasiac> Justin_o, would you know? ^
[09:52:08 CDT(-0500)] <ramil> anastasiac, I think that step 3 here: http://wiki.fluidproject.org/display/docs/Tutorial+-+Fat+Panel+UI+Options does not really work for cms
[09:53:57 CDT(-0500)] <Justin_o> anastasiac: i don't know specifically, but you can use demands and things like that⦠so perhaps you can switch it that way
[09:57:31 CDT(-0500)] <anastasiac> ramil, that tutorial has not yet been updated to the latest api. Modifying the links to the js is no longer necessary, which is why it's working for you. With version 1.5 and earlier, it would be necessary, whether it's a cms or not
[09:58:17 CDT(-0500)] <ramil> so i should just leave the original paths then?
[09:58:48 CDT(-0500)] <anastasiac> ramil, yes: It's working nicely for you, so you shouldn't need to make those changes.
[09:59:22 CDT(-0500)] <anastasiac> ramil, I think your next steps might be to get the reset button in place, and review your module code, tidy it up a bit, etc
[09:59:58 CDT(-0500)] <ramil> actually its causing some errors if you go to a url that's more than two directories deep
[10:01:14 CDT(-0500)] <ramil> anastasiac, can you try going to the front page and clicking on 'request new password' and tell me what happens?
[10:02:45 CDT(-0500)] <anastasiac> ooh, yes, ramil, that's not good it's getting lots of errors, and looping
[10:03:00 CDT(-0500)] <ramil> i find that Drupal seems to be adding whatever path after the root is to the path to that file
[10:03:18 CDT(-0500)] <ramil> so if its just the front page theres no problem
[10:03:47 CDT(-0500)] <ramil> localhost/drupal/ - this has the right path
[10:04:10 CDT(-0500)] <ramil> but localhost/drupal/user/something , drupal adds user/ to the path of that iframe
[10:04:25 CDT(-0500)] <anastasiac> yeah, ramil. I suspect that the path you've configured for UIO is a relative path, instead of something absolute. You should be able to set a "prefix" option that tells UIO where to find the templates. That prefix should not be relative to the current path
[10:04:51 CDT(-0500)] <ramil> its not relative it's php generated
[10:05:52 CDT(-0500)] <anastasiac> well, it seems the php is generating something wrong
[10:09:12 CDT(-0500)] <ramil> i think its the way drupal generates it's paths , everything is dynamically generated so setting something like that in the fatpaneluioptionsiframe.html relatively causes problems
[10:11:02 CDT(-0500)] <ramil> i could be wrong though, ill try to find out whats going on
[10:18:56 CDT(-0500)] <anastasiac> ramil, I don't think its the content of the iframe.html file. The errors indicate that the root path drupal is generating is different
[10:19:17 CDT(-0500)] <anastasiac> on the home page, the base path to the templates, generated by drupal, is http://localhost:8888/drupal-uio/sites/all/libraries...
[10:19:27 CDT(-0500)] <anastasiac> on the password page, it's http://localhost:8888/drupal-uio/user/sites/all/libraries...
[10:20:22 CDT(-0500)] <ramil> yeah I'll try to figure out what's going on
[10:22:25 CDT(-0500)] <anastasiac> ramil, in your functions.js, you're setting prefix to "settings.infusion_uio.library_path", which at first glance seems correct, but obviously, that variable is turning out to be relative, for some reason
[10:27:05 CDT(-0500)] <ramil_g> anastasiac, "settings.infusion_uio.library_path" is actually getting that information from php, if you look at around line 125 of infusion_uio.module, that's where that path is coming from using a php function to get the base path of the infusion library
[10:29:33 CDT(-0500)] <anastasiac> I see, ramil_g. Well, looks like you should dig a bit into the library loading, and see if you can understand why this variable is turning up relative
[10:32:05 CDT(-0500)] <ramil_g> anastasiac, can you tell me a little bit about how Infusion.js uses that path?
[10:39:28 CDT(-0500)] <anastasiac> ramil_g, sorry, I was at our stand-up meeting. The UIO component uses that path to find the HTML templates it uses for the settings inside the fat panel. It builds the url for the files using that path and submits an ajax query
[10:39:53 CDT(-0500)] <ramil_g> ok how does it build the path?
[10:42:04 CDT(-0500)] <anastasiac> checking
[10:44:27 CDT(-0500)] <anastasiac> ramil_g, I'm probably not the best person to ask this question. cindyli, might know better just how UIO uses the prefix option. Our demos use a relative path, but some of our integrations (e.g. into wordpress) use an absolute URL
[10:44:49 CDT(-0500)] <ramil_g> oh really?
[10:45:22 CDT(-0500)] <anastasiac> yes, our wordpress integration uses a wordpress-generated variable: <?php bloginfo('template_url'); ?>
[10:45:57 CDT(-0500)] <ramil_g> ok maybe i need to do something like that
[10:46:32 CDT(-0500)] <cindyli> ok, anastasiac and ramil_g, i will catch up your conversation and suggest on what i know about. before that, anastasiac, you were asking about the conditional root in IoC test, I don't know we have that. but there's something hopefully similar we did for enactor tests might help. digging out the code
[10:47:03 CDT(-0500)] <anastasiac> thanks, cindyli, on both counts
[10:47:23 CDT(-0500)] <anastasiac> ramil_g, perhaps drupal has some variable you can access that will be consistent
[10:47:53 CDT(-0500)] <cindyli> anastasiac: https://github.com/fluid-project/infusion/blob/master/src/webapp/tests/component-tests/uiOptions/js/EnactorTests.js#L516-584
[10:51:22 CDT(-0500)] <cindyli> ramil_g: first of all, a code example for UIO template prefix: https://github.com/fluid-project/infusion/blob/master/src/webapp/demos/uiOptions/js/uiOptionsDemo.js#L61-62
[10:52:01 CDT(-0500)] <cindyli> When should we use this prefix - if all of your UIO templates are in a common directory.
[10:53:52 CDT(-0500)] <cindyli> this path can be relative or absolute. but since you are accessing template thru a web server, absolute paths may not suitable or work, but you can certainly use the web absolute url such as "http://mysite/UIO-template-path" as long as your template directory is exposed to the public
[10:55:27 CDT(-0500)] <ramil_g> cindyli, checking the console on chrome , i checked the value of the variable that i put for that prefix, and it is the right path
[10:55:50 CDT(-0500)] <cindyli> ok. still not loading the template?
[10:56:48 CDT(-0500)] <ramil_g> this is one of the errors in firefox: "NetworkError: 404 Not Found - http://localhost/drupal7/node/sites/all/libraries/infusion/components/uiOptions/html/UIOptionsTemplate-lineSpacer.html?_=137243
[10:57:07 CDT(-0500)] <ramil_g> so the path is 'sites/all/libraries/infusion'
[10:57:53 CDT(-0500)] <ramil_g> but for some reason 'node/' is being added when you go to a page like localhost/drupal/node/...
[10:58:43 CDT(-0500)] <cindyli> you will need to look into drupal structure in terms of the path change on the fly, ramil_g
[10:59:04 CDT(-0500)] <ramil_g> ok I will do that
[11:47:44 CDT(-0500)] <Justin_o> anastasiac: i just tried video player in IE 11 from the windows 8.1 preview and the videos are showing up properly in the demo.. the captions don't work, but at least the video isn't green
[12:08:11 CDT(-0500)] <anastasiac> cindyli, thanks for the link to the enactor tests. I'm not sure that's enough for what I'm doing. In IE8, we don't support captions and we don't want the video player to add the captions settings panel. When the test file runs in non-IE8, the test needs to verify that the panel is there, but when it runs in IE8, the test needs to verify the opposite. I know I can use if/else, but I'm wondering if there's something cleaner
[12:08:17 CDT(-0500)] <yzen> jhernandez: ping
[12:10:09 CDT(-0500)] <anastasiac> cindyli1: ^
[12:10:55 CDT(-0500)] <Justin_o> fluid-everyone: we are going to start the discovery tool meeting. would anyone like to join remotely?
[12:11:07 CDT(-0500)] * anastasiac raises hand
[12:11:16 CDT(-0500)] <cindyli1> anastasiac: which notes was for me? video player IE11?
[12:11:47 CDT(-0500)] <anastasiac> no, cindyli1, the one after that, about testing captions settings in IE8
[12:11:53 CDT(-0500)] <anastasiac> or rather, not testing in IE8
[12:12:18 CDT(-0500)] <Justin_o> anastasiac: so the demands didn't work?
[12:12:24 CDT(-0500)] <cindyli1> i missed that with the connection dropping, anastasiac, what's that about
[12:12:38 CDT(-0500)] <anastasiac> thanks for the link to the enactor tests. I'm not sure that's enough for what I'm doing. In I