([BG1] Inconsistancy) Ulcaster's Book (0813)

Balquo

Current:
He wants you to retrieve a book from the ruins. This same book is also available from Gullykin and if you return with it he will accept it and you will complete the quest.

Expected:
For him to accept the book from the ruins only. Making it unique will solve that.

Using the same code from @CamDawg posted here

COPY_EXISTING ~book54.itm~ ~override/ulbook54.itm~ // History of the Sisters of Light and Darkness
 // SAY 0x0c ~Ulcaster's Dusty History Book~ 

COPY_EXISTING ~ar3901.are~ ~override~
  LPF REPLACE_AREA_ITEM STR_VAR
    old_item = book54
    new_item = ulbook54
    END

COPY_EXISTING ~ulcast.dlg~ ~override~	
  DECOMPILE_AND_PATCH BEGIN
    REPLACE_TEXTUALLY ~Book54~ ~Ulbook54~
  END

NOTE: The book is called "History of the Sisters of Light and Darkness" but maybe you want to change that a little since it will be unique to Ulcaster. I've given an example in the code but have commented it out... please feel free to change it.

SethDavis

Potentially fixed - @Balquo 's fix has been applied.

Tanthalas

Confirmed fixed for me.

1. The book in the Ulcaster ruins is Balquo's special book.
2. Delivering the special book to Ulcaster completes the quest.
3. Tried to give him the old History of the Sisters of Light and Darkness book and he wouldn't accept it.

NOTE:
@Dave
If possible could you write a new text for this book (Ulbook54)? Currently it has the title Balquo put in but it still has the old text of the "History of the Sisters of Light and Darkness" book.

Balquo gave the book the name "Ulcaster's Dusty History Book", so I don't think the new text needs to be anything fancy. You could probably change the name of the book if you thought it appropriate.

Avenger_teambg

Why do you need to change the internal text? It is supposed to be the same book (just with Ulcaster's own fingerprints)

Balquo

I just put that name in as a suggestion. You can change it of course.

Tanthalas

@Avenger_teambg
Well, I guess the text doesn't need to be changed.

Bhryaen

lol Ulcaster's Dusty History Book.

Funny stuff. I say we keep it the @Balquo way. :-)

Confirmed Fixed.

Avenger_teambg

OBSERVED:
Somehow the "most important" line is gone from the Ulcaster dialog: AddexperienceParty(1000)

FIX:
We need our XP. Add this line back.

@Balquo - do you know whom to blame?

Tanthalas

That's strange, it was working perfectly in the previous build.

Avenger_teambg

Yeah, and none of the suggested patches were about removing the xp line

Current dlg:

Vanilla in bg1:

Tanthalas

@Bhryaen

I can't move this discussion at the moment, but when you can, move it to confirmed not fixed as it got broken in Build 0803. Can probably merge this with the thread that Avenger_teambg created:

http://forum.baldursgate.com/discussion/2683/bgee-ulcaster-redux-no-xp

Balquo

I've no idea how that happened. The code doesn't even touch it. Anyway, here is a fix:

COPY_EXISTING ~ulcast.dlg~ ~override~	
  DECOMPILE_AND_PATCH BEGIN
    REPLACE_TEXTUALLY ~TakePartyItem("Ulbook54")~ ~AddexperienceParty(1000) TakePartyItem("Ulbook54")~
  END

SethDavis

Potentially fixed - the line has been restored. I must have accidentally had that line highlighted when i started typing

Avenger_teambg

Well, not all line will be missed that much, so watch what you highlight.

Avenger_teambg

Confirmed fixed - i think this is perfect now.
Anyone tried to give him the idol and the ancient armor?

Tanthalas

Confirmed Fixed in Build 0815

You now get the proper exp again.

@Avenger_teambg
I just tried that. CLUAConsole'd both items, went to AR4100 to kill the Doomsayer, came back and got the sword from talking to Ulcaster.

I never did this before, does he just give you the sword without any special text?

Avenger_teambg

Yes. I don't know how anyone would find this out (short of reading the dlg), or reading dudleyville

LadyRhian

I just tried to give him the book from the ruins, but he just repeats the line about the old school and doesn't react or give me anything.