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

Prefab instances have no local identifier in their scene file #17

Open
FlaShG opened this Issue Nov 15, 2014 · 12 comments

Comments

Projects
None yet
4 participants
@FlaShG
Copy link
Owner

FlaShG commented Nov 15, 2014

(Original issue title: "Test the tool with Unity 5")

The new scene management inside the editor might break a lot.

@FlaShG FlaShG added the help wanted label Nov 15, 2014

@FlaShG

This comment has been minimized.

Copy link
Owner

FlaShG commented Feb 10, 2015

It seems like the m_LocalIdentfierInFile property (used by the ObjectIDFinder) is 0 in Unity 5 for Objects not differing from their Prefab/Asset counterpart. This breaks the ObjectDictionaries system.

@randomPoison

This comment has been minimized.

Copy link
Contributor

randomPoison commented Feb 17, 2015

Heads up: It looks like Unity 5 is going to have a scene merge tool built in: http://blogs.unity3d.com/2015/02/17/production-workflow-improvements/

@FlaShG

This comment has been minimized.

Copy link
Owner

FlaShG commented Feb 17, 2015

Had to laugh about this line: "Of course all of the above only applies when you are using text serialisation, if you keep scenes and other serialized files in binary format, conflict cannot be resolved and scenes cannot be merged."

@NickolausDS

This comment has been minimized.

Copy link

NickolausDS commented Feb 17, 2015

The article doesn't appear to mention whether their merging tool will be a pro feature or a free one. Is it too early to tell, or would they reference that elsewhere?

@FlaShG

This comment has been minimized.

Copy link
Owner

FlaShG commented Feb 18, 2015

I can't tell. But the line i quoted makes me wonder if their solution is even better than comparable to GitMerge. I mean, form what I understood, their tool only does some pre-merge line magic, and maybe doesn't even help with the actual merging process. I think i will go and ask the folks over at Unity Tech about this.

@FlaShG FlaShG self-assigned this Mar 2, 2015

@FlaShG

This comment has been minimized.

Copy link
Owner

FlaShG commented Mar 4, 2015

Please vote here: http://feedback.unity3d.com/suggestions/re-add-local-identifiers-in-scene-file-for-prefab-instances
Without Unity Tech. changing this, the tool will not work in Unity 5 😞

@randomPoison

This comment has been minimized.

Copy link
Contributor

randomPoison commented Mar 5, 2015

I just tried with the release version of Unity 5, and it seems like m_LocalIdentfierInFile is working. The values seem valid and I can confirm by looking at the serialized file that they are still working. At the least merging prefabs still works, though I haven't tried merging scenes. Perhaps they added it back in before the full release?

@FlaShG

This comment has been minimized.

Copy link
Owner

FlaShG commented Mar 5, 2015

The problem is not in merging perfabs, but having prefab instances in a scene.
Those instances do not have their own id within the scene file.
Read more here: http://forum.unity3d.com/threads/how-to-get-the-local-identifier-in-file-for-scene-objects.265686/#post-1974654

I just downloaded 5.0.0f4, and the problem still exists.

@randomPoison

This comment has been minimized.

Copy link
Contributor

randomPoison commented Mar 5, 2015

Ah, okay. Sorry for the confusion.

@FlaShG

This comment has been minimized.

Copy link
Owner

FlaShG commented Apr 3, 2015

Nothing new concerning the feedback post. It has received enough attention to be on the front page in the "most activity" section for some time though. Thanks everyone for voting!

I hope we can have this problem solved so we can work on the tool again.

@kentmw

This comment has been minimized.

Copy link

kentmw commented Oct 11, 2016

Any update? Is this tool useful in the current state of Unity?

@FlaShG

This comment has been minimized.

Copy link
Owner

FlaShG commented Oct 11, 2016

I haven't heard of this being changed in Unity, and they probably want to push their own solutions anyway. I also haven't tested in quite a while. If someone finds that this issue is (or can be) solved, I would start working on this project again.

@FlaShG FlaShG changed the title Test the tool with Unity 5 Prefab instances have no local identifier in their scene file Jan 17, 2018

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