Changeset 13282

Timestamp:

06/05/08 20:06:48 (5 years ago)

Author:

trapezoid

Message:

直後にページ遷移すると遷移後のページがブックマークされる可能性があったのを修正しました

Files:

• lang/javascript/vimperator-plugins/trunk/direct_bookmark.js (modified) (11 diffs)

lang/javascript/vimperator-plugins/trunk/direct_bookmark.js

@@ -1 +1 @@
 // Vimperator plugin: 'Direct Post to Social Bookmarks'
-// Version: 0.09
-// Last Change: 04-Jun-2008. Jan 2008
+// Version: 0.10
+// Last Change: 05-Jun-2008. Jan 2008
 // License: Creative Commons
 // Maintainer: Trapezoid <trapezoid.g@gmail.com> - http://unsigned.g.hatena.ne.jp/Trapezoid
@@ -44 +44 @@
 //  ':bentry'
 //      Goto Bookmark Entry Page
-//  ':bicon'
-//      Show Bookmark Count as Icon
 (function(){
     var evalFunc = window.eval;
@@ -55 +53 @@
             }
         }
-    } catch(e) { liberator.log('warning: wassr.js is working with unsafe sandbox.'); }
+    } catch(e) { liberator.log('warning: direct_bookmark.js is working with unsafe sandbox.'); }
 
     var useServicesByPost = liberator.globalVariables.direct_sbm_use_services_by_post || 'hdl';
@@ -323 +321 @@
             loginPrompt:{ user:'', password:'', description:'Enter username and password.' },
             entryPage:'http://b.hatena.ne.jp/entry/%URL%',
-            poster:function(user,password,url,comment,tags){
+            poster:function(user,password,url,title,comment,tags){
                 var tagString = tags.length > 0 ? '[' + tags.join('][') + ']' : "";
                 var request =
@@ -370 +368 @@
             loginPrompt:{ user:'', password:'', description:'Enter username and password.' },
             entryPage:'http://del.icio.us/url/%URL::MD5%',
-            poster:function(user,password,url,comment,tags){
-                var title = liberator.buffer.title;
+            poster:function(user,password,url,title,comment,tags){
                 var request_url = 'https://api.del.icio.us/v1/posts/add?' + [
                     ['url', url], ['description', title], ['extended', comment], ['tags', tags.join(' ')]
@@ -419 +416 @@
             loginPrompt:{ user:'', password:'apikey', description:'Enter username and apikey.\nyou can get "api-key" from\n\thttp://clip.livedoor.com/config/api' },
             entryPage:'http://clip.livedoor.com/page/%URL%',
-            poster:function(user,password,url,comment,tags){
-                var title = liberator.buffer.title;
+            poster:function(user,password,url,title,comment,tags){
                 var request_url = 'http://api.clip.livedoor.com/v1/posts/add?' + [
                     ['url', url], ['description', title], ['extended', comment], ['tags', tags.join(' ')]
@@ -458 +454 @@
             loginPrompt:null,
             entryPage:'%URL%',
-            poster:function(user,password,url,comment,tags){
+            poster:function(user,password,url,title,comment,tags){
                 var request_url = 'http://www.google.com/bookmarks/mark';
                 var params = [
-                    ['bkmk', url], ['title', liberator.buffer.title], ['labels', tags.join(',')]
+                    ['bkmk', url], ['title', title], ['labels', tags.join(',')]
                 ].map(function(p) p[0] + '=' + encodeURIComponent(p[1])).join('&');
                 return Deferred.http({
@@ -481 +477 @@
             loginPrompt:null,
             entryPage:'%URL%',
-            poster:function(user,password,url,comment,tags){
+            poster:function(user,password,url,title,comment,tags){
                 const taggingService = Cc["@mozilla.org/browser/tagging-service;1"].getService(Ci.nsITaggingService);
                 var nsUrl = Cc["@mozilla.org/network/standard-url;1"].createInstance(Ci.nsIURL);
@@ -487 +483 @@
                 taggingService.tagURI(nsUrl,tags);
                 try{
-                    Application.bookmarks.tags.addBookmark(liberator.buffer.title, nsUrl);
+                    Application.bookmarks.tags.addBookmark(title, nsUrl);
                 }catch(e){
                     throw "Places: faild";
@@ -590 +586 @@
             }
 
+            var url = liberator.buffer.URL;
+            var title = liberator.buffer.title;
+
             targetServices.split(/\s*/).forEach(function(service){
                 var user, password, currentService = services[service] || null;
@@ -595 +594 @@
                 d = d.next(function() currentService.poster(
                     user,password,
-                    isNormalize ? getNormalizedPermalink(liberator.buffer.URL) : liberator.buffer.URL,
+                    isNormalize ? getNormalizedPermalink(url) : url,title,
                     comment,tags
                 ));