Добавление новых механик зажигалке синидката #13355
Conversation
…унов, а также добавлены в рандомные предметы разбросаные по тех тонелям на карте Box <!-- Читать: https://github.com/TauCetiStation/TauCetiClassic/blob/master/.github/wiki/STYLING_OF_PR.md --> Удалил с однаковых спаунов очки и изолционные перчатки, но добавил в рандомный спаун по тех тонелям Меньше манча Riverz <!-- В случае порта с другого билда - укажите источник (репозиторий или номер PR-а). Если это оригинальный PR - укажите первоисточник/авторство спрайтов и звуков. Укажите лицензию для звуков. --> <!-- В чеинжлог стоит писать изменения, которые будут заметны игрокам. И так, чтобы они были понятны игрокам. Ключевые слова для чеинжлога: bugfix, rscadd, rscdel, image, sound, spellcheck, tweak, balance, map, performance, experiment :cl: - bugfix: Пофикшен такой-то баг. - map: Перемаплен такой-то отсек. - image: Обновлен такой-то спрайт. -->:cl: Riverz - tweak: Удалены изоляционные перчатки и солнцезащитные очки со одинаковых спаунов, а также добавлены в рандомные предметы разбросаные по тех тонелям на карте Box
Changelog status: 🔘 No changelog |
|
босс тут что-то не так с изменениями |
…овых спаунов, а также добавлены в рандомные предметы разбросаные по тех тонелям на карте Box" This reverts commit 036f959.
Да, почему-то у меня уже 2ой ПР подряд ветка старая |
| icon_state = initial(icon_state) | ||
| active = FALSE | ||
| set_light(0) | ||
| if(!istype(src, /obj/item/weapon/weldingtool/experimental/sindi)) |
There was a problem hiding this comment.
Я взял этот код с экспериментальной сварки поменяв на свои значение вот у неё тоже самое:
hitsound = initial(hitsound)
if(icon_state != "welder") // Check that the sprite is correct, if it isnt, it means toggle() was not called
force = 3
damtype = BRUTE
icon_state = initial(icon_state)
active = FALSE
set_light(0)
if(!istype(src, /obj/item/weapon/weldingtool/experimental))
STOP_PROCESSING(SSobj, src)
return
|
Я кстати щас увидел что спрайт зажигалке не посредственно в самой руке на мобе при зажигании пропадает, щас попроавлю |
| /obj/item/weapon/weldingtool/experimental/syndicate_zippo/process() | ||
| if(active) | ||
| hitsound = SOUNDIN_LASERACT | ||
| if(icon_state != "syndizippo1") // Check that the sprite is correct, if it isnt, it means toggle() was not called | ||
| force = 15 | ||
| damtype = BURN | ||
| icon_state = initial(icon_state) + "1" | ||
| if(prob(5)) // passive fuel burning | ||
| use(1) | ||
| light_color = LIGHT_COLOR_NUKE_OPS | ||
| set_light(2) | ||
| else | ||
| hitsound = initial(hitsound) | ||
| if(icon_state != "syndizippo") // Check that the sprite is correct, if it isnt, it means toggle() was not called | ||
| force = 3 | ||
| damtype = BRUTE | ||
| icon_state = initial(icon_state) | ||
| active = FALSE | ||
| set_light(0) | ||
| if(!istype(src, /obj/item/weapon/weldingtool/experimental/syndicate_zippo)) | ||
| STOP_PROCESSING(SSobj, src) | ||
| return | ||
|
|
||
| /obj/item/weapon/weldingtool/experimental/syndicate_zippo/process() | ||
| ..() | ||
| if((get_fuel() < max_fuel) && (next_refuel_tick < world.time) && !active) | ||
| next_refuel_tick = world.time + 2.5 SECONDS | ||
| reagents.add_reagent("fuel", 1) | ||
| if(!active && (get_fuel() == max_fuel)) | ||
| STOP_PROCESSING(SSobj, src) |
There was a problem hiding this comment.
зачем на 2 прока разделять?
There was a problem hiding this comment.
Я сделал это также потому что в экспериментальной сварке также стоит
There was a problem hiding this comment.
необходим более глубокий анализ, никто не гарантирует что как сделано до тебя - сделано хорошо.
если никакого отличия в поведении между синдикат зажигалкой и экспериментальной сваркой нет - зачем копировать-вставлять поведение?
There was a problem hiding this comment.
Потому что это соответствует моей задумке, зажигалка на бензине работающая как сварка, а разница есть в размерах, ну и в получении
There was a problem hiding this comment.
если есть разница в размерах и получении - это можно поменять, вопросов нет
тут к тебе вопрос по поводу реализации поведения этой зажигалки (в частности прок process)
There was a problem hiding this comment.
Есть догадка что если удалить вот эти строки кода то задумка будет осуществлена, а копипаста исчезнет.
There was a problem hiding this comment.
там логика с if(icon_state != "syndizippo1") похерится если наследовать же
There was a problem hiding this comment.
возможно, надо переделать сварку чтобы нужные переменные менялись при включении-выключении тогда.
но всё ещё не копипастить процесс()
There was a problem hiding this comment.
Я это оставил потому что у меня зелёный огонь там с помощью этого кода появляется
|
Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад. |
|
ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув |
Описание изменений
Добавил синдикатовскую зажигалку в аплинк, теперь она работает на бензине, а также её можно использовать в качестве сварочного аппарата, она может самоперезаряжаться, а также её можно получить взломав автомат с сигеретами емагом
Почему и что этот ПР улучшит
Новые механики, новый гемплей
Авторство
Riverz
Чеинжлог
cl: