LocalizedString.GetLocalizedString throws an error if no ChangeHandler is registered

Unity Engine
,
1

Using LocalizedString.GetLocalizedString without setting a ChangeHandler results in the following error. It’s not clear if this is intended behavior.

Exception: RefreshString should be used with RegisterChangeHandler however no change handler has been registered.
UnityEngine.Localization.LocalizedString.RefreshString () (at Library/PackageCache/com.unity.localization@0.5.0-preview/Runtime/Localized Reference/LocalizedString.cs:81)
UnityEngine.Localization.LocalizedString.AutomaticLoadingCompleted (UnityEngine.ResourceManagement.AsyncOperations.AsyncOperationHandle`1[TObject] loadOperation) (at Library/PackageCache/com.unity.localization@0.5.0-preview/Runtime/Localized Reference/LocalizedString.cs:162)
DelegateList`1[T].Invoke (T res) (at Library/PackageCache/com.unity.addressables@1.3.3/Runtime/ResourceManager/Util/DelegateList.cs:69)
UnityEngine.Debug:LogException(Exception)
DelegateList`1:Invoke(AsyncOperationHandle`1) (at Library/PackageCache/com.unity.addressables@1.3.3/Runtime/ResourceManager/Util/DelegateList.cs:73)
UnityEngine.ResourceManagement.ChainOperation`2:OnWrappedCompleted(AsyncOperationHandle`1)
DelegateList`1:Invoke(AsyncOperationHandle`1) (at Library/PackageCache/com.unity.addressables@1.3.3/Runtime/ResourceManager/Util/DelegateList.cs:69)
UnityEngine.ResourceManagement.ResourceManager:Update(Single)
MonoBehaviourCallbackHooks:Update() (at Library/PackageCache/com.unity.addressables@1.3.3/Runtime/ResourceManager/Util/MonoBehaviourCallbackHooks.cs:19)
2

Yes that’s intentional. Refresh string is not needed if you call get localized string as it generates the string each time,(if using formatting).
I’ll try and make the error message clearer.

3

It seems like GetLocalizedString is calling in it’s RefreshString in it’s callback, and calling GetLocalizedString without first initializing it with a ChangeHandler causes it to fail.

So the expected flow is to always provide a ChangHandle before calling GetLocalizedString?

4

Thats really strange. Are you registering a null change handler?
If you just want to use GetLocalizedString then you do not need to use anything to do with ChangeHandlers or RefreshString.

I have added a null check in RegisterChangeHandler.

5

Discovered the issue, had nothing to do with GetLocalizedString at all. I am using LocalizeStringBehaviour on my main menu. By default it loads the entire scene, which has all of the menu GameObjects active, then deactivates all of them but the active menu. What this functionally results in is a OnEnable call, registering the handler (including the AutomaticLoadingCompleted callback), then instantly disables them, clearing the change handler. This launches the loading operation, but then removes the change handler for it, and AutomaticLoadingCompleted is still subscribed, which results in it calling a RefreshString without that handler.

6

Makes sense. We just need to cancel any pending loading operations in ClearChangeHandler. Thanks for spotting this.