Closed (fixed)
Project:
Disqus
Version:
6.x-1.x-dev
Component:
Code
Priority:
Minor
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
27 Mar 2011 at 21:33 UTC
Updated:
12 Apr 2011 at 15:01 UTC
disqus_footer outputs
$output .= 'var disqus_def_name = '. drupal_to_js($user->name) .';';
$output .= 'var disqus_def_email = '. drupal_to_js($user->mail) .';';
This prevents static page caching strategies for logged in users.
Doesn't disqus maintain its own logged-in/logged-out information via its own cookies?
Comments
Comment #1
robloachIf the Disqus cookies don't exist, this will fill in the Disqus username and email fields for the user. It should be moved into Drupal.settings though.
Comment #2
robloachhttp://drupalcode.org/project/disqus.git/commit/e588f8a
Requires lots of testing. This was already part of the 7.x branch.