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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikelutz created an issue. See original summary.

mikelutz’s picture

mikelutz
He/Him
Primary language English
Location Michigan, USA
CreditAttribution: mikelutz as a volunteer and at ICON Interactive 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
CreditAttribution: mikelutz as a volunteer and at ICON Interactive commented
Assigned: mikelutz » Unassigned
Status: Active » Needs review
FileSize
3082624-3.drupal.Properly-deprecate-ThemeHandlersetDefault.patch3.59 KB
andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Status: Needs review » Reviewed & tested by the community

Nice catch

larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
CreditAttribution: larowlan at PreviousNext 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.