From c5b3560fcab316163678f9b4b064e8646f3c4431 Mon Sep 17 00:00:00 2001
From: Sanjay B <b@pad.ma>
Date: Sat, 21 Nov 2009 20:45:32 +0530
Subject: [PATCH] converted to jquery method, added .hasOwnProperty checks

---
 index.html    |  6 ++++++
 jquery.srt.js | 52 ++++++++++++++++++++++++++++++++-------------------
 2 files changed, 39 insertions(+), 19 deletions(-)

diff --git a/index.html b/index.html
index 96b4fe7..624bdf3 100644
--- a/index.html
+++ b/index.html
@@ -7,6 +7,12 @@
       cortado_location = "http://footage.stealthisfilm.com/static/cortado.video.jar";
     </script>
     <script type="text/javascript" src="jquery.srt.js"></script>
+    <script type="text/javascript">
+$(document).ready(function() {
+  $('.srt').srt();
+}); 
+
+    </script>
     <style>
 body {
   color: #000;
diff --git a/jquery.srt.js b/jquery.srt.js
index 6fe057a..f29b5cc 100644
--- a/jquery.srt.js
+++ b/jquery.srt.js
@@ -9,15 +9,19 @@
  */
 /*
   usage:
+    html:
     <video src="example.ogg" id="examplevideo" />
     <div class="srt" data-video="examplevideo" data-srt="example.srt"></div>
 
-  jquery.srt.js will try to load subtitles in all elements with 'srt' class.
+    js:
+    $(document).ready(function() { $('.srt').srt(); });
+
+  in the above example, jquery.srt.js will try to load subtitles in all elements with 'srt' class.
   'data-video' atribute is used to link to the related video,
   if no data-srt is provided, the contents of the div is parsed as srt.
 */
 
-$(document).ready(function() {
+(function() {
   function toSeconds(t) {
     var s = 0.0
     if(t) {
@@ -27,19 +31,22 @@ $(document).ready(function() {
     }
     return s;
   }
+
   function strip(s) {
     return s.replace(/^\s+|\s+$/g,"");
   }
+
   function playSubtitles(subtitleElement) {
     var videoId = subtitleElement.attr('data-video');
     var srt = subtitleElement.text();
-    subtitleElement.text('');
-    srt = srt.replace(/\r\n|\r|\n/g, '\n')
+    subtitleElement.html('');
+    srt = srt.replace('\r\n|\r|\n', '\n')
     
     var subtitles = {};
     srt = strip(srt);
     var srt_ = srt.split('\n\n');
     for(s in srt_) {
+      if (srt_.hasOwnProperty(s)) {
         st = srt_[s].split('\n');
         if(st.length >=2) {
           n = st[0];
@@ -54,15 +61,20 @@ $(document).ready(function() {
           os = toSeconds(o);
           subtitles[is] = {i:i, o: o, t: t};
         }
+      }
     }
+
     var currentSubtitle = -1;
+
     var ival = setInterval(function() {
       var currentTime = document.getElementById(videoId).currentTime;
       var subtitle = -1;
       for(s in subtitles) {
-        if(s > currentTime)
-          break
-        subtitle = s;
+        if (subtitles.hasOwnProperty(s)) {
+          if(s > currentTime)
+            break
+          subtitle = s;
+        }
       }
       if(subtitle > 0) {
         if(subtitle != currentSubtitle) {
@@ -74,17 +86,19 @@ $(document).ready(function() {
       }
     }, 100);
   }
-  $('.srt').each(function() {
-    var subtitleElement = $(this);
-    var videoId = subtitleElement.attr('data-video');
-    if(!videoId) return;
-    var srtUrl = subtitleElement.attr('data-srt');
-    if(srtUrl) {
-      $(this).load(srtUrl, function (responseText, textStatus, req) { playSubtitles(subtitleElement)})
-    } else {
-      playSubtitles(subtitleElement);
-    }
-  });
 
-});
+  jQuery.fn.srt = function() {
+    this.each(function() {
+      var subtitleElement = $(this);
+      var videoId = subtitleElement.attr('data-video');
+      if(!videoId) return;
+      var srtUrl = subtitleElement.attr('data-srt');
+      if(srtUrl) {
+        $(this).load(srtUrl, function (responseText, textStatus, req) { playSubtitles(subtitleElement)});
+      } else {
+        playSubtitles(subtitleElement);
+      }      
+    });
+  };
+})();