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

fix: fluid type must be within enum #1678

Merged
merged 3 commits into from
Oct 10, 2023
Merged

fix: fluid type must be within enum #1678

merged 3 commits into from
Oct 10, 2023

Conversation

un000000
Copy link
Contributor

@un000000 un000000 commented Oct 4, 2023

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

Behaviour

Actual

Whenever auction is ended, the item count is set to 255, resulting in fluid type 255.

Expected

Whenever auction is ended, resulting fluid type should be between 0 and the size of fluid type enum.

Fixes #1535

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@luan luan merged commit c8f7233 into opentibiabr:main Oct 10, 2023
20 checks passed
luancolombo added a commit to luancolombo/nemesis that referenced this pull request Oct 13, 2023
luancolombo added a commit to luancolombo/nemesis that referenced this pull request Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Client crash(possible rollback)/game crash(if abused correctly) - Fix on description
3 participants