Last modified: 2011-12-23 18:54:43 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T20960, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 18960 - mvTextInterface.js error on loading CMML <mediaSource>
mvTextInterface.js error on loading CMML <mediaSource>
Status: RESOLVED WONTFIX
Product: MediaWiki extensions
Classification: Unclassified
MetaVidWiki (Other open bugs)
unspecified
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-05-27 18:57 UTC by George Chriss
Modified: 2011-12-23 18:54 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Comment out code that expects "&t=h:mm:ss/h:mm:ss" style URLs (1.67 KB, patch)
2009-05-28 00:37 UTC, George Chriss
Details

Description George Chriss 2009-05-27 18:57:03 UTC
I am able to view the captions provided in mwEmbed's timed text examples, but the showTextInterface() box hangs on "loading ..." when I supply a <mediaSource ... src="http://.../captions.xml"> CMML track in the file indicated by <video roe="http://.../mwEmbed_roe.xml">.  Both (static) XML files are valid, well-formatted, have content-type text/xml, and only two tracks are present.

I am able to trace the problem using Firebug's console:

load timed text from roe: http://.../mwEmbed_roe.xml
row data ready (no roe request)
mv_txt_load_:SHOW mv_txt_load_
video/ogg is not suported timed text fromat 
adding Track: Ht_en to v0
do load timed text: Ht_en
textCMML: loading track: undefined

pcurl.queryKey.t is undefined
        load()("source", function())
        loadAndDisplay()("Ht_en")
        (?)()(1, Object)
        each()([Object, Object 0=Object 1=Object], function(), undefined)
        getParseTimedText_rowReady()()
        getTimedTextTracks()()
        init()(div#v0)
        mvTextInterface()(div#v0)
        (?)()()
        doLoad()(undefined, undefined)
        mv_embed.js()
          var req_time = pcurl.queryKey['t'].split('/'); 
          ^-->JS Error from line 361 in libTimedText/mvTextInterface.js

native:onended:
base:onClipDone
mvEmbed:stop:v0


I believe that mvTextInterface.js is re-parsing the original mwEmbed_roe.xml, then retrieving the first <mediaSource> element (the video itself) rather than the second <mediaSource> listed under <track id="t" provides="text layers">.  I am not aware of a solution or workaround, or for that matter how my test case differs from the provided examples.

Any ideas?
-George


Minefield 3.6a1pre build 20090416052000
mvEmbed SVN 50996
Comment 1 George Chriss 2009-05-28 00:37:00 UTC
Created attachment 6163 [details]
Comment out code that expects "&t=h:mm:ss/h:mm:ss" style URLs
Comment 2 George Chriss 2009-05-28 00:45:00 UTC
The problem was caused by a lack of "&t=h:mm:ss/h:mm:ss"-style query strings in my URLs.
Reconfiguring web servers to work with these URLs takes a fair amount of effort, so hopefully
mwEmbed can be edited here and elsewhere (e.g., annotations and categories) to opportunistically
use these URLs.

The attached patch is a temporary workaround.

-George
Comment 3 George Chriss 2009-06-09 13:42:07 UTC
Michael Dale just added the following:
r1=51610&r2=51611">http://svn.wikimedia.org/viewvc/mediawiki/branches/new-upload/phase3/js2/mwEmbed/libTimedText/mvTextInterface.js?&pathrev=51611&r1=51610&r2=51611

However, the captions show up as blank box as of revision 51641.

Correction: "&t=h:mm:ss/h:mm:ss"-style query strings are just query strings, so no extra effort involved.
Comment 4 p858snake 2011-04-30 00:09:03 UTC
*Bulk BZ Change: +Patch to open bugs with patches attached that are missing the keyword*
Comment 5 Sumana Harihareswara 2011-12-23 18:10:19 UTC
Adding "need-review" keyword to indicate patch awaits review (Michael Dale, is that wrong?).  I'm sorry for the wait.
Comment 6 Michael Dale 2011-12-23 18:42:38 UTC
its from 2009 .. I don't think this project is maintained any longer.
Comment 7 Mark A. Hershberger 2011-12-23 18:54:43 UTC
MetaVidWiki work has moved to TimedMediaHandler ... use that instead.

https://www.mediawiki.org/wiki/Extension:TimedMediaHandler

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links