GNOME Bugzilla – Bug 725958
NMConnectionSection: don't remove connections that were never added
Last modified: 2014-03-10 16:52:05 UTC
NMApplet will call removeConnection() unconditionally on all sections, including those that had nothing to do with the connection in the first place. Fixes: Gjs-WARNING **: JS ERROR: TypeError: this._connectionItems.get(...) is undefined NMConnectionSection<.removeConnection@resource:///org/gnome/shell/ui/status/network.js:323 wrapper@resource:///org/gnome/gjs/modules/lang.js:169 NMApplet<._connectionRemoved@resource:///org/gnome/shell/ui/status/network.js:1885 wrapper@resource:///org/gnome/gjs/modules/lang.js:169
Created attachment 271340 [details] [review] NMConnectionSection: don't remove connections that were never added
Review of attachment 271340 [details] [review]: OK.
Review of attachment 271340 [details] [review]: Why don't they have a section in the first place? This seems like an NM bug workaround.
(In reply to comment #3) > Review of attachment 271340 [details] [review]: > > Why don't they have a section in the first place? > > This seems like an NM bug workaround. I don't follow you, who should have a section?
Review of attachment 271340 [details] [review]: Oh, I see. We don't store a device for a connection. Because connections don't have devices. I think the NM team wants to change that and pair connections with devices 1:1, but let's do this for now.
Attachment 271340 [details] pushed as 522f3bf - NMConnectionSection: don't remove connections that were never added