Database not updated when an asset is renamed about project-curator HOT 7 CLOSED

I'm very close to having finished refactoring it to use GUIDs, so you can check out the diff when it's done.

from project-curator.

This also applies to assets that are moved.

from project-curator.

This is coming up a fair bit in my team, so I'm happy to schedule a fix. Are you open to a PR @ogxd?

from project-curator.

Hi @rhys-vdw !
It is supposed to update when an asset is moved (and renamed I guess) but I'll check the reproduction steps when I have some time :)

Yes feel free to make a PR if you think you know where the issue lies!

from project-curator.

Good timing. I just started looking into it. I'll let you know how I go.

from project-curator.

It seems a fundamental issue here is that your AssetInfo class stores a list of paths instead of asset GUIDs. Is there a reason why this is the case?

If AssetInfo stored GUIDs then there would be no need to handle the move case at all, because there is no change to references resulting in a file move.

from project-curator.

Mhhh it has been a while since I wrote this, I don't recall the reasons for using a path instead of the Guid. I'll have a look at it! There are probably many improvements that could be done.

from project-curator.