Tweak modsuit holster module after bulky lasers #1770
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
m-dzianishchyts
requested changes
Feb 1, 2025
Drsmail
suggested changes
Feb 7, 2025
m-dzianishchyts
requested changes
Feb 9, 2025
Comment on lines
+45
to
+46
| for(var/type in overridebulky) | ||
| if(holding.type == type) |
There was a problem hiding this comment.
Этот цикл можно заменить на прок
Suggested change
| for(var/type in overridebulky) | |
| if(holding.type == type) | |
| is_type_in_list(holding, overridebulky) |
| ) | ||
|
|
||
| /obj/item/mod/module/holster/on_use() | ||
| if(!holstered) |
There was a problem hiding this comment.
Тут можно выполнить ранний return
Suggested change
| if(!holstered) | |
| if(holstered) | |
| return ..() |
There was a problem hiding this comment.
Точно у меня украл. Но я теперь вижу, как это всё можно было лучше по блоком расписать.
There was a problem hiding this comment.
Но у тебя там не возврат, а только вызов
There was a problem hiding this comment.
Я потом добавил, после того как ты написал мне в чате об этом :xdd
| if(!holstered) | ||
| var/obj/item/gun/holding = mod.wearer.get_active_hand() | ||
| if(!holding) | ||
| to_chat(mod.wearer, "<span class='warning'>Nothing to holster!</span>") |
There was a problem hiding this comment.
Для спанов надо бы использовать макросы
Suggested change
| to_chat(mod.wearer, "<span class='warning'>Nothing to holster!</span>") | |
| to_chat(mod.wearer, span_warning("Nothing to holster!") |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Что этот PR делает
Позволяет холстерить не так давно ставшие bulky-sized лазеры в кобуру модсьюта, которая ранее вмещала в себя их.
Почему это хорошо для игры
Потому что модуль перестаёт быть бесполезным для пользователей (СБ), ведь сейчас в него можно уместить разве что дизейблер, как и в обычную кобуру на комбинезон.
Тестирование
Попытался засунуть bulky комбат дробовик - не смог. Попытался засунуть укороченный дробовик размера normal - смог.
Попытался засунуть bulky лвап - не смог. Попытался засунуть обычный bulky лазган - смог. And so on.
Changelog
🆑
tweak: Кобура модсьюта вновь вмещает в себя большинство bulky-sized лазеров, как это и было ранее.
/:cl:
Summary by Sourcery
Bug Fixes: