-
June 27th, 2022, 20:06 #21
I will verify that your safety checks work and Polymorphism no longer has any issues when having damage failed concentration save expiring the concentration effect and triggering the polymorph change of nodes. So thanks for that. I'll have to recheck all my extensions individually as addNPC is now giving some nil call errors in things like Death Indicators. I could tell lots of things got shifted around so will have to sort through and figure out how to get things back to working.
Are there any other major extension breaking changes to TEST coming up before july 12?Free(Forums/Forge) Extension(FGU 5E):
Paid (Forge) Extension(FGU 5E):
-
June 27th, 2022, 20:07 #22
Supreme Deity
- Join Date
- Mar 2007
- Posts
- 20,665
Regarding post #17 and #18, I'm talking about removing CombatManager.setCustomDeleteCombatantEffectHandle r call in your code, then updating the replacement function to look like this:
Code:function onDeleteCombatantEffectEvent(nodeEffectList) .... Do your normal stuff .... .... Call your polymorphism specific code that you used to register with setCustomDeleteCombatantEffectHandler .... end
JPG
-
June 27th, 2022, 20:12 #23
Supreme Deity
- Join Date
- Mar 2007
- Posts
- 20,665
Regarding post #21, we're working on streamlining the process for registering new record types for combat, in order to support vehicles in combat for 5E. (i.e. upcoming Spelljammer release).
For the addNPC, there are now hooks to get/set the record type handlers, as well as postAdd record type handlers. (See 5E ruleset for CombatRecordManager.setRecordTypePostAddCallback).
The next step for adding 5E vehicle combat support is updating the CT UI to support vehicles; as well as updating all actor/roll functions to correctly handle vehicles as actors/combatants. I'm not sure if any of those changes will be in test yet, but I'm leaning towards after this Test release doing an immediate second cycle to finalize for Spelljammer (which releases in mid-August).
Regards,
JPG
-
June 27th, 2022, 20:13 #24
No need your guards in those areas fixed all the problems. Now I'm hunting why all my NPC action lines are blank in the NPC change to Ape - and why on change back 1/2 are blank in the original NPC it was polymorphing from. I'm assuming this last load of changes has shifted all sorts of things around messing up the overrides involving those areas - which means a number of my extensions will need to be rettested (and likely have this same issue). I'm guessing someone shifted the npc record xml or lua stuff around rendering any overrides bugged.
Free(Forums/Forge) Extension(FGU 5E):
Paid (Forge) Extension(FGU 5E):
-
June 27th, 2022, 20:16 #25
Supreme Deity
- Join Date
- Mar 2007
- Posts
- 20,665
The overrides are continuing to work in the layered rulesets (because I specifically tested for layered rulesets which have not been updated); so not sure why yours wouldn't work.
Can you provide another specific campaign example? Or will the previous example show the issue?
Regards,
JPG
-
June 27th, 2022, 20:27 #26
The sample will work fine - when the kobold is turned into a Baboon just hit CT (sword) to bring up the actions list - they will all be blank. Transform the baboon back to its original kobold entry and you will see all or some blank (in my usual campaign I have more complex NPCs with spells and things and not all the action lines were blank).
BlankActionLines.pngFree(Forums/Forge) Extension(FGU 5E):
Paid (Forge) Extension(FGU 5E):
-
June 27th, 2022, 20:30 #27
Note - a lot of functions got deprecated and shifted around when I did a diff on latest code drop in TEST - so this may all be tied to stuff I just have to adjust too. For sure I've seen error in another extension with addNPC being nil which was not there last release.
Free(Forums/Forge) Extension(FGU 5E):
Paid (Forge) Extension(FGU 5E):
-
June 27th, 2022, 21:58 #28
How is line 347 getting a stack overflow after this last TEST release? I fixed one of my new issues caused by CombatManager2.addNPC being removed - but this one I'm not understanding. The function should still be there and working as it did previously.
Code:header: Line 24: local saveaddNPC = nil; onInit() stuff: Line 151: saveaddNPC = CombatManager.addNPC; Line 214: CombatManager.addNPC = saveaddNPC; Functionality: Line 347: local nodeNewCTEntry = saveaddNPC(sClass, nodeNPC, sName);
local nodeCT = CombatManager.addNPC(sClass, nodeNPC, sName);
Something is broken in this.
[Postedit - did not find it - error occurs in new way this is working in TEST - but its a stack overflow where a print shows it calling it for the same operation over and over till it gets the stack overflow]Last edited by SilentRuin; June 28th, 2022 at 10:34.
Free(Forums/Forge) Extension(FGU 5E):
Paid (Forge) Extension(FGU 5E):
-
June 27th, 2022, 22:37 #29
Supreme Deity
- Join Date
- Mar 2007
- Posts
- 20,665
Actually, it does get returned if you're overriding the old CombatManager.addNPC:
Code:function addNPC(sClass, nodeNPC, sName) local tCustom = { sRecordType = "npc", nodeRecord = nodeNPC, sClass = sClass, sName = sName }; CombatRecordManager.addNPC(tCustom); return tCustom.nodeCT; end
-
June 28th, 2022, 01:09 #30
[QUOTE=Moon Wizard;654296]Actually, it does get returned if you're overriding the old CombatManager.addNPC:
[Postedit - did not find it - error occurs in new way this is working in TEST - but its a stack overflow where a print shows it calling it for the same operation over and over till it gets the stack overflow]Last edited by SilentRuin; June 28th, 2022 at 10:34.
Free(Forums/Forge) Extension(FGU 5E):
Paid (Forge) Extension(FGU 5E):
Thread Information
Users Browsing this Thread
There are currently 1 users browsing this thread. (0 members and 1 guests)
Bookmarks