Posted by Scott Reynolds on November 5, 2009 at 6:25pm
| Project: | Kaltura |
| Version: | 6.x-1.4 |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Scott Reynolds |
| Status: | needs work |
Issue Summary
For performance, themes can put all the Javascript files in the footer. this causes hardcoded script tags stuff to break. So this envasive patch removes all the cruft for the player for JS, adds a new file for drupal_kaltura helper that implements with behaviors.
similar to this: #623416: Don't use jQuery(document.ready())
Don't put script tags in your html, its will break a bunch of configurations.
Comments
#1
forgot to add the patch..
#2
followup
#3
+ $('div.kaltura-player:not(.kaltura-player-processed').each (function (i) {is missing a closing ) on the 'not'.
#4
Thanks a lot, Scott. I wonder why they didn't include the patch in the module by now. It's two months since you posted. On our site kaltura wouldn't work without the patch, because we load our javascript as late as possible for performance reasons. Like lots of other sites do.
#5
Scott, your patches are no longer working. The code of the module was changed since you posted (without integrating your patches).
#6
Same info applies here, these two patches need to be rerolled into one patch. Also this patch doesn't address remix nodes, only kaltura_entry nodes.
#7
I have taken some of the work here and integrated something similar into my version of this module:
http://github.com/univate/drupal-kaltura
The commit related to this issue is here:
http://github.com/univate/drupal-kaltura/commit/85a0c44ecc57828df8f1190e...