Ticket #2531339 (closed defect)

Reporter


Stuart Colville
Opened: 10/26/11
Last modified: 04/17/12
Status: closed
Type: defect
Resolution: fixed

Owner


Eric Ferraiuolo
Target Release: 3.5.0
Priority: P3 (normal)
Summary: YUI Cache drops values when using UniqueKeys = true
Description:

Cache shifts values off the cache array rather than remove the existing item so it can be added again.

See test-case url below.

Type: defect Observed in Version: 3.4.1
Component: Cache Severity: S3 (normal)
Assigned To: Eric Ferraiuolo Target Release: 3.5.0
Location: Library Code Priority: P3 (normal)
Tags: Relates To:
Browsers: All
URL: http://jsfiddle.net/muffinresearch/7g8us/
Test Information:

Change History

Stuart Colville

YUI Contributor

Posted: 10/26/11

I've added a pull request with a patch here: https://github.com/yui/yui3/pull/39

Eric Ferraiuolo

YUI Developer

Posted: 10/27/11
  • location changed to Library Code
  • milestone changed to 3.5.0
  • priority changed to P3 (normal)
  • status changed from new to accepted

Eric Ferraiuolo

YUI Developer

Posted: 10/27/11
  • estimated changed from 0 to 0.1
  • remaining changed from 0 to 0.1
  • sprint changed to sprint 1

Stuart Colville

YUI Contributor

Posted: 10/27/11

I've updated the pull request [1] with an improved patch and a test to cover checking the values.

[1] https://github.com/yui/yui3/pull/39

Jenny Donnelly

YUI Developer

Posted: 11/9/11
  • sprint changed from sprint 1

Jenny Donnelly

YUI Developer

Posted: 11/10/11

FYI, Stuart's CLA has been received.

Eric Ferraiuolo

YUI Developer

Posted: 02/21/12
  • milestone changed from 3.5.0 to 3.6.0

Punting until 3.6.0 when I expect to do some work on the Cache utility.

Eric Ferraiuolo

YUI Developer

Posted: 02/21/12
  • milestone changed from 3.6.0 to 3.5.0
  • status changed from accepted to checkedin

Instead of waiting I merged this Pull Request and cleaned it up.

Eric Ferraiuolo

YUI Developer

Posted: 02/21/12
  • completed changed from 0 to 0.1
  • remaining changed from 0.1 to 0

Jenny Donnelly

YUI Developer

Posted: 04/17/12
  • resolution changed to fixed
  • status changed from checkedin to closed

Closing out bugs checked in to 3.5.0.