-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126714/
-----------------------------------------------------------
Review request for Network Management and Lamarque Souza.
Bugs: 357816
http://bugs.kde.org/show_bug.cgi?id=357816
Repository: plasma-nm
Description
-------
Reproducible with l2tp connections (you can use http://freel2tpvpn.com/). In
case you activate it, NM creates automaticaly ppp0 connection which gets the
default route, but this connection is type of generic, thus getting device for
it in setIcons() leads to setDisconnectedIcon() because we care only about
certain devices. What we actually have to do is to go through all active
connections and find the one with hightest probability of being the main one
(e.g. a wired connection is probably used as the main one than some wireless
connection), also if there is an active VPN connection then it's probably the
one for which NM created a ppp0 connection and it's the one we are interested
about.
Diffs
-----
libs/declarative/connectionicon.cpp c000a8f
Diff: https://git.reviewboard.kde.org/r/126714/diff/
Testing
-------
Tested with L2TP.
Thanks,
Jan Grulich
_______________________________________________
kde-networkmanager mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-networkmanager
