Фикс блюспейс рюкзака и пересмотренное возвращение механики неисправности интерфейса #13724

hyyda · 2025-01-11T20:53:29Z

Описание изменений

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

Почему и что этот ПР улучшит

Исправляет баги и возвращает старую механику в менее грифозном виде

Авторство

hyda, danistans

Чеинжлог

🆑 HyDa, Danistans

bugfix: Рюкзак корректно исчезает при засовывании сам в себя.
bugfix: Аномалии созданные не ивентом теряли свои свойства.
rscadd: Блюспейс рюкзак теперь при засовывании в другой блюспейс рюкзак создает аномалию.

TauKitty · 2025-01-11T20:53:32Z

Changelog status: ❌ Empty changelog (check markdown correctness)

KIBORG04 · 2025-01-11T21:56:50Z

code/game/objects/items/weapons/storage/backpack.dm

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

KIBORG04 · 2025-01-11T21:57:51Z

code/game/objects/items/weapons/storage/backpack.dm

 		return

-	return ..()
+/obj/item/weapon/storage/backpack/holding/proc/Make_Anomaly(delay_time, current_anomaly)


Функцию бы вызывать асинхронно, так как есть слип. А я не очень хочу, чтобы attackby и handle_item_insertion останавливались

KIBORG04 · 2025-01-11T21:58:00Z

code/game/objects/items/weapons/storage/backpack.dm

@@ -49,17 +52,46 @@
 	max_w_class = SIZE_NORMAL
 	max_storage_space = 56

+	var/global/IsHoldingMalfunction = FALSE


Не оч похоже на глобал

KIBORG04 · 2025-01-11T21:58:12Z

code/game/objects/effects/anomalies.dm

 	QDEL_NULL(warp)
 	return ..()

+/obj/effect/anomaly/grav/process() // Спасибо ДаркВотеру


У нас принято писать комменты на инглише

KIBORG04 · 2025-01-11T21:58:25Z

code/game/objects/items/weapons/storage/backpack.dm

@@ -28,6 +28,9 @@
 	S.opened = !S.opened

 /obj/item/weapon/storage/backpack/attackby(obj/item/I, mob/user, params)
+	if(I == src)


Это как

KIBORG04 · 2025-01-11T22:00:08Z

code/game/objects/items/weapons/storage/backpack.dm

+/obj/item/weapon/storage/backpack/holding/handle_item_insertion(obj/item/W, prevent_warning = FALSE, NoUpdate = FALSE)
+	. = ..()
+	if(W == src)
+		Destroy() // каким-то образом удаляет экшен меню при попытке воспроизвести баг и фиксит его


У нас принято писать комменты на инглише

KIBORG04 · 2025-01-11T22:00:50Z

code/game/objects/items/weapons/storage/backpack.dm

+/obj/item/weapon/storage/backpack/holding/handle_item_insertion(obj/item/W, prevent_warning = FALSE, NoUpdate = FALSE)
+	. = ..()
+	if(W == src)
+		Destroy() // каким-то образом удаляет экшен меню при попытке воспроизвести баг и фиксит его


Хотя бы честно. А теперь разберись или пробуй без этого костыля

Как вообще можно вставить свой же рюкзак в рюкзак?

баг причиной которого и стал пр, если Е прожать с открытым блюспейс рюкзаком и занятым слотом спины рюкзак исчезнет, а экшен меню с которым можно взаимодействовать останется

Крч, вестимо нужно не ЭТО делать, а тыкнуть куда-то проверку в /mob/living/carbon/human/verb/quick_equip(), чтобы нельзя было положить себя в себя

KIBORG04 · 2025-01-11T22:01:17Z

code/game/objects/items/weapons/storage/backpack.dm

+		Destroy() // каким-то образом удаляет экшен меню при попытке воспроизвести баг и фиксит его
+		to_chat(usr, "<span class='red'>Рюкзак засасывается сам в себя и исчезает.</span>")
+		return
+	if(istype(W, /obj/item/weapon/storage/backpack/holding/) && !IsHoldingMalfunction && !(W == src))


Suggested change

if(istype(W, /obj/item/weapon/storage/backpack/holding/) && !IsHoldingMalfunction && !(W == src))

if(istype(W, /obj/item/weapon/storage/backpack/holding) && !IsHoldingMalfunction && !(W == src))

KIBORG04 · 2025-01-11T22:01:59Z

code/game/objects/items/weapons/storage/backpack.dm

+	var/obj/effect/anomaly/anomaly = new current_anomaly(targloc)
+	IsHoldingMalfunction = TRUE
+	sleep(delay_time)
+	anomaly.Destroy()


Suggested change

anomaly.Destroy()

qdel(anomaly)

Для любого удаления чего-то используй qdel()

KIBORG04 · 2025-01-11T22:02:54Z

code/game/objects/items/weapons/storage/backpack.dm

+/obj/item/weapon/storage/backpack/holding/proc/Adjust_Malfunction_Global_Timer()
+	var/timer = 30 MINUTES
+	for(timer)
+		timer -= 1 SECOND
+		if(!timer)
+			IsHoldingMalfunction = FALSE


?????????????????????????????????

Ты в курсе, что оно сработает практически мгновенно?

Что это и зачем? Если хочешь IsHoldingMalfunction = FALSE сетать, то просто перенесни к тому слипу в Make_Anomaly

KIBORG04 · 2025-01-11T22:05:08Z

code/game/objects/items/weapons/storage/backpack.dm

 		return

-	return ..()
+/obj/item/weapon/storage/backpack/holding/proc/Make_Anomaly(delay_time, current_anomaly)


А еще, грав аномалия как-то скучно, надо бы че-то пожосче

KIBORG04 · 2025-01-11T22:13:16Z

code/game/objects/items/weapons/storage/backpack.dm

+/obj/item/weapon/storage/backpack/holding/handle_item_insertion(obj/item/W, prevent_warning = FALSE, NoUpdate = FALSE)
+	. = ..()
+	if(W == src)
+		Destroy() // каким-то образом удаляет экшен меню при попытке воспроизвести баг и фиксит его
+		to_chat(usr, "<span class='red'>Рюкзак засасывается сам в себя и исчезает.</span>")
+		return
+	if(istype(W, /obj/item/weapon/storage/backpack/holding/) && !IsHoldingMalfunction && !(W == src))
+		to_chat(usr, "<span class='red'>The Bluespace interfaces of the two devices conflict and malfunction.</span>")
+		Make_Anomaly(150 SECONDS, /obj/effect/anomaly/grav)
 		return


Suggested change

/obj/item/weapon/storage/backpack/holding/handle_item_insertion(obj/item/W, prevent_warning = FALSE, NoUpdate = FALSE)

. = ..()

if(W == src)

Destroy() // каким-то образом удаляет экшен меню при попытке воспроизвести баг и фиксит его

to_chat(usr, "Рюкзак засасывается сам в себя и исчезает.")

return

if(istype(W, /obj/item/weapon/storage/backpack/holding/) && !IsHoldingMalfunction && !(W == src))

to_chat(usr, "The Bluespace interfaces of the two devices conflict and malfunction.")

Make_Anomaly(150 SECONDS, /obj/effect/anomaly/grav)

return

/obj/item/weapon/storage/backpack/holding/handle_item_insertion(obj/item/W, prevent_warning = FALSE, NoUpdate = FALSE)

. = ..()

if(W == src)

Destroy() // каким-то образом удаляет экшен меню при попытке воспроизвести баг и фиксит его

to_chat(usr, "Рюкзак засасывается сам в себя и исчезает.")

return FALSE

if(istype(W, /obj/item/weapon/storage/backpack/holding/) && !IsHoldingMalfunction && !(W == src))

to_chat(usr, "The Bluespace interfaces of the two devices conflict and malfunction.")

Make_Anomaly(150 SECONDS, /obj/effect/anomaly/grav)

return FALSE

return TRUE

hyyda added 6 commits January 10, 2025 23:48

Update backpack.dm

6e7a810

Update backpack.dm

afda9a8

Update backpack.dm

ad941bc

Update anomalies.dm

7b044b5

Update backpack.dm

52d1fd9

Update backpack.dm

f0e0b0f

KIBORG04 reviewed Jan 11, 2025

View reviewed changes

hyyda added 4 commits January 12, 2025 03:03

Update backpack.dm

c8c09d5

Update backpack.dm

851a1e6

Update anomalies.dm

b3a5bca

Update anomalies.dm

e573fb6

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Фикс блюспейс рюкзака и пересмотренное возвращение механики неисправности интерфейса #13724

Фикс блюспейс рюкзака и пересмотренное возвращение механики неисправности интерфейса #13724

hyyda commented Jan 11, 2025 •

edited

Loading

TauKitty commented Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

hyyda Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

KIBORG04 Jan 11, 2025

	if(istype(W, /obj/item/weapon/storage/backpack/holding/) && !IsHoldingMalfunction && !(W == src))
	if(istype(W, /obj/item/weapon/storage/backpack/holding) && !IsHoldingMalfunction && !(W == src))

Фикс блюспейс рюкзака и пересмотренное возвращение механики неисправности интерфейса #13724

Are you sure you want to change the base?

Фикс блюспейс рюкзака и пересмотренное возвращение механики неисправности интерфейса #13724

Conversation

hyyda commented Jan 11, 2025 • edited Loading

Описание изменений

Почему и что этот ПР улучшит

Авторство

Чеинжлог

TauKitty commented Jan 11, 2025

Changelog status: ❌ Empty changelog (check markdown correctness)

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hyyda commented Jan 11, 2025 •

edited

Loading