Put back set_with_expire that executes SET and EXPIRE in a MULTI bloc…

…k, but with a recommendation to use #setex.

lib/redis.rb

@@ -333,6 +333,14 @@ def setex(key, ttl, value)
     @client.call(:setex, key, ttl, value)
   end
 
+  def set_with_expire(key, value, ttl)
+    self.class.deprecate('Redis: Use the setex method instead of set_with_expire if your version of Redis supports the SETEX command')
+    multi do
+      set(key, value)
+      expire(key, ttl)
+    end
+  end
+
   def mset(*args)
     @client.call(:mset, *args)
   end

test/redis_test.rb

@@ -275,6 +275,20 @@ def redis.call(*attrs)
       assert_equal nil, @r.get("foo")
     end
 
+    test "SET with EXPIRE" do
+      capture_stderr do
+        @r.set_with_expire("bar", "s2", 1)
+
+        assert $stderr.string["Redis: Use the setex method instead of set_with_expire if your version of Redis supports the SETEX command"]
+
+        assert_equal "s2", @r.get("bar")
+
+        sleep 2
+
+        assert_equal nil, @r.get("bar")
+      end
+    end
+
     test "GETSET" do
       @r.set("foo", "bar")