Soul Effect 3.0.3

[vipclient]

vipclient добавил(а) новый ресурс:
Soul Effect - Душа отправляется в рай, показ кастомных моделей.

После гибели игрока его модель отрывается от тела и плавно поднимается
вверх, угасая. Поддерживается наблюдение от первого лица: сам игрок
видит, как его «душа» улетает, — через автоматическую камеру, которая
следит за ней до полного исчезновения.

Работает с любыми кастомными моделями без зависимостей от сторонних
плагинов.

Спойлер: Особенности

...

Узнать больше об этом ресурсе...

 

[ArKaNeMaN]

Правки для одобрения:

• Вынести список кваров в раздел "Настройка"
• Указать версию плагина в соотв. поле ресурса (там сейчас дата) и убрать её из названия
• Убрать название и номер версии из описания (дублируют соотв. отдельные поля)
• Убрать избыточное форматирование (зачем всё описание жирным шрифтом?)
• Вместо дефисов сделать горизонтальные линии специальной кнопкой в редакторе (см. скрин ниже). То же с маркированными списками, но это не так критично

Некритичные замечания:

• "Все параметры настраиваются через CVAR без перезапуска" - сами по себе квары подразумевают такое поведение, считаю излишним это уточнение
• "Нет зависимостей от внешних плагинов" - отсутствие оных в разделе требований уже это говорит
• "Совместимо с AMXX 1.9 / 1.10" - так же уже описано разделом требований
• И в целом не особо понимаю зачем особенности скрыты под спойлером, я бы вынес их из под него
• Квары можно оформить табличкой

Сообщение автоматически объединено: Вчера в 17:01

И ещё замечания чисто по коду:

1. Зачем нужна своя реализация clamp, если есть встроенная? При чём и для int и для float.
2. Почему бы не использовать биндинг кваров?
3. plugin_end тут по сути не нужен, AmxModX сам всё это чистит
4. Не совсем понимаю логику использования модификатора stock для функций, которые не могут быть использованы нигде, кроме этого плагина

 

[vipclient]

Правки для одобрения:

• Вынести список кваров в раздел "Настройка"
• Указать версию плагина в соотв. поле ресурса (там сейчас дата) и убрать её из названия
• Убрать название и номер версии из описания (дублируют соотв. отдельные поля)
• Убрать избыточное форматирование (зачем всё описание жирным шрифтом?)
• Вместо дефисов сделать горизонтальные линии специальной кнопкой в редакторе (см. скрин ниже). То же с маркированными списками, но это не так критично

Посмотреть вложение 1471

Некритичные замечания:

• "Все параметры настраиваются через CVAR без перезапуска" - сами по себе квары подразумевают такое поведение, считаю излишним это уточнение
• "Нет зависимостей от внешних плагинов" - отсутствие оных в разделе требований уже это говорит
• "Совместимо с AMXX 1.9 / 1.10" - так же уже описано разделом требований
• И в целом не особо понимаю зачем особенности скрыты под спойлером, я бы вынес их из под него
• Квары можно оформить табличкой

Сообщение автоматически объединено: Вчера в 17:01

И ещё замечания чисто по коду:

1. Зачем нужна своя реализация clamp, если есть встроенная? При чём и для int и для float.
2. Почему бы не использовать биндинг кваров?
3. plugin_end тут по сути не нужен, AmxModX сам всё это чистит
4. Не совсем понимаю логику использования модификатора stock для функций, которые не могут быть использованы нигде, кроме этого плагина

спасибо за совет, перезалил новую версию

 

[wopox1337]

В описании очень бы пригодилось .gif изображение с отображением эффекта, но не обязательно

 

[ArKaNeMaN]

@vipclient

1. Если уж не табличкой, то оберни хотя бы в блок кода квары.
2. Укажи в разделе настройки путь до файла настроек (который создаётся через AutoExecConfig).
3. Перезалей архив без .amxx файла.

 

[vipclient]

@vipclient

1. Если уж не табличкой, то оберни хотя бы в блок кода квары.
2. Укажи в разделе настройки путь до файла настроек (который создаётся через AutoExecConfig).
3. Перезалей архив без .amxx файла.

перезалил, gif прикрепил

 

[ArKaNeMaN]

@vipclient, Путь в описании неверный, судя по коду.

Спойлер

/**
 * Specifies that the given config file should be executed after plugin load.
 *
 * @note OnConfigsExecuted() will not be called until the config file has executed,
 *       but it will be called if the execution fails.
 * @note The name parameter should not contain dots, otherwise file will not be executed.
 *
 * @param autoCreate    If true, and the config file does not exist, such a config
 *                      file will be automatically created and populated with
 *                      information from the plugin's registered cvars.
 * @param name          Name of the config file, excluding the .cfg extension.
 *                      If empty, <plugin.filename.cfg> is assumed.
 * @param folder        Folder under plugins/ to use.
 *
 * @noreturn
 */
native AutoExecConfig(bool:autoCreate = true, const name[] = "", const folder[] = "");

Файл создаётся в папке amxmodx/configs/plugins/*

 

[vipclient]

@vipclient, Путь в описании неверный, судя по коду.

Спойлер

/**
 * Specifies that the given config file should be executed after plugin load.
 *
 * @note OnConfigsExecuted() will not be called until the config file has executed,
 *       but it will be called if the execution fails.
 * @note The name parameter should not contain dots, otherwise file will not be executed.
 *
 * @param autoCreate    If true, and the config file does not exist, such a config
 *                      file will be automatically created and populated with
 *                      information from the plugin's registered cvars.
 * @param name          Name of the config file, excluding the .cfg extension.
 *                      If empty, <plugin.filename.cfg> is assumed.
 * @param folder        Folder under plugins/ to use.
 *
 * @noreturn
 */
native AutoExecConfig(bool:autoCreate = true, const name[] = "", const folder[] = "");

Файл создаётся в папке amxmodx/configs/plugins/*

спасибо, исправил.

Сообщение автоматически объединено: Вчера в 22:15

Изменил:

• Убрал белый пересвет: kRenderFxGlowShell заменён на kRenderFxNone.
• Дефолт рендера: soul_rendermode 1 (обычная прозрачность без засвета).
• Дефолт прозрачности: soul_renderamt 150.
• Дефолт скорости подъёма: soul_speed 60.0.
• Версия поднята до 3.0.2.
• поправил создание конфига в папке configs/soul_effect.cfg

 

[vipclient]

Обновил версию 3.0.3
Конфиг теперь создается по верному пути configs/soul_effect.cfg

 

[wopox1337]

Конфиг был по верному пути, у вас просто описание было не корректным. Система AmxModX имеет встроенную возможность автоматического создания файла в нужной директории, и вам об этом указали выше:

@param autoCreate If true, and the config file does not exist, such a config * file will be automatically created and populated with * information from the plugin's registered cvars.

 

[CSLover]

Beautiful!