Skip to content

Commit

Permalink
Refs #1845, display current site's host in help text/description of t…
Browse files Browse the repository at this point in the history
…racking code admin page's inputs.
  • Loading branch information
diosmosis committed Feb 27, 2013
1 parent a06d961 commit 3cee71c
Show file tree
Hide file tree
Showing 3 changed files with 23 additions and 2 deletions.
14 changes: 14 additions & 0 deletions plugins/CoreAdminHome/Controller.php
Expand Up @@ -133,6 +133,20 @@ public function trackingCodeGenerator()
$view->defaultReportSiteName = Piwik_Site::getNameFor($view->idSite);
$view->defaultSiteRevenue = Piwik::getCurrency($view->idSite);

$allUrls = Piwik_SitesManager_API::getInstance()->getSiteUrlsFromId($view->idSite);
if (isset($allUrls[1]))
{
$aliasUrl = $allUrls[1];
}
else
{
$aliasUrl = 'x.domain.com';
}
$view->defaultReportSiteAlias = $aliasUrl;

$mainUrl = Piwik_Site::getMainUrlFor($view->idSite);
$view->defaultReportSiteDomain = @parse_url($mainUrl, PHP_URL_HOST);

// get currencies for each viewable site
$view->currencySymbols = Piwik_SitesManager_API::getInstance()->getCurrencySymbols();

Expand Down
7 changes: 7 additions & 0 deletions plugins/CoreAdminHome/templates/jsTrackingGenerator.js
Expand Up @@ -302,6 +302,13 @@ $(document).ready(function() {
});

getSiteData(site.id, '#js-code-options', function() {
var siteHost = getHostNameFromUrl(siteUrls[site.id][0]);
$('.current-site-host', '#optional-js-tracking-options').each(function() {
$(this).text(siteHost);
});

$('.current-site-alias').text(siteUrls[site.id][1] || 'x.domain.com');

resetGoalSelectItems(site.id, 'js-tracker-goal');
generateJsCode();
});
Expand Down
4 changes: 2 additions & 2 deletions plugins/CoreAdminHome/templates/jsTrackingGenerator.tpl
Expand Up @@ -42,7 +42,7 @@
<label for="javascript-tracking-all-subdomains">{'CoreAdminHome_JSTracking_MergeSubdomains'|translate} <span class='current-site-name'>{$defaultReportSiteName}</span></label>

<div class="small-form-description">
{'CoreAdminHome_JSTracking_MergeSubdomainsDesc'|translate:'x.domain.com':'y.domain.com'}
{'CoreAdminHome_JSTracking_MergeSubdomainsDesc'|translate:"x.<span class='current-site-host'>$defaultReportSiteDomain</span>":"y.<span class='current-site-host'>$defaultReportSiteDomain</span>"}
</div>
</div>

Expand All @@ -52,7 +52,7 @@
<label for="javascript-tracking-all-aliases">{'CoreAdminHome_JSTracking_MergeAliases'|translate} <span class='current-site-name'>{$defaultReportSiteName}</span></label>

<div class="small-form-description">
{'CoreAdminHome_JSTracking_MergeAliasesDesc'|translate:'x.domain.com'}
{'CoreAdminHome_JSTracking_MergeAliasesDesc'|translate:"<span class='current-site-alias'>$defaultReportSiteAlias</span>"}
</div>
</div>

Expand Down

0 comments on commit 3cee71c

Please sign in to comment.