[CB-10204] Fix getCurrentPosition options on Android

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Component/s: cordova-plugin-geolocation
• Labels: Android Triaged

Environment

Cordova 3.5.0, Plugin Geolocation 2.0.0

• Assignee: Unassigned
• Reporter: Timo Salola
• Created: 16/Dec/15 07:44
• Updated: 23/Aug/18 06:59
• Resolved: 05/Jan/16 06:12

Description

In the latest release there was created separate JS file for android. In getCurrentPosition it does not verify that options are given resulting javascript error when trying to access undefined as an array at www/android/geolocation.js lines 33 and 34:

    getCurrentPosition: function(success, error, args) {
        var win = function() {
          var geo = cordova.require('cordova/modulemapper').getOriginalSymbol(window, 'navigator.geolocation');
          geo.getCurrentPosition(success, error, {
            enableHighAccuracy: args[0],
            maximumAge: args[1]
          });
        };
        exec(win, error, "Geolocation", "getPermission", []);
    }

Workaround this is to give empty array to getCurrentPosition as options:

navigator.geolocation.getCurrentPosition(onSuccess, onError, []);