Skip to content

network: don't assume NMActiveConnection has a device

Will Thompson requested to merge wjt/gnome-shell:wjt/1406-ensureactiveconnectionprops-erroneously-assumes-connections-have-at-least-one-device into master

In practice this has been seen to fail:

JS ERROR: TypeError: active.get_devices(...)[0] is undefined
ensureActiveConnectionProps@resource:///org/gnome/shell/ui/status/network.js:73:22
_getMainConnection@resource:///org/gnome/shell/ui/status/network.js:1791:13
_syncMainConnection@resource:///org/gnome/shell/ui/status/network.js:1809:32

I haven't tested this patch, I'm afraid.

Fixes #1406 (closed).

Merge request reports