add: grey rework #6108
add: grey rework #6108
Conversation
| /datum/action/innate/research_scanner/Trigger(left_click = TRUE) | ||
| if(IsAvailable()) | ||
| owner.research_scanner = !owner.research_scanner | ||
| to_chat(owner, "<span class='notice'>Research analyzer is now [owner.research_scanner ? "active" : "deactivated"].</span>") | ||
| return TRUE | ||
| if(!IsAvailable()) | ||
| return | ||
|
|
||
| owner.research_scanner = !owner.research_scanner | ||
| to_chat(owner, span_notice("Вы [owner.research_scanner ? "включили" : "отключили"] исследовательский анализатор.")) | ||
|
|
||
| return TRUE |
There was a problem hiding this comment.
Trigger стоит поменять на /Activate(), убрать isAvailable
| if(!IsAvailable()) | ||
| return |
There was a problem hiding this comment.
Стоит вызывать родителя, если в будущем будут что то менять для всех экшенс. Если не хочется, что бы вызывался attack_self у предмета в случае активации кнопочки - переведи переменную экшенса
attack_self = FALSE
| if(isgrey(user)) // no water?? | ||
| var/mob/living/carbon/human/H = user | ||
| var/grey_message = pick("Вы не ожидали, что в раковине окажется вода!", "Вы слишком поздно понимаете, что совершили ошибку!", "Вы чувствуете адскую боль по всему телу!") | ||
| H.adjustFireLoss(30 * H.get_permeability_protection()) | ||
| to_chat(H, span_danger("[grey_message]")) | ||
| if(H.has_pain()) | ||
| H.emote("scream") | ||
|
|
||
| return |
There was a problem hiding this comment.
стоит ввести сигнал, к примеру COMSIG_SINK_ACT, чем расширять истайпы на расы
| for(var/path in subtypesof(/obj/item/translator_chip)) | ||
| var/obj/item/translator_chip/chip = path |
There was a problem hiding this comment.
| for(var/path in subtypesof(/obj/item/translator_chip)) | |
| var/obj/item/translator_chip/chip = path | |
| for(var/obj/item/translator_chip/chip as anything in subtypesof(/obj/item/translator_chip)) |
Вполне можно сократить. Главное в таком случае не забывать делать игнорирование типа
| var/stamina_damage = 0 | ||
| var/small_pushed = TRUE | ||
| // Handle pulling all non /obj/item stuff or tiny mobs | ||
| if(pulling && istype(pulling, /mob/living)) |
There was a problem hiding this comment.
| if(pulling && istype(pulling, /mob/living)) | |
| if(pulling && isliving(pulling)) |
| for(var/path in subtypesof(/obj/item/translator_chip)) | ||
| var/obj/item/translator_chip/chip = path |
There was a problem hiding this comment.
| for(var/path in subtypesof(/obj/item/translator_chip)) | |
| var/obj/item/translator_chip/chip = path | |
| for(var/obj/item/translator_chip/chip as anything in subtypesof(/obj/item/translator_chip)) |
Можно сократить.
| @@ -334,6 +332,19 @@ | |||
| poison_amount = 10 | |||
| origin_tech = "materials=4;powerstorage=3;biotech=3" | |||
|
|
|||
|
|
|||
| /obj/item/organ/internal/cyberimp/chest/nutriment/insert(mob/living/carbon/human/target, special = ORGAN_MANIPULATION_DEFAULT) | |||
There was a problem hiding this comment.
| /obj/item/organ/internal/cyberimp/chest/nutriment/insert(mob/living/carbon/human/target, special = ORGAN_MANIPULATION_DEFAULT) | |
| /obj/item/organ/internal/cyberimp/chest/nutriment/plus/insert(mob/living/carbon/human/target, special = ORGAN_MANIPULATION_DEFAULT) |
Ошибка. В Тз также изменения обычной помпы - нет, только плюс
| . = ..() | ||
|
|
||
|
|
||
| /obj/item/organ/internal/cyberimp/chest/nutriment/remove(mob/living/carbon/human/target, special = ORGAN_MANIPULATION_DEFAULT) |
There was a problem hiding this comment.
| /obj/item/organ/internal/cyberimp/chest/nutriment/remove(mob/living/carbon/human/target, special = ORGAN_MANIPULATION_DEFAULT) | |
| /obj/item/organ/internal/cyberimp/chest/nutriment/plus/remove(mob/living/carbon/human/target, special = ORGAN_MANIPULATION_DEFAULT) |
Также, как и комментарий выше
| for(var/path in subtypesof(/obj/item/translator_chip)) | ||
| var/obj/item/translator_chip/chip = path |
There was a problem hiding this comment.
| for(var/path in subtypesof(/obj/item/translator_chip)) | |
| var/obj/item/translator_chip/chip = path | |
| for(var/obj/item/translator_chip/chip as anything in subtypesof(/obj/item/translator_chip)) |
Стоит сократить
|
|
||
| /obj/item/translator_chip/wingdings | ||
| icon_state = "chip_wingdings" | ||
| stored_language = TRAIT_WINGDINGS //weird but works |
There was a problem hiding this comment.
А так ли стоит миксовать трейты с языками? Выглядит не очень.
There was a problem hiding this comment.
Спасибо за ревью. Я могу перенести трейт в отдельную переменную внутри чипа. Просто насколько это лучше(создавание переменной для целого одного случая) - хз. Если думаешь что лучше или есть другие идеи - сделаю, не вопрос.
Описание
Небольшой реворк греев, сделанный на основе двух предложек
Если постараться кратко, то: добавлен "ретранслятор псионического голоса" греев, который заменяет старый дешифратор вингдингс, а так же голосовые связки греев. Добавлены чипы с языками для установки в него же. Прописан весь код для лёгкого добавления имплантов-переводчиков в игру, изменены некоторые цифры у греев, теперь греи получают стаминаурон от перетаскивания существ/объектов, лучше принимают импланты
Весь список изменений есть в предложках ниже
Автор всех спрайтов(discord): vitalya_kramsatel
Ссылка на предложение/Причина создания ПР
https://discord.com/channels/617003227182792704/755125334097133628/1291932121291817030
https://discord.com/channels/617003227182792704/755125334097133628/1291932183333830748
Тесты
Много дней, проведённых на локалке в отлавливании багов. Тыкался долго, все найденные проблемы были устранены