Skip to content

fix(skymp5-server): allow FURN re-activation + silence script-noise logs#2726

Open
Wendys-nitgh wants to merge 1 commit into
skyrim-multiplayer:mainfrom
Wendys-nitgh:crownfall/furn-bypass
Open

fix(skymp5-server): allow FURN re-activation + silence script-noise logs#2726
Wendys-nitgh wants to merge 1 commit into
skyrim-multiplayer:mainfrom
Wendys-nitgh:crownfall/furn-bypass

Conversation

@Wendys-nitgh
Copy link
Copy Markdown

CrownfallRP patches to MpObjectReference.cpp.

  • CheckIfObjectCanStartOccupyThis: drop the FURN early-return so the existing occupant can re-activate the same furniture (stand up / use again). Vanilla Skymp blocked this, preventing all sit / sleep / lean / craft RP mechanics.

  • Demote four info/warn log lines to trace level. Each fired on every cell load or activation with no actionable signal:

    • "occupant is already this object" (non-FURN re-activate)
    • "Script '{}' not found in the script storage" (vanilla scripts not compiled into Skymp's storage)
    • "Skipping non-Sweet scripts for exterior form {:x}"
    • "Skipping script {}" (the explicit blocklist lambda) The script-filter behavior itself is unchanged.

CrownfallRP patches to MpObjectReference.cpp.

- CheckIfObjectCanStartOccupyThis: drop the FURN early-return so the
  existing occupant can re-activate the same furniture (stand up /
  use again). Vanilla Skymp blocked this, preventing all sit / sleep
  / lean / craft RP mechanics.

- Demote four info/warn log lines to trace level. Each fired on every
  cell load or activation with no actionable signal:
    * \"occupant is already this object\" (non-FURN re-activate)
    * \"Script '{}' not found in the script storage\" (vanilla scripts
      not compiled into Skymp's storage)
    * \"Skipping non-Sweet scripts for exterior form {:x}\"
    * \"Skipping script {}\" (the explicit blocklist lambda)
  The script-filter behavior itself is unchanged.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant