local: Перевод 5 предметов из автолата в разделе ИНСТРУМЕНТЫ.#8543
local: Перевод 5 предметов из автолата в разделе ИНСТРУМЕНТЫ.#8543AL5900 wants to merge 19 commits intoss220-space:master220from
Conversation
…ute от Доминатора к Мультифазке.
…има execute от Доминатора к Мультифазке." This reverts commit de9ea6e.
There was a problem hiding this comment.
Code Review
Аналитический отчёт от Большого Брата. Обработка запроса завершена.
Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит требующей доработок.
Сводка анализа:
- 🔴 Критических замечаний: 0
- 🟡 Рекомендаций: 11
- 💡 Предложений: 0
🟡 Рекомендации по улучшению:
code/game/objects/items/crayons.dm:90: Использование[target.name]не обеспечивает корректное склонение. Рекомендуется использоватьdeclent_ru().code/game/objects/items/crayons.dm:99: Использование[name]не обеспечивает локализацию и склонение. Рекомендуется использоватьDECLENT_RU_CAP().code/game/objects/items/crayons.dm:294: Некорректное склонение в винительном падеже для неодушевлённого существительного.code/game/objects/items/crayons.dm:307: Неверно выбран падеж для конструкции «у [чего-то]». Требуется родительный падеж.code/game/objects/items/crayons.dm:352: Неверно выбран падеж для глагола «израсходовать».code/game/objects/items/devices/scanners/gas_analyzer.dm:52: Неверно выбран падеж для действия «сканировать [чем-то]».code/game/objects/items/devices/scanners/gas_analyzer.dm:57: В сообщении о состоянии барометра утеряна ссылка на сам объект, что снижает ясность. Это также относится к строкам 69, 80, 85, 91, 93, 102.code/game/objects/items/devices/scanners/gas_analyzer.dm:255: Имя объекта «Газоанализатор» захардкожено. Рекомендуется использоватьDECLENT_RU_CAP()для поддержки переименований и других состояний объекта.code/modules/clothing/head/misc_special.dm:64: В имени объектаnameприсутствует точка в конце, что является опечаткой.code/modules/power/cable_coil.dm:107: Строка для меню крафтаmulti z cable hub (10)не переведена, что нарушает консистентность локализации.code/modules/power/cable_coil.dm:249: Конструкция «с помощью [родительный падеж]» грамматически верна, но более краткой и идиоматичной является конструкция с творительным падежом.
Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации.
Спасибо за ваш вклад в проект!
|
|
||
| if(!isturf(T) || !T.can_lay_cable()) | ||
| to_chat(user, span_warning("You can only lay cables on catwalks and plating!")) | ||
| to_chat(user, span_warning("Вы можете прокладывать проводку только по обшивке и мосткам!")) |
There was a problem hiding this comment.
это склонение от "мосток" или пропущенная "и" в слове "мостикам"?
There was a problem hiding this comment.
Это склонение от "Мосток"
pilygun
left a comment
There was a problem hiding this comment.
Стоит добавить балун алерты в этих местах, для ебейшего QOL
По поводу функции барометра, я ей не пользовался ни разу, по коду понял что работает через альт клик
ЕСЛИ ЭТО НЕ ТАК и оно активируется через ТУГИ, то закрой мои комиты связанные с барометром
| /obj/item/analyzer/click_alt(mob/living/user) //Barometer output for measuring when the next storm happens | ||
| if(cooldown) | ||
| to_chat(user, span_warning("[src]'s barometer function is prepraring itself.")) | ||
| to_chat(user, span_warning("Функция барометра готовится к работе.")) |
There was a problem hiding this comment.
Замени на балун алерт "перезарядка" опционально добавить цифры времени КД
|
|
||
| if(!user_area.outdoors) | ||
| to_chat(user, span_warning("[src]'s barometer function won't work indoors!")) | ||
| to_chat(user, span_warning("Функция барометра не будет работать в помещении!")) |
There was a problem hiding this comment.
Балун алерт, "не работает в помещении"
| if(ongoing_weather) | ||
| if((ongoing_weather.stage == MAIN_STAGE) || (ongoing_weather.stage == WIND_DOWN_STAGE)) | ||
| to_chat(user, span_warning("[src]'s barometer function can't trace anything while the storm is [ongoing_weather.stage == MAIN_STAGE ? "already here!" : "winding down."]")) | ||
| to_chat(user, span_warning("Функция барометра не может ничего отследить, пока буря. [ongoing_weather.stage == MAIN_STAGE ? "идёт!" : "стихает."]")) |
There was a problem hiding this comment.
Балун алерт, "не работает во время бури"
| return CLICK_ACTION_BLOCKING | ||
|
|
||
| to_chat(user, span_warning("The next [ongoing_weather] will hit in [butchertime(ongoing_weather.next_hit_time - world.time)].")) | ||
| to_chat(user, span_warning("Следующая [ongoing_weather] начнется [butchertime(ongoing_weather.next_hit_time - world.time)].")) |
There was a problem hiding this comment.
ту чат на балун, текст не менять
| to_chat(user, span_warning("Следующая [ongoing_weather] начнется [butchertime(ongoing_weather.next_hit_time - world.time)].")) | ||
| if(ongoing_weather.aesthetic) | ||
| to_chat(user, span_warning("[src]'s barometer function says that the next storm will breeze on by.")) | ||
| to_chat(user, span_warning("Функция барометра информирует, что следующая буря пройдёт мимо.")) |
|
|
||
| if(get_dist(T,user.loc) > 1) // Too far | ||
| to_chat(user, span_warning("You can't lay cable at a place that far away!")) | ||
| to_chat(user, span_warning("Вы не можете проложить проводку так далеко!")) |
| for(var/obj/structure/cable/LC in T) | ||
| if(LC.d2 == dirn && LC.d1 == 0) | ||
| to_chat(user, span_warning("There's already a cable at that position!")) | ||
| to_chat(user, span_warning("В этом месте уже есть проводка!")) |
|
|
||
| if(get_dist(C, user) > 1) // make sure it's close enough | ||
| to_chat(user, span_warning("You can't lay cable at a place that far away!")) | ||
| to_chat(user, span_warning("Вы не можете проложить проводку так далеко!")) |
| for(var/obj/structure/cable/LC in U) // check to make sure there's not a cable there already | ||
| if(LC.d1 == fdirn || LC.d2 == fdirn) | ||
| to_chat(user, span_warning("There's already a cable at that position!")) | ||
| to_chat(user, span_warning("В этом месте уже есть проводка!")) |
| continue | ||
| if((LC.d1 == nd1 && LC.d2 == nd2) || (LC.d1 == nd2 && LC.d2 == nd1)) // make sure no cable matches either direction | ||
| to_chat(user, span_warning("There's already a cable at that position!")) | ||
| to_chat(user, span_warning("В этом месте уже есть проводка!")) |
|
@pilygun Ну я завтра посмотрю что за балуны, если разберусь - попробую сделать. |
Что этот ПР делает
Переводит имя, описание и разные действия 5 предметов из категории "Инструменты" в автолате, а именно:
Analyzer
Cable Coil
Nanotrasen-Brand Rapid Paint Applicator
T-Ray Scanner
Welding Helmet
Тестирование
Перевод протестила, работает, ошибок (вроде как) не обнаружено.