Summoning Planetar/Deva - Bug?

[devSin]

Turns out to be a combination of the above:

COPY_EXISTING ~sword01.cre~  ~override/plangood.cre~
 WRITE_SHORT 0x0028 0x7f3b
 WRITE_BYTE  0x0275 0x05

results in the celestial cap being observed; changing only the gender or only the animation doesn't; renaming any of plangood.cre or spplan.eff or spwi924.spl results in no cap.

(sigh) I'm surprised they didn't also checksum the files and compare against the default (expected) version.

 

OMG!!! 1-byte difference in plansuck CRE!!! NO CAP!!!

[Nythrun]

I was so hoping we'd get to add a plansuck.cre

[Topaze]

I assume you've found the source of the problem. Could you please explain it in layman's terms and, more importantly, suggest how to get rid of it?

[devSin]

You can delete PLANGOOD.CRE, PLANEVIL.CRE, DEVAGOOD.CRE, and DEVAEVIL.CRE from your override folder to restore the original behavior (optionally, you can simply choose to summon only one at a time until this is fixed).

 

Basically, the engine checks the gender (which we apparently change) when determining if the celestial is subject to the one-per-party rule; since we've changed it, the rule no longer applies (changing the gender back to 5 should get things working again; Nythrun may be along to give you TP2 code to run to reset the genders if you're comfortable doing it, or you could use an editor if you have access and know how to edit the resources -- otherwise, deleting the files from override is the quickest way to restore the original behavior).

[Topaze]

I see. Thanks a lot!

[Nythrun]

COPY_EXISTING ~plangood.cre~ ~override~
		  ~planevil.cre~ ~override~
		  ~devaevil.cre~ ~override~
		  ~devagood.cre~ ~override~
 WRITE_BYTE 0x0275 0x05

 

Running this at any point after the fixpack will clean this up. You could also cut and paste this into the fixpack any time after the gender patch (which runs from line 6,872 to line 6,958) and reinstall the fixpack. Alternatively, you can comment out the changes in the existing patch thusly, and reinstall:

// gender fixes
COPY_EXISTING ~AMMONK08.CRE~ ~override~
		  ~AR18ARCH.CRE~ ~override~
		  ~C6CATTI.CRE~  ~override~
		  ~C6CATTI2.CRE~ ~override~
		  ~C6CLER3.CRE~  ~override~
		  ~C6DEL.CRE~	~override~
		  ~C6GUEN2.CRE~  ~override~
		  ~CEFALD02.CRE~ ~override~
		  ~CHICKDEF.CRE~ ~override~
		  ~CHICKE.CRE~   ~override~
		  ~CHICKER.CRE~  ~override~
		  ~COPLION.CRE~  ~override~
		  ~COW.CRE~	  ~override~
		  ~CSHECK1.CRE~  ~override~
		  ~CSHECK3.CRE~  ~override~
		  ~CSSUPP2.CRE~  ~override~
		  ~DADROW10.CRE~ ~override~
		  ~DADROW23.CRE~ ~override~
		  ~DAGHAUN1.CRE~ ~override~
		  ~DAGHAUN2.CRE~ ~override~
		  ~DEADCOW1.CRE~ ~override~
		  ~DEADCOW2.CRE~ ~override~
		  ~DEER01.CRE~   ~override~
		//  ~DEVAGOOD.CRE~ ~override~
		//  ~DEVAEVIL.CRE~ ~override~
		  ~DEVAST01.CRE~ ~override~
		  ~DEVMON01.CRE~ ~override~
		  ~DPFEMALE.CRE~ ~override~
		  ~DPSTAT05.CRE~ ~override~
		  ~DROFOD04.CRE~ ~override~
		  ~DROW04.CRE~   ~override~
		  ~DSBODY01.CRE~ ~override~
		  ~FANGEL01.CRE~ ~override~
		  ~FINSOL01.CRE~ ~override~
		  ~GEMCH01.CRE~  ~override~
		  ~GEMCH02.CRE~  ~override~
		  ~GORSTA05.CRE~ ~override~
		  ~HLSHYR.CRE~   ~override~
		  ~HSPECTR1.CRE~ ~override~
		  ~JADE1.CRE~	~override~
		  ~JAHEI1.CRE~   ~override~
		  ~JAREV1.CRE~   ~override~
		  ~JAMERONI.CRE~ ~override~
		  ~MGKHOL01.CRE~ ~override~
		  ~NCHICK.CRE~   ~override~
		  ~NCOW.CRE~	 ~override~
		  ~PIRFSH02.CRE~ ~override~
		  ~PIRSAL02.CRE~ ~override~
		  ~PLANET01.CRE~ ~override~
		//  ~PLANGOOD.CRE~ ~override~
		//  ~PLANEVIL.CRE~ ~override~
		  ~PLANWISH.CRE~ ~override~
		  ~PPBODHI3.CRE~ ~override~
		  ~PPMAG01.CRE~  ~override~
		  ~PPVALEN.CRE~  ~override~
		  ~REMAGE01.CRE~ ~override~
		  ~SENDAI7.CRE~  ~override~
		  ~SUENDEL2.CRE~ ~override~
		  ~SUENDEL4.CRE~ ~override~
		  ~SUENDEL6.CRE~ ~override~
		  ~SUENDEL8.CRE~ ~override~
		  ~TOBPAR02.CRE~ ~override~
		  ~TRFTOW04.CRE~ ~override~
		  ~TRSKIN01.CRE~ ~override~
		  ~UDDOOR06.CRE~ ~override~
		  ~UDDROW35.CRE~ ~override~
		  ~VAMANC01.CRE~ ~override~
		  ~VAMEMI01.CRE~ ~override~
		  ~VAMMAT01.CRE~ ~override~
		  ~VAMPAT01.CRE~ ~override~
		  ~VAMVER01.CRE~ ~override~
		  ~VAMPAER.CRE~  ~override~
		  ~VAMPJAH.CRE~  ~override~
		  ~VAMPVIC.CRE~  ~override~
		  ~VVBODHI.CRE~  ~override~
		  ~VVDEL.CRE~	~override~
		  ~VVPARIS.CRE~  ~override~
		  ~VVSALIA.CRE~  ~override~
		  ~VVTANOV.CRE~  ~override~
		  ~WPWENCH1.CRE~ ~override~
		  ~YARMY01.CRE~  ~override~
		  ~YSMAGE02.CRE~ ~override~
 PATCH_IF (SOURCE_SIZE > 0x2d3) THEN BEGIN // protects against invalid files
WRITE_BYTE 0x0275 2 // female
 END
 BUT_ONLY_IF_IT_CHANGES

 

planwish is the evil planetar that can crash your party if you haggle naively with the Wish Djinni of Sharp Dealings; it doesn't need to be changed back.

 

It's rather charming that it took this long for someone to notice - accidentally - that Summoning Cap: The Sequel was broken

[Salk]

It's soon Xmas and I am beginning to wish for some strings to be pulled around the Mods I love...

[CamDawg]

Thanks for the research everyone.

 

Turns out to be a combination of the above:

COPY_EXISTING ~sword01.cre~  ~override/plangood.cre~
 WRITE_SHORT 0x0028 0x7f3b
 WRITE_BYTE  0x0275 0x05

results in the celestial cap being observed; changing only the gender or only the animation doesn't; renaming any of plangood.cre or spplan.eff or spwi924.spl results in no cap.

Presumably the anti-solar animation works as well, yes?

 

devagood, devaevil, plangood, and planevil have been dropped from the gender changes.

 

It's possible to change HLA tables in midstream,

How? HLA tables are set by class/kit, and fallen paladins and rangers are not handled as a true kit by the engine (they get their own CLAB, but that's about it). You could force the issue with a kit that gets applied by their fallen CLAB, but that'll probably just kill any restoration of their original class/kit.

[Nythrun]

I was thinking more along the lines of four kits, and draw&quartering the RegainPaladinhood() calls into four blocks with a ChangeKit in there first.

 

It'll work, it just stinks. I'd wait for someone to code the

<<<<<<<<yousuck.baf
IF
 FallenPaladin(LastSummonerOf(Myself))
THEN
 RESPONSE #100
DisplayStringHead(Myself,~God hates you and so do I!~)
ReallyForceSpell(LastSummonerOf(Myself),JUST_TEN_DAMAGE)
DestroySelf()
END
>>>>>>>>
EXTEND_BOTTOM devagood.bcs yousuck.baf

tweak instead.

[WizWom]

EXTEND_BOTTOM devagood.bcs yousuck.baf

tweak instead.

heh... but it should EXTEND_TOP since you want it to happen with priority. Be a shame to let the paladin wait until after the Deva has smote the evil already.