It’s an issue if you have an integer tied with a string in singular.
It’s a coding issue
stefano_89027889
Don’t be too hard on them, they are not used to doing updates
SweHun
Poor tormented millionaires and billionaires at the smol indie company cant even write a proper if statement
Codacc69420
They must have tested this with two people lol
iFluffy_
Give that one dev guy some slack. Poor guy.
Impressive_Tomato362
Terrorae
wigneyr
Is smol indie company, leave alone pls
montjoye
you can’t have fixes AND spellchecks
eskannspecsein
Pretty sure none of these fucks actually care. They came in yesterday to change a few lines of code for the „new season update“ and probably went home after an hour. These devs are disgusting.
Arisa_kokkoro
no one cares about this.
readthetda
Warning: speculation ahead
Historically, it seems they’d just split singular and plural localisation strings into separate entries. You can see this in:
“`
Notice_Bonus_Shorthanded_Eligibility
Your team will be eligible for short-handed income in %s1 rounds
Notice_Bonus_Shorthanded_Eligibility_Single
Your team will be eligible for short-handed income in 1 round
“`
This can be seen in the localisation string they’ve added for this update:
“`
Team_Cash_Award_Dead_Terrorists:p{s2}
+$%s1 team income for %s2 eliminated terrorist#|#
+$%s1 team income for %s2 eliminated terrorists
“`
I don’t know how their localisation engine fully works, but I presume that :p{s2} tests whether the value of s2 is greater than 1, and if so will then use the string after the #|# delimiter. I also presume the reason it’s not working properly is because they’re testing s2 – which is a string – and they should maybe be using “d” which is what’s typically used to represent integers in C-style string formatting, and is also what is used in their own documentation in the link above.
Tl;dr: I think they’re trying to test whether literally “4” > 1 which it’s not because it’s a string and not an integer.
A potential fix would be:
“`
Team_Cash_Award_Dead_Terrorists:p{d}
+$%s1 team income for %d eliminated terrorist#|#
+$%s1 team income for %d eliminated terrorists
“`
nicey_v
Can someone tell me where the mistake is?
mafga1
Not enough people who are working on CS2, sad to say…
Najeeb1316
I feel like if this is something that actually bothers you then you need to touch grass more
pageofswrds
you guys are the whiniest motherfuckers out there lmao
Vast_Amphibian5933
Leave the small indie dev company alone, they barely make money
PingHangsLow
Indie company give them a break
cocobolo_table
Making a thread about this is god damn pathetic.
AgreeableBroomSlayer
Valve hires fiver devs so english is not their primary language
22 Comments
[deleted]
Its a small indie-company, cut them some slack.
It’s an issue if you have an integer tied with a string in singular.
It’s a coding issue
Don’t be too hard on them, they are not used to doing updates
Poor tormented millionaires and billionaires at the smol indie company cant even write a proper if statement
They must have tested this with two people lol
Give that one dev guy some slack. Poor guy.
Terrorae
Is smol indie company, leave alone pls
you can’t have fixes AND spellchecks
Pretty sure none of these fucks actually care. They came in yesterday to change a few lines of code for the „new season update“ and probably went home after an hour. These devs are disgusting.
no one cares about this.
Warning: speculation ahead
Historically, it seems they’d just split singular and plural localisation strings into separate entries. You can see this in:
“`
Notice_Bonus_Shorthanded_Eligibility
Your team will be eligible for short-handed income in %s1 rounds
Notice_Bonus_Shorthanded_Eligibility_Single
Your team will be eligible for short-handed income in 1 round
“`
But with Source 2 Panorama it seems to be a lot more flexible and [allows for more than simple string substitution](https://developer.valvesoftware.com/wiki/Dota_2_Workshop_Tools/Panorama/Localization).
This can be seen in the localisation string they’ve added for this update:
“`
Team_Cash_Award_Dead_Terrorists:p{s2}
+$%s1 team income for %s2 eliminated terrorist#|#
+$%s1 team income for %s2 eliminated terrorists
“`
I don’t know how their localisation engine fully works, but I presume that :p{s2} tests whether the value of s2 is greater than 1, and if so will then use the string after the #|# delimiter. I also presume the reason it’s not working properly is because they’re testing s2 – which is a string – and they should maybe be using “d” which is what’s typically used to represent integers in C-style string formatting, and is also what is used in their own documentation in the link above.
Tl;dr: I think they’re trying to test whether literally “4” > 1 which it’s not because it’s a string and not an integer.
A potential fix would be:
“`
Team_Cash_Award_Dead_Terrorists:p{d}
+$%s1 team income for %d eliminated terrorist#|#
+$%s1 team income for %d eliminated terrorists
“`
Can someone tell me where the mistake is?
Not enough people who are working on CS2, sad to say…
I feel like if this is something that actually bothers you then you need to touch grass more
you guys are the whiniest motherfuckers out there lmao
Leave the small indie dev company alone, they barely make money
Indie company give them a break
Making a thread about this is god damn pathetic.
Valve hires fiver devs so english is not their primary language
Literally unplayable.