Merge lp:~alfonsosanchezbeato/nuntium/lp1471740 into lp:nuntium/packaging

[Alfonso Sanchez-Beato]

Track properly org.ofono.PushNotificationAgent (LP: #1471740)

Track properly org.ofono.PushNotificationAgent (LP: #1471740)

[Manuel de la Peña (mandel)]

Looks good as per review in github.

[Alfonso Sanchez-Beato (alfonsosanchezbeato)]

review: Approve

[Manuel de la Peña (mandel)]

review: Approve

=== modified file 'debian/changelog'
--- debian/changelog	2015-07-01 16:36:42 +0000
+++ debian/changelog	2015-07-07 09:48:00 +0000
@@ -1,3 +1,9 @@
+nuntium (1.4+15.10.20150707-0ubuntu1) UNRELEASED; urgency=medium
+
+  * Track properly org.ofono.PushNotificationAgent (LP: #1471740) 
+
+ -- Alfonso Sanchez-Beato (email Canonical) <[e-mail]>  Tue, 07 Jul 2015 11:42:07 +0200
+
 nuntium (1.4+15.10.20150701.2-0ubuntu1) wily; urgency=medium
 
   [ Alfonso Sanchez-Beato (email Canonical) ]
 
=== modified file 'ofono/modem.go'
--- ofono/modem.go	2015-07-01 16:33:07 +0000
+++ ofono/modem.go	2015-07-07 09:48:00 +0000
@@ -207,16 +207,17 @@
 }
 
 func (modem *Modem) updatePushInterfaceState(interfaces dbus.Variant) {
-	origState := modem.pushInterfaceAvailable
+	nextState := false
 	availableInterfaces := reflect.ValueOf(interfaces.Value)
 	for i := 0; i < availableInterfaces.Len(); i++ {
 		interfaceName := reflect.ValueOf(availableInterfaces.Index(i).Interface().(string)).String()
 		if interfaceName == PUSH_NOTIFICATION_INTERFACE {
-			modem.pushInterfaceAvailable = true
+			nextState = true
 			break
 		}
 	}
-	if modem.pushInterfaceAvailable != origState {
+	if modem.pushInterfaceAvailable != nextState {
+		modem.pushInterfaceAvailable = nextState
 		log.Printf("Push interface state: %t", modem.pushInterfaceAvailable)
 		if modem.pushInterfaceAvailable {
 			modem.PushInterfaceAvailable <- true