Changeset 25326 for lang/javascript/vimperator-plugins

Timestamp:

11/29/08 12:38:06 (6 weeks ago)

Author:

janus_wel

Message:

the process to convert type from string to boolean cut off to function stringToBoolean().

Files:

• lang/javascript/vimperator-plugins/trunk/clock.js (modified) (5 diffs)

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

@@ -5 +5 @@
  * @description-ja  とけい。
  * @author          janus_wel <janus_wel@fb3.so-net.ne.jp>
- * @version         0.12
+ * @version         0.13
  * @minversion      2.0pre
  * @maxversion      2.0pre
@@ -54 +54 @@
 const format   = liberator.globalVariables.clock_format   || '[%t]';
 const position = liberator.globalVariables.clock_position || 'liberator-commandline-command';
-let atemp      = liberator.globalVariables.clock_after;
-const after =   (atemp === undefined)             ? true
-              : (atemp.toLowerCase() === 'false') ? false
-              : (/^\d+$/.test(atemp))             ? parseInt(atemp, 10)
-              :                                     true;
+const after    = stringToBoolean(liberator.globalVariables.clock_after, true);
 
 // class definitions
@@ -228 +224 @@
 clock.generate();
 
-// appendChild
+// insert
 after
     ? insertNodeAfterSpecified(clock.instance, insertBase)
@@ -235 +231 @@
 // register command
 [
-    [['clockhide'],   'hide clock',   function () clock.hide(), ],
+    [['clockhide'],   'hide clock',   function () clock.hide(),   ],
     [['clockappear'], 'clock appear', function () clock.appear(), ],
-    [['clockstart'],  'start clock',  function () clock.start(), ],
-    [['clockstop'],   'stop clock',   function () clock.stop(), ],
-].forEach( function ([names, desc, func]) commands.addUserCommand(names, desc, func, {}) );
+    [['clockstart'],  'start clock',  function () clock.start(),  ],
+    [['clockstop'],   'stop clock',   function () clock.stop(),   ],
+].forEach( function ([n, d, f]) commands.addUserCommand(n, d, f, {}) );
 
 
@@ -282 +278 @@
     }
 }
+
+// type conversion
+function stringToBoolean(str, defaultValue) {
+return !str                          ? (defaultValue ? true : false)
+     : str.toLowerCase() === 'false' ? false
+     : /^\d+$/.test(str)             ? (parseInt(str) ? true : false)
+     :                                 true;
+}
 } )();