Problem/Motivation

Currently running delete:all-delete-entities under Drush 10.x errors out when attempting use removed drush methods(drush_get_option, drush_set_error).

Steps to reproduce

Run 'drush delete:all-delete-entities...' with type and bundle options.

Proposed resolution

replace 'drush_get_option' with 'Drush::input()->hasOption('...')'
and 'drush_set_error' with 'Drush::logger()->error(...)'
see attached patch

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 3200447-3.patch2.32 KBsdrong
#2 3200447.patch1.32 KBsdrong
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sdrong created an issue. See original summary.

sdrong’s picture

FileSize
1.32 KB
sdrong’s picture

Issue summary: View changes
sdrong’s picture

FileSize
2.32 KB
uniquename’s picture

Works fine for me.

nicholasThompson’s picture

Status: Active » Reviewed & tested by the community

Worked for me too - couldn't delete taxonomy_terms before, but with patch #3 I can.

  • dipakmdhrm committed e88a855 on 2.x authored by sdrong
    Issue #3200447 by sdrong, uniquename, nicholasThompson: Remove use of...
dipakmdhrm’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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