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.

Files: 
CommentFileSizeAuthor
apps-check-empty-multiple-servers.patch592 bytespopulist

Comments

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

Status:Postponed (maintainer needs more info)» Fixed

Looking good!

Status:Fixed» Closed (fixed)

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