Almost nothing about the D7 amazon_media module theming works right. It may need to be rethought completely.

CommentFileSizeAuthor
#2 amazon.fix_theming.patch33.21 KBrfay

Comments

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented

1. I think amazon_media_theme_registry_alter() does its job correctly.

2. But the method of providing hints about where tpl.php files can be found has changed. That is going to affect amazon.module. And me. I'm going to have to understand the theme hinting thing. And I think you can't do it in D7 at the module level. On the other hand, maybe that's what theme_paths is.

See drupal_find_theme_functions().

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Active » Needs review
StatusFileSize
new amazon.fix_theming.patch33.21 KB

OK, here's the result of a day's learning. Seems to work pretty well now.

rfay’s picture

rfay
he,him
Primary language English
Location Palisade, CO, USA
commented
Status: Needs review » Fixed

Committed to HEAD: http://drupal.org/cvs?commit=417514

I think there will be more to learn from this one, but lead on.

Status: Fixed » Closed (fixed)

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