ensure newly created assets get sync'd to rc_client (#1049)

src/services/AchievementRuntime.cpp

@@ -163,6 +163,7 @@ AchievementRuntime::AchievementRuntime()
 #endif
 
     rc_client_set_event_handler(m_pClient.get(), EventHandler);
+    m_pClient->state.allow_leaderboards_in_softcore = true;
 
     rc_client_set_unofficial_enabled(m_pClient.get(), 1);
 }

src/ui/viewmodels/AssetListViewModel.cpp

@@ -1651,6 +1651,11 @@ void AssetListViewModel::RevertSelected()
         }
     }
 
+    // sync assets before calling EndUpdate to ensure anything watching for TriggerProperty to change
+    // can find the new definition (i.e. asset editor)
+    auto& pRuntime = ra::services::ServiceLocator::GetMutable<ra::services::AchievementRuntime>();
+    pRuntime.SyncAssets();
+
     pAssets.EndUpdate();
 
     // update the local file
@@ -1748,6 +1753,11 @@ void AssetListViewModel::CreateNew()
         }
     }
 
+    // sync assets before calling EndUpdate to ensure anything watching for TriggerProperty to change
+    // can find the new definition (i.e. asset editor)
+    auto& pRuntime = ra::services::ServiceLocator::GetMutable<ra::services::AchievementRuntime>();
+    pRuntime.SyncAssets();
+
     FilteredAssets().EndUpdate();
 
     UpdateTotals();
@@ -1865,6 +1875,11 @@ void AssetListViewModel::CloneSelected()
         }
     }
 
+    // sync assets before calling EndUpdate to ensure anything watching for TriggerProperty to change
+    // can find the new definition (i.e. asset editor)
+    auto& pRuntime = ra::services::ServiceLocator::GetMutable<ra::services::AchievementRuntime>();
+    pRuntime.SyncAssets();
+
     FilteredAssets().EndUpdate();
 
     UpdateTotals();

tests/ui/viewmodels/AssetListViewModel_Tests.cpp

@@ -3132,7 +3132,7 @@ TEST_CLASS(AssetListViewModel_Tests)
         Assert::AreEqual(AssetState::Active, pAchievement->GetState());
     }
 
-    TEST_METHOD(TestCreateNew)
+    TEST_METHOD(TestCreateNewAchievement)
     {
         AssetListViewModelHarness vmAssetList;
         vmAssetList.mockUserContext.Initialize("User1", "FOO");
@@ -3189,6 +3189,18 @@ TEST_CLASS(AssetListViewModel_Tests)
         pAsset = vmAssetList.FilteredAssets().GetItemAt(1);
         Expects(pAsset != nullptr);
         Assert::IsTrue(pAsset->IsSelected());
+
+        // both achievements should be loaded in the runtime
+        auto* pClient = vmAssetList.mockRuntime.GetClient();
+        auto* pAch1 = (rc_client_achievement_info_t*)rc_client_get_achievement_info(pClient, 111000001U);
+        Expects(pAch1 != nullptr);
+        Assert::IsNull(pAch1->trigger); // trigger not set until activated
+        Assert::AreEqual({0}, pAch1->public_.points);
+
+        auto* pAch2 = (rc_client_achievement_info_t*)rc_client_get_achievement_info(pClient, 111000002U);
+        Expects(pAch2 != nullptr);
+        Assert::IsNull(pAch2->trigger);
+        Assert::AreEqual({0}, pAch2->public_.points);
     }
 
     TEST_METHOD(TestCreateNewHardcore)