Fixed the client-side VScript "player" instance not being set once more after having moved around the VM init code again

Author: Blixibon

sp/src/game/client/c_baseplayer.cpp

@@ -855,6 +855,14 @@ void C_BasePlayer::PostDataUpdate( DataUpdateType_t updateType )
 			// changed level, which would cause the snd_soundmixer to be left modified.
 			ConVar *pVar = (ConVar *)cvar->FindVar( "snd_soundmixer" );
 			pVar->Revert();
+
+#ifdef MAPBASE_VSCRIPT
+			// Moved here from LevelInitPostEntity, which is executed before local player is spawned.
+			if ( g_pScriptVM )
+			{
+				g_pScriptVM->SetValue( "player", GetScriptInstance() );
+			}
+#endif
 		}
 	}
 

sp/src/game/client/vscript_client.cpp

@@ -576,13 +576,6 @@ bool VScriptClientInit()
 			if( g_pScriptVM )
 			{
 #ifdef MAPBASE_VSCRIPT
-				// Moved here from LevelInitPostEntity, which is executed before local player is spawned.
-				// This is executed after C_World::OnDataChanged, which is after C_BasePlayer::Spawn
-				if ( C_BasePlayer *pPlayer = C_BasePlayer::GetLocalPlayer() )
-				{
-					g_pScriptVM->SetValue( "player", pPlayer->GetScriptInstance() );
-				}
-
 				CGMsg( 0, CON_GROUP_VSCRIPT, "VSCRIPT CLIENT: Started VScript virtual machine using script language '%s'\n", g_pScriptVM->GetLanguageName() );
 #else
 				Log( "VSCRIPT: Started VScript virtual machine using script language '%s'\n", g_pScriptVM->GetLanguageName() );