[gnome-shell] NetworkMenu: actually add new access point to the network list
- From: Giovanni Campagna <gcampagna src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell] NetworkMenu: actually add new access point to the network list
- Date: Mon, 19 Dec 2011 15:55:39 +0000 (UTC)
commit 77afd6782fcf5b646e9fc30dc9c1882958e9c55c
Author: Giovanni Campagna <gcampagna src gnome org>
Date: Sat Dec 17 23:54:45 2011 +0100
NetworkMenu: actually add new access point to the network list
Previously the code in _accessPointAdded was iterating over the
the network list to find a good place, and at that time, added both
the network to the list and the item to the menu. When I refactored
to call queueCreateSection, I forgot to add code to insert the
network in the list.
Add it now, using the new Util.insertSorted function.
https://bugzilla.gnome.org/show_bug.cgi?id=666429
js/ui/status/network.js | 7 +++++--
1 files changed, 5 insertions(+), 2 deletions(-)
---
diff --git a/js/ui/status/network.js b/js/ui/status/network.js
index 863792d..fcf0835 100644
--- a/js/ui/status/network.js
+++ b/js/ui/status/network.js
@@ -1234,7 +1234,6 @@ const NMDeviceWireless = new Lang.Class({
accessPoints: [ accessPoint ]
};
apObj.ssidText = ssidToLabel(apObj.ssid);
- needsupdate = true;
}
// check if this enables new connections for this group
@@ -1249,7 +1248,11 @@ const NMDeviceWireless = new Lang.Class({
}
}
- if (needsupdate) {
+ if (pos == -1 || needsupdate) {
+ if (pos != -1)
+ this._networks.splice(pos, 1);
+ pos = Util.insertSorted(this._networks, apObj, this._networkSortFunction);
+
this._clearSection();
this._queueCreateSection();
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]