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 |