[EmandM]

Purpose of this PR

Ensure required NetworkManager methods are protected virtual so the base versions can be called when overridden.

Jira ticket

UUM-159597

Changelog

• Fixed: Resolves exceptions being thrown when a derived NetworkManager implements their own Awake.

Documentation

• No documentation changes or additions were necessary.

Testing & QA (How your changes can be verified during release Playtest)

Functional Testing

Manual testing :

• Manual testing done

Automated tests:

• Covered by existing automated tests
• Covered by new automated tests

Does the change require QA team to:

• Review automated tests?
• Execute manual tests?
• Provide feedback about the PR?

If any boxes above are checked the QA team will be automatically added as a PR reviewer.

Up-port

Needs up-port

Backports

n/a

[u-pr]

💡 Harness Review

The logger fallbacks consistently cover all three startup modes, and the lifecycle methods now allow derived managers to invoke their base implementations.

_{Reviewed commit e6693e9}

_{🤖 Helpful? 👍/👎}

[u-pr]

A future OnDestroy override that disposes a native allocation can dispose it twice when a listening derived manager quits. OnApplicationQuit manually calls OnDestroy, which now dispatches to the override. Unity also invokes OnDestroy when destroying the active component. Previously, the manual call reached only NetworkManager's private implementation. Extract the manual shutdown work into a private method. Call that method from OnApplicationQuit and OnSceneUnloaded. Reserve virtual OnDestroy for Unity's destruction callback.

_{🤖 Helpful? 👍/👎}