Internal calls to query() in sql commands should run even in --simulate ...

[weitzman]

Currently, sql-dump --simulate does not work properly as table selection queries do not run (because we are in simulate mode). That can be fixed in an ugly way by this PR or perhaps we should just use drush_invoke_process() with override-simulated backend option instead. My annoyance with that approach is that you have to decide whether or not to $integrate the log for the backend call. I wish we could integrate only in the case of errors. Because usually, $integrate just adds noise.

[weitzman]

Assigning to Greg. Anyone else is welcome to chime in.

[greg-1-anderson]

I think that in the short term, we should just commit the patch submitted. A better solution would be to make the query function take a parameter that controlled whether or not the function was simulated or not. In order to do this, though, I think we'd want to implement the object-based context system, and pass the active context around as a parameter to most worker functions. Then, we would have $context->get_option(...) instead of drush_get_option(). This would make it easy for code that needs to do special-purpose operations, like override simulated, easy to manage. This would take a fairly sweeping change to Drush, though, which is why I favor just putting in the quick fix.