SetSaveData and GetSaveData use wrong addon id when mounting multiple addons

[girlglock]

Im not sure if this is the intended behavior but when mounting an addon like ie map_workshop id1 de_nuke in order to load onto de_nuke with additional assets (like a prefab with the script etc) SetSaveData will save into the save_id1.txt file

however if we were to mount two addons like ie map_workshop id1,id2 some_cool_other_map where id1 is again our prefab with the script and id2 is some other workshop map, SetSaveData will save into the save_id2.txt file. (also the order in which we mount the addons does not matter for saving data it seems id1,id2 or id2,id1 will both use the id of the some_cool_other_map map)

this will potentially override the id2's own save file if it had any and make it impossible to store stuff synced between maps

my suggestion is to somehow make the script aware of what addonid its running from and always use that id for SetSaveData and GetSaveData

alternatively use the last mounted id, which in the case of map_workshop id1,id2 some_cool_other_map would be id2 for the save

[girlglock]

after investigating this a bit more it seems that some addons are being favored over others for SetSaveData

initially i thought its just taking the bigger file size as the preferred one because it appeared to be the case with 5 addons i mounted.. until i mounted a 6th one which was smaller than the other 5 and suddenly cs_script started to favour it (also the order in which we mount the addons seems to not play any role at all, cs_script will still pick favorites)

[girlglock]

update: seems like the favorable addonid is always the oldest (out of the mounted) ones the user has subscribed to based of "Date modified" in the \steamapps\workshop\content\730 dir

i might be wrong tho but every time the script has favored the older one based on the addon local file age