Meta issue: #2205673: [meta] Remove all @deprecated functions marked "remove before 8.0"

List_themes function is no longer used and is deprecated, so let's remove it.

CommentFileSizeAuthor
remove_list_themes.patch1.17 KBJeroenT
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, remove_list_themes.patch, failed testing.

Status: Needs work » Needs review

JeroenT queued remove_list_themes.patch for re-testing.

JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT commented
Issue summary: View changes
LinL’s picture

LinL CreditAttribution: LinL commented
Status: Needs review » Reviewed & tested by the community

No list_themes() in core other than this, CR done. Looks good to go.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Beta evaluation is in the meta issue. Committed a08f3af and pushed to 8.0.x. Thanks!

  • alexpott committed a08f3af on 8.0.x 
    Issue #2421677 by JeroenT: Remove list_themes function

Status: Fixed » Closed (fixed)

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