enhance "player_weaponstrip" entity

[Vaqtincha]

Test map attached
test.zip

[wopox1337]

This PR break aim_deagle weapon stripper.

[etojuice]

На aim_deagle не срабатывает player_weaponstrip, потому что нет активатора. Оружие убиралось через game_player_equip, а перестало потому что добавили новый spawnflags для этого.

[Vaqtincha]

На aim_deagle не срабатывает player_weaponstrip, потому что нет активатора. Оружие убиралось через game_player_equip, а перестало потому что добавили новый spawnflags для этого.

Там ничего не должно сломаться. Тк только фишки добавлены.

"Оружие убиралось через game_player_equip, а перестало потому что добавили новый spawnflags для этого."
Через game_player_equip ничего невозможно было убирать раньше. Новый спавнфлаг не причем.

[etojuice]

"Оружие убиралось через game_player_equip, а перестало потому что добавили новый spawnflags для этого."
Через game_player_equip ничего невозможно было убирать раньше. Новый спавнфлаг не причем.

ты прав.
конкретно на aim_deagle у game_player_equip почему-то установлен spawnflags на 1, вот всё и "поломалось"

[Vaqtincha]

"Оружие убиралось через game_player_equip, а перестало потому что добавили новый spawnflags для этого."
Через game_player_equip ничего невозможно было убирать раньше. Новый спавнфлаг не причем.

ты прав.
конкретно на aim_deagle у game_player_equip почему-то установлен spawnflags на 1, вот всё и "поломалось"

Снова мимо. Флаг 1 это SF_PLAYEREQUIP_USEONLY который вообще не менялся обработка в коде.
Это проблема недомаппера.

[Vaqtincha]

Возможно в этой карте вообще нету player_weaponstrip и у game_player_equip в поле targetname стоит game_playerspawn и флаг у него UseOnly. Это неправильно. Проще карту фиксит.

[etojuice]

да, в карте не активируется player_weaponstrip... поставил у game_player_equip spawnflags 0, стало лучше

[Vaqtincha]

Можно ставить 3 так правильнее будет работать.

[wopox1337]

@Vaqtincha "карты, которые раньше работали нормально - теперь не работают нормально после обновления ReGameDLL". Да, карты сделаны коряво, являются legacy. Однако, раньше GameDLL не умел правильно обрабатывать поэтому ошибочное поведение не было замечено и считалось нормой. Сомневаюсь что лишь несколько карт имеют проблему. Тогда проблема станет массовой.

Быть может, следует добавить CVar mp_legacy_game_player_equip с следованием lagacy поведению, для сохранения совместимости? (кстати, CVar оказался бесполезным, ибо никто не будет включать новое поведение, чтобы сломать себе карты, или пользуется минимальный проект людей)

[Vaqtincha]

@wopox1337 Я не собираюсь ничего добавлять. Никаких кваров и костылей. Это проблема мапперов которые не создали player_weaponstrip на своем уг карте. Исправь сам если горишь желанием. Быстрее принимают.