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

Перевод слипера и названий реагентов (и немного бутылочек) #12584

Closed
wants to merge 32 commits into from

Conversation

maleyvich
Copy link
Contributor

@maleyvich maleyvich commented Dec 7, 2023

Описание изменений

переведен аппарат диализа (Sleeper) и названия реагентов, части кода связанные с баночками
Resolves TauCetiStation-TranslationDepartment#8
фотокарточки:
image
image
image

Почему и что этот ПР улучшит

перевод

Авторство

maleyvich, трансдеп таукиты

Чеинжлог

🆑

  • ru: Аппарат "Sleeper", названия реагентов и часть кода связанная с наполнением бутылок/таблеток были переведены на русский язык.

@TauKitty
Copy link
Contributor

TauKitty commented Dec 7, 2023

Changelog status: ✔️

@NinjaPikachuska
Copy link
Contributor

здесь даже половины нет от всех реагентов

@AndroBetel
Copy link
Member

почему не перевести слипер как слипер, если все его так называют, какая нахуй камера диализа

@Kiromaka
Copy link
Contributor

Kiromaka commented Dec 7, 2023

почему не перевести слипер как слипер, если все его так называют, какая нахуй камера диализа

Он уже давно не слипер по функционалу, + хуита. Камера Диализа мне тоже не нравится, но что есть, то есть.

@maleyvich
Copy link
Contributor Author

почему не перевести слипер как слипер, если все его так называют, какая нахуй камера диализа

по результатам обсуждения трансдепа

@Kiromaka
Copy link
Contributor

Kiromaka commented Dec 7, 2023

Нейэмы переведены... не нравиться мне это.... падежи нужны падежи...

code/game/machinery/Sleeper.dm Outdated Show resolved Hide resolved
@AzzyDreemurr13
Copy link
Contributor

В описание изменений добавь:
Resolves TauCetiStation-TranslationDepartment#8

@maleyvich
Copy link
Contributor Author

здесь даже половины нет от всех реагентов

теперь есть содержание пра согласно описанию...

Copy link
Contributor

@AzzyDreemurr13 AzzyDreemurr13 left a comment

Choose a reason for hiding this comment

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

Указал, что можно было бы поменять. А также насчёт неймов... нам же наверное нужны будут падежи для каждого реагента, чтобы не было проблем.

code/game/machinery/Sleeper.dm Outdated Show resolved Hide resolved
code/game/machinery/Sleeper.dm Outdated Show resolved Hide resolved
code/game/machinery/Sleeper.dm Outdated Show resolved Hide resolved
code/game/machinery/Sleeper.dm Outdated Show resolved Hide resolved
code/game/machinery/Sleeper.dm Outdated Show resolved Hide resolved
code/game/machinery/Sleeper.dm Outdated Show resolved Hide resolved
code/game/machinery/Sleeper.dm Outdated Show resolved Hide resolved
code/game/machinery/Sleeper.dm Outdated Show resolved Hide resolved
code/game/machinery/Sleeper.dm Outdated Show resolved Hide resolved
@maleyvich
Copy link
Contributor Author

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

@AzzyDreemurr13
Copy link
Contributor

@maleyvich в описание ещё один резовл добавь
Resolves #12613

..(target)

/obj/machinery/sleeper/proc/inject_chem(mob/user, chem)
if(occupant && occupant.reagents)
if(occupant.reagents.get_reagent_amount(chem) + 10 <= 20 * efficiency)
occupant.reagents.add_reagent(chem, 10)
var/units = round(occupant.reagents.get_reagent_amount(chem))
to_chat(user, "<span class='notice'>Occupant now has [units] unit\s of [chem] in their bloodstream.</span>")
to_chat(user, "<span class='notice'>Пациенту было введено [units] юнит(ов) [chem] в организм.</span>")
Copy link
Contributor

Choose a reason for hiding this comment

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

Пациенту было введено 10 юнит(ов) anti_toxin в организм)))

Copy link
Contributor

Choose a reason for hiding this comment

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

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

хм я подумал как можно было бы решить эту проблему

Copy link
Contributor Author

Choose a reason for hiding this comment

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

хм идея не сработала

Copy link
Contributor

Choose a reason for hiding this comment

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

chem.name ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ne rabotaer

Copy link
Contributor

@Chip11-n Chip11-n Dec 15, 2023

Choose a reason for hiding this comment

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

Тайп пропиши химикатам и окончание юнитам сделай нормальное, есть инструмент для этого

@maleyvich
Copy link
Contributor Author

@maleyvich в описание ещё один резовл добавь Resolves #12613

он не фиксит эту проблему

name = "Serotrotium"
name = "Серотротий"
Copy link
Member

Choose a reason for hiding this comment

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

Если надо, CASE можно на уровень датума перенести

Copy link
Contributor

Choose a reason for hiding this comment

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

Ты не хочешь, чтобы в реагентах был перевод неймов?

Copy link
Member

Choose a reason for hiding this comment

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

а, я понял вас. Наверно. Надо только почекать, чтоб это ничего не сломало

Copy link
Contributor

Choose a reason for hiding this comment

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

@maleyvich - проверь ещё раз, и ответь.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

из самого неприятного это во всяких автоматах барменских и в химии будет слегка неконсинстентно, но химию надо определенно позже затрагивать.
а так я проверял ещё неделю назад, всё нормально работает

Copy link
Member

Choose a reason for hiding this comment

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

почекать - стоит именно весь билд проверить, где может использоваться name реагентов, возможно где-то есть хардкод со старыми названиями. Это лучше кому-то опытному, ну или я ближе к мержу прогляжу

Copy link
Member

Choose a reason for hiding this comment

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

в общем я пробежался, как минимум поиском по R.name:

  1. /obj/item/mecha_parts/mecha_equipment/syringe_gun/ использует name и английские названия реагентов
  2. /obj/machinery/chem_master/Topic проверяет кровь по английскому названию name
  3. в нескольких местах в логах используется get_english_list(injected), где injected - список имен, подразумевающихся английских
  4. get_imprinter_data в рнд консоли

в целом /datum/reagent в билде упомянут 936 раз, и проверять всё слишком долго. Я бы всё же предложил не изобретать велосипеды, а делать, как уже делали - через CASE. Это решает и проблему совместимости, и проблему склонений реагентов


var/datum/browser/popup = new(user, "sleeper", "Sleeper Console", 520, 605) //Set up the popup browser window
var/datum/browser/popup = new(user, "капсула диализа", "консоль капсулы диализа", 520, 605) //Set up the popup browser window
Copy link
Contributor

Choose a reason for hiding this comment

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

а зачем айдишник окна переводить? и почему название окошка с маленькой

Copy link
Contributor Author

Choose a reason for hiding this comment

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

айдишник переведен чтобы прям точно всё было на русском? а маленькие буквы исправил, да.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

оно ничего не ломает а название окошечка исправлено

Copy link
Contributor

Choose a reason for hiding this comment

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

айдишник переводить не стоит
это не то, что показывает юзеру

Copy link
Contributor Author

Choose a reason for hiding this comment

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

хм хорошо перевод айдишника уберу

Copy link
Contributor

Choose a reason for hiding this comment

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

Может быть стоит это просто назвать медицинской капсулой, так как по сути в ней много функционала, введение в стазис, диализ, а также ввод медикаментов различного назначения. В той же научной фантастика название вписывается довольно органично.

Copy link
Member

@volas volas left a comment

Choose a reason for hiding this comment

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

https://github.com/TauCetiStation/TauCetiClassic/pull/12584/files#r1436260540

переместите cases на уровень датумов или реагентов, и переводите их. Name реагентов всё еще может быть проблемно делать, ПР и так большой чтоб еще по всему билду исправления вносить

Copy link

github-actions bot commented Jan 9, 2024

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

@AzzyDreemurr13
Copy link
Contributor

текс, теперь cases на датумах, можно продолжить

@AzzyDreemurr13
Copy link
Contributor

@maleyvich тчк сюда, пока бот не закрыл

@maleyvich
Copy link
Contributor Author

пр жив

@volas
Copy link
Member

volas commented Jan 17, 2024

пр пока скорее мертв, вернетесь к нему - слапните, я переоткрою

@volas volas closed this Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Translation/Localization Перевод на русский
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Review]Перевод Sleeper.dm