Welcome to the AFK Mods bug tracker. In order to report an issue, you need to have a VALIDATED account to post one. Once you have followed the link the registration email sent you, please select a project from the drop down menu below. Select “Open New Issue” and fill out the form with as many details as possible.

Issue Data
Status: Closed
Issue Type: Bug Report
Project: Unofficial Skyrim Legendary Edition Patch
Component: Skyrim LE: Vanilla
Category: Placed References
Assigned To: Nobody
Platform: Windows PC
Severity: Low
Votes: 0
Watching: N/A
Opened By RobinHood70 on Nov 23, 2013 2:12 am
Closed By Arthmoor on Apr 8, 2014 8:45 pm
Resolution: Fixed

Issue #14203: Sacred Witness and Sithis disappear

 
The scripting on the skill books Sacred Witness and Sithis causes them to disappear everywhere if they were read in either of the Dark Brotherhood sanctuaries.

Comments

9 comment(s) [Closed]
PrinceShroob said:
 
The script suggests that this is intentional behavior.

RobinHood70 said:
 
Sorry, I missed the ! in the scripting. They'll disappear only if you *haven't* read them in the DB sanctuaries yet, or one that's in a container. Once you've acquired or read one of the available ones, they should appear everywhere.

RobinHood70 said:
 
@PrinceShroob: According to the script, this is supposed to only affect the books that are in the Dark Brotherhood in order to prevent them from duplicating during the DB10 fire. I don't think it's supposed to affect the ones in the Twilight Sepulcher, Deepwood Redoubt, etc., but it does.

Arthmoor said:
 
I don't understand why it would matter if you're messing with one outside of a sanctuary. There's two treasure containers out in the world that will, on load, add one copy of whichever book they're set for. No restriction has been placed on whether or not it should do that again later if you take the book.

<pre class="_prettyXprint _linenums:0">Event OnCellLoad()
if (!dunSkillBooksQST.GetStageDone(dunSkillBooksQSTStage))
if (Self.GetCurrentLocation() == DarkBrotherhoodSanctuaryLocation &amp;&amp; !DB10.GetStageDone(1))
Self.Enable()
ElseIf (Self.GetCurrentLocation() == DawnstarSanctuaryLocation &amp;&amp; DB11.GetStageDone(80))
Self.Enable()
Else
Self.Disable()
EndIf
EndIf
EndEvent</pre>
So I'm pretty sure that Self.Disable() call has no business being there.

RobinHood70 said:
 
It's not the ones in the treasure containers we're worried about, they work fine. It's the ones that are out in the open, like Deepwood Redoubt and the Twilight Sepulchur.

Instead of showing up there all the time, as was almost certainly intended, they only show up after you've already read them in the DB Sanctuary, which makes the ones elsewhere rather pointless to be there.

Edit: Oh, just noticed the line below the code. I haven't really looked into the mechanics, but my guess is that when you move from one DB Sanctuary to the other, if you hadn't read (picked up?) the books, they were supposed to transfer to the new DB Sanctuary and be disabled at the old one. Though, if that's the case, it seems like a lot of work to me for very little gain. Why not just leave them enabled at both DBs? It's not like a 70ish gp book is a big deal past about level 2.

RobinHood70 said:
 
I'm pretty sure all that needs to be done is to remove the inherited script from the books at all locations where they're not in containers (and not in one of the DB Sanctuaries).

Arthmoor said:
 
Honestly I'm not seeing the point in the script at all. As far as I can tell, no other skill book has this nonsense going on.

Unless the intent was to be that the DB only has one copy of "Sithis" or something. If that's the case then just the silly disable statement needs to come out.

RobinHood70 said:
 
Yeah, that's my impression of what was supposed to happen. Like I said, it seems like a lot of effort for so little.

Arthmoor said:
 
Fixed for USKP 2.0.4.

Script was rewritten so that it will only affect the two sanctuary based copies.

Showing Comments 1 - 9 of 9