http://www.couchbase.com/issues This file is an XML representation of an issue en-us 5.2.4 845 26-12-2012 http://www.couchbase.com/issues/browse/PCBC-154 Couchbase PHP client library I&#39;m really not sure what the behavior should be, but it&#39;s worth clarifying. <br/> <br/> What does touch with an argument of 0 imply -- it is effectively a delete. <br/> <br/> Perhaps this should be a warning etc? <br/> <br/> Please post your comments. PCBC-154

test for touch with expiration needs updating (maybe)

Bug Major Resolved Won't Fix Public Trond Norbye Mark Nunberg Mon, 3 Dec 2012 13:48:01 -0600 Tue, 16 Apr 2013 02:06:32 -0500 Tue, 16 Apr 2013 02:06:32 -0500 library 0 2 Given that using an expiration of 0 means &quot;forever&quot;, I would think a touch of 0 would extend the time to forever. <br/> <br/> In fact, the documentation says that: <br/> <a href="http://www.couchbase.com/docs/couchbase-devguide-2.0/cb-touch.html">http://www.couchbase.com/docs/couchbase-devguide-2.0/cb-touch.html</a> <br/> <br/> Trond: can you verify we do the right thing here? See also a commit in limbo: <br/> <a href="http://review.couchbase.org/#/c/21848/">http://review.couchbase.org/#/c/21848/</a> This is not a bug in the PHP extension, but a bug in the server. I filed <a href="http://www.couchbase.com/issues/browse/MB-7342" title="Touch with expiry time 0 does not work"><strike>MB-7342</strike></a> for the issue. Thanks for getting that addressed Trond. The underlying question on this test has been resolved. There was a server bug and a touch with expiration of 0 should not return an error, but should rather extend it from whatever the current time is to forever.<br/> <br/> A good functional test would be:<br/> - Set with expiration of 3 secs<br/> - Touch with expiration of 0<br/> - Check 3 secs later to see if it&#39;s still there.<br/> <br/> Trond: can you review what, if anything more, needs to be done here? I think it&#39;s closeable. I verified that this worked as intended in a test program. I don&#39;t want to add more tests than necessary to the test suite that does sleep, and this test is not testing the client but verifies the server and should therefore be done elsewhere... Rank 9758 Time In Status