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

Created on 13 November 2020, almost 5 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 11 days 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