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

Vehicle restriction removal #187

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Pavlovvn
Copy link

@Pavlovvn Pavlovvn commented Feb 28, 2025

Что этот PR делает

Убирает запрет на покупку танка если уже есть единица техники.

Почему это хорошо для игры

Добавляем больше техники в раунд!

Изображения изменений

Тестирование

Changelog

🆑
add: Добавил возможность марам покупать бтр и танк в одном раунде
balance: добавил марам возможность покупать и бтр, и танк в одном и том же раунде
/:cl:

@ss220app ss220app bot added the 📜 CL валиден Этот чейнджлог будет успешно опубликован. label Feb 28, 2025
return FALSE
//if(gearcomp.selected_vehicle == "TANK")
// to_chat(unlocking_mob, SPAN_WARNING ("A vehicle has already been selected for this operation."))
// return FALSE
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ты проверил количество закупок? А то щас окажется что можно сто тыщ танков купить

Говорил же также сделать ограничитель по типу техники + её количество.

Choose a reason for hiding this comment

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

Должно нормально работать, у лифта есть свойство spent, которое ставится TRUE при подъеме техники, ARC при разблокировке перезаписывает лист техники на только ARC и ставит spent FALSE

Copy link
Collaborator

Choose a reason for hiding this comment

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

Нам нужен список "Сколько техники можно поднять по лифту", а не просто перезапись.

@PhantornRU
Copy link
Collaborator

ПР переделать. Нужно не просто закомменчивание, а проверку на то, сколько техники уже было поднято с учетом ограничений по типу техники.
Дополнительно в будущем мы сможем это использовать для "покупок техники".
Покупку АРКа привязать к этому списку ограничения (Купил - значит максимальное число заказа "АРКов" повысилось с 0 до 1. Поднял на лифте АРК, значит число 1 из 1)

@PhantornRU PhantornRU marked this pull request as draft March 1, 2025 15:23
@PhantornRU PhantornRU added the :feelsgood: Частичная модульность Не всегда получается всё впихнуть в модуль, увы. label Mar 1, 2025
@PhantornRU
Copy link
Collaborator

PhantornRU commented Mar 2, 2025

Тебе нужно подзапариться немного и сделать списки доступной для вызове техники.
Объясняю:

  • Изначально доступен 0/1 танка.
  • Берешь 1 танк - всё. Больше не можешь взять танк. 1/1
  • Ты можешь купить/заказать доп технику (АРК). Изначально у всей остальной техники 0/0
  • Покупаешь АРК - становится 0/1.
  • Теперь ты можешь также заказать АРК 1/1.
    И ничто друг друга не пересекает. В этом плане понятно что делать нужно?

условно делаешь:

    var/static/list/available_vehicle_categories = list(
        VEHICLE_HEAVY_AVAILABLE = 1,
        VEHICLE_LIGHT_AVAILABLE = 0,
        VEHICLE_SUPPORT_AVAILABLE = 0
    )

И при вызове техники - отнимаешь отсюда единичку
Соответственно при добавлении новой техники - добавляешь сюда эту единичку.
После пишешь доступность вызова техники с выводом в консоль.

Да, нужно будет немного selected_vehicle тоже переписать

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:feelsgood: Частичная модульность Не всегда получается всё впихнуть в модуль, увы. 📜 CL валиден Этот чейнджлог будет успешно опубликован.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants