Changeset 5453

Timestamp:

01/25/08 01:04:17 (7 months ago)

Author:

drry

Message:

lang/javascript/userscripts/hatebuclippingsiteremover.user.js: tiny refactoring.

Files:

• lang/javascript/userscripts/hatebuclippingsiteremover.user.js (modified) (2 diffs)

lang/javascript/userscripts/hatebuclippingsiteremover.user.js

@@ -7 +7 @@
 // ==/UserScript==
 (function(){
-    var CLIPPINGSITELIST = [
+    const CLIPPINGSITELIST = [
                     'http://d.hatena.ne.jp/ringod/',
                     'http://d.hatena.ne.jp/johzan/',
@@ -31 +31 @@
     function is_clippingsite(url) {
         for (var i = CLIPPINGSITELIST.length; i-->0; ) {
-            if (url.indexOf(CLIPPINGSITELIST[i], 0) == 0) return true;
+            if (url.indexOf(CLIPPINGSITELIST[i]) == 0) return true;
         }
         return false;
     }
 
-    var bookmarklist = getElementsByXPath("//ul[@class='bookmarklist']");
+    var bookmarklist = getElementsByXPath('//ul[@class="bookmarklist" and not(@id="bookmarked_user")]//li/a[@href]');
     if (!bookmarklist) return;
-    if (bookmarklist[0].id == 'bookmarked_user') bookmarklist.shift();
-    if (!bookmarklist.length) return;
 
-    for (var ul_i = bookmarklist.length; ul_i-->0; ) {
-        var li = getElementsByXPath('li/a', bookmarklist[ul_i]);
-        if (!li) continue;
-        for (var li_i = li.length; li_i-->0; ) {
-            if (is_clippingsite(li[li_i].href)) {
-                li[li_i].parentNode.innerHTML = 'removed to clipping site';
-            }
-        }
+    for (var i = bookmarklist.length; i-->0; ) {
+        if (is_clippingsite(bookmarklist[i].href))
+            bookmarklist[i].parentNode.innerHTML = 'removed to clipping site';
     }
 })();