View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0005244 | The Dark Mod | Coding | public | 05.05.2020 15:28 | 08.08.2022 00:32 |
Reporter | Bikerdude | Assigned To | nbohr1more | ||
Priority | normal | Severity | normal | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Platform | PC | OS | All | ||
Product Version | TDM 2.10 | ||||
Target Version | TDM 2.11 | Fixed in Version | TDM 2.11 | ||
Summary | 0005244: Can't move in noclip with shouldered NPC's | ||||
Description | see below | ||||
Steps To Reproduce | - Fly around in noclip. - knock out an and pickup and AI. - go to move and your stuck in place. - you have type noclip twice in the console to be able to move again while shouldering an Ai, | ||||
Tags | No tags attached. | ||||
The start shouldering animation immobilizes the player. No shouldering animation is shown during noclip, so I assumed the immobilization is never cleared. The attached patch makes it so the player is not immobilized while shouldering and using noclip. fix-noclip-on-shoulder-v1-r10063.diff (757 bytes)
diff --git game/physics/Physics_Player.cpp game/physics/Physics_Player.cpp index 586d2ff..826a7c4 100644 --- game/physics/Physics_Player.cpp +++ game/physics/Physics_Player.cpp @@ -6072,7 +6072,10 @@ void idPhysics_Player::StartShouldering(idEntity const * const pBody) | EIM_MANTLE | EIM_FROB_COMPLEX | EIM_MOVEMENT | EIM_CROUCH_HOLD | EIM_CROUCH | EIM_JUMP | EIM_FROB | EIM_FROB_HILIGHT | EIM_LEAN; - pPlayer->SetImmobilization("ShoulderingAnimation", iImmobilization); + if (!pPlayer->noclip) + { + pPlayer->SetImmobilization("ShoulderingAnimation", iImmobilization); + } // Check height of body: If heigher than crouched, do not go to crouched state const float fBodyHeight = pBody->GetPhysics()->GetOrigin() * (-gravityNormal); |
|
I worked on this a bit more. The reason the immobilization is not cleared is that ShoulderingMove() is never called while using noclip. I've attached version 2 of the patch, which is cleaner in my opinion, because it doesn't change the Physics_Player.cpp file and overall state changes less during runtime. fix-noclip-on-shoulder-v2-r10063.diff (426 bytes)
diff --git game/Player.cpp game/Player.cpp index 28decdb..fd463d0 100644 --- game/Player.cpp +++ game/Player.cpp @@ -4483,7 +4483,8 @@ void idPlayer::OnStartShoulderingBody(idEntity* body) m_bShoulderingBody = true; // STiFU #3607: Trigger shouldering viewport animation - physicsObj.StartShouldering(body); + if (!noclip) + physicsObj.StartShouldering(body); } void idPlayer::OnStopShoulderingBody(idEntity* body) |
|
Thank you! | |
Indeed @Daft Mugi, thanks for sorting this. | |
Rev 10064 ( This is fantastic ! ) |
|
Date Modified | Username | Field | Change |
---|---|---|---|
05.05.2020 15:28 | Bikerdude | New Issue | |
09.10.2020 21:14 | Bikerdude | Product Version | TDM 2.07 => TDM 2.08 |
09.10.2020 21:14 | Bikerdude | Steps to Reproduce Updated | |
06.08.2022 14:07 | Bikerdude | Summary | Can't move in noclip whith shouldered AI => Can't move in noclip with shouldered NPC's |
06.08.2022 14:07 | Bikerdude | Product Version | TDM 2.08 => TDM 2.10 |
07.08.2022 05:19 | Daft Mugi | Note Added: 0015142 | |
07.08.2022 05:19 | Daft Mugi | File Added: fix-noclip-on-shoulder-v1-r10063.diff | |
07.08.2022 06:51 | Daft Mugi | Note Added: 0015143 | |
07.08.2022 06:51 | Daft Mugi | File Added: fix-noclip-on-shoulder-v2-r10063.diff | |
07.08.2022 11:17 | nbohr1more | Note Added: 0015145 | |
07.08.2022 11:17 | nbohr1more | Assigned To | => nbohr1more |
07.08.2022 11:17 | nbohr1more | Status | new => assigned |
07.08.2022 11:52 | Bikerdude | Note Added: 0015146 | |
08.08.2022 00:31 | nbohr1more | Note Added: 0015147 | |
08.08.2022 00:32 | nbohr1more | Reproducibility | have not tried => always |
08.08.2022 00:32 | nbohr1more | Status | assigned => resolved |
08.08.2022 00:32 | nbohr1more | Resolution | open => fixed |
08.08.2022 00:32 | nbohr1more | OS | Windows => All |
08.08.2022 00:32 | nbohr1more | OS Version | 10 => |
08.08.2022 00:32 | nbohr1more | Fixed in Version | => TDM 2.11 |
08.08.2022 00:32 | nbohr1more | Target Version | => TDM 2.11 |