Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GPS Information - the Connection is not restored #21294

Closed
qgib opened this issue Aug 19, 2015 · 5 comments
Closed

GPS Information - the Connection is not restored #21294

qgib opened this issue Aug 19, 2015 · 5 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! GPS

Comments

@qgib
Copy link
Contributor

qgib commented Aug 19, 2015

Author Name: Steven Mizuno (Steven Mizuno)
Original Redmine Issue: 13233
Affected QGIS version: master
Redmine category:gps_live_tracking


GPS Information panel

problem: on QGIS start the Autodetect connection is always selected even though another, like Serial device, was previously used and should have been restored. This is rather annoying.
This occurs on Windows. I believe Linux or Mac versions are also affected.

Also, the 'H accurancy' (sic) and 'V accurancy' fields always show -1 when GPS is connected.

I found in qgsgpsinformationwidget.cpp that there is a test for HAVE_QT_MOBILITY_LOCATION not defined that forces the Autodetect to be selected (negating the restored choice) and the Internal button to be disabled.
Note that that the labels are misspelled as '... accurancy' intead of '... accuracy' -- the ui file needs to be fixed.

As the test is for a compile-time setting the user has absolutely no influence on, I consider that the Internal radiobutton being displayed when there is no Internal availability is just noise. Therefore it should be hidden.

Additionally, I find that the H accurancy and V accurancy display fields' data are from the internal location module. These are also noise as they don't get any data if there is no internal connection.

I offer a patch to change the behavior to:

  1. setting Autodetect only if Internal just happened to be the stored connection (it shouldn't happen in normal use)
  2. hiding the Internal radiobutton
  3. hiding the H and V accuracy labels/text boxes
    when HAVE_QT_MOBILITY_LOCATION is not defined.

The patch was taken against final-2_10_1 tag, but should be applied to head. And perhaps back port to other branches that may have further releases.


@qgib
Copy link
Contributor Author

qgib commented Sep 10, 2015

Author Name: Giovanni Manghi (@gioman)


Hi,

if possible please submit this patch as pull request on github, thanks!


  • status_id was changed from Open to Feedback

@qgib
Copy link
Contributor Author

qgib commented Sep 27, 2015

Author Name: Steven Mizuno (Steven Mizuno)


see PR #2338

@qgib
Copy link
Contributor Author

qgib commented Nov 7, 2015

Author Name: Giovanni Manghi (@gioman)


  • fixed_version_id removed Version 2.12

@qgib
Copy link
Contributor Author

qgib commented Dec 20, 2015

Author Name: Giovanni Manghi (@gioman)


  • status_id was changed from Feedback to Open

@qgib
Copy link
Contributor Author

qgib commented Mar 23, 2017

Author Name: Steven Mizuno (Steven Mizuno)


Fixed in changeset "9d5f28921d185705303ce689fee84757f8990a3c".


  • status_id was changed from Open to Closed

@qgib qgib added Bug Either a bug report, or a bug fix. Let's hope for the latter! GPS labels May 25, 2019
@qgib qgib closed this as completed May 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! GPS
Projects
None yet
Development

No branches or pull requests

1 participant