Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"Stealing" checks are too aggressive #77239

Open
Demivan opened this issue Oct 21, 2024 · 14 comments
Open

"Stealing" checks are too aggressive #77239

Demivan opened this issue Oct 21, 2024 · 14 comments
Labels
(S2 - Confirmed) Bug that's been confirmed to exist

Comments

@Demivan
Copy link
Contributor

Demivan commented Oct 21, 2024

Describe the bug

You can no longer read signs at the exodii base.

You can no longer move creates and lockers at refugee center. This prevents completing the first merchant quest to kill zombies (Clear Back Bay).

Attach save file

N/A

Steps to reproduce

  1. Try grabbing crates at the refugee center

  2. This makes everyone hostile

  3. Try activating signs at the exodii base to read them.

  4. This triggers a warning too

Expected behavior

Reading signs should be allowed.

Moving some crates and lockers at the refugee center should be allowed.

Screenshots

image
image

Versions and configuration

  • OS: Windows
    • OS Version: 10.0.26100.2033 (24H2)
  • Game Version: 2357a3c [64-bit]
  • Graphics Version: Tiles
  • Game Language: System language []
  • Mods loaded: [
    Dark Days Ahead [dda],
    Disable NPC Needs [no_npc_food],
    Portal Storms Ignore NPCs [personal_portal_storms],
    Slowdown Fungal Growth [no_fungal_growth]
    ]

Additional context

No response

@Demivan Demivan added the (S1 - Need confirmation) Report waiting on confirmation of reproducibility label Oct 21, 2024
@AudBobb
Copy link
Contributor

AudBobb commented Oct 22, 2024

There was also a reddit post talking about pulping zombies at the Tacoma ranch making them angry

@RenechCDDA
Copy link
Member

RenechCDDA commented Oct 22, 2024

You can no longer read signs at the exodii base.

Yes, that's a problem.

You can no longer move creates and lockers at refugee center. This prevents completing the first merchant quest to kill zombies (Clear Back Bay).

Nope, they clear the entrance for you. You don't need to move any furniture at all.

@RenechCDDA RenechCDDA added (S2 - Confirmed) Bug that's been confirmed to exist and removed (S1 - Need confirmation) Report waiting on confirmation of reproducibility labels Oct 22, 2024
@Demivan
Copy link
Contributor Author

Demivan commented Oct 22, 2024

Nope, they clear the entrance for you. You don't need to move any furniture at all.

Strange, I haven't seen that. How do you trigger it? Just accepting the quest does not do it. I have always dismantled the baricade manually.

@EvanBalster
Copy link
Contributor

I did that quest a few days ago and was able to move the lockers out of the way with no warning or NPC hostility. My character was too weak to grab/drag the crates easily.

I have encountered the following interactions flagged as stealing:

  1. Looting the body of the refugee center informant. (Upon careful inspection, you're stealing from the biker faction and the refugee center people do not care)
  2. Looting the bodies of various other bandits who are in any case hostile to you.
  3. Attempting to read or digitize books in the refugee center. (YMMV on whether this is truly an issue)
  4. Exodii signs as mentioned before.
  5. The door winch in the refugee center, after it is enabled by completing a quest and the player is welcomed to use it.

@Demivan
Copy link
Contributor Author

Demivan commented Oct 22, 2024

Attempting to read or digitize books in the refugee center. (YMMV on whether this is truly an issue)

This is working as intended: #65927 (comment)

@Demivan
Copy link
Contributor Author

Demivan commented Oct 27, 2024

You cannot activate fences at a lumbermill to climb over them

@RenechCDDA
Copy link
Member

RenechCDDA commented Oct 27, 2024

You cannot activate fences at a lumbermill to climb over them

100% working as intended. You should be entering through intended entrances only. Climbing over the fences meant to keep you out is not very polite...

The only fence entrance was locked, that was just bad mapgen. That is being handled - #77371 / 7a42024

@failboat78
Copy link

You cannot smash corpses near npc bases like Tacoma or scrapper.

@RenechCDDA
Copy link
Member

Nope, they clear the entrance for you. You don't need to move any furniture at all.

Strange, I haven't seen that. How do you trigger it? Just accepting the quest does not do it. I have always dismantled the baricade manually.

It was added in the same PR which introduced this mechanic.

It is possible you accepted the quest beforehand but did not complete it before updating to a version with these checks.

@RenechCDDA
Copy link
Member

RenechCDDA commented Oct 29, 2024

With #77371 and #77426 (and #77428) the only concern I have left for this issue is being unable to read the signs, or any other possible 'benign' examine action which we want to explicitly allow.

I'm open to suggestions on how to resolve this.

@Night-Pryanik
Copy link
Contributor

Make explicit check for those "benign" iexamine actions?

@RenechCDDA
Copy link
Member

Make explicit check for those "benign" iexamine actions?

I would like to avoid hardcoding a list of allowed/disallowed actions, but that might have to be the approach.

It's troublesome to even attach it to the iexamine/iuse actor, because at the time of check we don't even have information on which actor it is. This might be solvable, but I'd have to look into it. It looks like examine_actor is protected, but we could have a function which returns whether the examine_actor is considered 'allowed' and set that value in json.

@osuphobia
Copy link
Contributor

2024-11-01 222134
Now I need to use Probability Travel CBM to enter the territory of Prisoners, otherwise they got mad and attack me.
Meanwhile, if I offend one prisoner and let him attacks me, then kill him, they feel nothing :)
2024-11-01 223254
Furthermore, if I climbed a tree and want to climb down, the relationship between me and the Prisoners will be challenged😀

@kevingranade
Copy link
Member

I don't think this adresses your actor problem, but probably we should have a method for "only examine, don't interact with in any way" that doesn't trigger theft checks to make it distinct from other interactions that Prentiss involve acquiring or interacting with an object in a way that should trigger those checks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
(S2 - Confirmed) Bug that's been confirmed to exist
Projects
None yet
Development

No branches or pull requests

8 participants