Skip to content

Conversation

@Nord1cWarr1or
Copy link
Contributor

Add CVar mapm_extend_map_if_no_votes, which allows you to extend the current map if no one has voted.

...which allows you to extend the current card if no one has voted.
@Mistrick

This comment has been minimized.

@Mistrick Mistrick self-requested a review September 9, 2021 06:14
Copy link
Owner

@Mistrick Mistrick left a comment

Choose a reason for hiding this comment

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

Всё должно быть реализовано в планировщике. В ядро лишь добавить недостающий API для реализации.

@Nord1cWarr1or
Copy link
Contributor Author

@Mistrick done.

@Takanashi-Rikka1
Copy link

I can not open my own request for some reason.
anyway @Mistrick

Add cvar mapm_change_type 3
its means after vote ending let the owners choose the time they want map change in minutes
now if you set "0", It replaces a map instantly.

Nomination:
option to search map like

in the chat
nom dust

open menu with all maps containing this name.

categories to /maps
By Name(a-z) , Date Added, Last Played, Size, Most popular

that's all!, if the project still active could be nice if all this be in the votemap.
Good Luck!

Copy link
Owner

@Mistrick Mistrick left a comment

Choose a reason for hiding this comment

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

Измени версию планировщика.
На пустом сервере будет вечное продление одной карты. Для меня это ломает ротацию карт. Должно ли быть больше продлений чем указано в кваре mapm_extended_map_max?

@Nord1cWarr1or
Copy link
Contributor Author

Nord1cWarr1or commented Oct 21, 2021

Измени версию планировщика.

Хотел тебе оставить управление версиями. Ведь в других PR я тоже вносил в планировщик правки.

На пустом сервере будет вечное продление одной карты.

Именно этого я и добивался.

Должно ли быть больше продлений чем указано в кваре mapm_extended_map_max?

Не знаю. Можно добавить ещё один режим квара mapm_extend_map_if_no_votes, где будет учитываться значение mapm_extended_map_max, и если лимит - менять карту. Что скажешь?

@JUNESYNGOTOFLEX
Copy link

Измени версию планировщика.

Хотел тебе оставить управление версиями. Ведь в других PR я тоже вносил в планировщик правки.

На пустом сервере будет вечное продление одной карты.

Именно этого я и добивался.

Должно ли быть больше продлений чем указано в кваре mapm_extended_map_max?

Не знаю. Можно добавить ещё один режим квара mapm_extend_map_if_no_votes, где будет учитываться значение mapm_extended_map_max, и если лимит - менять карту. Что скажешь?

you must use 3rd-party plugins for this. сам же делал что продлевает timelimit при пустом сервере

@Nord1cWarr1or
Copy link
Contributor Author

@JUNESYNGOTOFLEX ты вообще о чём?

@JUNESYNGOTOFLEX
Copy link

На пустом сервере будет вечное продление одной карты.

Именно этого я и добивался.

https://dev-cs.ru/resources/581/

@Nord1cWarr1or
Copy link
Contributor Author

@JUNESYNGOTOFLEX

  1. Причем тут сейчас сторонний плагин?
  2. Функционал, который я предлагаю в этом реквесте, продлит текущую карту, в отличие от моего плагина, который сменит её на стандартную.

@JUNESYNGOTOFLEX
Copy link

@JUNESYNGOTOFLEX

  1. Причем тут сейчас сторонний плагин?
  2. Функционал, который я предлагаю в этом реквесте, продлит текущую карту, в отличие от моего плагина, который сменит её на стандартную.

я не доглядел что он меняет карту кроме как устанавливает timelimit в 0, это почти одно и тоже в лучшем случае как я сказал это написать сторонний плагин на пару строк даже не использовав апи от этой системы.

@Nord1cWarr1or
Copy link
Contributor Author

@Mistrick я добавил зависимость от mapm_extend_map_if_no_votes

Copy link
Owner

@Mistrick Mistrick left a comment

Choose a reason for hiding this comment

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

Версию плагина нужно апнуть.


new extend_map_no_votes = get_num(EXTEND_MAP_IF_NO_VOTES);

new bool:bNeedExtend = bool:(!total_votes && (extend_map_no_votes && g_iExtendedNum < get_num(EXTENDED_MAX)));
Copy link
Owner

Choose a reason for hiding this comment

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

Измени bNeedExtend на can_be_extend
Внутренняя пара скобок ничего не делает и не особо разграничивает условие.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Mistrick
Может тогда вообще так:

new bool:can_be_extend = bool:(equali(map, g_sCurMap) || !total_votes && extend_map_no_votes && g_iExtendedNum < get_num(EXTENDED_MAX));

    // map extended
    if(can_be_extend) {

@Nord1cWarr1or Nord1cWarr1or requested a review from Mistrick January 5, 2022 15:12
@Mistrick Mistrick merged commit 995db9a into Mistrick:master Jan 5, 2022
@Nord1cWarr1or Nord1cWarr1or deleted the zero-votes branch January 5, 2022 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants