Demo Recording Filename about cod4x_server

[Iswenzz]

hi, you can use exec("record <client> <demoname>"); in GSC and make the demoname the player ID

[matt-alton]

@Iswenzz thanks for the suggestion. How would I implement this so it works?

I tried the following in player.gsx as part of a welcome message block but it appears to have had no effect. No errors logged either.

    exec( "record " + self.name + " demo_" + self.clientid + "_" );

Regards,
Matt.

[matt-alton]

@Iswenzz I've since managed to work out where the variable is set and have created a PR with the very simple change.

[Iswenzz]

You should use the client number instead of name, and the Pr doesn't seem to make sense, it's a steam ID function

[matt-alton]

Hmm I see your point. This likely needs to check a dvar to check what type of ID is being used or a conditional statement to fallback to codguid if steamid is null. The code at present is hardcoded to only use the steamid in the autorecord filename.

https://github.com/callofduty4x/CoD4x_Server/blob/master/src/sv_client.c#L1212-L1220

	SV_SApiSteamIDToString(client->steamid, psti, sizeof(psti));

	//It was never intended to make a new demo for each fast_restart.
	//SV_SpawnServer() stops the demo and cleans the name which did not happen here which resulted in strange naming bug
	if(sv_autodemorecord->boolean && !client->demorecording && (client->netchan.remoteAddress.type == NA_IP || client->netchan.remoteAddress.type == NA_IP6))
	{
		SV_RecordClient(client, va("demo_%s_", psti));
	}

[Iswenzz]

Ye a fallback sounds good