СПУ теперь можно перекрасить баллончиком краски. #11720
Conversation
Changelog status: ✔️ |
|
Этот запрос на слияние был упомянут в Tau Ceti Station Forum. Там могут быть соответствующие подробности: https://forum.taucetistation.org/t/kidajte-melkie-fichi-i-tviki-syuda/40032/14 |
Ушел же. |
| @@ -325,17 +326,34 @@ | |||
| to_chat(user, "<span class='warning'>Take the cap off first!</span>") | |||
| return | |||
| if(iscarbon(target) && uses - 10 >= 0) | |||
There was a problem hiding this comment.
предыдущая хардкод проверка, что uses > 10, но использовать может теперь 15 и мы можем выйти в минус.
По хорошему нужно переделать на use_tool, но для быстроты я бы предложил оставить использование 10 юнитов для всего, как было. Это не критично, и сохранит время на рефакторинге старого кода, если ты к этому не готов.
| H.lip_style = "spray_face" | ||
| H.lip_color = colour | ||
| H.update_body() | ||
| if(user.a_intent == INTENT_PUSH && H.get_species() == IPC) |
There was a problem hiding this comment.
проверка на интент не очевидна, это слишком скрытая фича получится, на которую попадут разве что случайно. Я бы оставил только на species
| else if (iscarbon(C)) | ||
| uses -= 10 | ||
| C.blurEyes(3) | ||
| C.eye_blind = max(C.eye_blind, 1) |
There was a problem hiding this comment.
лучше не делать лесенок с if-ами, а использовать концепт раннего возврата. Типо где-то в начале
if(!ishuman(C))
uses -= 10
C.blurEyes(3)
C.eye_blind = max(C.eye_blind, 1)
return
var/mob/living/carbon/human/H = C
...и дальше поведение работы с хьюманом, но уже без всяких лесенок...
На твоё усмотрение, переделывать или нет, тут остальной код такой же, так что не горит
|
Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад. |
|
ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув |
Описание изменений
Тайтл
Почему и что этот ПР улучшит
Лучше спросить у MIDI или Ксеновизора.
Авторство
Рукожопый - Я
Идея от MMDI
Чеинжлог
🆑