Hello,

This is a path to support Domain Access (http://drupal.org/project/domain) subdomains.

click_heatmap.module :

@@ -107,7 +107,12 @@
  * @return Site name.
  */
 function click_heatmap_site_name() {
-  return substr(conf_path(), strrpos(conf_path(), '/') + 1);
+  if (module_exists('domain')) {
+    $domain = domain_get_domain();
+    return $domain['subdomain'];
+  } else {
+    return substr(conf_path(), strrpos(conf_path(), '/') + 1);
+  }
 }

Comments

john franklin’s picture

john franklin commented
Status: Patch (to be ported) » Active

That causes the data to be stored correctly, but the reports show only one domain, regardless of the current domain. That is, http://example.org/admin/reports/click_heatmap and http://site-a.example.org/admin/reports/click_heatmap show the same background graphic, even if they're configured to be two different domains. More work needs to be done.

BTW, "patch (to be ported)" is used when a patch has been accepted, but needs to be ported for another version of the module, usually the d6 or d7 version. "Needs review" is for when a patch has been attached as a file. See this page for details.

john franklin’s picture

john franklin commented
Status: Active » Fixed

On further investigation, it does work. The page to view heatmaps does not default to the current domain, which is confusing but another issue.

Thanks for the patch.

Status: Fixed » Closed (fixed)

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