ThemeKey does not work properly due to the conflict with Revisions RSS

The logs look strange, it shows no match for the one that works and says it uses the default theme but it users the matched theme. it shows match for the second one doesn't work.

This one uses path property and works : http://salek.ws/projects/drupal/works.txt
This one users tid_and_childs and does not work : http://salek.ws/projects/drupal/not_works.txt

Nice debugging feature BTW ;)

They're from different vocabularies, 307 is a tag

Note that i used to work after this #584982: Does not respect hierarchy of terms

@mkalkbrenner I traced the code and i'm sure that it detect the taxonomy correctly.
The problem is with assigning the theme. I'm also sure that it fill $custom_theme with the
correct theme. but i don't know why it doesn't work!!
Any kind of tips is highly appreciated :)

Thanks that helped,
There is a conflict with Revisions RSS module, least expected!, i'm trying to figure why it's happening.

I found it, the following code causes this issue.
Solutoins
- Commenting drupal_add_feed... !!
- Disabling revisions_rss modlue !!
- And changing the default weight of themekey module to -1 to decrease the possibility of conflicting with other module in future. As you know most modules' weight is 0 so it can happen again.

There is something i don't understand , why calling drupal_add_feed prevents themekey from setting the theme?

<?php
/**
 * Implementation of hook_init().
 */
function revisions_rss_init() {
  // Expose node feed on node view page.
  if ((arg(0) == 'node') && is_numeric($nid = arg(1)) && is_null(arg(2)) &&
      ($node = node_load($nid)) && revision_rss_check_type($node) &&
      user_access(REV_RSS_PERM)) {
    
    drupal_add_feed(url("node/{$node->nid}/revisions/rss"),
                    t('Updates to "@title"', array('@title' => $node->title))
    );
    
  }
}
?>

I understand now,
I think we should move this, it's good example of what issues it causes. I also changed the title.

Thanks for your help @mkalkbrenner.

Revisions RSS maintainers please read comment #12.

Thanks

I've decided to write the patch.
This patch replaces node_load with simpler alternative, It also fixes a performance issue. The previous code used to call node_load for almost every page now it only calls it if it when all other conditions met.

Awesome , thanks