fix(advtool): removing necessary parents

[BaraBarax]

Создавать целый отдельный подтип инструментов ради одного крутого мультитула это конечно круто, но почему бы просто не засунуть его в сыновья обычного для меньшей мозгоёбки?

close #12856
close #12044

Чейнджлог

🆑BaraBara
bugfix: Продвинутый мультитул теперь может взаимодействовать с телекоммами.
/🆑

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

[TobyThorne]

ОП не осилил факт, что смысл тула - в переключении между мультитулом, отвёрткой и кусачками.

[BaraBarax]

А он потерял эту возможность? Он в первую очередь мультитул, в смысле хуйня для тыканья в проводки, к которому приделали отвёртку и кусачки

[Filatelele]

Честно? Проблема не в коде адвтула. И решаться должна не так.

/obj/machinery/telecomms/attack_hand(mob/user)

	if(stat & (BROKEN|NOPOWER))
		return TRUE

	var/obj/item/device/multitool/P = get_multitool(user)

	if(!P)
		return TRUE

/proc/get_multitool(mob/user)
	var/obj/item/device/multitool/MT = null
	if(!issilicon(user))
		if(isMultitool(user.get_active_hand()))
			MT = user.get_active_hand()
	else if(isAI(user))
		var/mob/living/silicon/ai/AI = user
		MT = AI.aiMulti
	else if(isrobot(user))
		if(isMultitool(user.get_active_hand()))
			MT = user.get_active_hand()
	return MT

Здесь нахуй что-то не то, потому что get_multitool() возвращает любой айтем у которого тул бихейвиор == TOOL_MULTITOOL

Но только device/multitool умеет в get_buffer() и аналогичное

И адекватным решением будет перевод поведения мультитула на компонент, что во-первых позволяет делать ну совсем продвинутую ржаку типа бесконтактных мультитулов, а во вторых - уменьшает количество говна в кодбазе.

[BaraBarax]

Ну я рвать жопу ради того чтобы тыкнуть более крутым мультитулом в телекоммы не собираюсь (может быть, когда-нибудь, наверно, возможно займусь этим потом), ну и если по этой причине ПР не принимается, то я его закрываю просто

[Filatelele]

Не обязательно. На будущее. Есть крутое решение, чтобы не ебаться с присвоением переменных когда чет дропаешь в атом.

Переопределяешь Entered() и когда чет попадает внутрь - смотришь по изтайпу и присваиваешь в нужную переменную. Аналогично с Exited() - обнуляешь ссылки.

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

[Filatelele]

А нахуя я ревью делал если автор все закрыл?
Сука.

[BaraBarax]

Уэээ, ну я подумал что если все молчат то так и будет

[Filatelele]

Уэээ, ну я подумал что если все молчат то так и будет

Ну решение очевидно субоптимальное но я хз того рот ебал
Пусть Игорь решает мне впадлу думать ревув кода я дал
Но компонент с сигналами это сто строк кода не более