Issue #539 - Fixing toString check with fixed properties

jpwsutton · jpwsutton · commit 822d254cf6dd · 2018-05-29T11:25:16.000+01:00
Signed-off-by: James Sutton &lt;james.sutton@uk.ibm.com&gt;
diff --git a/org.eclipse.paho.mqttv5.common/src/main/java/org/eclipse/paho/mqttv5/common/packet/MqttProperties.java b/org.eclipse.paho.mqttv5.common/src/main/java/org/eclipse/paho/mqttv5/common/packet/MqttProperties.java
@@ -1358,13 +1358,13 @@ public String toString() {
 		if (serverReference != null) {
 			sb.append(", serverReference=" + serverReference);
 		}
-		if (wildcardSubscriptionsAvailable) {
+		if (wildcardSubscriptionsAvailable != null) {
 			sb.append(", wildcardSubscriptionsAvailable=" + wildcardSubscriptionsAvailable);
 		}
-		if (subscriptionIdentifiersAvailable) {
+		if (subscriptionIdentifiersAvailable != null) {
 			sb.append(", subscriptionIdentifiersAvailable=" + subscriptionIdentifiersAvailable);
 		}
-		if (sharedSubscriptionAvailable) {
+		if (sharedSubscriptionAvailable != null) {
 			sb.append(", sharedSubscriptionAvailable=" + sharedSubscriptionAvailable);
 		}
 		if (sessionExpiryInterval != null) {

Original file line number	Diff line number	Diff line change
`@@ -1358,13 +1358,13 @@ public String toString() {`
`1358`	`1358`	`if (serverReference != null) {`
`1359`	`1359`	`sb.append(", serverReference=" + serverReference);`
`1360`	`1360`	`}`
`1361`		`- if (wildcardSubscriptionsAvailable) {`
	`1361`	`+ if (wildcardSubscriptionsAvailable != null) {`
`1362`	`1362`	`sb.append(", wildcardSubscriptionsAvailable=" + wildcardSubscriptionsAvailable);`
`1363`	`1363`	`}`
`1364`		`- if (subscriptionIdentifiersAvailable) {`
	`1364`	`+ if (subscriptionIdentifiersAvailable != null) {`
`1365`	`1365`	`sb.append(", subscriptionIdentifiersAvailable=" + subscriptionIdentifiersAvailable);`
`1366`	`1366`	`}`
`1367`		`- if (sharedSubscriptionAvailable) {`
	`1367`	`+ if (sharedSubscriptionAvailable != null) {`
`1368`	`1368`	`sb.append(", sharedSubscriptionAvailable=" + sharedSubscriptionAvailable);`
`1369`	`1369`	`}`
`1370`	`1370`	`if (sessionExpiryInterval != null) {`