Existing sites were found on this platform. These sites will need to be deleted before this platform can be deleted.

This is not always true. Provision only checks to see if a settings.php file exists. This doesn't mean there is an actual site present.

Same is true of the "Import" task. It only checks to see if a settings.php file exists, and then tries to import it, even if there is no actual database present.

We should have provision try to bootstrap these sites before assuming there are actual sites running in the codebase.

Comments

Jon Pugh created an issue.

helmo’s picture

+1 for refactoring :)

One thing to test when doing this is if you can still delete a 'broken' platform. E.g. with a syntax error in the code.