M41-A Фикс #317
M41-A Фикс #317
Conversation
WalkthroughВ данном пулл-реквесте внесены изменения в локализацию и конфигурацию сущностей, связанных с магазином для винтовки M41-A. Основные изменения заключаются в обновлении обозначения калибра с ".20" на ".30" для всех типов магазинов и описания винтовки. Эти изменения затрагивают файлы локализации на русском языке и YAML-файлы, при этом структура сущностей остается неизменной. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant Game
participant Localization
participant YAML
User->>Game: Запрос на использование M41-A
Game->>Localization: Получить описание M41-A
Localization-->>Game: "30-го калибра"
Game->>YAML: Получить конфигурацию магазина
YAML-->>Game: "magazine M41-A (.30 rifle)"
Game-->>User: Предоставить информацию о M41-A
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
github-actions
bot
added
Изменения: Локализация
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (4)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/weapons/guns/ammunition/magazines/M41-A_mag.ftl(1 hunks)Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/weapons/guns/rifles/M41-A.ftl(1 hunks)Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Ammunition/Magazines/M41-A_mag.yml(1 hunks)Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Rifles/M41-A.yml(1 hunks)
🧰 Additional context used
📓 Path-based instructions (4)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/weapons/guns/ammunition/magazines/M41-A_mag.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/weapons/guns/rifles/M41-A.ftl (1)
Pattern **/*.ftl: Все повторяющиеся переводы должны быть написаны так:
Оригинальный перевод - id1 = text1
Повторяющийся, но с изменениями id2 = { id1 }
и если требуется .desc должен прописываться как { id1.desc }
А многострочные переводы начинающиеся со спецсимвола должны идти сначала id = спецсимвол текст и потом следующая строчка. Пример
id = спецсимвол текст
текст
спецсимвол текст
Все переводы начинающиеся с ent- должны быть с большой буквы после =. Даже после .desc и .suffix
Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Ammunition/Magazines/M41-A_mag.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Rifles/M41-A.yml (1)
Pattern **/*.yml: так же в прототипах запрещены любые переводы на русский. Только на английском или ссылкой на перевод.
Все добавляемые прототипы должны быть в нашей папке.
Никогда не предлагать ничего по отступам
При удалении дополнительного дополнения по типу prob:, можно оставить оригинальную часть с - id:.
🪛 yamllint
Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Ammunition/Magazines/M41-A_mag.yml
[error] 1-1: wrong new line character: expected \n
(new-lines)
[error] 7-7: wrong indentation: expected at least 3
(indentation)
[error] 24-24: wrong indentation: expected 6 but found 4
(indentation)
[error] 40-40: wrong indentation: expected 4 but found 2
(indentation)
[error] 44-44: wrong indentation: expected 6 but found 4
(indentation)
[error] 54-54: wrong indentation: expected 4 but found 2
(indentation)
[error] 58-58: wrong indentation: expected 6 but found 4
(indentation)
[error] 68-68: wrong indentation: expected 4 but found 2
(indentation)
[error] 72-72: wrong indentation: expected 6 but found 4
(indentation)
[error] 83-83: wrong indentation: expected 4 but found 2
(indentation)
[error] 87-87: wrong indentation: expected 6 but found 4
(indentation)
Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Rifles/M41-A.yml
[error] 1-1: wrong new line character: expected \n
(new-lines)
[error] 7-7: wrong indentation: expected at least 3
(indentation)
🔇 Additional comments (10)
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/weapons/guns/rifles/M41-A.ftl (2)
1-1: Перевод названия сущности корректен!
Название сущности следует правилам именования и правильно капитализировано.
2-2: Описание и изменение калибра корректны!
Изменение калибра с 20-го на 30-й соответствует целям PR. Описание грамматически верно и сохраняет согласованность с другими файлами локализации.
Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/weapons/guns/ammunition/magazines/M41-A_mag.ftl (2)
3-10: Корректное использование ссылок на описания
Правильное использование шаблона для повторяющихся описаний через ссылку на базовое описание.
1-2:
Исправьте заглавную букву после знака равенства
Согласно правилам кодирования, все переводы, начинающиеся с ent-, должны начинаться с заглавной буквы после знака равенства.
Примените следующие изменения:
-ent-LPPBaseMagazineRifleM41A = Магазин для M41-A (.30 винтовочные)
+ent-LPPBaseMagazineRifleM41A = Магазин для M41-A (.30 Винтовочные)Likely invalid or redundant comment.
Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Rifles/M41-A.yml (1)
5-5: Изменение калибра подтверждено.
Изменение описания с .20 на .30 соответствует целям PR и выполнено корректно. Описание оставлено на английском языке, что соответствует требованиям к прототипам.
Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Ammunition/Magazines/M41-A_mag.yml (5)
1-33: Изменение калибра в базовом прототипе подтверждено!
Базовая сущность корректно обновлена с учетом нового калибра .30.
🧰 Tools
🪛 yamllint
[error] 1-1: wrong new line character: expected \n
(new-lines)
[error] 7-7: wrong indentation: expected at least 3
(indentation)
[error] 24-24: wrong indentation: expected 6 but found 4
(indentation)
35-48: Изменение калибра в полном магазине подтверждено!
Конфигурация полного магазина корректно обновлена для калибра .30.
🧰 Tools
🪛 yamllint
[error] 40-40: wrong indentation: expected 4 but found 2
(indentation)
[error] 44-44: wrong indentation: expected 6 but found 4
(indentation)
49-62: Изменение калибра в тренировочном магазине подтверждено!
Конфигурация тренировочного магазина корректно обновлена для калибра .30.
🧰 Tools
🪛 yamllint
[error] 54-54: wrong indentation: expected 4 but found 2
(indentation)
[error] 58-58: wrong indentation: expected 6 but found 4
(indentation)
63-76: Изменение калибра в резиновом магазине подтверждено!
Конфигурация резинового магазина корректно обновлена для калибра .30.
🧰 Tools
🪛 yamllint
[error] 68-68: wrong indentation: expected 4 but found 2
(indentation)
[error] 72-72: wrong indentation: expected 6 but found 4
(indentation)
78-90: Изменение калибра в урановом магазине подтверждено!
Конфигурация уранового магазина корректно обновлена для калибра .30.
🧰 Tools
🪛 yamllint
[error] 83-83: wrong indentation: expected 4 but found 2
(indentation)
[error] 87-87: wrong indentation: expected 6 but found 4
(indentation)
| ent-LPPBaseMagazineRifleM41A = Магазин для M41-A (.30 винтовочные) | ||
| .desc = { ent-BaseItem.desc } | ||
| ent-LPPBaseMagazineRifleM41AFull = Магазин для M41-A (.30 винтовочные Боевые) | ||
| .desc = { ent-LPPBaseMagazineRifleM41A.desc } | ||
| ent-LPPBaseMagazineRifleM41APractice = Магазин для M41-A (.30 винтовочные Учебные) | ||
| .desc = { ent-LPPBaseMagazineRifleM41A.desc } | ||
| ent-LPPBaseMagazineRifleM41ARubber = Магазин для M41-A (.30 винтовочные Резиновые) | ||
| .desc = { ent-LPPBaseMagazineRifleM41A.desc } | ||
| ent-LPPBaseMagazineRifleM41AUranium = Магазин для M41-A (.30 винтовочные Урановые) | ||
| .desc = { ent-LPPBaseMagazineRifleM41A.desc } |
There was a problem hiding this comment.
💡 Codebase verification
Обнаружены несоответствия в калибре M41-A между локализациями
В английской локализации (en-US) все ещё используется старый калибр .20, в то время как в русской локализации (ru-RU) и файлах прототипов используется новый калибр .30:
Resources/Locale/en-US/ss14-ru/prototypes/_lostparadise/entities/objects/weapons/guns/ammunition/magazines/M41-A_mag.ftl: калибр .20Resources/Locale/ru-RU/ss14-ru/prototypes/_lostparadise/entities/objects/weapons/guns/ammunition/magazines/M41-A_mag.ftl: калибр .30Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Ammunition/Magazines/M41-A_mag.yml: калибр .30
Необходимо обновить английскую локализацию для соответствия новому калибру .30.
🔗 Analysis chain
Проверка согласованности изменения калибра
Необходимо убедиться, что изменение калибра с .20 на .30 согласовано во всех связанных файлах.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск возможных упоминаний старого калибра (.20) в связанных файлах
rg -g "*.yml" -g "*.ftl" "\.20.*M41-A"
# Поиск всех упоминаний M41-A для проверки согласованности
rg -g "*.yml" -g "*.ftl" "M41-A.*\.(20|30)"
Length of output: 2408
| - type: entity | ||
| name: M41-A | ||
| parent: BaseWeaponRifle | ||
| id: LPPWeaponRifleM41 | ||
| description: An old rifle used during the colonial expansion. Uses .30 rifle ammo. | ||
| components: | ||
| - type: Sprite | ||
| sprite: _LostParadise/Objects/Weapons/Guns/Rifles/M41-A.rsi | ||
| layers: | ||
| - state: base | ||
| map: ["enum.GunVisualLayers.Base"] | ||
| - state: mag-0 | ||
| map: ["enum.GunVisualLayers.Mag"] | ||
| - type: Gun | ||
| fireRate: 7 | ||
| soundGunshot: | ||
| path: /Audio/_LostParadise/Weapons/Guns/Gunshots/M41-A.ogg | ||
| - type: ChamberMagazineAmmoProvider | ||
| soundRack: | ||
| path: /Audio/Weapons/Guns/Cock/ltrifle_cock.ogg | ||
| - type: ItemSlots | ||
| slots: | ||
| gun_magazine: | ||
| name: Magazine | ||
| startingItem: LPPBaseMagazineRifleM41AFull | ||
| insertSound: /Audio/Weapons/Guns/MagIn/ltrifle_magin.ogg | ||
| ejectSound: /Audio/Weapons/Guns/MagOut/ltrifle_magout.ogg | ||
| priority: 2 | ||
| whitelist: | ||
| tags: | ||
| - MagazineRifleM41A | ||
| gun_chamber: | ||
| name: Chamber | ||
| startingItem: CartridgeLightRifle | ||
| priority: 1 | ||
| whitelist: | ||
| tags: | ||
| - CartridgeLightRifle | ||
| - type: ContainerContainer | ||
| containers: | ||
| gun_magazine: !type:ContainerSlot | ||
| gun_chamber: !type:ContainerSlot | ||
| - type: MagazineVisuals | ||
| magState: mag | ||
| steps: 1 | ||
| zeroVisible: true |
There was a problem hiding this comment.
Требуется исправление символов новой строки.
В файле используются некорректные символы переноса строки. Необходимо использовать \n вместо текущих символов.
Предлагаю выполнить конвертацию символов новой строки в редакторе с поддержкой UNIX-style line endings (LF) или использовать команду:
dos2unix Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Rifles/M41-A.yml🧰 Tools
🪛 yamllint
[error] 1-1: wrong new line character: expected \n
(new-lines)
[error] 7-7: wrong indentation: expected at least 3
(indentation)
| - type: entity | ||
| id: LPPBaseMagazineRifleM41A | ||
| name: "magazine M41-A (.30 rifle)" | ||
| parent: BaseItem | ||
| abstract: true | ||
| components: | ||
| - type: Tag | ||
| tags: | ||
| - MagazineRifleM41A | ||
| - type: Item | ||
| size: Small | ||
| - type: BallisticAmmoProvider | ||
| mayTransfer: true | ||
| whitelist: | ||
| tags: | ||
| - CartridgeRifle | ||
| capacity: 99 | ||
| - type: ContainerContainer | ||
| containers: | ||
| ballistic-ammo: !type:Container | ||
| - type: Sprite | ||
| sprite: _LostParadise/Objects/Weapons/Guns/Ammunition/Magazine/M41-A_mag.rsi | ||
| layers: | ||
| - state: base | ||
| map: ["enum.GunVisualLayers.Base"] | ||
| - state: mag-1 | ||
| map: ["enum.GunVisualLayers.Mag"] | ||
| - type: MagazineVisuals | ||
| magState: mag | ||
| steps: 5 | ||
| zeroVisible: false | ||
| - type: Appearance | ||
| # Magazines | ||
| - type: entity | ||
| id: LPPBaseMagazineRifleM41AFull | ||
| name: "magazine M41-A (.30 rifle)" | ||
| parent: LPPBaseMagazineRifleM41A | ||
| components: | ||
| - type: BallisticAmmoProvider | ||
| proto: CartridgeRifle | ||
| - type: Sprite | ||
| layers: | ||
| - state: red | ||
| map: ["enum.GunVisualLayers.Base"] | ||
| - state: mag-1 | ||
| map: ["enum.GunVisualLayers.Mag"] | ||
| - type: entity | ||
| id: LPPBaseMagazineRifleM41APractice | ||
| name: "magazine M41-A (.30 rifle practice)" | ||
| parent: LPPBaseMagazineRifleM41A | ||
| components: | ||
| - type: BallisticAmmoProvider | ||
| proto: CartridgeRiflePractice | ||
| - type: Sprite | ||
| layers: | ||
| - state: practice | ||
| map: ["enum.GunVisualLayers.Base"] | ||
| - state: mag-1 | ||
| map: ["enum.GunVisualLayers.Mag"] | ||
| - type: entity | ||
| id: LPPBaseMagazineRifleM41ARubber | ||
| name: "magazine M41-A (.30 rifle rubber)" | ||
| parent: LPPBaseMagazineRifleM41A | ||
| components: | ||
| - type: BallisticAmmoProvider | ||
| proto: CartridgeRifleRubber | ||
| - type: Sprite | ||
| layers: | ||
| - state: rubber | ||
| map: ["enum.GunVisualLayers.Base"] | ||
| - state: mag-1 | ||
| map: ["enum.GunVisualLayers.Mag"] | ||
| - type: entity | ||
| id: LPPBaseMagazineRifleM41AUranium | ||
| name: "magazine (.30 rifle uranium)" | ||
| parent: LPPBaseMagazineRifleM41A | ||
| components: | ||
| - type: BallisticAmmoProvider | ||
| proto: CartridgeRifleUranium | ||
| - type: Sprite | ||
| layers: | ||
| - state: uranium | ||
| map: ["enum.GunVisualLayers.Base"] | ||
| - state: mag-1 | ||
| map: ["enum.GunVisualLayers.Mag"] |
There was a problem hiding this comment.
💡 Codebase verification
Обнаружено несоответствие калибров в конфигурациях
В файле Resources/Prototypes/Entities/Objects/Weapons/Guns/Ammunition/Cartridges/rifle.yml все базовые патроны определены как калибр .20, в то время как магазин M41-A настроен на использование этих патронов, но описан как калибр .30. Это создаст несоответствие в игре.
- Необходимо либо обновить калибр базовых патронов до .30
- Либо исправить описание магазина M41-A на калибр .20
🔗 Analysis chain
Необходима проверка согласованности калибра во всей кодовой базе
Давайте убедимся, что все связанные конфигурации боеприпасов также обновлены до калибра .30.
🏁 Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash
# Поиск оставшихся упоминаний калибра .20 в конфигурациях
echo "Searching for remaining .20 caliber references..."
rg -l "\.20\s*(rifle|caliber)" --type yaml
# Поиск конфигураций M41-A
echo "Checking M41-A related configurations..."
rg -l "M41-A" --type yaml
Length of output: 1514
Script:
#!/bin/bash
# Проверим содержимое файлов с упоминанием калибра .20
echo "Checking rifle ammunition configurations..."
rg "\.20\s*(rifle|caliber)" -C 2 Resources/Prototypes/Entities/Objects/Weapons/Guns/Ammunition/Cartridges/rifle.yml
echo -e "\nChecking M41-A specific configurations..."
rg "caliber|CartridgeRifle" Resources/Prototypes/_LostParadise/Entities/Objects/Weapons/Guns/Rifles/M41-A.yml
Length of output: 1162
🧰 Tools
🪛 yamllint
[error] 1-1: wrong new line character: expected \n
(new-lines)
[error] 7-7: wrong indentation: expected at least 3
(indentation)
[error] 24-24: wrong indentation: expected 6 but found 4
(indentation)
[error] 40-40: wrong indentation: expected 4 but found 2
(indentation)
[error] 44-44: wrong indentation: expected 6 but found 4
(indentation)
[error] 54-54: wrong indentation: expected 4 but found 2
(indentation)
[error] 58-58: wrong indentation: expected 6 but found 4
(indentation)
[error] 68-68: wrong indentation: expected 4 but found 2
(indentation)
[error] 72-72: wrong indentation: expected 6 but found 4
(indentation)
[error] 83-83: wrong indentation: expected 4 but found 2
(indentation)
[error] 87-87: wrong indentation: expected 6 but found 4
(indentation)
Описание обновления:
Фикс патронов с .20 на .30
Проверки
Изменения
🆑 PowerKAT1