rewritten 0.6.0 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4bbca7b948eee5641bd22eb57aec66f65a2501fc
-  data.tar.gz: 9eba6f5b82e869fef3c50a487f8ba0e60f793d3e
+  metadata.gz: 6b219a3b39bc494eb23d7768fd6de8fbab2b7933
+  data.tar.gz: 2f1ee10125acebf0d1b007e1d65a7f2b6648c6c7
 SHA512:
-  metadata.gz: 7b0ab0e5fc9379e0ea051040a864fbdeb5f78662fa92f24d9d15bc3c933a78313518160f2e0c20cce80bfbe36d4ec62d3cb9bde81eb3bb24dfc4f31f6ea66331
-  data.tar.gz: 4b237a6666a91216654ac7f894bce5882dbcf73274e8cc07b9ed237f2be64c9e987663ca2106fde7c783957c13706ac800f3e5fdf13b1065523538d80b796df3
+  metadata.gz: e15bbb18a1f5bf571c0846049ca57daba0ffd1665f7ee5d4035decbbb16389921abf28a9b6be4c9b03e6959a7e1bf9def345dce6f9402a00db00ffe057b3b8fb
+  data.tar.gz: 5e4bee52c3efef72088ca7ebd0326c7168cc17848b8c62bf2a4d4753363bbf0d0df28d4b9dc9e2b156e8d7cc966b0e058c9518f9ffbc05d5ab287bd524d25be7

data/HISTORY.rdoc

@@ -1,3 +1,7 @@
+== 0.7.0
+
+  * use sorted sets for the targets instead of lists
+
 == 0.6.0 
 
   * backwards translations (from resource to current translation)

data/lib/rewritten.rb

@@ -130,7 +130,10 @@ module Rewritten
     redis.set("from:#{from}", to)
     redis.sadd(:froms, from) 
     redis.sadd(:tos, to) 
-    redis.rpush("to:#{to}", from) 
+
+
+    score = redis.zcard("to:#{to}") || 0
+    redis.zadd("to:#{to}", score, from)  
   end
 
   def add_translations(to, froms)
@@ -138,13 +141,13 @@ module Rewritten
   end
 
   def num_translations(to)
-    Rewritten.redis.llen("to:#{to}")
+    Rewritten.redis.zcard("to:#{to}")
   end
 
   def remove_translation(from, to)
     Rewritten.redis.del("from:#{from}")
     Rewritten.redis.srem(:froms, from)
-    Rewritten.redis.lrem("to:#{to}", 0, from)
+    Rewritten.redis.zrem("to:#{to}", from)
     Rewritten.redis.srem(:tos, to) if num_translations(to) == 0
  end
 
@@ -176,11 +179,11 @@ module Rewritten
   end
 
   def get_all_translations(to)
-    Rewritten.redis.lrange("to:#{to}", 0, -1)
+    Rewritten.redis.zrange("to:#{to}", 0, -1)
   end
 
   def get_current_translation(path)
-    translation = Rewritten.list_range("to:#{path}", -1)  
+    translation = Rewritten.z_range("to:#{path}", -1)
     return translation if translation
     return path
   end
@@ -209,13 +212,11 @@ module Rewritten
 
   # Does the dirty work of fetching a range of items from a Redis list
   # and converting them into Ruby objects.
-  def list_range(key, start = 0, count = 1)
+  def z_range(key, start = 0, count = 1)
     if count == 1
-      #decode redis.lindex(key, start)
-      redis.lindex(key, start)
+      redis.zrange(key, start, start)[0]
     else
-      Array(redis.lrange(key, start, start+count-1)).map do |item|
-        #decode item
+      Array(redis.zrange(key, start, start+count-1)).map do |item|
         item
       end
     end

data/lib/rewritten/server.rb

@@ -201,6 +201,7 @@ module Rewritten
         # delete old translations
         Rewritten.remove_all_translations(@old_to)
         # create new translations
+        
         Rewritten.add_translations(@to, @translations)
 
         redirect u("/to?to=#{escape(@to)}")
@@ -222,7 +223,8 @@ module Rewritten
     end
 
     get "/to" do
-      translations = Rewritten.list_range(params[:to], 0, -1) 
+      #translations = Rewritten.list_range(params[:to], 0, -1) 
+      translations = Rewritten.z_range(params[:to], 0, -1) 
       show "to" 
     end
 

data/lib/rewritten/server/views/to.erb

@@ -17,7 +17,7 @@
     <th>Action</th>
   </tr>
 
-  <% for translation in Rewritten.list_range("to:#{params[:to]}", 0, 100) %>
+  <% for translation in Rewritten.z_range("to:#{params[:to]}", 0, 100) %>
     <tr>
       <td class=''><a href="<%= translation %>" target="new"><%= translation %></td>
         <td class=''>

data/lib/rewritten/version.rb

@@ -1,4 +1,4 @@
 module Rewritten
-  VERSION = "0.6.0"
+  VERSION = "0.7.0"
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rewritten
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.0
 platform: ruby
 authors:
 - Kai Rubarth