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.

Sign up for GitHub

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

Jump to bottom

Atmos meter can not be wrenched #306

Closed
Artorp opened this issue Feb 13, 2015 · 3 comments
Closed

Atmos meter can not be wrenched #306

Artorp opened this issue Feb 13, 2015 · 3 comments
Labels
Bug This is a bug in our game code

Comments

@Artorp
Copy link
Contributor

Artorp commented Feb 13, 2015

Current workaround everyone is doing is unscrewing whatever pipe the meter is connected to.

Apparently this happens because Click() overwrites attack_by() in https://github.com/ParadiseSS13/Paradise/blob/master/code/game/machinery/atmoalter/meter.dm#L98-L124
@Artorp
Copy link
Contributor Author

Artorp commented Feb 13, 2015

Looks like this will be fixed by #12

@mvanalphen mvanalphen added the Bug This is a bug in our game code label Feb 13, 2015
@mvanalphen mvanalphen mentioned this issue Feb 13, 2015
Merged
@FalseIncarnate
Copy link
Contributor

Meters could be un-wrenched with double click, though Mark already has a PR to probably simplify and standardize this.

@Artorp
Copy link
Contributor Author

Artorp commented Feb 14, 2015

Huh, I did not now that. All those times I've been unscrewing pipes... Anyway, that was fixed quite swiftly, good job Mark!

@Artorp Artorp closed this as completed Feb 14, 2015
mattgroy pushed a commit to mattgroy/Paradise that referenced this issue Nov 10, 2023
@AyIong @Furrior
Fix: Card Destroyer Fax (ParadiseSS13#306)
6002813 
<!-- Пишите **НИЖЕ** заголовков и **ВЫШЕ** комментариев, иначе что то
может пойти не так. -->
<!-- Вы можете прочитать Contributing.MD, если хотите узнать больше. -->

## Что этот PR делает
Факс больше не уничтожает карты.

<!-- Вкратце опишите изменения, которые вносите. -->
<!-- Опишите **все** изменения, так как противное может сказаться на
рассмотрении этого PR'а! -->
<!-- Если вы исправляете Issue, добавьте "Fixes ParadiseSS13#1234" (где 1234 - номер
Issue) где-нибудь в описании PR'а. Это автоматически закроет Issue после
принятия PR'а. -->

## Почему это хорошо для игры
Факс больше не уничтожает карты.

<!-- Опишите, почему, по вашему, следует добавить эти изменения в игру.
-->

## Изображения изменений

![image](https://github.com/ss220club/Paradise-SS220/assets/69762909/9878b367-7e82-4922-90bd-80ba528a1354)

<!-- Если вы не меняли карту или спрайты, можете опустить эту секцию.
Если хотите, можете вставить видео. -->

## Тестирование
Факс больше не уничтожает карты.

<!-- Как вы тестировали свой PR, если делали это вовсе? -->

## Changelog

:cl:
fix: Факс больше не уничтожает карты.
/:cl:

<!-- Оба :cl:'а должны быть на месте, что-бы чейнджлог работал! Вы
можете написать свой ник справа от первого :cl:, если хотите. Иначе
будет использован ваш ник на ГитХабе. -->
<!-- Вы можете использовать несколько записей с одинаковым префиксом
(Они используются только для иконки в игре) и удалить ненужные. Помните,
что чейнджлог должен быть понятен обычным игроком. -->
<!-- Если чейнджлог не влияет на игроков(например, это рефактор), вы
можете исключить всю секцию. -->

---------

Co-authored-by: Furior <68264134+Furrior@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug This is a bug in our game code
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@Artorp @mvanalphen @FalseIncarnate