Merge lp:~jonas-drange/ubuntu-system-settings/rtm-1399723 into lp:ubuntu-system-settings/rtm-14.09

[Jonas G. Drange]

[about] use networkinfo to pull the wifi device MAC

[about] use networkinfo to pull the wifi device MAC

[Ken VanDine (ken-vandine)]

Looks good!

review: Approve

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:955
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-ci/20/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/21
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/21/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-rtm-14.09-ci/20/rebuild

review: Approve (continuous-integration)

=== modified file 'plugins/about/PageComponent.qml'
--- plugins/about/PageComponent.qml	2014-11-12 16:03:25 +0000
+++ plugins/about/PageComponent.qml	2015-01-09 13:20:46 +0000
@@ -61,6 +61,10 @@
         id: network
     }
 
+    NetworkInfo {
+        id: wlinfo
+    }
+
     Flickable {
         id: scrollWidget
         anchors.fill: parent
@@ -122,9 +126,10 @@
             }
 
             ListItem.SingleValue {
+                property string address: wlinfo.macAddress(NetworkInfo.WlanMode, 0)
                 text: i18n.tr("Wi-Fi address")
-                value: network.networkMacAddresses[0]
-                visible: network.networkMacAddresses.length > 0
+                value: address ? address.toUpperCase() : ""
+                visible: address
                 showDivider: bthwaddr.visible
             }
 

=== modified file 'plugins/about/networkabout.cpp'
--- plugins/about/networkabout.cpp	2014-08-28 10:33:41 +0000
+++ plugins/about/networkabout.cpp	2015-01-09 13:20:46 +0000
@@ -28,27 +28,9 @@
     m_systemBusConnection(QDBusConnection::systemBus())
 {
 
-    setupNetworkMacAddresses();
     setupBluetoothMacAddress();
 }
 
-void NetworkAbout::setupNetworkMacAddresses()
-{
-    QList<QNetworkInterface> ifaces = QNetworkInterface::allInterfaces();
-    int count = ifaces.count();
-
-    QString str = QString();
-    for(int i=0;i<count;i++)
-    {
-        QNetworkInterface iface = ifaces.at(i);
-        str = iface.hardwareAddress();
-        // add interfaces that are not loopback
-        if (!iface.flags().testFlag(QNetworkInterface::IsLoopBack)) {
-            m_networkMacAddresses << str;
-        }
-    }
-}
-
 void NetworkAbout::setupBluetoothMacAddress()
 {
     QDBusInterface bluezManagerInterface (
@@ -84,11 +66,6 @@
     m_bluetoothMacAddress = props.value()["Address"].toString();
 }
 
-QStringList NetworkAbout::networkMacAddresses()
-{
-    return m_networkMacAddresses;
-}
-
 QString NetworkAbout::bluetoothMacAddress()
 {
     return m_bluetoothMacAddress;

=== modified file 'plugins/about/networkabout.h'
--- plugins/about/networkabout.h	2014-08-28 10:33:41 +0000
+++ plugins/about/networkabout.h	2015-01-09 13:20:46 +0000
@@ -23,33 +23,25 @@
 
 #include <QObject>
 #include <QStringList>
-#include <QtNetwork>
 #include <QtDBus/QDBusInterface>
 
 
 class NetworkAbout : public QObject
 {
     Q_OBJECT
-    Q_PROPERTY( QStringList networkMacAddresses
-                READ networkMacAddresses
-                NOTIFY networkMacAddressesChanged )
     Q_PROPERTY( QString bluetoothMacAddress
                 READ bluetoothMacAddress
                 NOTIFY bluetoothMacAddressChanged )
 
 public:
     explicit NetworkAbout (QObject *parent = 0);
-    QStringList networkMacAddresses();
     QString bluetoothMacAddress();
 
 Q_SIGNALS:
-    void networkMacAddressesChanged(QStringList addresses);
     void bluetoothMacAddressChanged(QString addresses);
 
 private:
-    QStringList m_networkMacAddresses;
     QString m_bluetoothMacAddress;
-    void setupNetworkMacAddresses();
     void setupBluetoothMacAddress();
     QDBusConnection m_systemBusConnection;
 };