Changeset 1568

Timestamp:

01/25/11 06:38:31 (2 years ago)

Author:

jeroen

Message:

added support for BOTR Facebook embeds and fixed an issue with url encoding

Location:

plugins/sharing/v5

Files:

• build/build.sh (modified) (1 diff)
• sharing.swf (modified) (previous)
• src/com/longtailvideo/plugins/sharing/Sharing.as (modified) (3 diffs)
• test/assets/playlist.xml (modified) (1 diff)

plugins/sharing/v5/build/build.sh

@@ -8 +8 @@
 $FLEXPATH/bin/mxmlc ../src/com/longtailvideo/plugins/sharing/Sharing.as -sp ../src -o ../sharing.swf -library-path+=../lib -load-externs=../lib/jwplayer-5-classes.xml -use-network=false -optimize=true -incremental=false
 
+# echo "Compiling library ..."
+
 # $FLEXPATH/bin/compc -source-path ../src -output ../sharing.swc -compiler.library-path $FLEXPATH/frameworks/libs -compiler.library-path ../lib -include-classes com.longtailvideo.plugins.sharing.Sharing -link-report='../sharing-classes.xml'

plugins/sharing/v5/src/com/longtailvideo/plugins/sharing/Sharing.as

@@ -164 +164 @@
         /** hack for backward-compatible BOTR embeds **/
         private function getBotrEmbed():String {
-            Logger.log(_player.config.playlistfile,'sharing');
             var code:String = '<embed src="';
             code += _player.config.playlistfile.substr(0,_player.config.playlistfile.indexOf('/',8));
@@ -173 +172 @@
             code += ' allowfullscreen="true" />';
             return code;
+        };
+
+
+        /** hack for backward-compatible BOTR embeds **/
+        private function getBotrPreview():String {
+            Logger.log(_player.config.playlistfile,'sharing');
+            var link:String = '';
+            link += _player.config.playlistfile.substr(0,_player.config.playlistfile.indexOf('/',8));
+            link += '/previews/' + _player.playlist.currentItem['image'].substr(-12,8);
+            link += '-' + _config['code'];
+            return link;
         };
 
@@ -266 +276 @@
                 var title:String = _player.playlist.currentItem['title'];
                 if (_shareScreen.mouseX < 75) {
-                    url = 'http://www.facebook.com/share.php?u='+encodeURI(_shareField.text);
+                    if(_config['code'].toString().length == 8) {
+                        url = 'http://www.facebook.com/share.php?u='+getBotrPreview();
+                    } else { 
+                        url = 'http://www.facebook.com/share.php?u='+encodeURIComponent(_shareField.text);
+                    }
                     navigateToURL(new URLRequest(url));
                 } else if (_shareScreen.mouseX < 150) {
                     if(title) { 
-                        url = 'http://twitter.com/home?status='+ title +': '+encodeURI(_shareField.text);
+                        url = 'http://twitter.com/?status='+ title +': '+encodeURIComponent(_shareField.text);
                     } else {
-                        url = 'http://twitter.com/home?status=' + encodeURI(_shareField.text);
+                        url = 'http://twitter.com/?status=' + encodeURIComponent(_shareField.text);
                     }
                     navigateToURL(new URLRequest(url));
                 } else if (_shareScreen.mouseX < 225) {
                     if(_config['code']) {
-                        url = 'http://www.myspace.com/Modules/PostTo/Pages/?c='+encodeURI(_embedField.text);
+                        url = 'http://www.myspace.com/Modules/PostTo/Pages/?c='+encodeURIComponent(_embedField.text);
                     } else {
-                        url = 'http://www.myspace.com/Modules/PostTo/Pages/?c='+encodeURI(_shareField.text);
+                        url = 'http://www.myspace.com/Modules/PostTo/Pages/?c='+encodeURIComponent(_shareField.text);
                     }
-                    if(title) { url += '&t=' + title; }
+                    if(title) { url += '&t=' + encodeURIComponent(title); }
                     navigateToURL(new URLRequest(url));
                 } else {
-                    url = 'mailto:?body='+encodeURI(_shareField.text);
-                    if(title) { url += '&subject=' + title; }
+                    url = 'mailto:?body='+encodeURIComponent(_shareField.text);
+                    if(title) { url += '&subject=' + encodeURIComponent(title); }
                     navigateToURL(new URLRequest(url),'_self');
                 }

plugins/sharing/v5/test/assets/playlist.xml

@@ -15 +15 @@
                         <description>This entry has no code defined, so the plugin uses the embed code of the current player.</description>
                         <enclosure url="http://content.bitsontherun.com/videos/nPripu9l-327.mp4" />
-                        <jwplayer:sharing.link>http://www.longtailvideo.com</jwplayer:sharing.link>
+                        <jwplayer:sharing.link><![CDATA[http://www.longtailvideo.com?foo=bar&hash=#]]></jwplayer:sharing.link>
                 </item>