Skip to content

[ALL] Fix functionality of ent_setname command #1221

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

FlaminSarge
Copy link

Description

ent_setname's docs state that it accepts arguments like so:
ent_setname {new name} {old name/class/use aim target if empty}

However, the logic doesn't handle the 2nd parameter properly and tries to search by the first argument instead of the second. It also accepts input even if there are no arguments provided, where it should instead print the help text.

This PR fixes the above issues and restores functionality for the command.

@FlaminSarge
Copy link
Author

I just noticed that #1217 has some of these fixes, but is missing the ArgC handling/missing arguments fix.

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.

1 participant