Skip to content

fix: AnticipatedNetworkVariable not updating previous value [MTTB-1036] #3306

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

NoelStephensUnity
Copy link
Collaborator

@NoelStephensUnity NoelStephensUnity commented Feb 19, 2025

This fixes the issue where the AnticipatedNetworkVariable previous value was never updated on the non-authority/write permission instances when the OnAuthoritativeValueChanged callback was invoked.

MTTB-1036

Back port: #3322

Changelog

  • Fixed: Issue where the AnticipatedNetworkVariable previous value was never updated on the non-authority/write permission instances when the OnAuthoritativeValueChanged callback was invoked.

Testing and Documentation

  • Includes integration test NetworkVariableAnticipationTests.PreviousValueIsMaintainedProperly.
  • No documentation changes or additions were necessary.

Fixing AnticipatedNetworkVariable issue where the previous value was not being updated for non-authority/write permission instances.
adding comment
@NoelStephensUnity NoelStephensUnity added the port:1.x-needed This issue needs to be ported to 1.X branch label Feb 19, 2025
Adding a test to validate this fix.
adding change log entry.
@NoelStephensUnity NoelStephensUnity marked this pull request as ready for review February 24, 2025 21:22
@NoelStephensUnity NoelStephensUnity requested a review from a team as a code owner February 24, 2025 21:22
@NoelStephensUnity NoelStephensUnity added the port:1.x-completed This issue was ported to 1.X branch label Feb 24, 2025
@NoelStephensUnity NoelStephensUnity enabled auto-merge (squash) February 25, 2025 19:20
@NoelStephensUnity NoelStephensUnity merged commit 096c22c into develop-2.0.0 Feb 25, 2025
27 checks passed
@NoelStephensUnity NoelStephensUnity deleted the fix/anticipatednetworkvariable-not-updating-previousvalue branch February 25, 2025 20:34
NoelStephensUnity added a commit that referenced this pull request Feb 25, 2025
…6] (back port) (#3322)

Back-port of #3306.

This fixes the issue where the `AnticipatedNetworkVariable` previous
value was never updated on the non-authority/write permission instances
when the `OnAuthoritativeValueChanged` callback was invoked.

[MTTB-1036](https://jira.unity3d.com/browse/MTTB-1036)

<!-- Add RFC link here if applicable. -->

## Changelog

- Fixed: Issue where the `AnticipatedNetworkVariable` previous value was
never updated on the non-authority/write permission instances when the
`OnAuthoritativeValueChanged` callback was invoked.

## Testing and Documentation

- Includes integration test
`NetworkVariableAnticipationTests.PreviousValueIsMaintainedProperly`.
- No documentation changes or additions were necessary.

<!-- Uncomment and mark items off with a * if this PR deprecates any
API:
### Deprecated API
- [ ] An `[Obsolete]` attribute was added along with a `(RemovedAfter
yyyy-mm-dd)` entry.
- [ ] An [api updater] was added.
- [ ] Deprecation of the API is explained in the CHANGELOG.
- [ ] The users can understand why this API was removed and what they
should use instead.
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
port:1.x-completed This issue was ported to 1.X branch port:1.x-needed This issue needs to be ported to 1.X branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants