Add conversion to FString for SentryId (#161)

* Add string conversion for Sentry ID on desctom, iOS and Android

* Update changelog

Author: tustanivsky

CHANGELOG.md

@@ -5,6 +5,7 @@
 ### Features
 
 - Add debug symbols upload settings ([#94](https://github.com/getsentry/sentry-unreal/pull/94))
+- Add conversion to FString for SentryId ([#161](https://github.com/getsentry/sentry-unreal/pull/161))
 - Add editor menu for automatic CRC configuration ([#152](https://github.com/getsentry/sentry-unreal/pull/152))
 
 ### Fixes

plugin-dev/Source/Sentry/Private/Android/SentryIdAndroid.cpp

@@ -2,6 +2,8 @@
 
 #include "SentryIdAndroid.h"
 
+#include "Infrastructure/SentryMethodCallAndroid.h"
+
 SentryIdAndroid::SentryIdAndroid()
 {
 	JNIEnv* Env = FAndroidApplication::GetJavaEnv();
@@ -26,4 +28,9 @@ SentryIdAndroid::~SentryIdAndroid()
 jobject SentryIdAndroid::GetNativeObject()
 {
 	return IdAndroid;
+}
+
+FString SentryIdAndroid::ToString() const
+{
+	return SentryMethodCallAndroid::CallStringMethod(IdAndroid, "toString", "()Ljava/lang/String;");
 }

plugin-dev/Source/Sentry/Private/Android/SentryIdAndroid.h

@@ -15,6 +15,8 @@ class SentryIdAndroid : public ISentryId
 
 	jobject GetNativeObject();
 
+	virtual FString ToString() const override;
+
 private:
 	jobject IdAndroid;
 };

plugin-dev/Source/Sentry/Private/Desktop/SentryIdDesktop.cpp

@@ -21,3 +21,10 @@ sentry_uuid_t SentryIdDesktop::GetNativeObject()
 {
 	return IdDesktop;
 }
+
+FString SentryIdDesktop::ToString() const
+{
+	char IdString[37];
+	sentry_uuid_as_string(&IdDesktop, IdString);
+	return FString(IdString);
+}

plugin-dev/Source/Sentry/Private/Desktop/SentryIdDesktop.h

@@ -15,6 +15,8 @@ class SentryIdDesktop : public ISentryId
 
 	sentry_uuid_t GetNativeObject();
 
+	virtual FString ToString() const override;
+
 private:
 	sentry_uuid_t IdDesktop;
-};
+};

plugin-dev/Source/Sentry/Private/IOS/SentryIdIOS.cpp

@@ -24,3 +24,8 @@ SentryId* SentryIdIOS::GetNativeObject()
 {
 	return IdIOS;
 }
+
+FString SentryIdIOS::ToString() const
+{
+	return FString(IdIOS.sentryIdString);
+}

plugin-dev/Source/Sentry/Private/IOS/SentryIdIOS.h

@@ -15,6 +15,8 @@ class SentryIdIOS : public ISentryId
 
 	SentryId* GetNativeObject();
 
+	virtual FString ToString() const override;
+
 private:
 	SentryId* IdIOS;
 };

plugin-dev/Source/Sentry/Private/Interface/SentryIdInterface.h

@@ -8,4 +8,6 @@ class ISentryId
 {
 public:
 	virtual ~ISentryId() = default;
+
+	virtual FString ToString() const = 0;
 };

plugin-dev/Source/Sentry/Private/SentryId.cpp

@@ -8,6 +8,8 @@
 #include "Android/SentryIdAndroid.h"
 #elif PLATFORM_IOS
 #include "IOS/SentryIdIOS.h"
+#elif PLATFORM_WINDOWS || PLATFORM_MAC || PLATFORM_LINUX
+#include "Desktop/SentryIdDesktop.h"
 #endif
 
 USentryId::USentryId()
@@ -18,10 +20,20 @@ USentryId::USentryId()
 		SentryIdNativeImpl = MakeShareable(new SentryIdAndroid());
 #elif PLATFORM_IOS
 		SentryIdNativeImpl = MakeShareable(new SentryIdIOS());
+#elif PLATFORM_WINDOWS || PLATFORM_MAC || PLATFORM_LINUX
+		SentryIdNativeImpl = MakeShareable(new SentryIdDesktop());
 #endif
 	}
 }
 
+FString USentryId::ToString() const
+{
+	if(!SentryIdNativeImpl)
+		return FString();
+
+	return SentryIdNativeImpl->ToString();
+}
+
 void USentryId::InitWithNativeImpl(TSharedPtr<ISentryId> idImpl)
 {
 	SentryIdNativeImpl = idImpl;

plugin-dev/Source/Sentry/Public/SentryId.h

@@ -17,6 +17,10 @@ class SENTRY_API USentryId : public UObject
 public:
 	USentryId();
 
+	/** Gets string representation of the event ID. */
+	UFUNCTION(BlueprintPure, Category = "Sentry")
+	FString ToString() const;
+
 	void InitWithNativeImpl(TSharedPtr<ISentryId> idImpl);
 	TSharedPtr<ISentryId> GetNativeImpl();