Makes it possible to purge + force, so delete a platform even if
- folder is missing
- provision "thinks" there are still sites in there etc.

Basically i remove the drupal bootstrap as it is not primary needed and only check for sites if purge is not set ( not set by default ).
For the UI part check the issue http://drupal.org/node/1074918

Optional:
add clie docu for purge

CommentFileSizeAuthor
#6 purgeoption.patch2.12 KBeugenmayer
purgeoption.patch1.6 KBeugenmayer

Comments

anarcat’s picture

anarcat commented
Status: Needs review » Needs work

There is a --force flag in drush already that we use elsewhere (for example in provision-backup) and that I think we should reuse here, so I encourage you to use that too..

Also, try to use "false" instead of 0 when you mean false (as in the default value for that flag).

I wonder if provision_drupal_find_sites() works without bootstrapping the platform or changing that bootstrap level will break site deletion?

Otherwise this seems like a good idea.

eugenmayer’s picture

eugenmayer commented

drupal_find_sites does not work without, will always return "found one".

i will try to reuse force, and report back. Thank

anarcat’s picture

anarcat commented

drupal_find_sites does not work without, will always return "found one".

I'm sorry, I don't understand that.

eugenmayer’s picture

eugenmayer commented

I wonder if provision_drupal_find_sites() works without bootstrapping the platform or changing that bootstrap level will break site deletion?

No it does not work without a bootstrap, will always return "found". but in addition its one task of purge (force) to delete in any case.

eugenmayer’s picture

eugenmayer commented
Status: Needs work » Needs review

incorporated our suggestion anarcat and fixed bootstrap for "site check".

eugenmayer’s picture

eugenmayer commented
StatusFileSize
new purgeoption.patch2.12 KB

woops, patch

anarcat’s picture

anarcat commented
Version: » 6.x-0.4-alpha3
Status: Needs review » Patch (to be ported)

Incredibly enough, this patch still applies, and looks about sound.

patch c0d53c7 applied to 2.x for a testing period, merge when tested.

j0nathan’s picture

j0nathan commented

Subscribing.

anarcat’s picture

anarcat commented
Status: Patch (to be ported) » Fixed

merged in 1.x.

Status: Fixed » Closed (fixed)

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