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

SayText and TextMsg #1859

Closed
GLoOoccK opened this issue Oct 25, 2024 · 6 comments · Fixed by Velaron/cs16-client#102
Closed

SayText and TextMsg #1859

GLoOoccK opened this issue Oct 25, 2024 · 6 comments · Fixed by Velaron/cs16-client#102

Comments

@GLoOoccK
Copy link

Problem displaying messages with plugins and modules that use #Spec_PlayerItem

Spec_PlayerItem

alliedmodders/amxmodx#776 (comment)
https://github.com/WPMGPRoSToTeMa/SafeNameAndChat/blob/master/Main.cpp#L990-L1029
https://github.com/OciXCrom/ChatManager/blob/master/scripting/include/cromchat.inc#L892

@a1batross
Copy link
Member

Engine version?

@a1batross
Copy link
Member

Are you sure it's also engine problem and not cs16-client specific?

@GLoOoccK
Copy link
Author

I'm using:

Xash3D FWGS 49/0.21 (win32-i386 build 3494)
https://github.com/FWGS/xash3d-fwgs/actions/runs/11507001497

I don't know if it's a specific problem with cs16-client

@Buggem
Copy link
Contributor

Buggem commented Oct 25, 2024

cs16-client used on windows/linux is very unstable sadly. It probably is a cs16-client problem.

@a1batross
Copy link
Member

#Spec_PlayerItem is defined NOT in titles.txt but in valve_english.txt.

Ideally, those strings should've been translated using pre-existing VGUI2 translation files support but instead it follows hardcoded array of strings, and #Spec_PlayerItem isn't one of them: https://github.com/Velaron/cs16-client/blob/8f325bc5e42c4e073cd46e9377118806fb7d59cf/cl_dll/saytext.cpp#L180

I probably wrote this part before I implemented VGUI2 translation files parsing and forgot to fix it.

cc @Velaron could you add Localize() call there or just turn #Spec_PlayerItem into a %s.

@a1batross
Copy link
Member

Closing it here as it's definitely a cs16-client bug. Reopen issue at https://github.com/Velaron/cs16-client.

@a1batross a1batross closed this as not planned Won't fix, can't repro, duplicate, stale Oct 27, 2024
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

Successfully merging a pull request may close this issue.

3 participants