Rename "depends" keyword in default content to "dependencies"

Created on 18 December 2024, 7 months ago

Problem/Motivation

Most of Drupal's APIs use the keyword "dependencies" to indicate other items of its type that are required and must be installed first.

In the default content system the keyword that provides this functionality is installed "depends".

Proposed resolution

Add "dependencies" as the keyword in default content for indicating the dependent content.
Deprecate and remove the old "depends" keyword.

Remaining tasks

Build the necessary changes.

User interface changes

n/a

Introduced terminology

n/a

API changes

In default content definitions, the list of dependent entities will be defined using the "dependencies" keyword instead of "depends".

Data model changes

In the "_meta" section of the default content, the "dependencies" keyword indicates the list of dependent entities, while "depends" is deprecated.

Release notes snippet

✨ Feature request
Status

Active

Version

11.0 🔥

Component

default content system

Created by

🇺🇸United States damienmckenna NH, USA

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

Merge Requests

Comments & Activities

  • Issue created by @damienmckenna
  • 🇺🇸United States damienmckenna NH, USA
  • Merge request !10615Issue #3494759:Rename depends keyword → (Open) created by Unnamed author
  • 🇮🇳India kulpratap2002

    Replaced the "depends" keyword with "dependencies" in default content and removed the old "depends" keyword.

  • Pipeline finished with Failed
    7 months ago
    Total: 657s
    #373378
  • 🇺🇸United States damienmckenna NH, USA

    The PR needs to do three things:
    1. Change the internal API to accept "dependencies" alongside "depends".
    2. Update tests to check for "dependencies", leaving some sample code to use the legacy "depends" keyword.
    3. Add a legacy test that supports the "depends" keyword.
    4. Mark usage of "depends" as deprecated.

    The PR only updates the sample code that's part of the tests, more work is needed.

  • 🇺🇸United States yujiman85
  • Pipeline finished with Failed
    7 months ago
    Total: 191s
    #374216
  • Pipeline finished with Failed
    7 months ago
    Total: 173s
    #374222
  • Pipeline finished with Failed
    7 months ago
    Total: 166s
    #374229
  • Pipeline finished with Failed
    7 months ago
    Total: 589s
    #374235
  • Pipeline finished with Failed
    7 months ago
    Total: 664s
    #374305
  • Pipeline finished with Success
    7 months ago
    Total: 509s
    #375140
  • 🇺🇸United States yujiman85

    My apologies for all the commits, I had to read some docs for proper implementation. I took a swing at this, not 100% that my approach is correct here. Needs a review.

  • 🇺🇸United States smustgrave
  • Pipeline finished with Failed
    7 months ago
    Total: 285s
    #375261
  • Pipeline finished with Failed
    7 months ago
    Total: 168s
    #375271
  • Pipeline finished with Failed
    7 months ago
    Total: 500s
    #375274
  • Pipeline finished with Failed
    7 months ago
    Total: 718s
    #375294
  • 🇺🇸United States yujiman85

    Okay, I think I've gone as far as I can on this one. Someone else will need to take a look and work this one out.

Production build 0.71.5 2024