bugfix: pAI bugfixes + localization #5874

Samirakis · 2024-09-07T22:23:49Z

Описание

Список исправленных ошибок:

Исправлены надписи у абилок, возвращена картинка у pai suicide
Исправлены проблемы с потерей памяти при сбросе модулей у СпИИ, убран ненужный костыль с памятью
Пофикшен баг при установке в уже работающий пИИ улучшения до СпИИ: модуль не записывался в слот апгрейда, из чего вытекали проблемы с софтом
Исправлена ошибка, при которой установка в уже активную личность улучшения СпИИ(НЕ из протолата) не выдавала новоиспечённому СпИИ ключ шифрования синдиката, оставляя его "глухим". Ключ выдавался только при установке в пустую карту без личности
Убран абуз, при котором отдельные от основного меню окна TGUI (GPS/Camera Bug) оставались открытыми после сброса модулей, позволяя пользоваться ими бесплатно
Починен ГПС пИИ
Убран второй ГПС пИИ от силиконов, всунутый в него апдейтом модулей боргов
Исправлена ошибка, при которй вынимание чипа улучшения СПИИ из карты отвёрткой удаляло чип вместо того, чтобы достать его
Убран абуз с загрузкой двух личностей на одно устройство пИИ
Убрана возможность открыть своё же меню карты пИИ через клик по устройству в меню TGUI справа
Мёртвый пИИ больше не умирает повторно от ЭМИ, при выгрузке пИИ выключается свет у карты

QoL изменения:

Добавлена возможность убирать заявку на становление пИИ, добавлен звук на включенные на поиск карты пИИ при отправке заявки(с кд)
Переведены все тексты(не ТГУИ/УИ интерфейсы), относящиеся к пИИ, заменены старые инпуты на тгуи инпуты
Добавлено приветственное сообщение для пИИ при загрузке

Тесты

Часы тестов на локалке с попыткой сломать пИИ любыми возможными способами, а затем проверок, всё ли работает должным образом

code/game/objects/items/devices/paicard.dm

code/modules/mob/living/silicon/pai/pai.dm

Samirakis · 2024-09-08T10:58:05Z

Мелкие правки перевода закинул, дилемму с ru_names() пусть разрешает главлокализатор как самый компетентный, ему этот вопрос уже задали
Ответ от главнердов получен, интелкарта пИИ остаётся

Ready to Review

pilygun · 2024-09-08T15:42:57Z

Тут лучше балун алерты и текст чутка сократить под балуны

Samirakis · 2024-09-08T15:52:37Z

Тут лучше балун алерты и текст чутка сократить под балуны

Ты бы вроде как этим занимался, мог бы и сам раньше этого..ну.. Сделаю вечерком

Den109G

По хорошему, надо бы разделить этот ПР на 2. Один на локализацию, второй - на багфикс. Пока тут больше перевода, чем всего - это по хорошему как переводу пройти через локализаторов.

Samirakis · 2024-09-21T17:17:39Z

Так я же обсуждал спорные моменты с локализатором и даже с главлороведом. Тем более если у кого либо из локализаторов(а живые есть вообще?) буду вопросыт, то я в тот же день смогу сделать ПР с правками.
Я уже не буду пилить этот фикс на два, как и не буду все свои наработки по текущим НЕ багфикс ПРам разбивать на "пр" и "локализация к ПРу". Но в будущем постараюсь делать именно багфиксы отдельно от переводов

commit eb10085 Merge: b380c2b bbfd8fd Author: Samirakis <[email protected]> Date: Thu Oct 3 01:37:51 2024 +0300 Merge remote-tracking branch 'upstream/master220' into pai_reset commit b380c2b Author: Samirakis <[email protected]> Date: Thu Oct 3 01:37:48 2024 +0300 review2 commit c6820d7 Author: Samirakis <[email protected]> Date: Tue Sep 10 11:19:42 2024 +0300 selfrewiew commit 074b07a Author: Samirakis <[email protected]> Date: Sun Sep 8 20:09:22 2024 +0300 translate2 commit ac29875 Author: Samirakis <[email protected]> Date: Sun Sep 8 13:52:44 2024 +0300 small translation fix commit a9f8b67 Author: Samirakis <[email protected]> Date: Sun Sep 8 01:20:46 2024 +0300 fixes2 commit b141abf Merge: 081bcb1 9fbb427 Author: Samirakis <[email protected]> Date: Sat Sep 7 14:49:09 2024 +0300 Merge remote-tracking branch 'upstream/master220' into pai_reset commit 081bcb1 Author: Samirakis <[email protected]> Date: Sat Sep 7 14:48:57 2024 +0300 bugfix + some translation

github-actions · 2024-10-06T22:30:51Z