Zend Framework

Zend_Cache_Core::getIdsMatchingAnyTags() not implemented

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.10.3
  • Component/s: Zend_Cache
  • Labels:
    None
  • Fix Version Priority:
    Should Have

Description

Cache backends implementing the extended interface all have a getIdsMatchingAnyTags() method. However, this is not implemented in Zend_Cache_Core, so the backend method can not be called.

A new method Zend_Cache_Core::getIdsMatchingAnyTags() should be created similar to Zend_Cache_Core::getIdsMatchingTags().

Activity

Hide
Satoru Yoshida added a comment -

Thank You for report, I will be happy if You try trunk code ( SVN r20815)

Show
Satoru Yoshida added a comment - Thank You for report, I will be happy if You try trunk code ( SVN r20815)
Hide
Sven Axelsson added a comment - - edited

The impementation looks fine.

However, there ought to be unit tests that test calling all extended backend functions through Zend_Cache_Core, not just load, save, test, clean. And that would mean having Zend_Cache_Backend_Test implement the extended interface to make those methods available.

I could do that if there's interest for it.

Show
Sven Axelsson added a comment - - edited The impementation looks fine. However, there ought to be unit tests that test calling all extended backend functions through Zend_Cache_Core, not just load, save, test, clean. And that would mean having Zend_Cache_Backend_Test implement the extended interface to make those methods available. I could do that if there's interest for it.
Hide
Satoru Yoshida added a comment -

Yes, it's right, Sven.
We should implement missing test and do several changes that the test needs.

Show
Satoru Yoshida added a comment - Yes, it's right, Sven. We should implement missing test and do several changes that the test needs.
Hide
Satoru Yoshida added a comment -

merged SVN trunk r20815 to 1.10 branch at SVN r21290.

Show
Satoru Yoshida added a comment - merged SVN trunk r20815 to 1.10 branch at SVN r21290.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: