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.

CommentFileSizeAuthor
apps-check-empty-multiple-servers.patch592 bytespopulist
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

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)

Thanks

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.