When using a module like Print I get errors like this:

It looks like profiles/openatrium/modules/contrib/print/assets/stylesheets/screen.css has not been created yet.

This is because Radix is using path_to_theme() to get the theme path, which is not correct when called within certain modules. The correct way to always get the current theme path is:

drupal_get_path('theme',$GLOBALS['theme']) as per https://api.drupal.org/api/drupal/includes%21theme.inc/function/path_to_...

Patch forthcoming.

CommentFileSizeAuthor
#1 radix_theme_path-2227537-1.patch904 bytesmpotter
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mpotter’s picture

mpotter CreditAttribution: mpotter commented
Status: Active » Needs review
FileSize
radix_theme_path-2227537-1.patch904 bytes

Here is the patch for Radix 2.x. Can be easily ported to 3.x.

  • Commit d085968 on 7.x-2.x authored by mpotter, committed by arshadcn: 
    Issue #2227537 by mpotter: Radix getting theme path incorrectly.

  • Commit 3c563ad on 7.x-3.x authored by mpotter, committed by arshadcn: 
    Issue #2227537 by mpotter: Radix getting theme path incorrectly.
shadcn’s picture

shadcn CreditAttribution: shadcn commented
Status: Needs review » Fixed

Thanks. Committed to both 2.x and 3.x.

Status: Fixed » Closed (fixed)

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