FIX: errors in qunit tests when version check info is missing

This commit is contained in:
Neil Lalonde 2019-09-10 16:38:23 -04:00
parent 82398907af
commit f0586ece32
2 changed files with 10 additions and 6 deletions

View File

@ -19,16 +19,18 @@ const VersionCheck = Discourse.Model.extend({
@computed("git_branch", "installed_sha") @computed("git_branch", "installed_sha")
gitLink(gitBranch, installedSHA) { gitLink(gitBranch, installedSHA) {
if (gitBranch) { if (gitBranch && installedSHA) {
return `https://github.com/discourse/discourse/compare/${installedSHA}...${gitBranch}`; return `https://github.com/discourse/discourse/compare/${installedSHA}...${gitBranch}`;
} else { } else if (installedSHA) {
return `https://github.com/discourse/discourse/tree/${installedSHA}`; return `https://github.com/discourse/discourse/tree/${installedSHA}`;
} }
}, },
@computed("installed_sha") @computed("installed_sha")
shortSha(installedSHA) { shortSha(installedSHA) {
return installedSHA.substr(0, 10); if (installedSHA) {
return installedSHA.substr(0, 10);
}
} }
}); });

View File

@ -6,9 +6,11 @@
<div class="version-number"> <div class="version-number">
<h4>{{i18n 'admin.dashboard.installed_version'}}</h4> <h4>{{i18n 'admin.dashboard.installed_version'}}</h4>
<h3>{{dash-if-empty versionCheck.installed_version}}</h3> <h3>{{dash-if-empty versionCheck.installed_version}}</h3>
<div class="sha-link"> {{#if versionCheck.gitLink}}
(<a href={{versionCheck.gitLink}} target="_blank">{{versionCheck.shortSha}}</a>) <div class="sha-link">
</div> (<a href={{versionCheck.gitLink}} target="_blank">{{versionCheck.shortSha}}</a>)
</div>
{{/if}}
</div> </div>
{{#if versionCheck.noCheckPerformed}} {{#if versionCheck.noCheckPerformed}}