Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yched’s picture

Status: Active » Needs review
FileSize
784 bytes
alexpott’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
789 bytes
402 bytes

Looks good and works when locally tested... small c&p error...

yched’s picture

lol @ c/p error - busted !

sun’s picture

Status: Reviewed & tested by the community » Fixed

Thanks for reporting, reviewing, and testing! Committed.

Piazza’s picture

Status: Needs work » Needs review

.

Status: Fixed » Needs work

The last submitted patch, admin_menu-register_cache-1950180.2.patch, failed testing.

yched’s picture

Status: Needs review » Fixed

Why reopen a fixed issue?

egger’s picture

Status: Fixed » Needs work
alexpott’s picture

Status: Needs work » Fixed

Nope it was fixed... and now it needs a new fix and a new issue :)

core has change to use yml for service declarations and for self creating cache bins... I would advise to hold off working on this until... #1167144: Make cache backends responsible for their own storage lands

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.