Fragmentation of comments when using clean-urls

Very true! The isssue is disqus_url. You're suppose to set it to an absolute URL of the post. This is quite easy to do, unless your domain changes, or your website is hosted on multiple domains. It also causes a problem when you're experiencing what you're doing, where the path to the content can change....

I brought it up a while ago on the Disqus forum here and it doesn't look like they made much progress. I guess the easiest solution here is to use:

disqus_url = 'http://example.com/node/nid';

... Because then even if you change the path, it will still be using the node ID. Once Disqus fixes the disqus_url problem, we could adapt to their solution. Thanks for bringing this up. I wouldn't of experienced it because I'm not using the Path module on my site.

Reported a comment: http://disqus.disqus.com/disqus_url/?826500#comment-826500

Need their new API.

Im not sure if you want to mark http://drupal.org/node/343363 as a duplicate.

I'm looking at using the Disqus module and was concerned about this. But I happen to have the Global Redirect module enabled and if someone tries to go to node/# it will redirect to the pathauto URL. So that might be a fix, at least temporarily?

Global Redirect project page: http://drupal.org/project/globalredirect

You can test it on my site:
http://throk.net/blog/601/today-friday
http://throk.net/node/601

I'm setting this to patch code (needs review) b/c while it's not a fix, it's sort of a solution...

would be less of an issue if I could divine the pathauto url in the view to build the Disqus URL around...

You can wrap a 'node/[nid]' path in a url() call and it'll sort out the appropriate pathauto url...

Note that you need to pass in the relative url, not the absolute url. Something like this should work:

$clean_url = url('node/' . $node->nid);

I'd rather not pass the PathAuto's or Path's node URL since that path can change. We should pass the node ID instead because that doesn't change. The Disqus URL can be a relative path? Here it is with relative node/nid.

This should be long fixed.