Uncaught TypeError: videojs is not a function error in 5.0, autoSetup #2386

lomok · 2015-07-20T20:48:13Z

I continue to get this message - I am using rc_2, 5.0.

It does not seem to affect anything - people I have shown it to believe it is a timing issue.

video.js:9407 Uncaught TypeError: videojs is not a function

In chrome browser, clicking on the error in output window takes one to line 9407 --, which is 👍 var player = videojs(mediaEl);

From the code:

// Check if any media elements exist
  if (mediaEls && mediaEls.length > 0) {

    for (var i = 0, e = mediaEls.length; i < e; i++) {
      var mediaEl = mediaEls[i];

      // Check if element exists, has getAttribute func.
      // IE seems to consider typeof el.getAttribute == 'object' instead of 'function' like expected, at least when loading the player immediately.
      if (mediaEl && mediaEl.getAttribute) {

        // Make sure this player hasn't already been set up.
        if (mediaEl.player === undefined) {
          var options = mediaEl.getAttribute('data-setup');

          // Check if data-setup attr exists.
          // We only auto-setup if they've added the data-setup attr.
          if (options !== null) {
            // Create new video.js instance.
line 9407 ----->>>>            var player = videojs(mediaEl);
          }
        }

The text was updated successfully, but these errors were encountered:

heff · 2015-09-15T01:22:21Z

This one should be fixed in the latest 5.0 RC. Let me know if you find differently.

clauso · 2015-10-11T09:52:16Z

Hi,

I get this message with the current 5.0.0 in Firefox (https://github.com/videojs/video.js/releases/tag/v5.0.0).
The error log points to these lines:

    // Check if data-setup attr exists.
    // We only auto-setup if they've added the data-setup attr.
    if (options !== null) {
        // Create new video.js instance.
        var player = videojs(mediaEl);
    }

Video plays and everything seems to work fine. However as we log client side js errors to our server through an API ( we want to notice if things are not working well ) this generates a whole lot of error messages.
My current work-around is to remove the empty data-setup attribute.
Claus

grebett · 2015-10-26T09:07:44Z

My personal workaround, in the video.js file, remove:

// l. 11283
var videojs = undefined; // remove this line

and

// l. 11344
var autoSetupTimeout = function autoSetupTimeout(wait, vjs) {
  videojs = vjs; // remove this line
  setTimeout(autoSetup, wait);
};

Since videojs is already defined in the file, and not defined again after manually set it to undefined for ie8 fix. Not sure to understand all the issues of this bug, and don't have time too, but may be the beginning of a solution.

yairpe · 2015-11-03T20:46:53Z

my workaround: change

      if (options !== '{}' && options !== null) {                   //  @@ if (options !== null) {
        // Create new video.js instance.
        var player = videojs(mediaEl);
       }

gkatsev · 2015-11-03T20:56:04Z

Does it still happen with the latest videojs release (5.0.2 or next release 5.1.0)?
Would someone be able to put up a reduced test case that makes this error happen? I have not been able to reproduce.

yairpe · 2015-11-04T06:41:04Z

I have just upgraded to 5.1.0 from 4.12.5 and the problem showed up.

ghost · 2015-11-16T19:43:36Z

I upgraded from 4.11.2 to 5.2.0 and and the problem showed up.

gkatsev · 2015-11-16T22:02:47Z

Reopening since @WebDevRN is still seeing this issue. Would you have to have a reduced test case?

DonDozone · 2015-11-20T13:45:56Z

Hi everyone!

I suspect that this has to do with jQuery. On my page, when I load video.js and then jQuery, I get the same error. When I FIRST load jQuery and THEN video.js, everything works fine.

So this is the setup that generates the problem:

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <link href="http://vjs.zencdn.net/5.2.1/video-js.css" rel="stylesheet">

  <script src="http://vjs.zencdn.net/5.2.1/video.js"></script> 
  <script type='text/javascript' src='http://code.jquery.com/jquery-1.11.3.js'></script>

</head>

<body>
  <video id="example_video_1" class="video-js vjs-default-skin"
           controls preload="auto" width="640" height="320"
           poster="http://video-js.zencoder.com/oceans-clip.png"
           data-setup='{ "autoplay": true, "controlBar": {"fullscreenToggle": false, "progressControl": true}}'>
        <source src="http://video-js.zencoder.com/oceans-clip.mp4" type='video/mp4' />
        <source src="http://video-js.zencoder.com/oceans-clip.webm" type='video/webm' />
        <source src="http://video-js.zencoder.com/oceans-clip.ogv" type='video/ogg' />
    </video>
</body>
</html>

Unfortunately, when I use the hosted jQuery, I can't reproduce the error consistently. When I load over the cache (CTRL + F5) I can only reproduce the error about 1 in 10 times. If I use the script hosted in my development site, I get it 4 out of 5 times. For the next 2 weeks or so you can test with my page:

<script type='text/javascript' src='http://46.101.144.93/wp-includes/js/jquery/jquery.js?ver=1.11.3'></script>

After that, I will shutdown this VM but I hope you can reproduce it yourself.

Best regards,
Georg

deidei · 2015-12-02T11:23:07Z

Please check setup.js. In function autoSetupTimeout, videojs is initialized before autoSetup is called.
But sometimes autoSetup is called anonymously. Then the error is thrown.

deidei · 2015-12-02T11:31:20Z

I think you can use setTimeout to wait the videojs is initialized. Or delete attribute data-setup of video tag in html. and initiate the video player in js.

peterhrdy · 2016-01-18T14:44:49Z

script defer="defer" src="http://vjs.zencdn.net/ie8/1.1.1/videojs-ie8.min.js
script defer="defer" src="http://vjs.zencdn.net/5.4.4/video.js

this worked for me

ghost · 2016-03-30T00:53:32Z

Consistently getting the same error on iPhone with just updated version 5.8.7. Surprisingly iPad and desktop browsers works fine...

sethborg · 2016-03-30T01:46:42Z

My bug report seems to be the same issue but I only experience it on IE8 when I use data-setup="{}" (doesn't have to be empty) rather than javascript initialization and if I have the meta charset="utf-8" tag in my document. I tested on IE11, Firefox 45, and iPhone 9.3 and they work as expected.

Start by first clearing your cache whenever you visit the page to test. When you visit it in IE8 you get the error message. If you refresh the page, usually once but sometimes it takes 2 refreshes, you'll get the play button but missing the play triangle (utf8 issue) but if you mouse over the play button it appears correctly. The video will then play fine too. It does seem to be a timing issue but the utf8 part must be part of it somehow.

Here's my minimal test case: http://ivcmiami.com/test/

ghost · 2016-03-30T01:50:38Z

@sethborg thanks for the example. Definitely not showing video-js on iPhone 6 Plus. I can see a native media player, but also there is no error.

sethborg · 2016-03-30T02:00:03Z

Pretty sure you're never going to see videojs controls on iPhone. I don't see them on videojs.com. I just included the test cause someone mentioned it not playing videos on iPhone for them.

@gkatsev

* upstream/stable: (77 commits) v5.9.2 @gkatsev grouped text track errors in the console, if we can. closes videojs#3259 v5.9.1 @gkatsev fixed text track tests for older IEs. closes videojs#3269 revert 75116d4 adding chrome to travis (videojs#3254) @forbesjo added back the background color to the poster. closes videojs#3267 @gkatsev fixed removeRemoteTextTracks not working with return value from addRemoteTextTracks. closes videojs#3253 @gkatsev made the first emulated text track enabled by default. closes videojs#3248 @mister-ben blacklisted Chrome for Android for playback rate support. closes videojs#3246 @benjipott updated IS_CHROME to not be true on MS Edge. closes videojs#3232 v5.9.0 @andyearnshaw updated document event handlers to use el.ownerDocument. closes videojs#3230 @chrisauclair added ARIA region and label to player element. closes videojs#3227 @MCGallaspy added vttjs to the self-hosting guide. closes videojs#3229 @forbesjo added chrome for PR tests. closes videojs#3235 @OwenEdwards improved handling of deprecated use of Button component. closes videojs#3236 v5.8.8 @seescode fixed dragging on mute toggle changing the volume. Fixes videojs#3036. Closes videojs#3228 @seescode fixed css failing on IE8 due to incorrect ie8 hack. Fixes videojs#3140. Closes videojs#3226. @vtytar fixed auto-setup failing if taking too long to load. Fixes videojs#2386. Closes videojs#3233. ...

bahiamartins · 2016-05-19T18:44:50Z

@grebett solution works

heff added this to the v5.0.0 milestone Sep 14, 2015

heff closed this as completed Sep 15, 2015

gkatsev mentioned this issue Nov 16, 2015

videojs is not a function #2818

Closed

gkatsev reopened this Nov 16, 2015

gkatsev mentioned this issue Nov 17, 2015

videojs is not a function #2822

Closed

gkatsev added needs: reduced test case A reproducible test case is needed. See https://stackoverflow.com/help/minimal-reproducible-example needs: more info Please make enough detailed information is added to be actionable. labels Nov 17, 2015

elflyfreely mentioned this issue Dec 28, 2015

A lot of times thumbnail (Poster) won't show #2950

Closed

This was referenced Mar 29, 2016

autoSetup problem #2788

Closed

"videojs is not a function" error - in 5.2.1 #2831

Closed

gkatsev changed the title ~~video.js:9407 Uncaught TypeError: videojs is not a function error in 5.0, rc_2~~ Uncaught TypeError: videojs is not a function error in 5.0, autoSetup Mar 29, 2016

gkatsev mentioned this issue Mar 29, 2016

video.min.js and video.js both broken in IE8 when using data-setup="{}" #3142

Closed

vtytar mentioned this issue Apr 1, 2016

do not reset videojs when autoSetupTimeout called on autoSetup retry #3233

Closed

gkatsev closed this as completed in 68bb36e Apr 4, 2016

github-actions bot locked as resolved and limited conversation to collaborators May 27, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uncaught TypeError: videojs is not a function error in 5.0, autoSetup #2386

Uncaught TypeError: videojs is not a function error in 5.0, autoSetup #2386

lomok commented Jul 20, 2015

heff commented Sep 15, 2015

clauso commented Oct 11, 2015

grebett commented Oct 26, 2015

yairpe commented Nov 3, 2015

gkatsev commented Nov 3, 2015

yairpe commented Nov 4, 2015

ghost commented Nov 16, 2015

gkatsev commented Nov 16, 2015

DonDozone commented Nov 20, 2015

deidei commented Dec 2, 2015

deidei commented Dec 2, 2015

peterhrdy commented Jan 18, 2016

ghost commented Mar 30, 2016

sethborg commented Mar 30, 2016

ghost commented Mar 30, 2016

sethborg commented Mar 30, 2016

bahiamartins commented May 19, 2016

Uncaught TypeError: videojs is not a function error in 5.0, autoSetup #2386

Uncaught TypeError: videojs is not a function error in 5.0, autoSetup #2386

Comments

lomok commented Jul 20, 2015

heff commented Sep 15, 2015

clauso commented Oct 11, 2015

grebett commented Oct 26, 2015

yairpe commented Nov 3, 2015

gkatsev commented Nov 3, 2015

yairpe commented Nov 4, 2015

ghost commented Nov 16, 2015

gkatsev commented Nov 16, 2015

DonDozone commented Nov 20, 2015

deidei commented Dec 2, 2015

deidei commented Dec 2, 2015

peterhrdy commented Jan 18, 2016

ghost commented Mar 30, 2016

sethborg commented Mar 30, 2016

ghost commented Mar 30, 2016

sethborg commented Mar 30, 2016

bahiamartins commented May 19, 2016