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

[CSGO] Using two team colors in one message is not allowed #8

Closed
Nerus87 opened this issue Feb 1, 2021 · 4 comments
Closed

[CSGO] Using two team colors in one message is not allowed #8

Nerus87 opened this issue Feb 1, 2021 · 4 comments

Comments

@Nerus87
Copy link

Nerus87 commented Feb 1, 2021

Hi, I have an error when using darkred color.

Translation:

"sm_qd_panel_defuse_nokit_incorrected_csgo"
	{
		"#format"		"{1:N},{2:s},{3:s},{4:s},{5:s}"
		"en"			"{blue}{1}{default} {darkred}detonated{default} the C4 with an incorrect {{2}}{3}{default} wire (7:8 odds).\nThe correct wire was {{4}}{5}{default}."
	}

Error:

L 02/02/2021 - 00:26:56: [SM] Exception reported: Using two team colors in one message is not allowed
L 02/02/2021 - 00:26:56: [SM] Blaming: quickdefuse.smx
L 02/02/2021 - 00:26:56: [SM] Call stack trace:
L 02/02/2021 - 00:26:56: [SM]   [0] ThrowError
L 02/02/2021 - 00:26:56: [SM]   [1] Line 410, D:\csgo-ffa\csgo\addons\sourcemod\scripting\include\multicolors/colors.inc::C_Format
L 02/02/2021 - 00:26:56: [SM]   [2] Line 85, D:\csgo-ffa\csgo\addons\sourcemod\scripting\include\multicolors/colors.inc::C_PrintToChat
L 02/02/2021 - 00:26:56: [SM]   [3] Line 104, D:\csgo-ffa\csgo\addons\sourcemod\scripting\include\multicolors.inc::CPrintToChatAll
L 02/02/2021 - 00:26:56: [SM]   [4] Line 229, .\quickdefuse.sp::PanelDefuseNoKit

Full code is on the GitHub QuickDefuse

@Nerus87 Nerus87 changed the title CSGO team colours problem [CSGO] Using two team colors in one message is not allowed Feb 1, 2021
@Bara
Copy link
Owner

Bara commented Feb 2, 2021

Normally it comes from combining red/blue, darkred should be fine. When I look at your code, it is possible with a red/blue combination here: https://github.com/Nerus87/QuickDefuse/blob/master/addons/sourcemod/scripting/quickdefuse.sp#L25 and https://github.com/Nerus87/QuickDefuse/blob/master/addons/sourcemod/scripting/quickdefuse.sp#L229

@Bara
Copy link
Owner

Bara commented Feb 2, 2021

And your "error" also exist in the base version of color.inc, you can try to change playername color to darkblue. And by removing this check with 2 tema colors it could make some troubles in other games then csgo, so I'll keep it as it is.

@Nerus87
Copy link
Author

Nerus87 commented Feb 20, 2021

Thanks, fixed, problem was on my plugin.

@Nerus87 Nerus87 closed this as completed Feb 20, 2021
@Bara
Copy link
Owner

Bara commented Feb 21, 2021

Thanks for your feedback!

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

No branches or pull requests

2 participants