Indicator Location

The location and gps check box don't stay checked

Bug #1248973 reported by Dave Morley on 2013-11-07

This bug affects 11 people

	Status	Importance	Assigned to
Indicator Location	Fix Released	High	Charles Kerr
indicator-location (Ubuntu)	Fix Released	High	Charles Kerr
location-service (Ubuntu)	Fix Released	High	Thomas Voß
platform-api (Ubuntu)	Fix Released	High	Thomas Voß

Bug Description

If you check the location and gps checkboxes from the indicator-location and then after a minute view the setting again you will see that they are nolonger checked.

Maguro image 14

STEPS TO REPRODUCE:
1. Pull down the location indicator
2. Select the checkboxes for Location and GPS
3. Push the indicator back up
4. Wait a minute
5. Pull down the location indicator.

EXPECTED RESULTS:
I should still have 2 boxes checked.

See original description

Tags:

Related branches

lp:~thomas-voss/platform-api/add_missing_agps_hooks_take_2

Superseded for merging into lp:platform-api

Charles Kerr (community): Needs Information on 2014-04-10

PS Jenkins bot: Approve (continuous-integration) on 2014-01-22

lp:~thomas-voss/platform-api/expose_accuracy_and_service_status_to_platform

Ready for review for merging into lp:platform-api

Gerry Boland (community): Needs Information on 2014-11-06

Charles Kerr (community): Needs Fixing on 2014-05-27

PS Jenkins bot: Needs Fixing (continuous-integration) on 2014-01-30

lp:~thomas-voss/location-service/add_controller_and_service_configuration

Merged into lp:location-service/trunk at revision 66

Manuel de la Peña (community): Approve on 2014-06-25

PS Jenkins bot: Needs Fixing (continuous-integration) on 2014-06-24

Charles Kerr (community): Needs Information on 2014-04-10

Timo Jyrinki: Pending requested 2014-02-03

Francis Ginther: Pending requested 2014-02-03

Diff: 24850 lines (+18319/-3465)

150 files modified

CMakeLists.txt (+22/-9)
data/CMakeLists.txt (+10/-1)
data/ubuntu-location-service-connectivity.pc.in (+12/-0)
data/ubuntu-location-service.pc.in (+1/-1)
debian/changelog (+6/-0)
debian/control (+28/-5)
debian/copyright (+18/-0)
debian/libubuntu-location-service-dev.install (+4/-2)
debian/libubuntu-location-service1.install (+1/-0)
debian/rules (+3/-0)
debian/ubuntu-location-service-bin.install (+1/-1)
debian/ubuntu-location-service-examples.install (+1/-0)
debian/ubuntu-location-service-tests.install (+1/-0)
doc/Doxyfile.in (+19/-19)
examples/CMakeLists.txt (+3/-5)
examples/service/CMakeLists.txt (+2/-7)
examples/service/client.cpp (+18/-17)
examples/service/service.cpp (+61/-21)
examples/standalone/connectivity/CMakeLists.txt (+41/-0)
examples/standalone/connectivity/connectivity.cpp (+201/-0)
include/CMakeLists.txt (+3/-3)
include/location_service/com/ubuntu/location/channel.h (+0/-41)
include/location_service/com/ubuntu/location/clock.h (+26/-0)
include/location_service/com/ubuntu/location/codec.h (+274/-309)
include/location_service/com/ubuntu/location/connectivity/bounded_integer.h (+209/-0)
include/location_service/com/ubuntu/location/connectivity/manager.h (+167/-0)
include/location_service/com/ubuntu/location/connectivity/radio_cell.h (+283/-0)
include/location_service/com/ubuntu/location/connectivity/wireless_network.h (+117/-0)
include/location_service/com/ubuntu/location/criteria.h (+33/-23)
include/location_service/com/ubuntu/location/default_provider_selection_policy.h (+6/-6)
include/location_service/com/ubuntu/location/engine.h (+108/-10)
include/location_service/com/ubuntu/location/heading.h (+3/-74)
include/location_service/com/ubuntu/location/logging.h (+1/-0)
include/location_service/com/ubuntu/location/optional.h (+35/-0)
include/location_service/com/ubuntu/location/position.h (+24/-43)
include/location_service/com/ubuntu/location/provider.h (+185/-82)
include/location_service/com/ubuntu/location/provider_enumerator.h (+49/-0)
include/location_service/com/ubuntu/location/provider_factory.h (+3/-3)
include/location_service/com/ubuntu/location/provider_selection.h (+62/-0)
include/location_service/com/ubuntu/location/provider_selection_policy.h (+11/-37)
include/location_service/com/ubuntu/location/providers/geoclue/geoclue.h (+26/-26)
include/location_service/com/ubuntu/location/providers/geoclue/provider.h (+4/-7)
include/location_service/com/ubuntu/location/providers/skyhook/provider.h (+2/-2)
include/location_service/com/ubuntu/location/proxy_provider.h (+10/-9)
include/location_service/com/ubuntu/location/satellite_based_positioning_state.h (+40/-0)
include/location_service/com/ubuntu/location/service/configuration.h (+4/-4)
include/location_service/com/ubuntu/location/service/default_configuration.h (+1/-1)
include/location_service/com/ubuntu/location/service/default_permission_manager.h (+1/-1)
include/location_service/com/ubuntu/location/service/interface.h (+134/-7)
include/location_service/com/ubuntu/location/service/permission_manager.h (+21/-6)
include/location_service/com/ubuntu/location/service/session/implementation.h (+4/-7)
include/location_service/com/ubuntu/location/service/session/interface.h (+78/-222)
include/location_service/com/ubuntu/location/service/session/skeleton.h (+72/-17)
include/location_service/com/ubuntu/location/service/session/stub.h (+6/-7)
include/location_service/com/ubuntu/location/service/skeleton.h (+107/-13)
include/location_service/com/ubuntu/location/service/stub.h (+6/-2)
include/location_service/com/ubuntu/location/space_vehicle.h (+123/-0)
include/location_service/com/ubuntu/location/units/units.h (+6/-0)
include/location_service/com/ubuntu/location/update.h (+34/-5)
include/location_service/com/ubuntu/location/velocity.h (+3/-74)
include/location_service/com/ubuntu/location/wgs84/altitude.h (+2/-2)
include/location_service/com/ubuntu/location/wgs84/coordinate.h (+5/-3)
include/location_service/com/ubuntu/location/wgs84/latitude.h (+2/-2)
include/location_service/com/ubuntu/location/wgs84/longitude.h (+2/-2)
include/location_service/com/ubuntu/location/wifi_and_cell_reporting_state.h (+36/-0)
src/location_service/com/ubuntu/location/CMakeLists.txt (+107/-8)
src/location_service/com/ubuntu/location/connectivity/cached_radio_cell.h (+489/-0)
src/location_service/com/ubuntu/location/connectivity/cached_wireless_network.h (+218/-0)
src/location_service/com/ubuntu/location/connectivity/nm.h (+519/-0)
src/location_service/com/ubuntu/location/connectivity/ofono.h (+699/-0)
src/location_service/com/ubuntu/location/connectivity/ofono_nm_connectivity_manager.cpp (+404/-0)
src/location_service/com/ubuntu/location/connectivity/ofono_nm_connectivity_manager.h (+114/-0)
src/location_service/com/ubuntu/location/connectivity/radio_cell.cpp (+118/-0)
src/location_service/com/ubuntu/location/connectivity/wireless_network.cpp (+47/-0)
src/location_service/com/ubuntu/location/criteria.cpp (+99/-0)
src/location_service/com/ubuntu/location/default_permission_manager.cpp (+1/-1)
src/location_service/com/ubuntu/location/default_provider_selection_policy.cpp (+36/-23)
src/location_service/com/ubuntu/location/engine.cpp (+94/-8)
src/location_service/com/ubuntu/location/init_and_shutdown.cpp (+6/-20)
src/location_service/com/ubuntu/location/position.cpp (+67/-81)
src/location_service/com/ubuntu/location/provider.cpp (+66/-103)
src/location_service/com/ubuntu/location/provider_factory.cpp (+2/-2)
src/location_service/com/ubuntu/location/providers/dummy/provider.cpp (+67/-30)
src/location_service/com/ubuntu/location/providers/dummy/provider.h (+72/-22)
src/location_service/com/ubuntu/location/providers/geoclue/CMakeLists.txt (+1/-1)
src/location_service/com/ubuntu/location/providers/geoclue/provider.cpp (+33/-48)
src/location_service/com/ubuntu/location/providers/gps/CMakeLists.txt (+12/-2)
src/location_service/com/ubuntu/location/providers/gps/android_hardware_abstraction_layer.cpp (+634/-0)
src/location_service/com/ubuntu/location/providers/gps/android_hardware_abstraction_layer.h (+278/-0)
src/location_service/com/ubuntu/location/providers/gps/hardware_abstraction_layer.h (+323/-0)
src/location_service/com/ubuntu/location/providers/gps/net_cpp_gps_xtra_downloader.h (+78/-0)
src/location_service/com/ubuntu/location/providers/gps/null_gps_xtra_downloader.h (+39/-0)
src/location_service/com/ubuntu/location/providers/gps/provider.cpp (+52/-173)
src/location_service/com/ubuntu/location/providers/gps/provider.h (+19/-16)
src/location_service/com/ubuntu/location/providers/skyhook/provider.cpp (+3/-3)
src/location_service/com/ubuntu/location/proxy_provider.cpp (+27/-37)
src/location_service/com/ubuntu/location/service/daemon.cpp (+467/-0)
src/location_service/com/ubuntu/location/service/daemon.h (+191/-0)
src/location_service/com/ubuntu/location/service/daemon_cli_main.cpp (+55/-0)
src/location_service/com/ubuntu/location/service/daemon_main.cpp (+55/-0)
src/location_service/com/ubuntu/location/service/default_configuration.cpp (+8/-4)
src/location_service/com/ubuntu/location/service/default_permission_manager.cpp (+3/-1)
src/location_service/com/ubuntu/location/service/harvester.cpp (+79/-0)
src/location_service/com/ubuntu/location/service/harvester.h (+97/-0)
src/location_service/com/ubuntu/location/service/ichnaea_reporter.cpp (+226/-0)
src/location_service/com/ubuntu/location/service/ichnaea_reporter.h (+155/-0)
src/location_service/com/ubuntu/location/service/implementation.cpp (+123/-43)
src/location_service/com/ubuntu/location/service/implementation.h (+41/-14)
src/location_service/com/ubuntu/location/service/main.cpp (+0/-148)
src/location_service/com/ubuntu/location/service/program_options.h (+62/-1)
src/location_service/com/ubuntu/location/service/session/implementation.cpp (+77/-31)
src/location_service/com/ubuntu/location/service/session/interface.cpp (+5/-32)
src/location_service/com/ubuntu/location/service/session/interface_p.h (+228/-0)
src/location_service/com/ubuntu/location/service/session/skeleton.cpp (+295/-161)
src/location_service/com/ubuntu/location/service/session/stub.cpp (+133/-27)
src/location_service/com/ubuntu/location/service/skeleton.cpp (+151/-204)
src/location_service/com/ubuntu/location/service/stub.cpp (+52/-10)
src/location_service/com/ubuntu/location/set_name_for_thread.cpp (+24/-0)
src/location_service/com/ubuntu/location/set_name_for_thread.h (+34/-0)
symbols.map (+16/-0)
tests/CMakeLists.txt (+67/-273)
tests/acceptance_tests.cpp (+803/-171)
tests/accuracy_test.cpp (+0/-35)
tests/accuracy_tests.cpp (+0/-52)
tests/connectivity_manager_test.cpp (+109/-0)
tests/controller_test.cpp (+85/-143)
tests/criteria_test.cpp (+50/-0)
tests/cross_process_sync.h (+0/-63)
tests/daemon_and_cli_tests.cpp (+258/-0)
tests/default_permission_manager_test.cpp (+3/-1)
tests/engine_test.cpp (+89/-33)
tests/fork_and_run.h (+0/-61)
tests/geoclue_provider_test.cpp (+2/-2)
tests/gps_provider_test.cpp (+818/-4)
tests/harvester_test.cpp (+125/-0)
tests/heading_test.cpp (+1/-1)
tests/ichnaea_reporter_test.cpp (+251/-0)
tests/mock_connectivity_manager.h (+89/-0)
tests/mongoose.c (+4986/-0)
tests/mongoose.h (+144/-0)
tests/null_provider_selection_policy.h (+40/-0)
tests/position_test.cpp (+34/-63)
tests/provider_factory_test.cpp (+1/-1)
tests/provider_selection_policy_test.cpp (+52/-22)
tests/provider_test.cpp (+93/-79)
tests/session_test.cpp (+2/-2)
tests/test_main.cpp (+5/-8)
tests/velocity_test.cpp (+0/-54)
tests/web_server.h (+125/-0)
tests/wgs84_test.cpp (+5/-5)

lp:~charlesk/indicator-location/sync-with-platform-api-changes-to-accuracy-and-service

Merged into lp:indicator-location/14.10 at revision 81

Ted Gould (community): Approve on 2014-05-27

PS Jenkins bot: Pending (continuous-integration) requested 2014-05-27

Superseded for merging into lp:indicator-location/14.04

PS Jenkins bot: Pending (continuous-integration) requested 2014-05-27

Ted Gould: Pending requested 2014-05-27

lp:ubuntu/utopic-proposed/indicator-location

Dave Morley (davmor2) on 2013-11-07

tags:

added: qa-touch

Dave Morley (davmor2) on 2013-11-07

description:

updated

Dave Morley (davmor2) on 2013-12-02

tags:

added: r100

Pat McGowan (pat-mcgowan) on 2013-12-11

Changed in indicator-location:
status:	New → Confirmed
assignee:	nobody → Charles Kerr (charlesk)
importance:	Undecided → High

Revision history for this message

Charles Kerr (charlesk) wrote on 2013-12-17:

I'm able to reproduce this issue on the phone & am looking into the cause.

Changed in indicator-location:
status:	Confirmed → In Progress

Revision history for this message

Joseph Mills (josephjamesmills) wrote on 2014-01-14:

I'm am also able to reproduce this issue on the phone.

Pat McGowan (pat-mcgowan) on 2014-01-23

Changed in indicator-location (Ubuntu):
status:	New → In Progress
importance:	Undecided → High

Selene ToyKeeper (toykeeper) on 2014-03-26

tags:

added: r261

Revision history for this message

Launchpad Janitor (janitor) wrote on 2014-03-27:

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in platform-api (Ubuntu):
status:	New → Confirmed

Selene ToyKeeper (toykeeper) on 2014-03-27

tags:

added: r263

Selene ToyKeeper (toykeeper) on 2014-04-04

tags:

added: r276

Revision history for this message

Selene ToyKeeper (toykeeper) wrote on 2014-04-29:

Reproduced on utopic r2.

tags:

added: u2

Revision history for this message

Launchpad Janitor (janitor) wrote on 2014-05-27:

This bug was fixed in the package indicator-location - 13.10.0+14.10.20140527.1-0ubuntu1

---------------
indicator-location (13.10.0+14.10.20140527.1-0ubuntu1) utopic; urgency=low

  [ Charles Kerr ]
  * Syncs indicator-location to the changed platform location service's
    API (LP: #1248973)
  * Add initial .pot file and update-pot.sh script

[ Ted Gould ]
* Synchronize process management across indicators
-- Ubuntu daily release <email address hidden> Tue, 27 May 2014 15:42:19 +0000

Changed in indicator-location (Ubuntu):
status:	In Progress → Fix Released

Pat McGowan (pat-mcgowan) on 2014-05-27

tags:

added: avengers-3

Charles Kerr (charlesk) on 2014-06-14

Changed in indicator-location:
status:	In Progress → Fix Released

Revision history for this message

Selene ToyKeeper (toykeeper) wrote on 2014-07-11:

Just confirming... this seems to be fixed in the last few phone images. Utopic 124 had no trouble keeping both location indicators checked.

Granted, I still can't get location to work, but the boxes at least stay checked now.

Revision history for this message

Selene ToyKeeper (toykeeper) wrote on 2014-08-02:

Well, it *was* fixed...

On Utopic 166, the location indicator checkboxes un-check themselves again after a minute or two. I can't get location functions to work at all.

Changed in indicator-location (Ubuntu):
status:	Fix Released → Confirmed
tags:	added: qa-daily-testing rtm14

Revision history for this message

Jean-Baptiste Lallement (jibel) wrote on 2014-08-05:

closing this issue since it has already been fixed several months ago and I opened bug 1352930 to track the regression on recent builds.

Changed in indicator-location (Ubuntu):
status:	Confirmed → Fix Released

Revision history for this message

Launchpad Janitor (janitor) wrote on 2014-08-08:

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in location-service (Ubuntu):
status:	New → Confirmed

Revision history for this message

Cris Dywan (kalikiana) wrote on 2014-09-08:

#10

I don't see this happening - but alas they revert after reboot. Not sure if that would be related.

Revision history for this message

Charles Kerr (charlesk) wrote on 2014-09-23:

#11

I'm not able to reproduce this bug and wonder if it might have been fixed as a side-effect of fixing bug #1348334 in mid-August.

jibel, toykeeper, davmor2, are any of you still experiencing this?

Revision history for this message

Charles Kerr (charlesk) wrote on 2014-09-23:

#12

After confirming with some of the bug commenters in #ubuntu-touch I'm closing this as Fix Released.

Changed in location-service (Ubuntu):
status:	Confirmed → Fix Released
Changed in platform-api (Ubuntu):
status:	Confirmed → Fix Released
Changed in indicator-location (Ubuntu):
assignee:	nobody → Charles Kerr (charlesk)
Changed in location-service (Ubuntu):
assignee:	nobody → Thomas Voß (thomas-voss)
Changed in platform-api (Ubuntu):
assignee:	nobody → Thomas Voß (thomas-voss)
Changed in location-service (Ubuntu):
importance:	Undecided → High
Changed in platform-api (Ubuntu):
importance:	Undecided → High