From 6c3300bc2f196c76b17ccd78804a9f2aabaff48e Mon Sep 17 00:00:00 2001
From: "Owen W. Taylor" <otaylor@fishsoup.net>
Date: Sat, 2 Apr 2011 15:05:06 -0400
Subject: [PATCH] network: fix logic bug in checking whether to activate or
 deactive

A cosmetic change recommended in review of the patch to fix the
VPN Connections switch ended up introducing a logic error that
made the switch not work properly. Fix.

https://bugzilla.gnome.org/show_bug.cgi?id=646380
---
 js/ui/status/network.js | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/js/ui/status/network.js b/js/ui/status/network.js
index 7616542c8..c78b08b03 100644
--- a/js/ui/status/network.js
+++ b/js/ui/status/network.js
@@ -254,12 +254,14 @@ NMWiredSectionTitleMenuItem.prototype = {
             return;
         }
 
+        let newState = this._switch.state;
+
         // Immediately reset the switch to false, it will be updated appropriately
         // by state-changed signals in devices (but fixes the VPN not being in sync
         // if the ActiveConnection object is never seen by libnm-glib)
         this._switch.setToggleState(false);
 
-        if (this._switch.state)
+        if (newState)
             this._device.activate();
         else
             this._device.deactivate();