The call to drush_invoke_process() in _drush_sql_get_db_table_list() has 'integrate' => FALSE set as a command option, when it should be set as a backend option.

As it stands, the full list of SQL tables to be dropped is printed out each time site-install or sql-sync commands are run.

This patch moves the option into the $backend_options array.

CommentFileSizeAuthor
#1 sql-table-list-2043661-1.patch658 byteskostajh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kostajh’s picture

Status: Active » Needs review
FileSize
658 bytes
greg.1.anderson’s picture

'override-simulated' is also in the wrong place.

moshe weitzman’s picture

Status: Needs review » Fixed

Committed, with enhancement from #2. I think I copied this pattern from elsewhere so we may not be done yet.

Status: Fixed » Closed (fixed)

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