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

Jan Grulich jgrulich at redhat.com
Sat Feb 28 10:41:56 UTC 2015



> 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.

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.


- 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/58a1d8e2/attachment.html>


More information about the Plasma-devel mailing list