prune works with numbers... who cares?

vol4ok · Feb 27, 2012 · 58cf4d1 · 58cf4d1
1 parent 7df0e74
commit 58cf4d1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/lib/underscore.string.js b/lib/underscore.string.js
@@ -325,11 +325,13 @@
      * @author github.com/sergiokas
      */
     prune: function(str, length, pruneStr){
-      var pruned, borderChar, template = (''+str).replace(/\W/g, function(ch){
+      str = ''+str; length = ~~length;
+      pruneStr = pruneStr != null ? ''+pruneStr : '...'; 
+
+      var pruned, borderChar, template = str.replace(/\W/g, function(ch){
         return (ch.toUpperCase() != ch.toLowerCase()) ? 'A' : ' '
       });
 
-      pruneStr = pruneStr || '...'; length = ~~length;
 
       borderChar = template[length];
 

diff --git a/test/strings.js b/test/strings.js
@@ -230,6 +230,8 @@ $(document).ready(function() {
     equals(_('Привет, мир').prune(8), 'Привет...');
     equals(_('Привет, жестокий мир').prune(16), 'Привет, жестокий...');
     equals(_('Привет, мир').prune(22), 'Привет, мир');
+    equals(_(123).prune(10), '123')
+    equals(_(123).prune(1,1), '11')
   });
 
   test('String: isBlank', function(){