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

Remove exception printing stacktrace from invalid name in getTown(). #4259

Merged
merged 1 commit into from Aug 12, 2020
Merged

Remove exception printing stacktrace from invalid name in getTown(). #4259

merged 1 commit into from Aug 12, 2020

Conversation

ghost
Copy link

@ghost ghost commented Aug 12, 2020

Description:

This fixes the error spam when do /t {name} where {name} fails the regex check.


Relevant Towny Issue ticket:

Closes #4257


  • I have tested this pull request for defects on a server.

By making this pull request, I represent that I have the right to waive copyright and related rights to my contribution, and agree that all copyright and related rights in my contributions are waived, and I acknowledge that the TownyAdvanced organization has the copyright to use and modify my contribution under the Towny License for perpetuity.

@LlmDl LlmDl merged commit b13884c into TownyAdvanced:master Aug 12, 2020
@LlmDl LlmDl added this to the 0.96.3.0 milestone Aug 12, 2020
LlmDl added a commit that referenced this pull request Aug 12, 2020
    - Improves locking of database for reads and writes in regards to
synchronization.
  - Remove exception printing stacktrace from invalid name in getTown(),
courtesy of Siris with PR #4259.
    - Closes #4257.
  - Hotfix/bankcap bug, courtesy of Siris with PR #4260.
    - Closes #4253.
@ghost ghost deleted the hotfix/getTown-Error branch August 19, 2020 05:24
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.

using /t <invalid town name> returns InvalidNameException warning
2 participants