fix(sprites): returning lost and null sprites

[i1yadobr]

Починил практически весь список из иссуя ниже. Немного добавил отсебятины в виде фигурки из дартпатронов и форму для ИИшной голограммы. Последнее, в теории, нигде не должно использоваться, поскольку это является родителем голограммной картинки.

close #6791

• Pull Request полностью завершен, мне не нужна помощь чтобы его закончить.
• Я внимательно прочитал(-а) все свои изменения и багов в них не нашёл(-ла).
• Я запускал(-а) сервер со своими изменениями локально и все протестировал(-а).
• Я ознакомился(-ась) c Guide to Contribute.

[TobyThorne]

И чтобы что? В апдейт_иконе иконка обновляется через initial(icon_state). В итоге ты это просто поломал.

[i1yadobr]

Не заметил, поправлю.

[Filatelele]

error: Icon state light1 not found in icons/effects/lighting_overlay.dmi!

Это фиксится ну явно не добавлением light1 в lighting_overlays.dmi

[Filatelele]

Ну и уж ТЕМ БОЛЕЕ подобное дерьмо, когда у спрайт аксессори указан неправильный айкон стейт, не должно исправляться пустым айкон стейтом. Ну, ты ведь лечишь не симптомы заболевания, а причину, верно?

[i1yadobr]

Ну и уж ТЕМ БОЛЕЕ подобное дерьмо, когда у спрайт аксессори указан неправильный айкон стейт, не должно исправляться пустым айкон стейтом. Ну, ты ведь лечишь не симптомы заболевания, а причину, верно?

Я не совсем понимаю, что должно быть на месте icon_state = "default", поскольку я даже не имею понятия для чего может использоваться в игре ветка /datum/sprite_accessory/skin, поэтому предпочту в таком случае просто не трогать данную ветку.
Я осознаю, что мой метод фикса не самый лучший. Ты правильно выразился, что "...лечишь не симптомы заболевания, а причину...".

[Filatelele]

Оно лезет вот отсюда.

//going to have to re-integrate this with surgery
//let the icon_state hold an icon preview for now
/datum/sprite_accessory/skin
	icon = 'icons/mob/human_races/r_human.dmi'

/datum/sprite_accessory/skin/human
	name = "Default human skin"
	icon_state = "default"

Ну, вернее - не лезет, потому что это не используется и не выбирается и эту легаси ересь можно смело вырезать.

[i1yadobr]

Оно лезет вот отсюда.

//going to have to re-integrate this with surgery
//let the icon_state hold an icon preview for now
/datum/sprite_accessory/skin
	icon = 'icons/mob/human_races/r_human.dmi'

/datum/sprite_accessory/skin/human
	name = "Default human skin"
	icon_state = "default"

Ну, вернее - не лезет, потому что это не используется и не выбирается и эту легаси ересь можно смело вырезать.

Хорошо. Спасибо большое за разъяснение.

[i1yadobr]

По хорошему на самом деле надо бы и лишние кнопки убрать, поскольку у них метод обновления спрайта настолько устарел, что из всех кнопок обновляется только "/obj/machinery/button/remote":

Или я не понимаю, как работает код. Скорее всего второе, но с частью первого.

[github-actions]

This pull request has conflicts, please resolve those before we can evaluate the pull request.