Исправляет выпадение антагов кадету СБ

[HMBGERDO]

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

Добавляет профессию кадет СБ в проверки при выборе антагонистов.

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

СБшники не могут быть антагами не просто так

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

Скомпилировал, запустил

Changelog

🆑
fix: Кадету СБ более недоступны антаг роли, как и офицерам СБ
/:cl:

[HMBGERDO]

пр альтернативен изменениям в #723 , так что принимать оба точно не стоит

[PhantornRU]

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

Альтернативу с полностью выпилом проффки и оставления альт-титтла реализовал тут:
#723

[PhantornRU]

Да, по модулям. У нас чтобы в будущем не возникало конфликтов, все изменения стоит выносить в модуль.
Я пробовал точно также делать в модуле и перезаписи. Но увы. Пришлось идти по сложному пути и вместо костыля делать работающую реализацию.

[HMBGERDO]

так, ну видимо мне нужно обосновать, почему это должно работать

возьмем ситуацию, когда у нас в префах выбран кадет на high и нам выпадает генокрад

посмотрим порядок, как это все работает

начинается все с функции pre_setup, в ней определяется, кто может быть антагусом и собственно выбираются игроки для ролей

на данный момент у игрока не выбрана профессия, не указан assigned_role, но записаны restricted_roles

далее по списку идет распределение профессий, а именно SSjobs.DivideOccupations()

в DivideOccupations() у нас идет распределение профессий и, среди прочих проверок, там есть вот такая:

сравнивается текущая профессия, которую хочет выдать система с тем, что выдавать нельзя
при этом среди текущих профессий также проходит и датум Security Cadet, но в restricted_roles до этого PR он указан не будет. Это можно увидеть в логах SSjobs:

Следовательно, проверка пройдет и игроку будет выдана профессия Security Cadet, несмотря на то, что он уже является, например, генокрадом

Ну а дальше вы сами знаете

вот вроде и всё
если есть ошибка - я слушаю

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

[PhantornRU]

Хорошо, он смотрит по Assigned и ты запретил Security Cadet. Что насчет Security Graduate? :)

[HMBGERDO]

У Security Graduate нет своего отдельного job/datum, из-за этого он никогда не будет в этой проверке

[HMBGERDO]

Протестировал с распределением ролей раунд стартом и через create antagonist, работает

[PhantornRU]

Тогда странно что с тем же кодом мне удалось за кадета получить антажку. Я внес все новичковые роли в качестве исключения. Ну да и ладно.

[PhantornRU]

А теперь реши проблему ДЖОБок, когда людей принудительно кидает на роль, если хочешь оставить этот вариант кода.
Либо моим вариантом мы просто отказываемся от них xD

[PhantornRU]

Извини. Но закрываем до переделки роли новичков.