-
-
Notifications
You must be signed in to change notification settings - Fork 362
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
Police patrol improvement #5809
base: master
Are you sure you want to change the base?
Conversation
If we aim to do one last "bug fix release" then this would be merged after that. So far, there's just been one bug fix merged since the 3.14 release. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for tackling this issue! I have a few concerns / changes regarding the comms messages presented to the player, but overall I think this is a very useful change.
e943552
to
9d01b7c
Compare
if ship:DistanceTo(patrol[i]) <= 4000000 then | ||
Comms.ImportantMessage(string.interp(l_ui_core.X_CANNOT_BE_TOLERATED_HERE, { crime = l_ui_core.PIRACY }, patrol[i].label)) | ||
if ship:DistanceTo(patrol[i]) <= lawEnforcedRange then | ||
Comms.ImportantMessage(string.interp(l_ui_core.X_CANNOT_BE_TOLERATED_HERE, { crime = l_ui_core.UNLAWFUL_WEAPONS_DISCHARGE }, patrol[i].label)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note for future reference that string.interp(l.SOME_STRING, { crime = ... })
is also available via a custom operator as l.SOME_STRING % { crime = ... }
. You don't need to amend this PR to prefer the use of either one however.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The updated code looks good to me! Provisionally approved pending playtesting from myself or another contributor.
Not entirely perfect. But an improvement.
Fixes #5603
It does not solve the police apathy problem described in #5202