ficool2 / HammerPlusPlus-Website

The website for Hammer++
93 stars 9 forks source link

Can't put a comma in IO System in non-CS:GO games, output gets offset after restarting Hammer or map compile #32

Closed 8BEATER-Il-S closed 2 years ago

8BEATER-Il-S commented 2 years ago

Describe the bug

The .VMF file stores wrong data about an output which contains a comma, which makes passing functions with multiple parameters through VScript quite troublesome in non-CS:GO games. It's noticeable only after saving the file and restarting Hammer++. This bug also breaks logic on compile and doesn't produce any errors in compile and game consoles.

Steps to reproduce

Steps to reproduce the behavior:

  1. Have an output which contains a comma, like this one: изображение
  2. Restart Hammer++.
  3. See the parameter be transferred over to the delay and fires limit. изображение

Expected behavior

Have parameters look fine like on the first screenshot. If it's a feature, then at least give a warning and an option to disable it, because the way it is now won't cut it. изображение изображение

8BEATER-Il-S commented 2 years ago

Sorry if this sounded way too aggressive, I was just angry at the moment, I don't mean any hate. I really appreciate this work on Hammer++, but this issue just got me by surprise and I spent about an hour on this before figuring it out. Still, I think there needs to be at least a warning.

hexaflexahexagon commented 2 years ago

I'll let someone smarter confirm/deny but I'm fairly sure this is just a limitation with the vmf file format in older engine games and can't be worked around without breaking backwards compatibility.