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

Add Delay to the "Change Weapon Mode" to stop it being used as spread reset. #32

Closed
fsRaideR opened this issue May 28, 2014 · 3 comments
Assignees
Milestone

Comments

@fsRaideR
Copy link
Member

[21:06] @RAIDER okay :)
[21:06] @RAIDER http://www.urban-zone.org/modules/Competition/media/demo_22462_1523847750.urtdemo
[21:06] @RAIDER 2nd Round of Turnpike
[21:07] @RAIDER check this out
[21:07] @RAIDER modegunchange; fire; gunmodechange; fire; gunmodechange
[21:07] @RAIDER it resets the spred
[21:07] @RAIDER and they have a script for it
[21:08] @RAIDER its a game breaking exploit
[21:08] @RAIDER since it sends every bullet you fire to 0,0
[21:08] @RAIDER fire has no place in "Scripting"
[21:09] @RAIDER it should be only bindable to a key
[21:09] @RAIDER that would fixate this issue
[21:09] @bladekiller good point
[21:09] @bladekiller remove fire from scripting

@danielepantaleone
Copy link

Fire is a widely used in scripts. Also I do not remeber correctly the code executing scripts, but at first guess I'd say it's a simple parse and execution (which means that the lines in the cfg files are parsed like they are entered into the console, so like binds). I guess it would be easier not to reset the spread on weapon mode change 😃

@Barbatos
Copy link
Member

I'll probably put a delay on weapon mode change.

@Barbatos Barbatos added 4.2.018 and removed 4.2.019 labels May 28, 2014
@fsRaideR
Copy link
Member Author

UPDATE: Decided to add delay on the weapon mode change, Barbie speaks wisdom!

Change: Ticket Title to reflect the change.

@fsRaideR fsRaideR changed the title Remove ability to have "Fire" in any UrT Scripts Add Delay to the "Change Weapon Mode" to stop it being used as spread reset. May 29, 2014
@Barbatos Barbatos added this to the 4.2.019 milestone May 31, 2014
@Barbatos Barbatos self-assigned this May 31, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants