Skip to content

Conversation

@Rushaway
Copy link
Contributor

Description

Fix #850 Comms : You can Insert invalid STEAMID format in DB

Motivation and Context

Check issue to understand.
Also added the name of the target returning in error. Usefull to know who aren't a valid STEAM ID Format when using multi target filter.

How Has This Been Tested?

Joining on server with Steam ID pending.
Perform mute with multi target
Valid steamid get muted
User without valid steamid format return this error :

sm_mute @ct 1 test
[SourceComms++] .Rushaway has not been verified. Please wait thirty seconds before retrying.
[SourceComms++] Ire has not been verified. Please wait thirty seconds before retrying.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.

@Hackmastr Hackmastr merged commit bf74d00 into sbpp:php81 Jan 15, 2023
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