Skip to content

Commit c6c848d

Browse files
authored
Merge pull request #741 from mikeller/fix_version_for_chrome
Fix Chrome issues with SemVer RC version.
2 parents 80e0bb5 + e1bbd7c commit c6c848d

File tree

4 files changed

+35
-8
lines changed

4 files changed

+35
-8
lines changed

eventPage.js

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ chrome.app.runtime.onLaunched.addListener(startApplication);
8080
chrome.runtime.onInstalled.addListener(function (details) {
8181
if (details.reason == 'update') {
8282
var previousVersionArr = details.previousVersion.split('.'),
83-
currentVersionArr = chrome.runtime.getManifest().version.split('.');
83+
currentVersionArr = getManifestVersion().split('.');
8484

8585
// only fire up notification sequence when one of the major version numbers changed
8686
if (currentVersionArr[0] > previousVersionArr[0] || currentVersionArr[1] > previousVersionArr[1]) {
@@ -91,7 +91,7 @@ chrome.runtime.onInstalled.addListener(function (details) {
9191
priority: 0,
9292
type: 'basic',
9393
title: manifest.name,
94-
message: chrome.i18n.getMessage('notifications_app_just_updated_to_version', [manifest.version]),
94+
message: chrome.i18n.getMessage('notifications_app_just_updated_to_version', [getManifestVersion(manifest)]),
9595
iconUrl: '/images/icon_128.png',
9696
buttons: [{'title': chrome.i18n.getMessage('notifications_click_here_to_start_app')}]
9797
};
@@ -110,3 +110,16 @@ chrome.notifications.onButtonClicked.addListener(function (notificationId, butto
110110
startApplication();
111111
}
112112
});
113+
114+
function getManifestVersion(manifest) {
115+
if (!manifest) {
116+
manifest = chrome.runtime.getManifest();
117+
}
118+
119+
var version = manifest.version_name;
120+
if (!version) {
121+
version = manifest.version;
122+
}
123+
124+
return version;
125+
}

js/backup_restore.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
function configuration_backup(callback) {
66
var activeProfile = null;
77

8-
var version = chrome.runtime.getManifest().version;
8+
var version = getManifestVersion();
99

1010
if (version.indexOf(".") === -1) {
1111
version = version + ".0.0";

main.js

Lines changed: 17 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@ $(document).ready(function () {
77
// alternative - window.navigator.appVersion.match(/Chrome\/([0-9.]*)/)[1];
88
GUI.log('Running - OS: <strong>' + GUI.operating_system + '</strong>, ' +
99
'Chrome: <strong>' + window.navigator.appVersion.replace(/.*Chrome\/([0-9.]*).*/, "$1") + '</strong>, ' +
10-
'Configurator: <strong>' + chrome.runtime.getManifest().version + '</strong>');
10+
'Configurator: <strong>' + getManifestVersion() + '</strong>');
1111

12-
$('#logo .version').text(chrome.runtime.getManifest().version);
12+
$('#logo .version').text(getManifestVersion());
1313
updateStatusBarVersion();
1414
updateTopBarVersion();
1515

@@ -378,7 +378,7 @@ $(document).ready(function () {
378378
});
379379

380380
function notifyOutdatedVersion(version) {
381-
if (semver.lt(chrome.runtime.getManifest().version, version)) {
381+
if (semver.lt(getManifestVersion(), version)) {
382382
GUI.log('You are using an old version of ' + chrome.runtime.getManifest().name + '. Version ' + version + ' is available online with possible improvements and fixes.');
383383
}
384384
}
@@ -533,7 +533,7 @@ function getFirmwareVersion(firmwareVersion, firmwareId, hardwareId) {
533533
}
534534

535535
function getConfiguratorVersion() {
536-
return chrome.i18n.getMessage('versionLabelConfigurator') + ': ' + chrome.runtime.getManifest().version;
536+
return chrome.i18n.getMessage('versionLabelConfigurator') + ': ' + getManifestVersion();
537537
}
538538

539539
function updateTopBarVersion(firmwareVersion, firmwareId, hardwareId) {
@@ -557,3 +557,16 @@ function updateStatusBarVersion(firmwareVersion, firmwareId, hardwareId) {
557557

558558
$('#status-bar .version').text(versionText);
559559
}
560+
561+
function getManifestVersion(manifest) {
562+
if (!manifest) {
563+
manifest = chrome.runtime.getManifest();
564+
}
565+
566+
var version = manifest.version_name;
567+
if (!version) {
568+
version = manifest.version;
569+
}
570+
571+
return version;
572+
}

manifest.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
{
22
"manifest_version": 2,
33
"minimum_chrome_version": "38",
4-
"version": "10.0.0-RC2",
4+
"version": "10.0.0",
5+
"version_name": "10.0.0-RC2",
56
"author": "Betaflight Squad",
67
"name": "Betaflight - Configurator",
78
"short_name": "Betaflight",

0 commit comments

Comments
 (0)