Merge pull request #164 from ovr/patch-3

HazAT · web-flow · commit de0a8b4e1e0e · 2017-07-24T08:36:24.000+02:00
captureBreadcrumb(), data: Skip toString for null, refs #163
diff --git a/android/src/main/java/io/sentry/RNSentryModule.java b/android/src/main/java/io/sentry/RNSentryModule.java
@@ -169,9 +169,7 @@ public void captureBreadcrumb(ReadableMap breadcrumb) {
         if (breadcrumb.hasKey("data") && breadcrumb.getMap("data") != null) {
             Map<String, String> newData = new HashMap<>();
             for (Map.Entry<String, Object> data : breadcrumb.getMap("data").toHashMap().entrySet()) {
-                if (data.getValue() != null) {
-                    newData.put(data.getKey(), data.getValue().toString());
-                }
+                newData.put(data.getKey(), data.getValue() != null ? data.getValue().toString() : null);
             }
             breadcrumbBuilder.setData(newData);
         }

Original file line number	Diff line number	Diff line change
`@@ -169,9 +169,7 @@ public void captureBreadcrumb(ReadableMap breadcrumb) {`
`169`	`169`	`if (breadcrumb.hasKey("data") && breadcrumb.getMap("data") != null) {`
`170`	`170`	`Map<String, String> newData = new HashMap<>();`
`171`	`171`	`for (Map.Entry<String, Object> data : breadcrumb.getMap("data").toHashMap().entrySet()) {`
`172`		`- if (data.getValue() != null) {`
`173`		`- newData.put(data.getKey(), data.getValue().toString());`
`174`		`- }`
	`172`	`+ newData.put(data.getKey(), data.getValue() != null ? data.getValue().toString() : null);`
`175`	`173`	`}`
`176`	`174`	`breadcrumbBuilder.setData(newData);`
`177`	`175`	`}`