An empty hook_theme() causes fatal errors in theme.inc

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.

Reroll the patch for the current version 10.1.x as latest patch was not compatible.

Re-uploading the patch and rerolldiff

Fixed test failure.

Fixed corrupt patch and added it as MR.

The Needs Review Queue Bot → tested this issue. It no longer applies to Drupal core. 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.

Patch against 10.1.x

Interdiff #52-#57

@Rishabh Vishwakarma what was the base for your re-roll ?

Can you post interdiff ?

As you can see there is already a MR raised against 10.x branch there is no need for patches. You can just continue working on the MR. Thanks.

This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.

This could use an issue summary update. What is the proposed solution as this is touching a number of files? May need framework review too
Recommend using issue summary template.

Left a review, there's a lot of out of scope changes here.