Bug #39368
Support unlimited lifetime - same way as caching framework does
| Status: | Closed | Start date: | 2012-07-30 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assignee: | Christian Kuhn | % Done: | 100% |
|
| Category: | - | |||
| Target version: | - | |||
| Votes: | 0 |
Description
Currently enetcache does not distinguish between lifetime NULL (= fallback to default) and 0 (= unlimited). Attached patch fixes that.
History
Updated by Nicole Cordes 6 months ago
I reviewed the patch and found this
if ($lifetime === 0) {
$lifetime = 0;
}
IMHO this is not necessary as it defines with an already given value?!
Updated by Nicole Cordes 6 months ago
- File 39368_default_value_for_null.diff added
I changed the patch and removed the reassignment of zero as lifetime.
Updated by Christian Kuhn 4 months ago
- Status changed from New to Accepted
- Assignee set to Christian Kuhn
Updated by Christian Kuhn 4 months ago
- Status changed from Accepted to Resolved
- % Done changed from 0 to 100
Thanks! Fixed in trunk revision 70243.
Updated by Christian Kuhn 4 months ago
- Status changed from Resolved to Closed
released with enetcache 1.1.0