Merge pull request #31 from session-foundation/fix-profile-ts-disabled-state

fix: allow profile update to be used when current ts is 0

Author: Bilb

package.json

@@ -2,7 +2,7 @@
   "main": "index.js",
   "name": "libsession_util_nodejs",
   "description": "Wrappers for the Session Util Library",
-  "version": "0.5.7",
+  "version": "0.5.8",
   "license": "GPL-3.0",
   "author": {
     "name": "Oxen Project",

src/contacts_config.cpp

@@ -145,7 +145,8 @@ void ContactsConfigWrapper::set(const Napi::CallbackInfo& info) {
 
         // if the saved profile info is older than the new one, update it and the profile fields
         // provided
-        if (contact.profile_updated < newProfileUpdateSeconds) {
+        if (contact.profile_updated.time_since_epoch().count() == 0 ||
+            contact.profile_updated < newProfileUpdateSeconds) {
             contact.profile_updated = newProfileUpdateSeconds;
 
             if (auto name = maybeNonemptyString(obj.Get("name"), "contacts.set name"))

src/groups/meta_group_wrapper.cpp

@@ -649,8 +649,11 @@ void MetaGroupWrapper::memberSetProfileDetails(const Napi::CallbackInfo& info) {
         auto updatedAtSeconds =
                 toCppSysSeconds(argsAsObj.Get("profileUpdatedSeconds"), "memberSetProfileDetails");
 
-        // if the profile details provided are more recent that the ones saved, update them
-        if (m && updatedAtSeconds > m->profile_updated) {
+        // if the profile details provided are more recent that the ones saved, update them.
+        // we also allow anything when our current value is 0, as it means we haven't got an updated
+        // profileDetails yet
+        if (m && (updatedAtSeconds > m->profile_updated ||
+                  m->profile_updated.time_since_epoch().count() == 0)) {
             m->profile_updated = updatedAtSeconds;
 
             auto profilePicture = profile_pic_from_object(argsAsObj.Get("profilePicture"));