diff --git a/src/browser/DeviceProxy.js b/src/browser/DeviceProxy.js index 881d3914..e80522e2 100644 --- a/src/browser/DeviceProxy.js +++ b/src/browser/DeviceProxy.js @@ -18,8 +18,6 @@ * under the License. * */ -var browser = require('cordova/platform'); - function getPlatform () { return 'browser'; } @@ -70,7 +68,6 @@ module.exports = { getDeviceInfo: function (success, error) { setTimeout(function () { success({ - cordova: browser.cordovaVersion, platform: getPlatform(), model: getModel(), version: getVersion(), diff --git a/src/ios/CDVDevice.h b/src/ios/CDVDevice.h index a146d882..5c63dcb2 100644 --- a/src/ios/CDVDevice.h +++ b/src/ios/CDVDevice.h @@ -23,8 +23,6 @@ @interface CDVDevice : CDVPlugin {} -+ (NSString*)cordovaVersion; - - (void)getDeviceInfo:(CDVInvokedUrlCommand*)command; @end diff --git a/src/ios/CDVDevice.m b/src/ios/CDVDevice.m index 73154897..bc81c2f5 100644 --- a/src/ios/CDVDevice.m +++ b/src/ios/CDVDevice.m @@ -93,17 +93,11 @@ - (NSDictionary*)deviceProperties @"platform": @"iOS", @"version": [device systemVersion], @"uuid": [self uniqueAppInstanceIdentifier:device], - @"cordova": [[self class] cordovaVersion], @"isVirtual": @([self isVirtual]), @"isiOSAppOnMac": @([self isiOSAppOnMac]) }; } -+ (NSString*)cordovaVersion -{ - return CDV_VERSION; -} - - (BOOL)isVirtual { #if TARGET_OS_SIMULATOR diff --git a/tests/tests.js b/tests/tests.js index 734d1a10..23b9eeb3 100644 --- a/tests/tests.js +++ b/tests/tests.js @@ -49,15 +49,6 @@ exports.defineAutoTests = function () { expect(String(window.device.cordova).length > 0).toBe(true); }); - it('should depend on the presence of cordova.version string', function () { - expect(window.cordova.version).toBeDefined(); - expect(String(window.cordova.version).length > 0).toBe(true); - }); - - it('should contain device.cordova equal to cordova.version', function () { - expect(window.device.cordova).toBe(window.cordova.version); - }); - it('should contain a model specification that is a string', function () { expect(window.device.model).toBeDefined(); expect(String(window.device.model).length > 0).toBe(true); diff --git a/www/device.js b/www/device.js index 7e67edbd..2256e75c 100644 --- a/www/device.js +++ b/www/device.js @@ -50,8 +50,6 @@ function Device () { channel.onCordovaReady.subscribe(function () { me.getInfo( function (info) { - // ignoring info.cordova returning from native, we should use value from cordova.version defined in cordova.js - // TODO: CB-5105 native implementations should not return info.cordova var buildLabel = cordova.version; me.available = true; me.platform = info.platform;