Skip to content

Conversation

Mirag1993
Copy link

@Mirag1993 Mirag1993 commented Sep 10, 2025

Описание / Что этот PR делает

Изменяет работу кобуры , теперь это не только дополнительный карман на рубашке а удобный предмет для быстрой работы с оружием.

Причина создания ПР / Почему это хорошо для игры

Мне всегда не нравилась механика кобур ТГ , открой доп карман , кликни мимо кобуры , умри пока доставал пушку , а если еще чтото в руке есть , то надо сначало сбросить , что не всегда из за лагов удается быстро сделать.
Стиль парадиз кобур вводит четкое обещание , после нажатия кнопки , то что было в кобуре , появится в активной руке.

Демонстрация изменений / Тестирование

1.Установить кейбинд на удобную кнопку
2.Надеть кобуру на одежду либо взять в руки( не сработает кейбинд , только в ручную работать )
3.Взять оружие подходящее для кобуры , нажать кейбинд либо кликнуть оружием по кобуре или одежде с кобурой
4.Оружие вложится в кобуру если позволяют условия
5.Достать можно либо повторным нажатием на кейбинд либо альт клик по униформе с кобурой либо кобуре , откроется ui окно с содержимым
6.Оружие снова появится в руке.

Список изменений

Изменена работа кобуры , теперь она имеет всего один слот.
В кобуру можно положить только оружие смолл размера и без авто режима.
Кобура nuke может принимать оружие большего размера.
Добавлена unbind кнопка для действия holster/unholster.

🆑
- add: Новая система кобур в стиле Paradise
- add: Поддержка клавиши для быстрого доступа к кобуре
- add: Проверка размера оружия (TINY/SMALL)
- add: Максимум 1 предмет в кобуре 
- add: Возможность положить в кобуру пистолет, револьвер и энергетическое оружие
- del: Старая система кобур Shiptest (закомментирована)
/🆑

Меняем работу кобуры на парадайз-стайл
@Celadon-Bot Celadon-Bot added 🧱 Код 🧱 Изменения .dm файлов 🪓 ТРОГАЛИ CORE-CODE 🪓 Ты трогал офовский кор код! Внимание! labels Sep 10, 2025
@github-actions github-actions bot requested a review from Vairkharst September 10, 2025 21:19
Copy link
Member

@KOCMODECAHTHUK KOCMODECAHTHUK left a comment

Choose a reason for hiding this comment

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

Это конечно хорошо, но почему не улучшить тогда обычный холстер без переписывания методов?

Мне не очень нравится вообще что у нас в билде существуют функциональные предметы что цепляются на одежду, это только путает.

Если хочешь улучшить попробуй через метод on_alt_click_async(datum/source, mob/user) что я разбирал в #2215
Ну и хоткей добавить отдельно с той же проверкой на холстер и оружие.

Comment on lines 178 to 204
// Chameleon holster
/obj/item/clothing/accessory/holster/chameleon
name = "syndicate holster"
desc = "A two pouched hip holster that uses chameleon technology to disguise itself and any guns in it."
var/datum/action/item_action/chameleon/change/chameleon_action

/obj/item/clothing/accessory/holster/chameleon/Initialize()
. = ..()

chameleon_action = new(src)
chameleon_action.chameleon_type = /obj/item/clothing/accessory
chameleon_action.chameleon_name = "Accessory"
chameleon_action.initialize_disguises()

/obj/item/clothing/accessory/holster/chameleon/Destroy()
QDEL_NULL(chameleon_action)
return ..()

/obj/item/clothing/accessory/holster/chameleon/emp_act(severity)
. = ..()
if(. & EMP_PROTECT_SELF)
return
chameleon_action.emp_randomise()

/obj/item/clothing/accessory/holster/chameleon/broken/Initialize()
. = ..()
chameleon_action.emp_randomise(INFINITY)
Copy link
Member

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.

зачем что ? зачем кобура хамелеон ? ну хз , сложный вопрос

Реворк чутка , добавлена возможность смотреть инвентарь кобуры , изменена проверка
@Celadon-Bot Celadon-Bot added the 🔊 Звуки 🔊 Изменения звуков label Sep 11, 2025
Смешные баги с бесконечным хранилищем пистолетов
@Mirag1993
Copy link
Author

Это конечно хорошо, но почему не улучшить тогда обычный холстер без переписывания методов?

Мне не очень нравится вообще что у нас в билде существуют функциональные предметы что цепляются на одежду, это только путает.

Если хочешь улучшить попробуй через метод on_alt_click_async(datum/source, mob/user) что я разбирал в #2215 Ну и хоткей добавить отдельно с той же проверкой на холстер и оружие.

Немного не так но по сути реализовано , хотя считаю что это лишнее

@KOCMODECAHTHUK
Copy link
Member

Я абстрактно говорил, не суть.

Copy link
Member

@KOCMODECAHTHUK KOCMODECAHTHUK left a comment

Choose a reason for hiding this comment

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

Тяжело, сильно много багов будет...
Это я ещё про дубляж кода умолчал.

@KOCMODECAHTHUK KOCMODECAHTHUK marked this pull request as draft September 18, 2025 12:17
Попытка рефактора , надеюсь удачная , чуть разнесли код и убрали косяки
Убираем лишнюю проверку
Исправляем претензии по коду
Надеюсь последние правки , косметику править уже не буду
Copy link
Member

@KOCMODECAHTHUK KOCMODECAHTHUK left a comment

Choose a reason for hiding this comment

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

Чет смущает подробнейшая логирование

Copy link
Member

Choose a reason for hiding this comment

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

Напомни а зачем нам такое подробное логирование кобуры?

Copy link
Member

Choose a reason for hiding this comment

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

Выглядит так как будто это лишнее

Убираем лишние логирование
@KOCMODECAHTHUK KOCMODECAHTHUK changed the title EDIT: Меняем работу кобуры на парадайз-стайл [TM] ADD: Меняем работу кобуры на парадайз-стайл Oct 4, 2025
@MysticalFaceLesS
Copy link
Member

MysticalFaceLesS commented Oct 4, 2025

Test merge deployment history:

Test Merge Deployed @ 10/04/2025 16:26:02 +00:00:
Server Instance

Shiptest_Alpha

Merged By

De-Grand

Revision

Origin: ed76e50
Pull Request: 58b0097
Server: eb3df9a1aa6913a39266b358bed8aa6bc5778524

Test Merge Updated @ 10/05/2025 15:52:02 +00:00:
Server Instance

Shiptest_Alpha

Merged By

Voiko

Revision

Origin: ed76e50
Pull Request: 58b0097
Server: 870a9bd67786db084512ff7dd29c6d9103a1d5fd

Test Merge Updated @ 10/05/2025 16:23:30 +00:00:
Server Instance

Shiptest_Alpha

Merged By

Voiko

Revision

Origin: ed76e50
Pull Request: 58b0097
Server: 4b1fdd3a523e5298a9a52b653df23293da4df0de

Test Merge Updated @ 10/05/2025 22:20:18 +00:00:
Server Instance

Shiptest_Alpha

Merged By

De-Grand

Revision

Origin: ed76e50
Pull Request: 58b0097
Server: 1e5a09a0ca695f7c5681f24ccf0650af4b16c002

Test Merge Updated @ 10/05/2025 22:43:56 +00:00:
Server Instance

Shiptest_Alpha

Merged By

Voiko

Revision

Origin: ed76e50
Pull Request: 58b0097
Server: a6df3d8e174e18f4ea1a15737ea4d1a4211eb25c

@KOCMODECAHTHUK KOCMODECAHTHUK added the ! TM ! PR в тестировании на лайв-сервере label Oct 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔊 Звуки 🔊 Изменения звуков 🧱 Код 🧱 Изменения .dm файлов 🪓 ТРОГАЛИ CORE-CODE 🪓 Ты трогал офовский кор код! Внимание! ! TM ! PR в тестировании на лайв-сервере
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants