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

Перевод фраз вендоматов и не только! #42

Closed
wants to merge 43 commits into from

Conversation

Frogich228
Copy link

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

Это мой первый ПР так что надеюсь всё сделано нормально.

Copy link
Author

Choose a reason for hiding this comment

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

не в курсах чё с текстом, в бранче всё нормально а тут белеберда какая то... попробую всё одним файлом закинуть надеюсь не доломаю всё к чертям

Copy link
Author

Choose a reason for hiding this comment

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

Воооось, теперь без PqiPdG вместо кириллицы

modular_ss220/modules/translations/code/vending_ru.dm Outdated Show resolved Hide resolved
modular_ss220/modules/translations/code/vending_ru.dm Outdated Show resolved Hide resolved
modular_ss220/modules/translations/code/vending_ru.dm Outdated Show resolved Hide resolved
modular_ss220/modules/translations/code/vending_ru.dm Outdated Show resolved Hide resolved
modular_ss220/modules/translations/code/vending_ru.dm Outdated Show resolved Hide resolved
modular_ss220/modules/translations/code/vending_ru.dm Outdated Show resolved Hide resolved
modular_ss220/modules/translations/code/vending_ru.dm Outdated Show resolved Hide resolved
modular_ss220/modules/translations/code/vending_ru.dm Outdated Show resolved Hide resolved
if(isliving(user))
var/mob/living/living_user = user
if(!(machine_stat & NOPOWER) && !istype(living_user.get_idcard(TRUE), /obj/item/card/id/advanced/prisoner))
speak("Не обнаружена действующая карта заключённого. Покупка не может быть произведена.")

Choose a reason for hiding this comment

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

Переопределять всю функцию полнстью - плохая идея. Подобные вещи лучше делать напрямую в коде

Copy link
Author

Choose a reason for hiding this comment

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

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

Choose a reason for hiding this comment

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

Твоя функция имеет приоритет при компиляции т.к. она стоит ниже кода апстрима. В твоём случае она полностью переопределяет оригинальную функцию. Когда будем снова обновляться, то подобные вставки могут сыграть злую шутку тем, что когда апстрим будет что-то там менять, то этот код не начнет возмущаться конфликтами слияния.

paying_scum_id.points -= price_to_use
return TRUE

//Автоматы с одеждой

Choose a reason for hiding this comment

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

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

-product_ads
+product_slogans

Copy link
Author

Choose a reason for hiding this comment

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

в плане под каждым вендором?
а не легче просто написать product_ads = "" ? аналогично паре

Copy link
Author

Choose a reason for hiding this comment

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

а бля понял о чём ты
сделаю

Frogich228 and others added 17 commits September 15, 2024 18:32
Frogich228 and others added 15 commits September 15, 2024 18:50
@Frogich228 Frogich228 closed this by deleting the head repository Sep 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants