Problem/Motivation

README contains information about dmu-clean drush command but drupalmoduleupgrader.drush.inc does not contain actual command and drush says that such command cannot be found.

I don't know whether this is a bug or not. Maybe the command itself was removed, but info in readme left as is. Or maybe someone removed the command by accident.

Proposed resolution

Restore the command code OR update README.txt.

CommentFileSizeAuthor
#8 dmu-readme-2598932-8.patch2.43 KBhongpong

Comments

pingwin4eg created an issue. See original summary.

plazik’s picture

plazik commented

It was removed for some reason http://cgit.drupalcode.org/drupalmoduleupgrader/commit/drupalmoduleupgra...

aswathyajish’s picture

aswathyajish commented

Me also got the same issue.

chipway’s picture

chipway commented

Hi,
Same issue. The command was removed by Phéna Proxima on 2015-05-20, but no issue explains why, and how to help.

liquidcms’s picture

liquidcms commented
Component: Miscellaneous » Code
Category: Support request » Bug report
Priority: Normal » Major

I am pretty sure the removal of this is a bug; unless possibly it was incorporated into dmu-upgrade, in which case still a bug since it isn't being done there either.

I am guessing part of this issue: i have 4 tables created by the upgrade process specific to the module i am upgrading and which i am sure are not required. I deleted them manually and my module still works.

joseph.olstad’s picture

joseph.olstad
Primary language French
commented
Parent issue: » #2949601: Drush 9 compatibility

**EDIT** sorry I think I posted to the wrong issue **EDIT**
drush 8.x works fine however drush 9 support needs a patch. If you are using drush 9, there is a patch, however that patch breaks compatibility with drush 8.

So for now, it will be left as a patch. Something should be added to the Readme.txt about this.

joseph.olstad’s picture

joseph.olstad
Primary language French
commented

**EDIT** sorry I think I posted to the wrong issue **EDIT**
if you want to use this functionality now without a patch, drush 8.1.17 works fine
if you have drush 9 , use the patch #2949601-20: Drush 9 compatibility

hongpong’s picture

hongpong commented
Title: Drush command 'dmu-clean' could not be found » Update readme: Drush command 'dmu-clean' could not be found
Status: Active » Needs review
StatusFileSize
new dmu-readme-2598932-8.patch2.43 KB

Updated readme to reflect this and some other things including better guidance for support requests. the clean command was removed because it was unreliable, that is my understanding.

greggmarshall’s picture

greggmarshall
Location Aurora/Denver, Colorado
commented

@HongPong, nice update.

Perhaps you could also remove the references to the cleaner plugin in /drupalmoduleupgrader/drupalmoduleupgrader.drush.inc and drupalmoduleupgrader/src/Commands/DrupalmoduleupgraderCommands.php which would also resolve #3128692: Plugins list outdated for drush dmu-list..

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Needs review » Closed (duplicate)

Modules now works with Drush 10 and 11 even :) I removed the mention of the clean command from the readme when I updated it to markdown as I realized that does not exist anymore. While I don't think these changes apply anymore, I'll credit you for the effort, thanks a lot!

See https://www.drupal.org/project/drupalmoduleupgrader/releases/2.0.0-alpha2 for the version that now supports Drupal 10 and 11!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.