Here is a quick patch that just uses empty() to check the value in the $manifests instead of !. When you have more than one app server, you get improper index notices.

apps-check-empty-multiple-servers.patch592 bytespopulist
Members fund testing for the Drupal project. Drupal Association Learn more


jec006’s picture

Status: Needs review » Postponed (maintainer needs more info)

I'm having a hard time reproducing this issue - are you still having this problem on the current dev version?

This seems like a good patch that just in general better meets coding standards, but I also want to make sure any issues get resolved.

(i'm committing the patch from the OP as a style fix)


populist’s picture

Status: Postponed (maintainer needs more info) » Fixed

Looking good!

Status: Fixed » Closed (fixed)

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