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

Blacklisted addiction hediff causes pawn to be stuck getting treated #30

Closed
MarsM0nd opened this issue Aug 25, 2020 · 6 comments · Fixed by #33
Closed

Blacklisted addiction hediff causes pawn to be stuck getting treated #30

MarsM0nd opened this issue Aug 25, 2020 · 6 comments · Fixed by #33
Labels

Comments

@MarsM0nd
Copy link

If you want to patch an addiction hediff, or any other hediff that will make ShouldPawnSeekMedPod return true, that will cause the pawn to be stuck in a treatment loop. They will enter for any reason, get healed, medpod see the blacklisted hediff and ignores it, after the treatment is finished, the pawn won't get up from the pod, as they belive they still have a treatable hediff, and it loops.
Can be easily reproduced with just this mod and one that patches a vanilla addiction into the blacklist.

@sumghai sumghai added the Bug label Aug 26, 2020
@sumghai
Copy link
Owner

sumghai commented Aug 26, 2020

Bug confirmed - I reproduced this issue by blacklisting AlcoholAddiction as an example.

I'll have to investigate this a bit further - looks like the diagnosis logic is wonky yet again.

@sumghai
Copy link
Owner

sumghai commented Aug 30, 2020

@MarsM0nd - Please test the following development build to see if it fixes your issue:

https://github.com/sumghai/MedPod/archive/3b9cb8ad5292b5ed3ea035e19a2ed825c35d7724.zip

@MarsM0nd
Copy link
Author

MarsM0nd commented Sep 1, 2020

That fixes the issue i had, yes. It's still possible to get this stuck behaviour by blacklisting MissingBodyPart, but that would be another issue, and probably more of a feature request.

@sumghai
Copy link
Owner

sumghai commented Sep 1, 2020

That fixes the issue i had, yes.

Excellent!

It's still possible to get this stuck behaviour by blacklisting MissingBodyPart, but that would be another issue, and probably more of a feature request.

A cursory glance of ShouldPawnSeekMedPod() suggests this could be fixed there as well, so I'm going to consider it as part of the same issue.

@sumghai
Copy link
Owner

sumghai commented Sep 2, 2020

Okay @MarsM0nd, now try this dev build, with and without blacklisting the MissingBodyPart hediff in your patches:

https://github.com/sumghai/MedPod/archive/518bc687522fb87b6e92c64797af032a9de6cd10.zip

@sumghai
Copy link
Owner

sumghai commented Sep 6, 2020

@MarsM0nd did not respond in a timely manner, so I only have my own testing to confirm that this issue has been fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants