Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pwolanin’s picture

Looks like a reasonable start. I'd do a little more method name cleanup.

We also don't even need to re-write the tree in the 1st method - I think it could just be:


if ($instance) {

...

}
else {
  unset($tree[$key])
}
pwolanin’s picture

Status: Needs review » Fixed
FileSize
4.6 KB
7.04 KB

Here's what I have in mind - incorporating this into the branch.

victor-shelepen’s picture

:) Why it is fixed if it has not been commited.

Status: Fixed » Closed (fixed)

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