bugfix/add: В синдикарго категория Miscellaneous не ломает консоль, можно заказать реверсивные карты в разном и добавление ядра МЭКов для синдикарго#8534
Conversation
Нету группы contraband в синди карго, раз он находиться в списке misc добавил туда
Взято с обычного карго
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
Аналитический отчёт от Большого Брата. Обработка запроса завершена. Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит удовлетворительно. Вы успешно исправили проблему с консолью при открытии категории Miscellaneous и переместили реверсивные карты в соответствующую категорию. Также был добавлен новый набор ядер МЭК для синдиката.
Сводка анализа:
- 🟡 Рекомендаций: 2
- 💡 Предложений: 0
Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации. Спасибо за ваш вклад в проект!
| /datum/syndie_supply_packs/science/modcore | ||
| name = "Ящик ядер МЭК" | ||
| contains = list( | ||
| /obj/item/mod/core/standard, | ||
| /obj/item/mod/core/standard, | ||
| /obj/item/mod/core/standard, | ||
| /obj/item/mod/core/standard, | ||
| ) | ||
| cost = 5000 | ||
| access = ACCESS_ROBOTICS | ||
| containertype = /obj/structure/closet/crate/secure/scisec | ||
| containername = "ящик с ядрами МЭК" | ||
| container_ru_names = list( | ||
| NOMINATIVE = "ящик с ядрами МЭК", | ||
| GENITIVE = "ящика с ядрами МЭК", | ||
| DATIVE = "ящику с ядрами МЭК", | ||
| ACCUSATIVE = "ящик с ядрами МЭК", | ||
| INSTRUMENTAL = "ящиком с ядрами МЭК", | ||
| PREPOSITIONAL = "ящике с ядрами МЭК", | ||
| ) |
There was a problem hiding this comment.
🟡 Рекомендация по читаемости и согласованности
Новый набор /datum/syndie_supply_packs/science/modcore наследуется от /datum/syndie_supply_packs/science, но не имеет явного определения group. Хотя базовый syndie_supply_packs по умолчанию использует SYNDIE_SUPPLY_MISC, для ясности и соответствия иерархии рекомендуется явно указать group = SYNDIE_SUPPLY_SCIENCE.
Предлагаемое изменение:
/datum/syndie_supply_packs/science/modcore
name = "Ящик ядер МЭК"
contains = list(
/obj/item/mod/core/standard,
/obj/item/mod/core/standard,
/obj/item/mod/core/standard,
/obj/item/mod/core/standard,
)
cost = 5000
access = ACCESS_ROBOTICS
group = SYNDIE_SUPPLY_SCIENCE
containertype = /obj/structure/closet/crate/secure/scisec
containername = "ящик с ядрами МЭК"
container_ru_names = list(
NOMINATIVE = "ящик с ядрами МЭК",
GENITIVE = "ящика с ядрами МЭК",
DATIVE = "ящику с ядрами МЭК",
ACCUSATIVE = "ящик с ядрами МЭК",
INSTRUMENTAL = "ящиком с ядрами МЭК",
PREPOSITIONAL = "ящике с ядрами МЭК",
)
AurothWyvern
left a comment
AurothWyvern
left a comment
There was a problem hiding this comment.
Русский язык в норме 👍
3 participants
Что этот ПР делает
Синдикарго:
Открытие Miscellaneous больше не ломает интерфейс консоли.
Меняет путь ящика с Реверсивными картами с несуществующей группы contraband на misc(из-за чего ломалась консоль).
Добавляет ящик ядер МЭКов, взято с обычного (возможно стоит подкорректировать цену).
Почему это хорошо для игры
Вновь возможно заказать в синдикарго что-то из категорий разного и реверсивные карты.
Ящик ядер МЭКов для тайпана, теперь можно собирать МЭКи на тайпане, не через плазмоядра.
Демонстрация изменений
Демонстрации изменений
Тестирование
На локалке рантаймов не заметил