new_weapon_for_sec #51

unDepresh · 2024-07-30T01:59:07Z

Убрано табельный mk-58 у таких ролей как: Офицер сб, смотритель, ГСБ.
Добавлен миниатюрный энергетический пистолет как табельное оружие офицеру сб и смотрителю, так за 2500 можно заказать ящик с двумя такими.
Добавлен Х-2 Двухфазовый энергетический лазерный пистолет. Появляется у ГСБ как табельное оружие в кармане
Removed the mk-58 from such roles as: Officer of the sb, caretaker, GSB.
Added a miniature energy pistol as a tactical weapon for the sec officer and warden, so for 2500 you can order a box with two of them.
Added X-2 Dual Phase Energy Laser Pistol. Appears in the HoS as a service weapon in his pocket

CLAassistant · 2024-07-30T01:59:13Z

All committers have signed the CLA.

github-actions · 2024-07-30T01:59:52Z

RSI Diff Bot; head commit ca151aa merging into e031716
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/Exodus/Objects/Weapons/Guns/Battery/MiniEnergyGun.rsi

State	Old	New	Status
base			Added
equipped-BELT			Added
equipped-SUITSTORAGE			Added
icon			Added
inhand-left-0			Added
inhand-left-1			Added
inhand-left-2			Added
inhand-left-3			Added
inhand-left			Added
inhand-right-0			Added
inhand-right-1			Added
inhand-right-2			Added
inhand-right-3			Added
inhand-right			Added
mag-unshaded-0			Added
mag-unshaded-1			Added
mag-unshaded-2			Added
mag-unshaded-3			Added
mag-unshaded-4			Added

Resources/Textures/Exodus/Objects/Weapons/Guns/Battery/MyltiLaserForHoS.rsi

State	Old	New	Status
base			Added
equipped-BELT			Added
equipped-SUITSTORAGE			Added
icon			Added
inhand-left-0			Added
inhand-left-1			Added
inhand-left-2			Added
inhand-left-3			Added
inhand-left			Added
inhand-right-0			Added
inhand-right-1			Added
inhand-right-2			Added
inhand-right-3			Added
inhand-right			Added
mag-unshaded-0			Added
mag-unshaded-1			Added
mag-unshaded-2			Added
mag-unshaded-3			Added
mag-unshaded-4			Added

Resources/Textures/Exodus/Objects/Weapons/Guns/Projectiles/ProjectilesForLaserByDepresh.rsi

State	Old	New	Status
impact_laser_greyscale			Added
omnilaser_greyscale			Added

Edit: diff updated after ca151aa

Fragoler

Необходимо немного подправить общий вид PR.

Fragoler · 2024-07-30T09:52:05Z

Resources/Prototypes/Catalog/Cargo/cargo_security.yml

@@ -77,3 +77,13 @@
  cost: 1000
  category: cargoproduct-category-name-security
  group: market
+
+- type: cargoProduct


Переместить в отдельный yml в Exodus/Catalog/Cargo/...yml

Fragoler · 2024-07-30T09:52:39Z

Resources/Prototypes/Catalog/Fills/Crates/security.yml

@@ -114,3 +114,14 @@
        amount: 4

 # Cosmetic Crates
+
+- type: entity


Аналогично переместить в отдельный yml

Fragoler · 2024-07-30T09:53:03Z

Resources/Prototypes/Catalog/Fills/Crates/security.yml

+- type: entity
+  id: CrateSecurityMiniEnergyGun
+  parent: CrateSecgear
+  name: Ящик с миниатюрными энергетическими пистолетами


Локализация через FTL

Fragoler · 2024-07-30T09:53:31Z

Resources/Prototypes/Entities/Objects/Weapons/Guns/Battery/MiniEnergyGunForSec.yml

@@ -0,0 +1,129 @@
+# Сам прототип оружия
+- type: entity
+  name: Миниатюрный энергетический пистолет 


Локализация. Переместить в Exodus

Fragoler · 2024-07-30T09:54:04Z

Resources/Prototypes/Entities/Objects/Weapons/Guns/Battery/MiniEnergyGunForSec.yml

+
+# продект лазера
+- type: entity
+  name: Режим лазера


Локализация

Fragoler · 2024-07-30T10:02:44Z

Resources/Prototypes/Roles/Jobs/Security/warden.yml

@@ -30,8 +30,7 @@
    eyes: ClothingEyesGlassesSecurity
    id: WardenPDA
    ears: ClothingHeadsetSecurity
-    pocket1: WeaponPistolMk58


Комментарий изменения кода оффов

Fragoler · 2024-07-30T10:04:37Z

Resources/Textures/Objects/Weapons/Guns/Battery/MyltiLaserForHoS.rsi/meta.json

@@ -0,0 +1,108 @@
+{


Разделить на смысловые части.
Переместить в Textures/Exodus/
Разделить на rsi

Fragoler · 2024-07-30T10:05:48Z

Resources/Audio/Weapons/HoSLaser.ogg

Переместить в Audio/Exodus/

Fragoler · 2024-07-30T10:05:56Z

Resources/Audio/Weapons/MiniEnergyGun.ogg

Переместить в Audio/Exodus/

Fragoler · 2024-07-30T10:06:19Z

Resources/Audio/Weapons/HoSLaser.ogg

Написать Atttribution.yml для звуков

Lokilife · 2024-07-30T12:25:27Z

Заменить текстурки самих лазеров на обесцвеченные (спойлер, одна из них уже есть) и задавать цвет уже в компоненте Sprite.

unDepresh · 2024-07-30T12:36:24Z

Заменить текстурки самих лазеров на обесцвеченные (спойлер, одна из них уже есть) и задавать цвет уже в компоненте Sprite.
А зачем вообще и в какой из них уже такая обесцвеченная есть? Я с текстурами пока особо не работал нормально, но этот момент у тебя в коде лазера осщ видел

github-actions · 2024-08-30T09:56:10Z

This pull request has conflicts, please resolve those before we can evaluate the pull request.

Fragoler

Переименовать, разрешить конфликт, да понять что-то с дробовиком и будет конфетка..)

Fragoler · 2024-08-30T10:21:52Z

.../Textures/Exodus/Objects/Weapons/Guns/Projectiles/ProjectilesForLaserByDepresh.rsi/meta.json

@@ -0,0 +1,25 @@
+{


Переименовать папку...

На кой чёрт этот RSI, если в сборке такие спрайты уже есть?

Fragoler · 2024-08-30T10:22:46Z

Resources/Prototypes/Roles/Jobs/Security/security_officer.yml

@@ -27,7 +27,7 @@
  equipment:
    eyes: ClothingEyesGlassesSecurity
    ears: ClothingHeadsetSecurity
-    pocket1: WeaponMiniEnergyGun
+    pocket1: WeaponMiniEnergyGun # Убрал "WeaponPistolMk58" и добавил "WeaponMiniEnergyGun"


Заменить на комментарий # Exodus - WeaponForHos

Fragoler · 2024-08-30T10:23:00Z

Resources/Prototypes/Roles/Jobs/Security/security_officer.yml

  storage:
-    back:
-    - Flash
+    back: # Убрал "MagazinePistol "


Заменить комментарий

Fragoler · 2024-08-30T10:23:08Z

Resources/Prototypes/Roles/Jobs/Security/warden.yml

@@ -30,7 +30,7 @@
    eyes: ClothingEyesGlassesSecurity
    id: WardenPDA
    ears: ClothingHeadsetSecurity
-    pocket1: WeaponMiniEnergyGun
+    pocket1: WeaponMiniEnergyGun # Убрал "WeaponPistolMk58" и добавил "WeaponMiniEnergyGun"


Заменить комментарий

Fragoler · 2024-08-30T10:23:11Z

Resources/Prototypes/Roles/Jobs/Security/warden.yml

  storage:
-    back:
-    - Flash
+    back: # Убрал "MagazinePistol"


Заменить комментарий

Fragoler · 2024-08-30T10:24:16Z

...es/Prototypes/Exodus/Entities/Objects/Weapons/Guns/Projectiles/ProjectilesForLaserForHos.yml

@@ -0,0 +1,82 @@
+# проджект лазера


Заменить название файла на LaserForHos.yml

❌
На energetic.yml, если файл уже существует, то закинуть в конец. Исправить косяки с опечатками. Нет такого слова Staner, есть Stunner, от слова Stun. И всё в таком духе.
Также эффекты от попаданий лежат во отдельном файле, impacts.yml

Fragoler · 2024-08-30T10:24:44Z

.../Prototypes/Exodus/Entities/Objects/Weapons/Guns/Projectiles/ProjectilesForMiniEnergyGun.yml

@@ -0,0 +1,79 @@
+# продект лазера


Заменить название на MiniEnergyGun.yml

К projectiles.yml, эффекты от попаданий в impacts.yml

Fragoler · 2024-08-30T10:25:54Z

Resources/Prototypes/Exodus/Catalog/Fills/Crates/securityWithEnergyGunForSec.yml

@@ -0,0 +1,10 @@
+- type: entity


Заменить название файла на security.yml

Fragoler · 2024-08-30T10:26:09Z

Resources/Prototypes/Exodus/Catalog/Cargo/cargo_securityWithMiniEnergyGun.yml

@@ -0,0 +1,9 @@
+- type: cargoProduct


Заменить название файла на security.yml

Lokilife · 2024-08-30T12:12:55Z

...es/Prototypes/Exodus/Entities/Objects/Weapons/Guns/Projectiles/ProjectilesForLaserForHos.yml

+# проджект станера
+- type: entity
+  name: Staner
+  id: stanerBullet


Здесь вообще нарушение конвенции имён энтитей, которые должны быть в PascalCase (начинается с большой буквы и каждое слово тоже с большой).
Да и в принципе здесь ID другой быть должен.

Lokilife · 2024-08-30T12:13:38Z

.../Prototypes/Exodus/Entities/Objects/Weapons/Guns/Projectiles/ProjectilesForMiniEnergyGun.yml

+
+# эффект от станера
+- type: entity
+  id: BulletImpactEffectLaserStan


BulletImpactEffectLaserCyan -> BulletImpactEffectStunnerCyan

Lokilife · 2024-08-30T12:13:54Z

Resources/Prototypes/Roles/Jobs/Security/head_of_security.yml

@@ -46,7 +46,8 @@
    id: HoSPDA
    gloves: ClothingHandsGlovesCombat
    ears: ClothingHeadsetAltSecurity
-    pocket1: WeaponX-2Dual-phaseEnergyLaserGun
+    pocket1: WeaponPistolMk58


Потерял коментарий

Lokilife · 2024-08-30T12:15:10Z

.../Prototypes/Exodus/Entities/Objects/Weapons/Guns/Projectiles/ProjectilesForMiniEnergyGun.yml

+# продект лазера
+- type: entity
+  name: Laser
+  id: LaserBulletForMEG


Никаких "AForB" в ID и названиях. -> LaserBulletMEG.

А ещё лучше отнести снаряд лазера к какой-либо категории, более унифицированно и отталкиваться уже от названия категории, а не от названия пушки, для которой он был сделан.

Lokilife · 2024-08-30T12:16:11Z

Resources/Prototypes/Exodus/Entities/Objects/Weapons/Guns/Battery/MiniEnergyGunForSec.yml

@@ -0,0 +1,53 @@
+# Сам прототип оружия
+- type: entity
+  name: Mini energy gun 


Название энтити с маленькой буквы быть должны

Да и название по-креативнее бы....

Lokilife · 2024-08-30T12:20:09Z

...es/Prototypes/Exodus/Entities/Objects/Weapons/Guns/Projectiles/ProjectilesForLaserForHos.yml

+
+# эффект от станера
+- type: entity
+  id: BulletImpactEffectDisablerRed


Ну хоть себе-то не ври.
BulletImpactEffectDisablerRed -> BulletImpactEffectDisablerPink

Lokilife · 2024-08-30T12:21:45Z

С названиями сплошная лажа.
@Fragoler внимательнее следи за кодом, который проверяешь и одобряешь.

github-actions · 2024-09-08T19:59:36Z

This pull request has conflicts, please resolve those before we can evaluate the pull request.

new_weapon_for_sec

09a19d8

github-actions bot added No C# Changes: Sprites labels Jul 30, 2024

Fragoler suggested changes Jul 30, 2024

View reviewed changes

new_weapon_for_secr_upd1

ca151aa

github-actions bot added the Changes: Localization label Aug 4, 2024

github-actions bot added the Merge Conflict label Aug 30, 2024

Fragoler reviewed Aug 30, 2024

View reviewed changes

Lokilife reviewed Aug 30, 2024

View reviewed changes

github-actions bot removed the Merge Conflict label Sep 6, 2024

Lokilife added the Status: Awaiting changes label Sep 6, 2024

github-actions bot added the Merge Conflict label Sep 8, 2024

new_weapon_for_sec #51

Are you sure you want to change the base?

new_weapon_for_sec #51

Conversation

unDepresh commented Jul 30, 2024

CLAassistant commented Jul 30, 2024 • edited Loading

github-actions bot commented Jul 30, 2024 • edited Loading

Fragoler left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Lokilife commented Jul 30, 2024

unDepresh commented Jul 30, 2024

github-actions bot commented Aug 30, 2024

Fragoler left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Lokilife Aug 30, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Lokilife commented Aug 30, 2024

github-actions bot commented Sep 8, 2024

CLAassistant commented Jul 30, 2024 •

edited

Loading

github-actions bot commented Jul 30, 2024 •

edited

Loading

Lokilife Aug 30, 2024 •

edited

Loading