Deprecate ThemeHandlerInterface::getBaseThemes and remove usages from core

The Needs Review Queue Bot → tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".

Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.

Consult the Drupal Contributor Guide → to find step-by-step guides for working with issues.

For good practice we should still use the standard issue summary template.

Still is tagged for change record updates from a few years back.

Updated the CR too

So I think this needs it's own change record.

The current one is published and said it was introduced in 8.7 which isn't the case for this. So methods have been deprecated in 8.7 and no 10.3 I'm just worried that would be confusing.

Will post in slack also.

Yes, this should have it's own change record. Any changes to the original change record should be moved to the new one so that the dates and version/branch of each change record is correct.

Updated remaining tasks in IS

Additional CR looks fine, made some tweaks to make it clear when it was deprecated and removed.

is deprecated in drupal:10.3.0 and is removed from drupal:12.0.0

What is the reason for keeping this until Drupal 12? We are OK to deprecate in 10.3.0 for removal in 11.0.0, except where we think things might be widely used or difficult to remove, but this one looks straightforward.

Believe there is a slack discussion around this. But moving to NW as the MR is unmergable.

Resolved conflicts / rebased

Fixed https://wiki.php.net/rfc/make-reflection-setaccessible-no-op

Applied a simple :void to the test, but deprecation seems correct.

Committed and pushed 1e80a22b06 to 11.x and 5e778473f4 to 10.3.x. Thanks!

Also published the change record.

longwave → closed merge request !5848

Third time lucky, issue credits got swallowed up by d.o somehow.

Automatically closed - issue fixed for 2 weeks with no activity.

We are also using this function in our custom module. But after deprecation there are no replacements suggested here.
So Is there any alternate workaround to achieve this?

Before looking for workarounds please consider if you really need this method as it very internal to ThemeExtensionList::doGetBaseThemes() which you can decorate/access in a hacky way

But custom code is not expected to use this method

Thanks for reply andy.
Yes for our use case it might be needed as we are creating some predefined block based to the base theme condition in hook_theme_installed() hook.

Hmm, my theme is heavily using the getBaseThemes and I would like to avoid a major refactor. So you're saying it's possible to decorate ThemeExtensionList::doGetBaseThemes() in order to do what getBaseThemes does?