1060178 - [ALA] check the settings DB for the location precision

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: General, defect)

Description

[marta]

Look at the settings DB for global settings of ALA on/off and the precision of location returned (or fake location).

Comment 1

[daniel.marecik]

Attachment: Bug1060178.patch

Comment 2

[Ed Morley [:emorley]]

Per the comments on:
https://github.com/mozilla/gecko-dev/pull/47
https://github.com/mozilla/gecko-dev/pull/48
https://github.com/mozilla/gecko-dev/pull/49

gecko-dev cannot accept pull requests, so please stop opening them - it's not the correct workflow for development work here.

The patch you have attached is sufficient on it's own - no need to open a PR (you can of course continue to work in your own user fork).

Comment 3

[daniel.marecik]

Reading global settings from the settings DB is ready.

Following tasks are required to create full feature functionality:
- reading app exception list (from setting DB),
- reading list of unblurred app (from a predefined file or setting DB).

These tasks should be created as separate bugs.

Comment 4

[daniel.marecik]

I will not create PR.

Comment 5

[marta]

Daniel, as have discussed during our call earlier this week, and as showed in other bugs there should be one DB that contains settings for all of the apps:
<ala.global.precise>  = true
<ala.fmd.precise> = true
<ala.weather.blurr> = 5
<ala.netflix.fake> = [51.2345;14.133]

Comment 6

[daniel.marecik]

Attachment: Bug1060178.patch

Comment 7

[daniel.marecik]

Reading global settings from the settings DB is ready.
Reading app exception list from the settings DB is ready.
Reading list of unblurred app from the settings DB is ready.

Global settings keys and values:
geolocation.blur.type = 1 (Precise) / 2 (Blur) / 3 (Custom)
geolocation.blur.radius = 6
geolocation.blur.coords = "@51.1234,14.1234"

App exception list:
geolocation.exceptions = { "app//app1.gaiamobile.org/manifest.webapp": { "type": 1 },
     "app//app2.gaiamobile.org/manifest.webapp": { "type": 2, "radius": 5 },
     "app//app3.gaiamobile.org/manifest.webapp": { "type":3, "coords": "@51.1234,14.1234" }}

List of unblurred app: 
geolocation.unblured = ["app//app5.gaiamobile.org/manifest.webapp",
    "app//app6.gaiamobile.org/manifest.webapp",
    "app//app7.gaiamobile.org/manifest.webapp"]

Comment 8

[Dave Huseby [:huseby]]

See Bug 1073419