Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.3.0
    • Fix Version/s: None
    • Component/s: mobile-spec
    • Labels:
      None

      Description

      When using the mobile spec, a NaN value is returned for Heading when testing the Cordova Geolocation plugin using the Location test page.

        Activity

        James Keshavarzi created issue -
        Josh Soref made changes -
        Field Original Value New Value
        Component/s mobile-spec [ 12317700 ]
        Hide
        Josh Soref added a comment -

        OK, it turns out that this is by design, see
        www/Coordinates.js
        var Coordinates = function(lat, lng, alt, acc, head, vel, altacc) {
        this.heading = (head !== undefined ? head : null);
        this.speed = (vel !== undefined ? vel : null);
        if (this.speed === 0 || this.speed === null)

        { this.heading = NaN; }

        Mobile Spec should be adjusted to indicate that "NaN" is expected if you aren't moving.

        Show
        Josh Soref added a comment - OK, it turns out that this is by design, see www/Coordinates.js var Coordinates = function(lat, lng, alt, acc, head, vel, altacc) { this.heading = (head !== undefined ? head : null); this.speed = (vel !== undefined ? vel : null); if (this.speed === 0 || this.speed === null) { this.heading = NaN; } Mobile Spec should be adjusted to indicate that "NaN" is expected if you aren't moving.
        Josh Soref made changes -
        Component/s BlackBerry [ 12316402 ]
        Josh Soref made changes -
        Summary [BlackBerry10] NaN returned for Cordova Geolocation heading value Document geolocation fields
        Hide
        ASF GitHub Bot added a comment -

        GitHub user jsoref opened a pull request:

        https://github.com/apache/cordova-mobile-spec/pull/52

        CB-6163 Document geolocation fields

        https://issues.apache.org/jira/browse/CB-6163

        Documenting the fields makes it easier for people testing to verify that what they see is what they should see...

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/jsoref/cordova-mobile-spec cb_6163

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/cordova-mobile-spec/pull/52.patch

        To close this pull request, make a commit to your master/trunk branch
        with (at least) the following in the commit message:

        This closes #52


        commit d3c0b5686ce27ef901dff3d56d4e0268f654b637
        Author: Jesse MacFadyen <purplecabbage@gmail.com>
        Date: 2014-03-19T21:44:24Z

        added localXHR test to prove CB-6299 fix is effective

        commit 456e388e246ff6b0395ca82d2811cb5088c1d4d1
        Author: Jesse MacFadyen <purplecabbage@gmail.com>
        Date: 2014-03-19T21:46:01Z

        fixed test id

        commit 5d09a17ac7f1da083c29ad2df93cfdbb6a136a0a
        Author: Jesse MacFadyen <purplecabbage@gmail.com>
        Date: 2014-03-19T21:46:25Z

        Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/cordova-mobile-spec

        commit 4bd62487bc5ab80a9de854fa53953298fe5a70de
        Author: Josh Soref <jsoref@blackberry.com>
        Date: 2014-03-19T21:54:01Z

        CB-6163 Document geolocation fields


        Show
        ASF GitHub Bot added a comment - GitHub user jsoref opened a pull request: https://github.com/apache/cordova-mobile-spec/pull/52 CB-6163 Document geolocation fields https://issues.apache.org/jira/browse/CB-6163 Documenting the fields makes it easier for people testing to verify that what they see is what they should see... You can merge this pull request into a Git repository by running: $ git pull https://github.com/jsoref/cordova-mobile-spec cb_6163 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/cordova-mobile-spec/pull/52.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #52 commit d3c0b5686ce27ef901dff3d56d4e0268f654b637 Author: Jesse MacFadyen <purplecabbage@gmail.com> Date: 2014-03-19T21:44:24Z added localXHR test to prove CB-6299 fix is effective commit 456e388e246ff6b0395ca82d2811cb5088c1d4d1 Author: Jesse MacFadyen <purplecabbage@gmail.com> Date: 2014-03-19T21:46:01Z fixed test id commit 5d09a17ac7f1da083c29ad2df93cfdbb6a136a0a Author: Jesse MacFadyen <purplecabbage@gmail.com> Date: 2014-03-19T21:46:25Z Merge branch 'master' of https://git-wip-us.apache.org/repos/asf/cordova-mobile-spec commit 4bd62487bc5ab80a9de854fa53953298fe5a70de Author: Josh Soref <jsoref@blackberry.com> Date: 2014-03-19T21:54:01Z CB-6163 Document geolocation fields
        Show
        Josh Soref added a comment - https://github.com/apache/cordova-mobile-spec/pull/52
        Hide
        ASF subversion and git services added a comment -

        Commit 4bd62487bc5ab80a9de854fa53953298fe5a70de in cordova-mobile-spec's branch refs/heads/master from Josh Soref
        [ https://git-wip-us.apache.org/repos/asf?p=cordova-mobile-spec.git;h=4bd6248 ]

        CB-6163 Document geolocation fields

        Show
        ASF subversion and git services added a comment - Commit 4bd62487bc5ab80a9de854fa53953298fe5a70de in cordova-mobile-spec's branch refs/heads/master from Josh Soref [ https://git-wip-us.apache.org/repos/asf?p=cordova-mobile-spec.git;h=4bd6248 ] CB-6163 Document geolocation fields
        Bryan Higgins made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Hide
        ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

        https://github.com/apache/cordova-mobile-spec/pull/52

        Show
        ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/cordova-mobile-spec/pull/52
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Closed Closed
        19d 21h 42m 1 Bryan Higgins 24/Mar/14 19:52

          People

          • Assignee:
            Unassigned
            Reporter:
            James Keshavarzi
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development