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

New ConVars: mp_item_respawn_time, mp_weapon_respawn_time, mp_ammo_respawn_time #983

Merged
merged 7 commits into from
Aug 3, 2024
Merged

New ConVars: mp_item_respawn_time, mp_weapon_respawn_time, mp_ammo_respawn_time #983

merged 7 commits into from
Aug 3, 2024

Conversation

SergeyShorokhov
Copy link
Member

@SergeyShorokhov SergeyShorokhov commented Aug 2, 2024 •
edited
Loading 

// The respawn time for items (such as health packs, armor, etc.).
// 0 - disable delay
//
// Default value: "30"
mp_item_respawn_time "30"

// The respawn time for weapons.
// 0 - disable delay
//
// Default value: "20"
mp_weapon_respawn_time "20"

// The respawn time for ammunition.
// 0 - disable delay
//
// Default value: "20"
mp_ammo_respawn_time "20"

@SergeyShorokhov SergeyShorokhov added the Type: 🚀 enhancement Improvement or addition of a new feature. label Aug 2, 2024
@SergeyShorokhov SergeyShorokhov requested a review from s1lentq August 2, 2024 22:21
@SergeyShorokhov SergeyShorokhov self-assigned this Aug 2, 2024
@SergeyShorokhov SergeyShorokhov merged commit 9d9c2de into master Aug 3, 2024
4 checks passed
@SergeyShorokhov SergeyShorokhov deleted the feature/convars-respawn-time branch August 3, 2024 17:17
jonathan-up added a commit to jonathan-up/ReGameDLL_CS that referenced this pull request Aug 4, 2024
@jonathan-up @SergeyShorokhov
Feature: ConVars for weapon/item/ammo respawn time (#13)
6507f41 
* Fix: `ammo`/`weapons` respawn behavior (rehlds#982)

* `CBasePlayerAmmo`: check spawnflags on `Spawn()`

* `CBasePlayerItem`: check spawnflags on `Materialize()`

* `CBasePlayerItem`: Add `Respawn()` item when hasn't specific spawnflags

* `CBasePlayerItem`: remove `SF_NORESPAWN` flag on `Respawn()`

* Use forgotten `AMMO_RESPAWN_TIME`

* Feature: ConVars for `weapon`/`item`/`ammo` respawn time (rehlds#983)

* `CBasePlayerAmmo`: check spawnflags on `Spawn()`

* `CBasePlayerItem`: check spawnflags on `Materialize()`

* `CBasePlayerItem`: Add `Respawn()` item when hasn't specific spawnflags

* `CBasePlayerItem`: remove `SF_NORESPAWN` flag on `Respawn()`

* Use forgotten `AMMO_RESPAWN_TIME`

* new ConVars: `mp_item_respawn_time`, `mp_weapon_respawn_time`, `mp_ammo_respawn_time`

---------

Co-authored-by: Sergey Shorokhov <[email protected]>
@SergeyShorokhov SergeyShorokhov changed the title Feature: ConVars for weapon/item/ammo respawn time New ConVars: mp_<weapon/item/ammo>_time Dec 12, 2024
@SergeyShorokhov SergeyShorokhov changed the title New ConVars: mp_<weapon/item/ammo>_time New ConVars: mp_<weapon/item/ammo>_respawn_time Dec 12, 2024
@SergeyShorokhov SergeyShorokhov added Priority: 🕒 low Low priority tasks that can be postponed for the future. Status: 🎮 testing Issue in testing stage. Difficulty: 🟢 easy Easy task that can be completed quickly. labels Dec 12, 2024
@SergeyShorokhov SergeyShorokhov changed the title New ConVars: mp_<weapon/item/ammo>_respawn_time New ConVars: mp_<weapon/item/ammo>_respawn_time Dec 12, 2024
@SergeyShorokhov SergeyShorokhov changed the title New ConVars: mp_<weapon/item/ammo>_respawn_time New ConVars: mp_item_respawn_time, mp_weapon_respawn_time, mp_ammo_respawn_time Dec 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@s1lentq s1lentq Awaiting requested review from s1lentq

Assignees

@SergeyShorokhov SergeyShorokhov

Labels
Difficulty: 🟢 easy Easy task that can be completed quickly. Priority: 🕒 low Low priority tasks that can be postponed for the future. Status: 🎮 testing Issue in testing stage. Type: 🚀 enhancement Improvement or addition of a new feature.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@SergeyShorokhov