dialogic icon indicating copy to clipboard operation
dialogic copied to clipboard

Allow multiple characters with same name

Open Jowan-Spooner opened this issue 2 years ago • 4 comments

This is important for saving events (mainly character and text).

Also the picker should be improved so that they can be easily distinguished.

Jowan-Spooner avatar Jul 30 '22 20:07 Jowan-Spooner

also mentioned in #1055 while building the 1.x -> 2.0 converter.

I'm sure im not the only one that currently or future wants to use multiple characters with the same name in different folders, but to copy and paste my specific reasons from the discord: "so for my RPG its for organization, I got folders for each of the towns and each of the buildings for that town, and like as the game progresses some NPCs are gonna be found in multiple places so I had intended to have a corresponding character for that location. this is mainly for my integrity tester, so it can check each one of the maps and tick of that the timelines, characters, variables, and anythign else i attach to each NPC on that map all exist and have the same uniform path

for my VN, i'm just using so many portraits that the portraits list was starting to get unweildly, so I broke them up into separate characters by bracnh of the VN, so there's duplicates of a few key characers in each branch's subfolder, with only the portraits for that specific branch assigned to it "

exelia-antonov avatar Jul 30 '22 20:07 exelia-antonov

A general rule of thumb is to use IDs in the form of integers and save it to the files. And maybe not stop at characters, but giving everything an ID.
Also opens up the road to performance improvements.
If that is desirable, then that would leave the question on whether one global ID counter, or one counter per type (like database tables do).

MJacred avatar Aug 06 '22 09:08 MJacred

im gonna leave a note here for later, with the get_name added in #1154, I think I actually have an idea on how to implement this.

exelia-antonov avatar Aug 24 '22 18:08 exelia-antonov

not actually implmeneted yet, #1161 was just preliminary change to make this one possible

exelia-antonov avatar Aug 28 '22 15:08 exelia-antonov

Oh whoops missed closing this one out with #1207

exelia-antonov avatar Sep 28 '22 18:09 exelia-antonov