Skip to content
Open
Show file tree
Hide file tree
Changes from 2 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions Resources/Locale/ru-RU/ADT/clothing/misc.ftl
Original file line number Diff line number Diff line change
@@ -0,0 +1,9 @@
ent-ADTClothingHeadHatBlackroseFlowerWreath = венок из черных роз
.desc = вено
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Опечатка в описании чёрного венка.

На Line 2 слово вено выглядит как обрезанная опечатка. В текущем виде строка попадёт игроку прямо в интерфейс.

Предлагаемая правка
 ent-ADTClothingHeadHatBlackroseFlowerWreath = венок из черных роз
-    .desc = вено
+    .desc = венок
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
ent-ADTClothingHeadHatBlackroseFlowerWreath = венок из черных роз
.desc = вено
ent-ADTClothingHeadHatBlackroseFlowerWreath = венок из черных роз
.desc = венок
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@Resources/Locale/ru-RU/ADT/clothing/misc.ftl` around lines 1 - 2, Поломанная
строка описания для ent-ADTClothingHeadHatBlackroseFlowerWreath содержит
опечатку ".desc = вено" — исправь значение .desc для ключа
ent-ADTClothingHeadHatBlackroseFlowerWreath, заменив "вено" на корректное слово
"венок" или на полный осмысленный текст (например "венок из чёрных роз") чтобы
описание отображалось правильно в интерфейсе.

ent-ADTClothingHeadHatMacFlowerWreath = маковый венок
.desc = Венок состоящий из мака
ent-ADTClothingHeadHatDandelionFlowerWreath = Желтый венок
.desc = Венок состоящий из ромашек
ent-ADTClothingHeadHatRoseFlowerWreath = Красный венок
.desc = Венок состоящий из красных роз
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Приведите локализацию к нижнему регистру.

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

Предлагаемая правка
 ent-ADTClothingHeadHatMacFlowerWreath = маковый венок
-    .desc = Венок состоящий из мака
+    .desc = венок состоящий из мака
- ent-ADTClothingHeadHatDandelionFlowerWreath = Желтый венок
+ ent-ADTClothingHeadHatDandelionFlowerWreath = желтый венок
-    .desc = Венок состоящий из ромашек
+    .desc = венок состоящий из ромашек
- ent-ADTClothingHeadHatRoseFlowerWreath = Красный венок
+ ent-ADTClothingHeadHatRoseFlowerWreath = красный венок
-    .desc = Венок состоящий из красных роз
+    .desc = венок состоящий из красных роз

Based on learnings: В локализации файлов формата .ftl имена сущностей должны быть написаны в нижнем регистре.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
ent-ADTClothingHeadHatMacFlowerWreath = маковый венок
.desc = Венок состоящий из мака
ent-ADTClothingHeadHatDandelionFlowerWreath = Желтый венок
.desc = Венок состоящий из ромашек
ent-ADTClothingHeadHatRoseFlowerWreath = Красный венок
.desc = Венок состоящий из красных роз
ent-ADTClothingHeadHatMacFlowerWreath = маковый венок
.desc = венок состоящий из мака
ent-ADTClothingHeadHatDandelionFlowerWreath = желтый венок
.desc = венок состоящий из ромашек
ent-ADTClothingHeadHatRoseFlowerWreath = красный венок
.desc = венок состоящий из красных роз
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@Resources/Locale/ru-RU/ADT/clothing/misc.ftl` around lines 3 - 8, The string
values for the listed localization keys use sentence-case; change them to
lowercase to match .ftl conventions: update
ent-ADTClothingHeadHatMacFlowerWreath,
ent-ADTClothingHeadHatDandelionFlowerWreath, and
ent-ADTClothingHeadHatRoseFlowerWreath values and their .desc entries so the
visible names and descriptions start with lowercase letters (e.g., "маковый
венок" stays but ensure first character is lowercase, "Желтый венок" -> "желтый
венок", "Венок состоящий из ромашек" -> "венок состоящий из ромашек", etc.),
preserving existing wording and punctuation.


58 changes: 57 additions & 1 deletion Resources/Prototypes/ADT/Entities/Clothing/Head/misc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -52,4 +52,60 @@
volume: -10
range: 10 #радиус голоса
sound:
path: /Audio/ADT/Ambience/Music/robotnik.ogg
path: /Audio/ADT/Ambience/Music/robotnik.ogg

- type: entity
parent: ClothingHeadBase
id: ADTClothingHeadHatBlackroseFlowerWreath
name: ClothingHeadHatBlackroseFlowerWreath-name
description: ClothingHeadHatBlackroseFlowerWreath-desc
components:
- type: Sprite
sprite: ADT/Clothing/Head/Misc/blackrose-wreath.rsi
- type: Clothing
sprite: ADT/Clothing/Head/Misc/blackrose-wreath.rsi
slots:
- HEAD
- neck

- type: entity
parent: ClothingHeadBase
id: ADTClothingHeadHatMacFlowerWreath
name: ClothingHeadHatMacFlowerWreath-name
description: ClothingHeadHatMacFlowerWreath-desc
components:
- type: Sprite
sprite: ADT/Clothing/Head/Misc/mac-wreat.rsi
- type: Clothing
sprite: ADT/Clothing/Head/Misc/mac-wreat.rsi
slots:
- HEAD
- neck

- type: entity
parent: ClothingHeadBase
id: ADTClothingHeadHatRoseFlowerWreath
name: ClothingHeadHatRoseFlowerWreath-name
description: ClothingHeadHatRoseFlowerWreath-desc
components:
- type: Sprite
sprite: ADT/Clothing/Head/Misc/rose-wreat.rsi
- type: Clothing
sprite: ADT/Clothing/Head/Misc/rose-wreat.rsi
slots:
- HEAD
- neck

- type: entity
parent: ClothingHeadBase
id: ADTClothingHeadHatDandelionFlowerWreath
name: ClothingHeadHatDandelionFlowerWreath-name
description: ClothingHeadHatDandelionFlowerWreath-desc
components:
- type: Sprite
sprite: ADT/Clothing/Head/Misc/dandelion-wreat.rsi
- type: Clothing
sprite: ADT/Clothing/Head/Misc/dandelion-wreat.rsi
slots:
- HEAD
- neck
31 changes: 30 additions & 1 deletion Resources/Prototypes/ADT/Loadouts/Miscellaneous/trinkets.yml
Original file line number Diff line number Diff line change
Expand Up @@ -66,9 +66,38 @@
storage:
back:
- ADTClothingMaskPaperBlank

- type: loadout
id: ADTHandheldStationMapUnpowered
storage:
back:
- HandheldStationMapUnpowered

# Flowers
- type: loadout
id: ADTFlowerWreath1
storage:
back:
- ADTClothingHeadHatBlackroseFlowerWreath
groupBy: "flowerwreath"

- type: loadout
id: ADTFlowerWreath2
storage:
back:
- ADTClothingHeadHatMacFlowerWreath
groupBy: "flowerwreath"

- type: loadout
id: ADTFlowerWreath3
storage:
back:
- ADTClothingHeadHatRoseFlowerWreath
groupBy: "flowerwreath"

- type: loadout
id: ADTFlowerWreath4
storage:
back:
- ADTClothingHeadHatDandelionFlowerWreath
groupBy: "flowerwreath"
Comment on lines +76 to +103
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Здесь нужен equipment, а не storage.back.

Сейчас венки добавляются как содержимое слота back, а не как экипируемый предмет. Для этих прототипов это ломает ожидаемое поведение лодаута: венок не будет надет на голову, хотя сами сущности объявлены как Clothing со слотами HEAD/neck.

Предлагаемая правка
 # Flowers
 - type: loadout
   id: ADTFlowerWreath1
-  storage:
-    back:
-    - ADTClothingHeadHatBlackroseFlowerWreath
+  equipment:
+    head: ADTClothingHeadHatBlackroseFlowerWreath
   groupBy: "flowerwreath"

 - type: loadout
   id: ADTFlowerWreath2
-  storage:
-    back:
-    - ADTClothingHeadHatMacFlowerWreath
+  equipment:
+    head: ADTClothingHeadHatMacFlowerWreath
   groupBy: "flowerwreath"

 - type: loadout
   id: ADTFlowerWreath3
-  storage:
-    back:
-    - ADTClothingHeadHatRoseFlowerWreath
+  equipment:
+    head: ADTClothingHeadHatRoseFlowerWreath
   groupBy: "flowerwreath"

 - type: loadout
   id: ADTFlowerWreath4
-  storage:
-    back:
-    - ADTClothingHeadHatDandelionFlowerWreath
+  equipment:
+    head: ADTClothingHeadHatDandelionFlowerWreath
   groupBy: "flowerwreath"
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
# Flowers
- type: loadout
id: ADTFlowerWreath1
storage:
back:
- ADTClothingHeadHatBlackroseFlowerWreath
groupBy: "flowerwreath"
- type: loadout
id: ADTFlowerWreath2
storage:
back:
- ADTClothingHeadHatMacFlowerWreath
groupBy: "flowerwreath"
- type: loadout
id: ADTFlowerWreath3
storage:
back:
- ADTClothingHeadHatRoseFlowerWreath
groupBy: "flowerwreath"
- type: loadout
id: ADTFlowerWreath4
storage:
back:
- ADTClothingHeadHatDandelionFlowerWreath
groupBy: "flowerwreath"
# Flowers
- type: loadout
id: ADTFlowerWreath1
equipment:
head: ADTClothingHeadHatBlackroseFlowerWreath
groupBy: "flowerwreath"
- type: loadout
id: ADTFlowerWreath2
equipment:
head: ADTClothingHeadHatMacFlowerWreath
groupBy: "flowerwreath"
- type: loadout
id: ADTFlowerWreath3
equipment:
head: ADTClothingHeadHatRoseFlowerWreath
groupBy: "flowerwreath"
- type: loadout
id: ADTFlowerWreath4
equipment:
head: ADTClothingHeadHatDandelionFlowerWreath
groupBy: "flowerwreath"
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@Resources/Prototypes/ADT/Loadouts/Miscellaneous/trinkets.yml` around lines 76
- 103, The four loadout prototypes ADTFlowerWreath1, ADTFlowerWreath2,
ADTFlowerWreath3 and ADTFlowerWreath4 currently place the wreath items under
storage.back; change each to use an equipment section instead so the wreaths are
equipped to the proper clothing slot (e.g., HEAD or neck as declared on the
Clothing prototypes) rather than stored in the back slot. Specifically, replace
the storage.back entries with an equipment mapping that references the same item
IDs (ADTClothingHeadHatBlackroseFlowerWreath, ADTClothingHeadHatMacFlowerWreath,
ADTClothingHeadHatRoseFlowerWreath, ADTClothingHeadHatDandelionFlowerWreath) so
the engine will equip them to the Clothing slots (HEAD/neck) declared on those
Clothing prototypes.

1 change: 1 addition & 0 deletions Resources/Prototypes/Loadouts/Miscellaneous/trinkets.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
storage:
back:
- ClothingHeadHatFlowerWreath
groupBy: "flowerwreath"
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Отсутствует обязательный комментарий ADT-Tweak.

Изменение находится вне папки /ADT/, поэтому согласно правилам проекта необходимо добавить маркер # ADT-Tweak: для одиночной строки.

🔧 Предлагаемое исправление
 - type: loadout
   id: FlowerWreath
   storage:
     back:
     - ClothingHeadHatFlowerWreath
-  groupBy: "flowerwreath"
+  groupBy: "flowerwreath" # ADT-Tweak: группировка с ADT венками

Согласно coding guidelines: "Все изменения вне папок /ADT/ должны быть прокомментированы... Одиночные строки можно просто помечать как # ADT-Tweak:"

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@Resources/Prototypes/Loadouts/Miscellaneous/trinkets.yml` at line 17,
Добавьте обязательный маркер комментария для изменения вне папки /ADT/: перед
строкой содержащей ключ groupBy: "flowerwreath" в файле trinkets.yml вставьте
одиночную строку комментария вида # ADT-Tweak: <краткая причина изменения>
(например, # ADT-Tweak: исправлена группировка для венков), чтобы пометить
изменение согласно правилам проекта.


- type: loadout
id: Hairflower
Expand Down
4 changes: 4 additions & 0 deletions Resources/Prototypes/Loadouts/loadout_groups.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,6 +74,10 @@
- ADTHandheldStationMapUnpowered
- ADTModStorageVoucher
- ADTClothingGeorgeRibbonPin
- ADTFlowerWreath1
- ADTFlowerWreath2
- ADTFlowerWreath3
- ADTFlowerWreath4
# ADT-Tweak-End

- type: loadoutGroup
Expand Down
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
{
"version": 1,
"license": "CC-BY-SA-4.0",
"copyright": "made by Prazat",
"size": {
"x": 32,
"y": 32
},
"states": [
{
"name": "icon"
},
{
"name": "equipped-NECK",
"directions": 4
},
{
"name": "equipped-HELMET",
"directions": 4
},
{
"name": "equipped-HELMET-vox",
"directions": 4
}
]
}
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
{
"version": 1,
"license": "CC-BY-SA-4.0",
"copyright": "made by Prazat",
"size": {
"x": 32,
"y": 32
},
"states": [
{
"name": "icon"
},
{
"name": "equipped-NECK",
"directions": 4
},
{
"name": "equipped-HELMET",
"directions": 4
},
{
"name": "equipped-HELMET-vox",
"directions": 4
}
]
}
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
26 changes: 26 additions & 0 deletions Resources/Textures/ADT/Clothing/Head/Misc/mac-wreat.rsi/meta.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
{
"version": 1,
"license": "CC-BY-SA-4.0",
"copyright": "made by Prazat",
"size": {
"x": 32,
"y": 32
},
"states": [
{
"name": "icon"
},
{
"name": "equipped-NECK",
"directions": 4
},
{
"name": "equipped-HELMET",
"directions": 4
},
{
"name": "equipped-HELMET-vox",
"directions": 4
}
]
}
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
26 changes: 26 additions & 0 deletions Resources/Textures/ADT/Clothing/Head/Misc/rose-wreat.rsi/meta.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
{
"version": 1,
"license": "CC-BY-SA-4.0",
"copyright": "made by Prazat",
"size": {
"x": 32,
"y": 32
},
"states": [
{
"name": "icon"
},
{
"name": "equipped-NECK",
"directions": 4
},
{
"name": "equipped-HELMET",
"directions": 4
},
{
"name": "equipped-HELMET-vox",
"directions": 4
}
]
}
Loading