cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CB-10204) Fix getCurrentPosition options on Android
Date Mon, 04 Jan 2016 08:05:40 GMT

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

ASF subversion and git services commented on CB-10204:
------------------------------------------------------

Commit 8cf37567c4c144303c59128664db71abe196bc23 in cordova-plugin-geolocation's branch refs/heads/master
from sgrebnov
[ https://git-wip-us.apache.org/repos/asf?p=cordova-plugin-geolocation.git;h=8cf3756 ]

CB-10204 Fix getCurrentPosition options on Android

github close #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-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org


Mime
View raw message