Skip to content

v0.5.3 Release Candidate#163

Merged
ultranoobian merged 26 commits intoDUWS-R-Team:masterfrom
ultranoobian:master
Dec 9, 2017
Merged

v0.5.3 Release Candidate#163
ultranoobian merged 26 commits intoDUWS-R-Team:masterfrom
ultranoobian:master

Conversation

@ultranoobian
Copy link
Copy Markdown
Contributor

@ultranoobian ultranoobian commented Dec 9, 2017

This change is Reviewable

ultranoobian and others added 26 commits July 12, 2017 14:00
When player rating goes below -2000, the player is deemed 'Traitor' and is
treated as an enemy by ALL factions and KOS.

This EH checks for negative rating (currently below zero) and resets back
to zero.
If the future rating value is negative, the rating change is set to zero
else continues normally.

Referencing: DUWS-R-Team#151
findEmptyPosition is less complicated and sensitive.

DUWS-R-Team#156
Missing brackets caused rating expression to be interpreted unexpectedly
Refactored vehicle spawning code into seperate file to reduce
duplication, enhance readability, and reduce error.

Order of values to be passed to the new 'spawnVehicle.sqf' are [CP Cost,
Vehicle Type]
# Conflicts:
#	source/dialog/request_vehicle.sqf
Previously would always decrease CP pool by four CP due to Copy-paste
error. Now it actually does reduce CP pool by passed parameter.
Reduced minimum spawning distance for vehicles from 50 to 15 metres

Added code guard in case it didn't find a valid spot for the vehicle,
instead it will abort and give the player a helpful message tell them
why the vehicle hasn't spawned
Moved mission parameter outside of the MP-Only code block, and defaulted
to Stamina System to be OFF until BI introduces an option to have
Mission parameters for SP missions
Changed "" into valid equiv. enum "NONE"
@ultranoobian
Copy link
Copy Markdown
Contributor Author

Reviewed 1 of 5 files at r4, 3 of 6 files at r5, 1 of 2 files at r6, 3 of 3 files at r7, 1 of 1 files at r8.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable

@ultranoobian ultranoobian merged commit 78e46b6 into DUWS-R-Team:master Dec 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants