Problem/Motivation

ThemeHandler::setDefault is deprecated but does not trigger an error.

Proposed resolution

remove usage and trigger the error.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#3 3082624-3.drupal.Properly-deprecate-ThemeHandlersetDefault.patch3.59 KBmikelutz

Comments

mikelutz created an issue. See original summary.

mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
commented
Related issues: +#2635784: Deprecate ThemeHandler::setDefault() in favour of configuration and replace usages
mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
commented
Assigned: mikelutz » Unassigned
Status: Active » Needs review
StatusFileSize
new 3082624-3.drupal.Properly-deprecate-ThemeHandlersetDefault.patch3.59 KB
andypost’s picture

andypost
he/him
Primary language Russian
commented
Status: Needs review » Reviewed & tested by the community

Nice catch

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Reviewed & tested by the community » Fixed

Committed 3bd3a19 and pushed to 8.8.x. Thanks!

  • larowlan committed 3bd3a19 on 8.8.x 
    Issue #3082624 by mikelutz: Properly deprecate ThemeHandler::setDefault

Status: Fixed » Closed (fixed)

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