API function names in module/theme install/uninstall space should match UI

Created on 13 November 2020, over 4 years ago
Updated 3 April 2024, over 1 year ago

Problem/Motivation

On meta issue
#2888657: [meta] Less confusing and more consistent wording needed in module/theme add/install/update β†’
we are updating the UI text for the actions of uploading files for a module/theme, and installing/uninstalling modules/themes.

That work is being done on these three child issues:
#2577407: Action of uploading module/theme files should consistently be called "Add", not "Install" β†’
πŸ“Œ [Meta] Use Install/Uninstall consistently for turning modules/themes on/off (not Enable) Fixed
πŸ› Do not use verb "Install" for things other than turning on modules/themes Fixed
Plus this one that is updating URLs:
#2888654: URLs for module/theme actions are inconsistent with UI text β†’

Once that is finished, we should update the API functions/classes for these actions so they are consistent with the UI terminology. This issue is postponed until all those others are done.

Proposed resolution

TBD.

Note that this will need a BC layer and/or deprecations.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

πŸ› Bug report
Status

Active

Version

11.0 πŸ”₯

Component
SystemΒ  β†’

Last updated about 6 hours ago

No maintainer
Created by

πŸ‡ΊπŸ‡ΈUnited States jhodgdon Spokane, WA, USA

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡³πŸ‡ΏNew Zealand quietone
  • πŸ‡³πŸ‡ΏNew Zealand quietone

    The commands I used to search for classes, traits and methods to change.

    git grep -i "^.*function.*module.*enable.*(" -- '*.php'
    git grep -i "^.*function.*theme.*enable.*(" -- '*.php'
    git grep -i "^.*function.*enable.*theme.*(" -- '*.php'
    git grep -i "^.*function.*enable.*module.*(" -- '*.php'
    git grep -i "^class.*enable.*module.*(" -- '*.php'
    git grep -i "^class.*enable.*theme.*(" -- '*.php'
    git grep -i "^class.*theme.*enable.*(" -- '*.php'
    git grep -i "^class.*module.*enable.*(" -- '*.php'
    git grep -i "^trait.*enable.*module.*{" -- '*.php'
    git grep -i "^trait.*enable.*theme.*{" -- '*.php'
    git grep -i "^trait.*theme.*enable.*{" -- '*.php'
    git grep -i "^trait.*module.*enable.*{" -- '*.php'
    
  • πŸ‡³πŸ‡ΏNew Zealand quietone

    This is the last child issue, which I am closing. I am re-parenting this to the grandparent issue.

Production build 0.71.5 2024