Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merge it already] Магические жезлы для мага #13542

Closed
wants to merge 13 commits into from

Conversation

Ro2tCrab
Copy link
Contributor

@Ro2tCrab Ro2tCrab commented Sep 13, 2024

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

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

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

Минусы:

  • Всего 1 заряд, который перезаряжается аналогично оригинальному заклинанию.
  • Может быть украден и использован в полную мощь против вас же.

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

Немного контента, да?

Авторство

  • Идея TG
  • Реализация моя.
  • Спрайты: WROTL

Чеинжлог

🆑

  • rscadd: Маг получает магические жезлы в арсенал.

@TauKitty
Copy link
Contributor

Changelog status: 🔘 No changelog

@TauKitty TauKitty requested a review from a team September 13, 2024 10:33
@WatlerJe
Copy link
Contributor

WatlerJe commented Sep 19, 2024

То есть не маги смогут использовать жезлы?

Напиши мне в дискорде спрайты каких палок нужны.

@Ro2tCrab
Copy link
Contributor Author

Ro2tCrab commented Oct 3, 2024

Чёт ПР немного умер, сейчас пришью спрайты и можно делать ревью.

@Ro2tCrab Ro2tCrab changed the title [WIP-HELP] Магические жезлы для мага [Ready] Магические жезлы для мага Oct 5, 2024
@Ro2tCrab
Copy link
Contributor Author

Да ёб, сейчас всё сделаю
Почему я откладываю всё травильное на 100 лет?

@TauKitty TauKitty requested a review from a team October 12, 2024 10:51
@Ro2tCrab Ro2tCrab marked this pull request as ready for review October 12, 2024 10:52
@Ro2tCrab Ro2tCrab changed the title [Ready] Магические жезлы для мага [Merge it already] Магические жезлы для мага Oct 17, 2024
@Ro2tCrab
Copy link
Contributor Author

Merge conflicts...

Fuuuuuuuu...

/mob/living/simple_animal/hostile/carp/magic/carp_randomify()
return 0

/mob/living/simple_animal/hostile/carp/magic/chaos //Only for shitspawn.. for now. -Ro2tCrab
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Может only for shitspawn удалить на сейчас?

Comment on lines +79 to +96
if(item_state_inventory_off != null){
if(charges > 0)
icon_state = item_state_inventory_on
item_state_inventory = item_state_inventory_on
else
icon_state = item_state_inventory_off
item_state_inventory = item_state_inventory_off
}

/obj/item/weapon/gun/magic/update_world_icon()

if(item_state_world_off != null){
if(charges > 0)
item_state_world = item_state_world_on
else
item_state_world = item_state_world_off
..()
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В бъонде не нужны { и }

return 1

/obj/item/weapon/gun/magic/update_icon()

if(item_state_inventory_off != null){
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну и, для безопасности мб впихнуть if(item_state_inventory_off && item_state_inventory_on)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if(item_state_inventory_off != null){
if(item_state_inventory_off && item_state_inventory_on)


/obj/item/weapon/gun/magic/update_world_icon()

if(item_state_world_off != null){
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
if(item_state_world_off != null){
if(item_state_world_off && item_state_world_on)

Comment on lines +141 to +143



Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

Comment on lines +144 to +148
/obj/item/weapon/gun/magic/wand/attack(mob/living/M, mob/living/user, def_zone)
if(user.a_intent != INTENT_HARM && M == user)
zap_self(user)
else
..()
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

А почему бы это не сделать через attack_self() тот же?


/obj/item/weapon/gun/magic/shoot_with_empty_chamber(mob/living/user)
to_chat(user, "<span class='warning'> Ваш [name] тихо свистит...</span>")
return
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
return


/obj/item/weapon/gun/magic/wand/proc/zap_self(mob/living/user)
if(charges > 0)
user.visible_message("<span class='danger'> [user] стреляет в себя из [src].</span>")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
user.visible_message("<span class='danger'> [user] стреляет в себя из [src].</span>")
user.visible_message("<span class='danger'>[user] стреляет в себя из [src].</span>")

charges--
update_icon()
else
to_chat(user, "<span class='warning'> Ваш [name] тихо свистит...</span>")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
to_chat(user, "<span class='warning'> Ваш [name] тихо свистит...</span>")
to_chat(user, "<span class='warning'>Ваш [name] тихо свистит...</span>")

Мб заменить [name] на "жезл"? Ты точно хочешь с падежами и склонениями возиться? Просто, сюда бы добавить штуку, чтобы от name ваш-ваше-ваша изменялось

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

лучше падежи сразу прописать, или оставить как есть и падежи потом кто-то другой сделает

item_state_world_off = "heal_off_world"
var/heal_power = -100
fire_sound = 'sound/magic/Staff_Healing.ogg'

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

item_state_world_off = "wall_off_world"
fire_sound = 'sound/magic/Staff_Door.ogg'
recharge_rate = 10

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

@@ -202,7 +207,7 @@

/obj/item/projectile/process(boolet_number = 1) // we add default arg value, because there is alot of uses of projectiles without guns (e.g turrets).
var/first_step = 1

original_loc = get_turf(original)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
original_loc = get_turf(original)
if(hitturf)
original_loc = get_turf(original)

Оптимизации ради

Comment on lines +347 to +351





Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

light_color = "#b63aa3"

/obj/item/projectile/magic/wand/magic_carp/do_effect(target)
var/final_loc = target
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Меня немного пугает, что механ "атаки по себе" и выстрела одинаковый, но код копипасщеный, че думаешь?

Copy link

github-actions bot commented Nov 2, 2024

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

Copy link

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

@github-actions github-actions bot closed this Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants