[Rust] Return `String` instead of `BnString` for cases where lossy conversion can be tolerated

Still need to go and audit all usage, but realistically the most important places to give the user control are with symbols, where the data can come from non utf8 sources

This is still incomplete, I just looked for usage of -> BnString so any other variant was omitted.

1 files changed, 10 insertions, 10 deletions

diff --git a/rust/src/collaboration/user.rs b/rust/src/collaboration/user.rs
index 0e3433d3..b08e9da4 100644
--- a/rust/src/collaboration/user.rs
+++ b/rust/src/collaboration/user.rs
@@ -28,24 +28,24 @@ impl RemoteUser {
     }
 
     /// Web api endpoint url
-    pub fn url(&self) -> BnString {
+    pub fn url(&self) -> String {
         let value = unsafe { BNCollaborationUserGetUrl(self.handle.as_ptr()) };
         assert!(!value.is_null());
-        unsafe { BnString::from_raw(value) }
+        unsafe { BnString::into_string(value) }
     }
 
     /// Unique id
-    pub fn id(&self) -> BnString {
+    pub fn id(&self) -> String {
         let value = unsafe { BNCollaborationUserGetId(self.handle.as_ptr()) };
         assert!(!value.is_null());
-        unsafe { BnString::from_raw(value) }
+        unsafe { BnString::into_string(value) }
     }
 
     /// User's login username
-    pub fn username(&self) -> BnString {
+    pub fn username(&self) -> String {
         let value = unsafe { BNCollaborationUserGetUsername(self.handle.as_ptr()) };
         assert!(!value.is_null());
-        unsafe { BnString::from_raw(value) }
+        unsafe { BnString::into_string(value) }
     }
 
     /// Set user's username. You will need to push the user to update the Remote
@@ -65,10 +65,10 @@ impl RemoteUser {
     }
 
     /// User's email address
-    pub fn email(&self) -> BnString {
+    pub fn email(&self) -> String {
         let value = unsafe { BNCollaborationUserGetEmail(self.handle.as_ptr()) };
         assert!(!value.is_null());
-        unsafe { BnString::from_raw(value) }
+        unsafe { BnString::into_string(value) }
     }
 
     /// Set user's email. You will need to push the user to update the Remote
@@ -88,10 +88,10 @@ impl RemoteUser {
     }
 
     /// String representing the last date the user logged in
-    pub fn last_login(&self) -> BnString {
+    pub fn last_login(&self) -> String {
         let value = unsafe { BNCollaborationUserGetLastLogin(self.handle.as_ptr()) };
         assert!(!value.is_null());
-        unsafe { BnString::from_raw(value) }
+        unsafe { BnString::into_string(value) }
     }
 
     /// If the user account is active and can log in