This would be handy for making something like a "theme previewer," and probably something that user/#/edit and OG module could use as well.

Comments

LAsan’s picture

LAsan CreditAttribution: LAsan commented
Version: x.y.z » 7.x-dev
Priority: Normal » Minor

Moving to cvs.

effulgentsia’s picture

effulgentsia CreditAttribution: effulgentsia commented
Version: 7.x-dev » 8.x-dev

@webchick: please revert this to D7 if you think it's not out of scope. Otherwise, I'm guessing even a non-breaking API addition is out of scope for D7 without a sufficiently compelling use-case.

It's not hard for a module to achieve the desired functionality with:
array_filter(list_themes(), 'drupal_theme_access')

But it would also be a pretty simple patch if you think it's worth getting into D7.

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
CreditAttribution: jhedstrom at Phase2 commented
Version: 8.0.x-dev » 8.1.x-dev
Issue summary: View changes
Issue tags: +Needs issue summary update
joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Title: Add a parameter to list_themes() to list only enabled themes » Add a parameter to Drupal::service('theme_handler')->listInfo() to list only enabled themes

\Drupal::service('theme_handler')->listInfo()

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Active » Fixed

Yeah I really don't think we need this.

joelpittet’s picture

joelpittet
Primary language English
Location Vancouver
CreditAttribution: joelpittet commented
Status: Fixed » Closed (won't fix)

Someone who thinks this is useful then can reopen or open a new one with a stronger case. Setting the status to reflect better the decision as this wasn't added.