Jump to content


Photo

Paladin Inquisitor bug


8 replies to this topic

#1 Dakk

Dakk
  • Members
  • 784 posts
  • Gender:Male

Posted 14 January 2012 - 02:05 PM

Inquisitor Mind Shield
Description: The Inquisitor does not receive protection from Charm spells (Mind Shield) at level 1.
Steps to reproduce:
1. Start a BGT game (with FixPack)
2. Choose BG1
3. Create an Inquisitor
4. Level up to level 2
Expected behaviour: The Inquisitor should start at level 1 with Mind Shield, and the icon displayed, just as the Cavalier starts with all its immunities (Mind Shield, Resist Fear, Protection from Poison) and the Undead Hunter Negative Plane protection).
Actual behaviour: The Inquisitor gains Mind Shield at level 2.
Other: From all the sources I can gather, immunity to charm spells (= Mind Shield) is intrinsic to the Inquisitor class and should be active from level 1 - just like it works for all other Paladin kits.

Weidu:
Spoiler


#2 aVENGER_(RR)

aVENGER_(RR)

    Sneaksie!

  • Fixpackers
  • 1008 posts
  • Gender:Male

Posted 14 January 2012 - 10:40 PM

Easily fixed.

// Fix Inquisitors not getting charm immunity at level 1

COPY_EXISTING ~clabpa03.2da~ ~override~ // Inquisitor ability table
  SET_2DA_ENTRY  3 2 3 ~****~
  SET_2DA_ENTRY 10 1 3 ~AP_SPCL233b~
  PRETTY_PRINT_2DA
BUT_ONLY_IF_IT_CHANGES

Retired modder

Rogue Rebalancing - Author
aTweaks - Author
Wizard Slayer Rebalancing - Co-Author
BG2 Fixpack - Contributor

#3 Dakk

Dakk
  • Members
  • 784 posts
  • Gender:Male

Posted 15 January 2012 - 05:29 AM

Finally I've made it big in the city - a contribution to the BG2 Fixpack (ok, I didn't actually code anything, but still ;))! Now I can retire :D

I'm surprised that's never been spotted, and thank you for coding it up aVENGER.

#4 devSin

devSin
  • Fixpackers
  • 3017 posts
  • Gender:Male

Posted 15 January 2012 - 01:05 PM

Are we sure this is a bug? Does it say anywhere that inquisitor kits start at Level 1 with charm immunity? If it was in fact intended to be a Level 2 buff, would they even note it (since you start at Level 7 anyway)?

I'm inclined not to mess with whatever they decided in the kit tables (unless it explicitly says it's a L1 benefit), especially since there's no source material that exactly matches the kits they implemented to know what they were thinking.

Edited by devSin, 15 January 2012 - 01:06 PM.


#5 Dakk

Dakk
  • Members
  • 784 posts
  • Gender:Male

Posted 15 January 2012 - 01:14 PM

The closest thing to source material would be the Baldurs Gate 2 manual, which states:
PALADIN
Special Abilities:Weapon Specialization, Lay Hands, Turn Undead, +2 all Saving Throws, Protection from Evil, Detect Evil

CAVALIER
Special Abilities: Bonus to hit demons and dragons, immune to fear and poison. 20% resistance to fire/acid.

INQUISITOR
Special Abilities: Immune to hold and charm. True sight, dispel magic.

UNDEAD HUNTER
Special Abilities: Immune to hold and level drain. Bonus to hit and damage vs undead.

All kits start with all immunities, capabilities and innates at level 1. LH, PfE and DE evolves with level, but are there from level 1. The Inquisitor also gains Dispel Magic and True Sight once per day per every 4 levels - and starts at 1st level with one use each. Having them gain Mind Shield at level 2 is contrary to what the manual says their Special Ability is.

EDIT: Could there potentially be an issue that the Inquisitor would have ten rows in CLABPA03 (Mind Shield would become ABILITY10)? Although CLABM01 has 14 rows so... Also, what is the ABILITYA in CLAPBPA03? :)

Edited by Dakk, 15 January 2012 - 01:21 PM.


#6 devSin

devSin
  • Fixpackers
  • 3017 posts
  • Gender:Male

Posted 15 January 2012 - 01:29 PM

The row count cap is higher (it's an arbitrary amount); the row labels are discarded ('A' is hex for 10).

If the inquisitor filled up his L1 column with nine abilities, then I'd agree that it was probably made L2 simply to keep the row count under 10 (but it can be higher, so it doesn't matter).

#7 Dakk

Dakk
  • Members
  • 784 posts
  • Gender:Male

Posted 15 January 2012 - 01:53 PM

Ah I see. Thank you for explaining the A = 10! Makes sense.

What do you think, an (over)cautions programmer was afraid of going over 9 rows? Regarding actual developer intent, I think the lack of "gain/acquire/receive immunity to charm" is telling of what was intended.

PS. In case I sound confrontational at times, I apologise. I'm really not :)

#8 devSin

devSin
  • Fixpackers
  • 3017 posts
  • Gender:Male

Posted 15 January 2012 - 02:14 PM

I think it was just done for formatting (they didn't have to worry about characters < L7). It's possible they didn't want to test whether more than 9 rows would work (or maybe it originally didn't).

As long as it works, I don't see issue with moving it to Level 1. Normal players won't ever see it.

#9 Dakk

Dakk
  • Members
  • 784 posts
  • Gender:Male

Posted 15 January 2012 - 02:30 PM

Cheers!
Yes, it would only show up in a BGT (Tutu?) game in BG1. But that's possibly the majority of all games in BGT, so it's a nice touch for Candlekeep (and any Sirines lurking nearby) :)



Reply to this topic



  


0 user(s) are reading this topic

0 members, 0 guests, 0 anonymous users