We need to bring the user instructions for webserver access up to date so that they match up with the new logic in apps_installer_has_write_access().

That being said... i think it would be great, for many reasons, to install apps in either sites/all/modules/apps or perhaps even a new sites/all/apps. The former being much easier and the latter being probably an interesting idea for the future. Stricter permissioning would be the main functional reason for both suggestions however i also think if we really want to push forward the idea of apps as being distinct from core/contrib and higher up in the stack then we might want to consider something like the latter.

Anyways...

Members fund testing for the Drupal project. Drupal Association Learn more

Comments

pirog’s picture

Status: Active » Needs review
FileSize
1.76 KB

here is the patch

webflo’s picture

Status: Needs review » Reviewed & tested by the community

Looks good! Experienced this problem during the installation of panopoly.

febbraro’s picture

Status: Reviewed & tested by the community » Fixed

Thanks @pirog!

Status: Fixed » Closed (fixed)

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