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

Bug: /give 10 medi gives 20 small medipacks and 20 large medipacks #1463

Closed
rr- opened this issue Aug 30, 2024 · 1 comment · Fixed by #1464
Closed

Bug: /give 10 medi gives 20 small medipacks and 20 large medipacks #1463

rr- opened this issue Aug 30, 2024 · 1 comment · Fixed by #1464
Assignees
Labels
TR1 TRX bug A bug with TRX
Milestone

Comments

@rr-
Copy link
Collaborator

rr- commented Aug 30, 2024

While it's OK to give both small and large medipacks since the command is ambiguous, it's not OK to hand them out twice like so:

363041700-a569bf1e-7433-4918-ae80-cc20d926b56e

The bug occurs because Object_IdsFromName does not ensure unique matches, e.g. if the input matches two aliases for the same object, the object appears twice in the returned value.

@aredfan can you please try and find out what was the version that introduced this bug?

@rr- rr- added TRX bug A bug with TRX Priority: medium labels Aug 30, 2024
@rr- rr- added this to the 4.4 milestone Aug 30, 2024
@rr- rr- self-assigned this Aug 30, 2024
@rr-
Copy link
Collaborator Author

rr- commented Aug 30, 2024

Never mind – it seems to be a regression from 3.0, that implemented this command for the first time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TR1 TRX bug A bug with TRX
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant