New features and fixes#614
New features and fixes#614RickyB505 wants to merge 6 commits intoTomGrobbe:supplemental-permissions-pilotfrom
Conversation
RickyB505
force-pushed
the
supplemental-permissions-pilot
branch
from
95003db to
eb04052
Compare
|
|
||
| #region Main Spawn Vehicle Function | ||
| public static int lastSpawnTime = 0; | ||
| public static int spawnTime = ConfigManager.GetSettingsInt(ConfigManager.Setting.vmenu_vehicle_spawn_delay, 5) * 1000; // make convar |
There was a problem hiding this comment.
nope just left in while writing it ill remove it
|
|
||
| if (!vehicle.Model.IsTrain) // to be extra fucking safe | ||
| { | ||
| // workaround of retarded feature above: |
There was a problem hiding this comment.
Moving this block means this comment makes no sense.
Because it was relating to line 1408:
//vehicle.Speed = speed; // retarded feature that randomly breaks for no fucking reason
So you can probably just cleanup both the commented out line and the comment here.
There was a problem hiding this comment.
And why did you move it up here btw? Now it gets it's speed before the vehicle mods, and the RPM is set (delayed) after the velocity.
There was a problem hiding this comment.
this was moved so if your not spawning in the vehicle it doesn't keep the momentum
| } | ||
| }), false); | ||
|
|
||
| RegisterCommand("vMenu:DV", new Action<dynamic, List<dynamic>, string>(async (source, args, rawCommand) => |
There was a problem hiding this comment.
Can't this command just trigger existing functionality to delete the vehicle? I'm assuming the option for the permission VODelete already has all the necessary checks implemented?
There was a problem hiding this comment.
it uses the default delete vehicle functionality partly, but it also has a raycast element to it so you dont need to be inside it to delete it
2 participants
No description provided.