Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Updated network transform message #3743

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

Open
Squashi11 wants to merge 2 commits into Unity-Technologies:develop-2.0.0
base: develop-2.0.0
Choose a base branch
Loading
from Squashi11:patch-1

Conversation

@Squashi11
Copy link

@Squashi11 Squashi11 commented Oct 26, 2025
edited
Loading

Updated network transform message to include the missing reference from this problem :

https://discussions.unity.com/t/targeted-network-transform-does-not-exist-but-it-does-exist-error-question/1637568/5

Purpose of this PR

Jira ticket

???

Changelog

Changed the way the error is displayed in console so it now correctly shows the problematic Network Objects name

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.

Backports

@Squashi11 Squashi11 requested a review from a team as a code owner October 26, 2025 17:45
Copy link

unity-cla-assistant commented Oct 26, 2025
edited by cla-assistant-unity bot
Loading

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Author

My first try at making a merge request, correct me in stuff please

Copy link
Member

@EmandM EmandM left a comment
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rather than adding information with no context, it's best if the existing debug log is updated to include the missing information.

Thank you for your contribution!

if (transform == null)
{
Debug.LogError($"[{nameof(NetworkTransformMessage)}][Invalid] Targeted {nameof(NetworkTransform)}, {nameof(NetworkBehaviour.NetworkBehaviourId)} ({networkBehaviourId}), does not exist! Make sure you are not spawning {nameof(NetworkObject)}s with disabled {nameof(GameObject)}s that have {nameof(NetworkBehaviour)} components on them.");
Debug.LogError(networkObject.gameObject.name);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I appreciate that this fixes the issue of finding the name of the game object, however that name is being logged without any context. We prefer to ensure that our error logs have enough context to understand what the error message is trying to communicate.

A better fix would be to edit the log underneath and add the gameobject name into the existing log.

Squashi11 reacted with hooray emoji
Copy link
Author

Hello just checking in on this, doesnt seem like it made its way into the latest version so far. Seems like an easy fix to me
{D10EA356-0DCA-4A6F-86F3-673557FD4F49}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@EmandM EmandM EmandM left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /