Skip to content

New CVar: mp_plant_c4_anywhere#692

Merged
wopox1337 merged 7 commits intorehlds:masterfrom
aleeperezz16:plant-anywhere
Oct 28, 2021
Merged

New CVar: mp_plant_c4_anywhere#692
wopox1337 merged 7 commits intorehlds:masterfrom
aleeperezz16:plant-anywhere

Conversation

@aleeperezz16
Copy link
Contributor

@aleeperezz16 aleeperezz16 commented Oct 24, 2021 •
edited by wopox1337
Loading

resolve #683

Basically lets players to plant c4 anywhere in the map. CTs are also allowed for mods.

This was referenced Oct 24, 2021
Closed
Closed
wopox1337
wopox1337 requested changes Oct 28, 2021
View reviewed changes
wopox1337
wopox1337 requested changes Oct 28, 2021
View reviewed changes
@wopox1337 wopox1337 added Priority: 🕒 low Low priority tasks that can be postponed for the future. Type: 🚀 enhancement Improvement or addition of a new feature. labels Oct 28, 2021
@wopox1337
Copy link
Member

wopox1337 commented Oct 28, 2021

Thank you for your contribution! Could you also implement here API member m_bPlantC4Anywhere (as in a similar PR) for this functionality?

It would be convenient to manage this from ReAPI (or smth similar).

@aleeperezz16
Copy link
Contributor Author

aleeperezz16 commented Oct 28, 2021

Yes, of course

@wopox1337
Copy link
Member

wopox1337 commented Oct 28, 2021 •
edited
Loading

It is worth understanding that this functionality will break the gameplay without adjusting mp_plant_c4_delay. That is, it is dependent. Otherwise, this CVar is not suitable for use by every map.

mp_plant_c4_delay should be implemented in a separate PR

@wopox1337
Copy link
Member

wopox1337 commented Oct 28, 2021

About naming. I guess we better not deviate from the original source of functionality ( CS:GO mp_plant_c4_anywhere). That way it will be easier to find the description in google and find it in ReGameDLL cmds list.

@wopox1337
Copy link
Member

wopox1337 commented Oct 28, 2021

Note for the following PRs. Each new functionality must be added at the end of the *.cfg file. (and in readme.md table of CVar's)

@wopox1337
Copy link
Member

wopox1337 commented Oct 28, 2021

I will test everything later (today or tomorrow) and add it. Thank you for your contributions.

@aleeperezz16
Copy link
Contributor Author

aleeperezz16 commented Oct 28, 2021

Note for the following PRs. Each new functionality must be added at the end of the *.cfg file. (and in readme.md table of CVar's)

Yeah, I know. I added that below mp_buy_anywhere as it has the "anywhere" and it's implementation is on the same method

@aleeperezz16
Copy link
Contributor Author

aleeperezz16 commented Oct 28, 2021

I will change these things you commented

@aleeperezz16 aleeperezz16 changed the title New CVar: mp_plant_anywhere New CVar: mp_plant_c4_anywhere Oct 28, 2021
wopox1337
wopox1337 requested changes Oct 28, 2021
View reviewed changes
@wopox1337 wopox1337 merged commit 112326a into rehlds:master Oct 28, 2021
@aleeperezz16 aleeperezz16 deleted the plant-anywhere branch October 28, 2021 13:02
yohimik pushed a commit to yohimik/ReGameDLL_CS that referenced this pull request Jun 5, 2025
@aleeperezz16 @wopox1337 @yohimik
New CVar: mp_plant_c4_anywhere (rehlds#692)
a59a4e2 
* Added new CVar `mp_plant_anywhere`

* API member for CVar `m_bCanPlantAnywhere`

Co-authored-by: Sergey Shorokhov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wopox1337 wopox1337 wopox1337 approved these changes

Assignees

@aleeperezz16 aleeperezz16

Labels

Priority: 🕒 low Low priority tasks that can be postponed for the future. 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.

Add CVar mp_plant_c4_anywhere

2 participants

@aleeperezz16 @wopox1337