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

Изменения доступных для вкладывания в кобуру вещей #3652

Merged
merged 13 commits into from
Jul 30, 2019

Conversation

FelixRuin
Copy link
Contributor

@FelixRuin FelixRuin commented Jul 22, 2019

Описание изменений

Переименован прок isHandgun в canHolster для понятности. Добавленна переменная для возможности уже в раунде менять возможность положить оружие в кобуру. Доработан список доступных для укладывания в кобуру оружия
image

Почему и что этот ПР улучшит

Данный пр отребалансит возможности кобуры, упростит работу с кодом оружия

Авторство

Felix Ruin - Winter Schock

Чеинжлог

🆑 Winter Schock

  • balance[link]: Измененён список оружия доступного для вкладывания в кобуру

@TauKitty
Copy link
Contributor

Changelog status: ✔️

@FelixRuin FelixRuin changed the title Guns guns guns Изменения доступных для вкладывания в кобуру вещей Jul 22, 2019
@ViDLiathan
Copy link

Наконец то не будет больше теслы в кобуре.
И что на счет игрушечного револьвера из аркады? Его почему то нельзя было в кобуру класть.

теперь можно положить в кобуру банан, игрушечное оружие
@FelixRuin
Copy link
Contributor Author

@ViDLiathan Done. Теперь в кобуру можно положить ещё банан, игрушечный пистолет и игрушечный дротикомёт

code/game/objects/items.dm Outdated Show resolved Hide resolved
code/modules/clothing/under/accessories/holster.dm Outdated Show resolved Hide resolved
@LudwigVonChesterfield
Copy link
Contributor

Думаю распиленный дробовик тоже должен умещаться.

ну и обрез в кобуре
@FelixRuin
Copy link
Contributor Author

поставил что и обрез можно положить

FelixRuin and others added 2 commits July 24, 2019 11:16
Co-Authored-By: Alexander V. <volas@ya.ru>
доделавание с обрезом и превращением в него
@SpaiR SpaiR added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Jul 28, 2019
@SpaiR SpaiR mentioned this pull request Jul 28, 2019
@TungstenOctopus
Copy link
Contributor

Обрез в кобуру? Серьезно? Ну разве что в кобуру 26 века, которая растягивается в три раза.

Кстати, isHandgun, по идее, понятие более широкое, чем canHolster. Довольно логично, что в кобуру для пистолета помещается только оружие, по форм-фактору близкое к пистолету, а за этот параметр собсно и отвечает прок isHandgun. То есть, если где-то нам надо будет проверить пистолетоподобность оружия, нам пригодится прок isHandgun, а canHolster будет здесь выглядеть неинтуитивно и узконаправленно. Короче переименование прока выглядит как-то бессмысленно и недальновидно. Но это чисто мое имхо.

@FelixRuin
Copy link
Contributor Author

переименование прока осмысленно на том что этот прок отвечает только за то чтоб поместить вещь в кобуру

@SpaiR SpaiR merged commit 372f747 into TauCetiStation:master Jul 30, 2019
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Jul 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants