geerlingguy / ansible-role-drupal

Ansible Role - Drupal
https://galaxy.ansible.com/geerlingguy/drupal/
MIT License
92 stars 51 forks source link

Issue #43: Fix site installed check always returning false on translated sites #44

Closed oxyc closed 7 years ago

oxyc commented 7 years ago

Like @azurams pointed out in https://github.com/geerlingguy/drupal-vm/issues/1399#issuecomment-305526777 the output of drush status bootstrap will always be empty if not installed.

Looking at https://github.com/drush-ops/drush/blob/8.1.11/commands/core/core.drush.inc#L592, to update the drupal_site_installed check, we may assume that if drush status bootstrap does not return any new line then bootstrap failed. If the string "Drupal bootstrap" is present in the output, then bootstrap succeeded ?

The code he references shows this and drush silently gives empty output for anything which doesn't have an entry in that status-table map. eg drush status foobar

geerlingguy commented 7 years ago

Restarted failing build.