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

Halloween things2 #301

Merged
merged 2 commits into from
Oct 25, 2024
Merged
Changes from 1 commit
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
Original file line number Diff line number Diff line change
@@ -1,10 +1,2 @@
ent-LPPCandy = конфета
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

Исправьте регистр первой буквы в переводе.

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

-ent-LPPCandy = конфета
+ent-LPPCandy = Конфета
📝 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-LPPCandy = конфета
ent-LPPCandy = Конфета

.desc = Обычная конфета состоящия из сахара и других вещей
ent-LPPCandy2 = { ent-LPPCandy }
.desc = { ent-LPPCandy.desc }
ent-LPPCandy3 = { ent-LPPCandy }
.desc = { ent-LPPCandy.desc }
ent-LPPCandy4 = { ent-LPPCandy }
.desc = { ent-LPPCandy.desc }
ent-LPPLolipop = конфета на палочке
.desc = Всё такая же конфета, но на палочке.
.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

Исправьте грамматические ошибки и пунктуацию.

В описании присутствуют следующие ошибки:

  1. "состоящия" -> "состоящая" (грамматическая ошибка)
  2. Отсутствует точка в конце предложения
-    .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
.desc = Обычная конфета состоящия из сахара и других вещей
.desc = Обычная конфета, состоящая из сахара и других вещей.

Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
ent-HalloweenTree01 = мёртвое дерево
ent-LPPHalloweenTree01 = мёртвое дерево
.desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
ent-HalloweenTree02 = мёртвое дерево
ent-LPPHalloweenTree02 = мёртвое дерево
.desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
ent-HalloweenTree03 = мёртвое дерево
ent-LPPHalloweenTree03 = мёртвое дерево
.desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
ent-HalloweenTree04 = мёртвое дерево
ent-LPPHalloweenTree04 = мёртвое дерево
.desc = Мёртвое дерево с остатками листьев на нём, выглядит жутко.
Original file line number Diff line number Diff line change
@@ -1,2 +1,14 @@
ent-LPPPSpiderDeco = паучки
.desc = Муляжи паучков которые выглядят так, будто готовы вас покусать.
ent-LPPPPumpkinDeco = тыква
.desc = Маленькая декоративная тыква.
Comment on lines +3 to +4
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

Исправьте регистр первой буквы в переводе.

Согласно правилам локализации, все переводы, начинающиеся с 'ent-', должны начинаться с заглавной буквы после знака =.

-ent-LPPPPumpkinDeco = тыква
+ent-LPPPPumpkinDeco = Тыква
📝 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-LPPPPumpkinDeco = тыква
.desc = Маленькая декоративная тыква.
ent-LPPPPumpkinDeco = Тыква
.desc = Маленькая декоративная тыква.

ent-LPPPPumpkinBigDeco = тыква
.desc = Декоративные тыквы разных размеров.
Comment on lines +5 to +6
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

Исправьте регистр первой буквы в переводе.

Аналогичная проблема с регистром первой буквы после знака =.

-ent-LPPPPumpkinBigDeco = тыква
+ent-LPPPPumpkinBigDeco = Тыква
📝 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-LPPPPumpkinBigDeco = тыква
.desc = Декоративные тыквы разных размеров.
ent-LPPPPumpkinBigDeco = Тыква
.desc = Декоративные тыквы разных размеров.

ent-LPPPottedPlantHalloween1 = растение в горшке
.desc = Высохшее растение в горшке, выглядит не особо живым.
Comment on lines +7 to +8
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

Исправьте регистр первой буквы в переводе.

Та же проблема с регистром первой буквы после знака =.

-ent-LPPPottedPlantHalloween1 = растение в горшке
+ent-LPPPottedPlantHalloween1 = Растение в горшке
📝 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-LPPPottedPlantHalloween1 = растение в горшке
.desc = Высохшее растение в горшке, выглядит не особо живым.
ent-LPPPottedPlantHalloween1 = Растение в горшке
.desc = Высохшее растение в горшке, выглядит не особо живым.

ent-LPPPottedPlantHalloween2 = { ent-LPPPottedPlantHalloween1 }
.desc = { ent-LPPPottedPlantHalloween1.desc }
ent-LPPPottedPlantHalloween3 = { ent-LPPPottedPlantHalloween1 }
.desc = { ent-LPPPottedPlantHalloween1.desc }
ent-LPPPottedPlantHalloween4 = { ent-LPPPottedPlantHalloween1 }
.desc = { ent-LPPPottedPlantHalloween1.desc }
Original file line number Diff line number Diff line change
@@ -6,7 +6,16 @@
components:
- type: Sprite
sprite: _LostParadise/Objects/Consumable/halloween.rsi
state: candy_1
layers:
- state: candy_1
map: [ "enum.DamageStateVisualLayers.Base" ]
- type: RandomSprite
available:
- enum.DamageStateVisualLayers.Base:
candy_1: Material
candy_2: Material
candy_3: Material
candy_4: Material
- type: Item
size: Tiny
shape:
@@ -22,54 +31,6 @@
Quantity: 1
- ReagentId: Theobromine
Quantity: 1
- type: Tag
tags:
- LPPCandy

- type: entity
parent: LPPCandy
id: LPPCandy2
components:
- type: Sprite
state: candy_2

- type: entity
parent: LPPCandy
id: LPPCandy3
components:
- type: Sprite
state: candy_3

- type: entity
parent: LPPCandy
id: LPPCandy4
components:
- type: Sprite
state: candy_4

- type: entity
name: lolipop
parent: LPPCandy
id: LPPLolipop
description: This is my kingdom come, this is my kingdom come...
components:
- type: Sprite
state: lollipop
- type: Item
size: Tiny
shape:
- 0, 0, 0, 1
- type: SolutionContainerManager
solutions:
food:
maxVol: 15
reagents:
- ReagentId: Sugar
Quantity: 8
- ReagentId: Nutriment
Quantity: 4
- ReagentId: Theobromine
Quantity: 3
- type: Tag
tags:
- LPPCandy
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
- type: entity
parent: BaseTree
id: HalloweenTree01
id: LPPHalloweenTree01
name: dead tree
description: Dead tree with leaves on it, looks creepy.
components:
@@ -10,7 +10,7 @@

- type: entity
parent: BaseTree
id: HalloweenTree02
id: LPPHalloweenTree02
name: dead tree
description: Dead tree with leaves on it, looks creepy.
components:
@@ -20,7 +20,7 @@

- type: entity
parent: BaseTree
id: HalloweenTree03
id: LPPHalloweenTree03
name: dead tree
description: Dead tree with leaves on it, looks creepy.
components:
@@ -30,7 +30,7 @@

- type: entity
parent: BaseTree
id: HalloweenTree04
id: LPPHalloweenTree04
name: dead tree
description: Dead tree with leaves on it, looks creepy.
components:
Original file line number Diff line number Diff line change
@@ -1,16 +1,18 @@
- type: entity
parent: BaseSign
id: LPPPSpiderDeco
name: spiders
description: Fake spiders that look like they're ready to bite you.
placement:
mode: SnapgridCenter
components:
- type: Sprite
sprite: _LostParadise/Objects/Decoration/halloween.rsi
snapCardinals: true
noRot: false
layers:
- state: spider_big
map: [ "enum.DamageStateVisualLayers.Base" ]
- type: InteractionOutline
- type: Clickable
- type: RandomSprite
available:
- enum.DamageStateVisualLayers.Base:
@@ -31,4 +33,93 @@
sound:
collection: WoodDestroyHeavy
- !type:DoActsBehavior
acts: [ "Destruction" ]
acts: [ "Destruction" ]

- type: entity
id: LPPPPumpkinDeco
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

Исправьте опечатку в названии тыквы

В названии сущности есть опечатка: "pumkin" должно быть "pumpkin"

Предлагаемые изменения:

  id: LPPPPumpkinDeco
- name: pumkin
+ name: pumpkin
📝 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
- type: entity
id: LPPPPumpkinDeco
name: pumkin
- type: entity
id: LPPPPumpkinDeco
name: pumpkin

name: pumkin
placement:
mode: SnapgridCenter
description: Small decorative pumpkin with brush marks on it
components:
- type: Sprite
sprite: _LostParadise/Objects/Decoration/halloween.rsi
noRot: false
snapCardinals: true
layers:
- state: pumkin_minis
map: [ "enum.DamageStateVisualLayers.Base" ]
- type: InteractionOutline
- type: Clickable
- type: RandomSprite
available:
- enum.DamageStateVisualLayers.Base:
pumkin_minis: ""
pumkin_minis2: ""
- type: Damageable
damageContainer: Inorganic
- type: Destructible
thresholds:
- trigger:
!type:DamageTrigger
damage: 10
behaviors:
- !type:PlaySoundBehavior
sound:
collection: WoodDestroyHeavy
- !type:DoActsBehavior
acts: [ "Destruction" ]

- type: entity
id: LPPPPumpkinBigDeco
name: pumkin
parent: BaseSign
description: Decorative pumpkins with brush marks on it
components:
- type: Sprite
sprite: _LostParadise/Objects/Decoration/halloween.rsi
state: pumkins
- type: Damageable
damageContainer: Inorganic
- type: Destructible
thresholds:
- trigger:
!type:DamageTrigger
damage: 10
behaviors:
- !type:PlaySoundBehavior
sound:
collection: WoodDestroyHeavy
- !type:DoActsBehavior
acts: [ "Destruction" ]

- type: entity
id: LPPPottedPlantHalloween1
parent: PottedPlantBase
description: Withered houseplant, looks dead.
components:
- type: Sprite
sprite: _LostParadise/Objects/Decoration/halloween.rsi
state: plant1

- type: entity
id: LPPPottedPlantHalloween2
parent: LPPPottedPlantHalloween1
components:
- type: Sprite
state: plant2

- type: entity
id: LPPPottedPlantHalloween3
parent: LPPPottedPlantHalloween1
components:
- type: Sprite
state: plant3

- type: entity
id: LPPPottedPlantHalloween4
parent: LPPPottedPlantHalloween1
components:
- type: Sprite
state: plant4

Original file line number Diff line number Diff line change
@@ -29,7 +29,7 @@
- type: Sprite
sprite: _LostParadise/Structures/Wallmounts/halloween.rsi
state: checkboxes
noRot: true
noRot: false

- type: entity
parent: BaseSign
@@ -44,4 +44,4 @@
- type: Sprite
sprite: _LostParadise/Structures/Wallmounts/halloween.rsi
state: light_bulbs
noRot: true
noRot: false
Original file line number Diff line number Diff line change
@@ -21,6 +21,28 @@
},
{
"name": "spider_mini3"
},
{
"name": "pumkin_minis"
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

Исправьте опечатку в названии состояний

В названиях состояний присутствует опечатка: "pumkin" вместо правильного "pumpkin".

Примените следующие изменения:

-            "name": "pumkin_minis"
+            "name": "pumpkin_minis"
-            "name": "pumkin_minis2"
+            "name": "pumpkin_minis2"

Also applies to: 29-29

},
{
"name": "pumkin_minis2"
},
{
"name": "pumkins",
"directions": 4
},
Copy link
Contributor

Choose a reason for hiding this comment

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

💡 Codebase verification

⚠️ Potential issue

Обнаружены проблемы со спрайтами и их именованием

В состоянии "pumkins" указано 4 направления (directions: 4), но:

  • Найден только один файл спрайта pumkins.png
  • Опечатка "pumkins" присутствует в именах файлов
  • Отсутствуют спрайты для всех 4 направлений

Необходимо:

  • Исправить опечатку в meta.json и переименовать файлы с "pumkins" на "pumpkins"
  • Добавить недостающие спрайты для всех 4 направлений
  • Убедиться, что все спрайты корректно отображаются в игре
🔗 Analysis chain

Исправьте опечатку и проверьте спрайты

  1. Присутствует та же опечатка в названии.
  2. Убедитесь, что для состояния подготовлены все 4 спрайта для разных направлений.

Примените исправление:

-            "name": "pumkins",
+            "name": "pumpkins",
             "directions": 4
🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Проверка наличия всех необходимых спрайтов
find Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi -name "pumkins*.png" | wc -l

Length of output: 103


Script:

#!/bin/bash
# Проверим наличие файлов с правильным именем (pumpkins)
find Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi -name "pumpkins*.png" | wc -l

Length of output: 104


Script:

#!/bin/bash
# Посмотрим все файлы в директории .rsi для полной картины
ls -la Resources/Textures/_LostParadise/Objects/Decoration/halloween.rsi/

Length of output: 1007

{
"name": "plant1"
},
{
"name": "plant2"
},
{
"name": "plant3"
},
{
"name": "plant4"
Comment on lines +35 to +45
Copy link
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick (assertive)

Рекомендация по улучшению именования растений

Текущие имена состояний (plant1, plant2, plant3, plant4) слишком общие. Рекомендуется использовать более описательные названия, отражающие внешний вид или тематику растений.

Пример более информативных названий:

-            "name": "plant1"
+            "name": "halloween_plant_thorny"
-            "name": "plant2"
+            "name": "halloween_plant_twisted"
-            "name": "plant3"
+            "name": "halloween_plant_dead"
-            "name": "plant4"
+            "name": "halloween_plant_spooky"
📝 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
{
"name": "plant1"
},
{
"name": "plant2"
},
{
"name": "plant3"
},
{
"name": "plant4"
{
"name": "halloween_plant_thorny"
},
{
"name": "halloween_plant_twisted"
},
{
"name": "halloween_plant_dead"
},
{
"name": "halloween_plant_spooky"

}
]
}
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.
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.