Skip to content

pm:enable should be pm:install #4588

Closed
Closed
pm:enable should be pm:install#4588

Description

@jhodgdon-drp
jhodgdon-drp
opened on Nov 13, 2020

Describe the bug
In Drupal 8.x and above, there is no longer a way to "enable/disable" modules. Instead we "install/uninstall" them.

However, Drush still has a pm:enable command along with pm:uninstall.

This is confusing (enable is the opposite of uninstall?) and inconsistent with the Drupal UI.

To Reproduce
List commands in Drush and you'll see that pm:enable turns a module on, and pm:uninstall turns it off (which also involves adding/removing configuration, database tables, and/or content).

Expected behavior
The pm:enable command should be named pm:install for consistency with the Drupal UI and with the pm:uninstall command.

System Configuration

Q A
Drush version? 10.x
Drupal version? 9.x/8.x
PHP version any
OS? any

Additional information
Drupal core issue about making the Drupal UI more consistent in this area:
https://www.drupal.org/project/drupal/issues/2891294
which is part of a meta issue:
https://www.drupal.org/project/drupal/issues/2888657

Activity

  1. greg-1-anderson commented on Nov 13, 2020

    @greg-1-anderson
    greg-1-anderson
    on Nov 13, 2020
    Member

    +1 we can keep pm:enable as an alias, so there is little cost to doing this.

  2. weitzman commented on Dec 5, 2020

    @weitzman
    weitzman
    on Dec 5, 2020
    Member

    Yeah, this is a good proposal ... The docs web page will change url. We could add a redirect plugin for mkdocs if we want to avoid a few 404s.

  3. rajeshreeputra commented on Feb 2, 2022

    @rajeshreeputra
    rajeshreeputra
    on Feb 2, 2022
    Contributor

    I have create PR #5061
    Also kept alias "install, pm-install, en, pm-enable, pm:enable" for pm:install
    @weitzman could you please review changes in PR.

    I have tested this on local and confirming that below all commands are working fine, as expected.

    drush en
drush pm-enable
drush pm:enable
drush install
drush pm-install
drush pm:install
  4. weitzman
    closed this as completedin 3cfabb5on Apr 15, 2022

  5. gitressa commented on Apr 16, 2022

    @gitressa
    gitressa
    on Apr 16, 2022
    Contributor

    This is an improvement, but for consistency, shouldn't we also add an un alias for pm:uninstall, just as pm:install has in?

    pm:install

    Aliases: in, install, pm-install, en, pm-enable, pm:enable

    pm:uninstall

    Aliases: pmu, pm-uninstall

  6. gitressa commented on Apr 16, 2022

    @gitressa
    gitressa
    on Apr 16, 2022
    Contributor

    I created a PR which adds a un alias, #5131.

    $ drush | grep pm:uninstall
  pm:uninstall (un, pmu)      Uninstall one or more modules and their dependent modules.

$ drush un diff
 [success] Successfully uninstalled: diff
  7. mrbaileys
    added a commit that references this issue on Jan 27, 2024

    Fix drush-ops#4588 pm:enable renamed to pm:install (backward compat p…

    3cfabb5
  8. petergerner
    added 2 commits that reference this issue on Jul 30, 2024

    add pm:install as alias

    72bd2d3

    add pm:install as alias (#2)

    9217a2f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions