Review Request 122736: [dataengines/geolocation] Use wifi data if available

[Jan Grulich]

> On Úno. 27, 2015, 9:42 dop., Jan Grulich wrote:
> > dataengines/geolocation/location_ip.cpp, line 149
> > <https://git.reviewboard.kde.org/r/122736/diff/1/?file=351954#file351954line149>
> >
> >     SSID is never empty (at least it shouldn't be), if it would be a hidden AP it wouldn't be listed between networks, but I think that newer NM versions make hidden APs visible anyway.
> 
> Martin Gräßlin wrote:
>     so how can I check for a hidden AP? They are not supposed to be sent to the service.
> 
> Jan Grulich wrote:
>     I'm afraid that this is not possible. 
>     According to NM documentation: GetAllAccessPoints ( ) ? ao - Get the list of all access points visible to this device, including hidden ones for which the SSID is not yet known.
>     
>     And there is no property or flag indicating whether the AP is hidden or not.

Thinking about it and reading the docu above, maybe it will be empty. I've never used hidden network so I'm not sure right now.


- Jan


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122736/#review76704
-----------------------------------------------------------


On Úno. 27, 2015, 9:13 dop., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122736/
> -----------------------------------------------------------
> 
> (Updated Úno. 27, 2015, 9:13 dop.)
> 
> 
> Review request for Plasma, Jan Grulich and Lukáš Tinkl.
> 
> 
> Repository: plasma-workspace
> 
> 
> Description
> -------
> 
> Adds a dependency to NetworkManager and queries the available wifi
> access points to include in the request to the location service.
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 8b1529e6aea7e10f89696a495c2791b6383d9489 
>   dataengines/geolocation/CMakeLists.txt efdcd14940b47a64b4e70dce69bdeeffdc6873f5 
>   dataengines/geolocation/geolocation.h fcc95086b97b6c96c0ab9bce3cac703d96d0b43e 
>   dataengines/geolocation/geolocation.cpp e02060e157a29902e594310102555096bd619e2e 
>   dataengines/geolocation/location_ip.cpp 3a13dc3018bff482c48098e1827f31c5fa3184cc 
> 
> Diff: https://git.reviewboard.kde.org/r/122736/diff/
> 
> 
> Testing
> -------
> 
> Tested with my available networks and checked position on openstreetmaps. Basically it even put me in the right room of the building I'm in.
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20150228/cf728a4e/attachment.html>