Ticket Details

`\lithium\action\Response::cache()` doesn't work as expected

BUG Ticket (closed)

`\lithium\action\Response::cache()` doesn't work properly with timestamp and string parameters as explained in API.

See this tests:

[http://dev.lithify.me/forks/farhadi/lithium/commits/view/91c0a96f52bad9a4c4689381ff6b2596ccf8b9a7](http://dev.lithify.me/forks/farhadi/lithium/commits/view/91c0a96f52bad9a4c4689381ff6b2596ccf8b9a7)
on 08.30.10 reported by: farhadi owned by: nate

Updates

on 08.30.10 by farhadi
I fixed it in here:
http://dev.lithify.me/forks/farhadi/lithium/commits/view/6690b825b0c61133c69751ea6229be2e6f5ad8d1
on 08.30.10 by farhadi
BTW, we should not use Last-Modified header when we want to disable cache.
Actually this header causes the browser to cache the response and the browser will use this header's value as a cache validator.

I fixed it in here:
[http://dev.lithify.me/forks/farhadi/lithium/commits/view/9dc78c717c056c84d62a5055221e9a54fd45d0ad](http://dev.lithify.me/forks/farhadi/lithium/commits/view/9dc78c717c056c84d62a5055221e9a54fd45d0ad)
(fixed) on 09.29.10 by nate
  • owner was changed to nate
  • status was changed to closed
  • resolution was changed to fixed
Pushed the commits to the x-encoding branch. Will be merged to master shortly. Thanks for your work!