[ 
https://issues.apache.org/jira/browse/CB-10204?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15080799#comment-15080799
 ] 

ASF GitHub Bot commented on CB-10204:
-------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/cordova-plugin-geolocation/pull/60


> Fix getCurrentPosition options on Android
> -----------------------------------------
>
>                 Key: CB-10204
>                 URL: https://issues.apache.org/jira/browse/CB-10204
>             Project: Apache Cordova
>          Issue Type: Bug
>          Components: Plugin Geolocation
>         Environment: Cordova 3.5.0, Plugin Geolocation 2.0.0
>            Reporter: Timo Salola
>            Assignee: Sergey Grebnov
>              Labels: Android, Triaged
>
> 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:
> {code:javascript}
>     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", []);
>     }
> {code}
> Workaround this is to give empty array to getCurrentPosition as options:
> {code:javascript}
> navigator.geolocation.getCurrentPosition(onSuccess, onError, []);
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org

Reply via email to