error in docs for getComponentNames()

Prem Suthar → made their first commit to this issue’s fork.

Thanks for the MR!

Could you just fix the one problem for this issue though, rather than expand the scope?

I am going to be filing an issue to fix the confusing use of 'component' to mean extensions, but it's going to involve method renaming too.

@Prem Suthar just FYI this was tagged for novice, meaning for new users to Drupal contribution, and based on your post history believe you could work on non novice issues please.

Believe the goal was just to update getComponentNames function.

I think we could expand the scope to include the other error that @Prem Suthar spotted, which is that the structure of the @param array is incorrectly documented:

> Array of component lists indexed by type

However, the current MR is removing the '(optional)' part.

I've filed 🐛 Fix incorrect use of word 'component' in locale module Active , which is a bigger task which covers both code and docs.

Sure but should be documented

Hmm on second thoughts,

> Array of component lists indexed by type

does sort of describe it -- it's an array of several lists, and the array is keyed by the type of the things in the list.

The other issue is going to rewrite that text anyway so let's just fix the incorrect 'update' here.

Hi, I am looking into this issue queue as part of Drupalcon Singapore 2024 Contribution day, and hope I can contribute in one way or another.

I have made a fresh installation of a Drupal 11 site to check on this, and tried to search the codebase for the code block shown in the Problem/Motivation. I realised that the only location I can find comes from core/modules/locale/src/LocaleConfigManager.php. I checked that merge request MR !6723, and found that the MR includes changes to both core/modules/locale/src/LocaleConfigManager.php and core/modules/locale/locale.bulk.inc. This confused me as I didn't understand the current state of the issue queue, and how I can contribute further.

It's core/modules/locale/src/LocaleConfigManager.php that needs fixing.

adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to hidden.

adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to active.

Working on it.

adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to hidden.

adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to hidden.

adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to active.

adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to hidden.

adwivedi008 → changed the visibility of the branch 3422919-error-in-docs to active.

Reverted the out-of-scope changes.
please review

Issue summary for good practice should still be filled out.

Also the parameter is still optional.