AM2PlayGround

AM2PlayGround

2.5k Downloads

[BUG] Eldritch Mind research unlocks wrong spell

Closed this issue ยท 7 comments

commented

Not sure if it's a server specific thing, but I've confirmed with other players that it doesn't seem to work.

commented

Not sure if it's a server specific thing, but I've confirmed with other players that it doesn't seem to work.

Yeah thanks, that was stupid ctrl+c ctrl+v issue. Seems that my API need to be reworked to avoid similar errors in the future...

ELDRITCHMIND = new PGResearchItem("ELDRITCHMIND", CATEGORY_AM2PG,)
researchToSkill.put(ELDRITCHMIND.key, PGSpells.eldritchMind);
INSIGHT = new PGResearchItem("INSIGHT", CATEGORY_AM2PG,)
researchToSkill.put(/*wrong key*/ELDRITCHMIND.key, PGSpells.insight);

Will be fixed in next release.

commented

yeah that sort of thing happens. please just make sure that whatever you write can work retroactively if that makes sense.
Also one thing I'd greatly appreciate is a proper config for the mod, but that's an entirely sense

commented

please just make sure that whatever you write can work retroactively if that makes sense.

I usually test my mod, but since there are no testers other than me, I usually overlook things.
So probably everything works fine... or not... I never know for sure.

Also one thing I'd greatly appreciate is a proper config for the mod, but that's an entirely sense

Config what?
There is [minecraft_root]\config\am2pg.cfg for AM2PG content and also [minecraft_root]\config\AM2\SkillConf.cfg for any AM2 spell parts.
If you need to add something else to the list of configurable things - tell me, don't be silent, I'm not a psychic.

commented

I specifically meant that am2pg spell parts don't show up in skillconf. sorry I should have been more specific

also if you need testing I've got time

commented

I specifically meant that am2 spell parts don't show up in skillconf.

B:VoidDamage=false works like everything else, but since you need to guess spellnames, I'll add them by-deafult with true flag for simplicity.

commented
commented

I am so very sorry. didn't see that there was a reply until now.

okay: the Eldritch mind part works perfectly, even retroactively.
and everything is also appearing in the config.