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

Antag xeno restrictions #312

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

Kernstall
Copy link
Contributor

@Kernstall Kernstall commented Oct 24, 2018

Крутись, колесо репрессий, крутись.

Надо бы хорошо проверить перед мержем, потому что потому.
🆑

  • tweak: СПУ потеряли возможность становиться культистами, блобами, магами, генокрадами, тенелингами. Дионы потеряли возможность становиться генокрадами.
  • tweak: Главы только для людей и скреллов.

@Skippeer12
Copy link
Contributor

Вместо того чтобы пихать это в инициализацию, нужно выпилить возможность расам получать данные роли вообще.

@LudwigVonChesterfield
Copy link
Contributor

И что, даже от педалей выпилить? Нормально сделано.

@Kernstall
Copy link
Contributor Author

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

@DieSiege
Copy link
Contributor

DieSiege commented Nov 1, 2018

Конфликтить начало.

segway wip
😏

…into antag-xeno-restrictions

# Conflicts:
#	code/game/gamemodes/changeling/changeling.dm
@Kernstall
Copy link
Contributor Author

С запретом на конверт определенных рас проблемы есть, так что не сегодня наверное

@Kernstall
Copy link
Contributor Author

@Miracler112 Этот референс ни к месту, потому что не исправит проблему замораживания машин. Он только не даст машинам быть лингами

@Miracler112
Copy link
Contributor

Показалось тогда что оно и это исправит, ладно, убираю тогда референс

@Kernstall
Copy link
Contributor Author

Здарова. Ксеноте выпилены главы, кроме скреллов. Листы с запрещенными ролями теперь в datum/species. Конверт неподходящей ксеноты в культ отловлен

@LudwigVonChesterfield
Copy link
Contributor

LudwigVonChesterfield commented Nov 18, 2018

Antag xeno restrictions

Здарова. Ксеноте выпилены главы, кроме скреллов.

А ты неплох.

@Skippeer12
Copy link
Contributor

Skippeer12 commented Nov 18, 2018

Never give up!

Xeno-scum must suffer!

code/game/gamemodes/blob/blob.dm Outdated Show resolved Hide resolved
@@ -55,6 +55,11 @@ var/list/blob_nodes = list()

return 1

/datum/game_mode/blob/check_species_restriction(var/mob/dead/new_player/player)
Copy link
Contributor

Choose a reason for hiding this comment

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

var/ - объявление того что аргумент переданный в функцию - переменная?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Спиздил и не заметил, во как бывает

code/game/gamemodes/blob/blob.dm Outdated Show resolved Hide resolved
@@ -75,6 +75,11 @@ var/list/possible_changeling_IDs = list("Alpha","Beta","Gamma","Delta","Epsilon"

return ..()

/datum/game_mode/changeling/check_species_restriction(var/mob/dead/new_player/player)
Copy link
Contributor

Choose a reason for hiding this comment

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

Вообще некрасиво выходит как-то каждый раз перезаписывать функцию. Почему нельзя у гейммода объявить переменную типо role_type, и в общей функции родителя проверять return S.restricted_roles[role_type]?

code/game/gamemodes/cult/cult_datums.dm Outdated Show resolved Hide resolved
@@ -66,6 +66,8 @@
var/flesh_color = "#FFC896" //Pink.
var/base_color //Used when setting species.

var/list/restricted_roles = list()
Copy link
Contributor

Choose a reason for hiding this comment

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

Ты объявил список запрещённых ролей, но нигде его не наполняешь. М?

@Kernstall Kernstall force-pushed the antag-xeno-restrictions branch from f23bc8a to 81c82ff Compare November 22, 2018 18:42
@Miracler112
Copy link
Contributor

Ну вроде в этот раз всё более менее нормально :pekapled:

@Phoenix4O4
Copy link
Contributor

Phoenix4O4 commented Nov 25, 2018

@Kernstall а культисты-дионеи? Там надо либо запрет на рисование кровью(вроде бы делали) либо запрет.

@Kernstall
Copy link
Contributor Author

Культисты-дионеи хоть и стремная вещь, но в принципе не рушит атмосферы. Запрет на рисованию прикольно, но не в этом пре

@Kernstall Kernstall closed this Nov 25, 2018
@Kernstall Kernstall reopened this Nov 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants