Updated #1730 to use the ready function instead of warning.

Added a line to the changelog for #1730. closes #1730
videojs · Dec 22, 2014 · f0d500a · f0d500a
1 parent fc0730d
commit f0d500a
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,7 @@ CHANGELOG
 ## HEAD (Unreleased)
 * @PeterDaveHello added a Traditional Chinese translation ([view](https://github.com/videojs/video.js/pull/1729))
 * @mmcc updated the hide/show functions to use a class instead of inline styles ([view](https://github.com/videojs/video.js/pull/1681))
+* @mister-ben added better handling of the additional videojs() arguments when the player is already initialized ([view](https://github.com/videojs/video.js/pull/1730))
 
 --------------------
 

diff --git a/src/js/core.js b/src/js/core.js
@@ -42,8 +42,9 @@ var vjs = function(id, options, ready){
       if (options) {
         vjs.log.warn ('Player "' + id + '" is already initialised. Options will not be applied.');
       }
+
       if (ready) {
-        vjs.log.warn ('Player "' + id + '" is already initialised. Ready function will not be executed.');
+        vjs.players[id].ready(ready);
       }
 
       return vjs.players[id];