In html.tpl.php sessions are being used to determine the theme color. This exposes a very serious performance impact as page caching is now bypassed in Drupal (see this ticket on removing anonymous sessions in Drupal 7 http://drupal.org/node/201122). It appears this may have been set up for the demo site and just left in, but it should be addressed considering the performance impact this exposes.

Comments

that0n3guy’s picture

that0n3guy commented

I've also noticed this cause some funky redirect issues. I had a bunch of nodes that I couldn't go to. When I clicked on the url, it would change my theme color and redirect me to the homepage... it was weird and took me forever to figure out.

If I just hard-coded color into the html.tpl.php (to blue) then everything worked fine. This is defiantly more than just a performance bug.

Jamie Holly’s picture

Jamie Holly commented
Priority: Major » Critical

I didn't notice that, but can see where it would happen. I'm going to bump this to critical as it can be a real site killer that can go unnoticed from the dev->prod transition.

danny englander’s picture

danny englander
(he/him)
commented

I have submitted a patch that addresses this, that code was not even necessary and in fact was hindering the choice of colors in the UI from rendering properly.

http://drupal.org/node/1473566#comment-5761156