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

Better player name lookups #1110

Open
talvo opened this issue Jan 4, 2017 · 3 comments
Open

Better player name lookups #1110

talvo opened this issue Jan 4, 2017 · 3 comments
Assignees

Comments

@talvo
Copy link
Member

talvo commented Jan 4, 2017

When listing mail with @mail *<player>, abbreviated names of online players aren't checked (should use match_player not lookup_player)

visible_short_page can fail when there's only one player you can see whose name matches a prefix, but there's also a dark player who matches that you can't see.

@talvo talvo self-assigned this Jan 4, 2017
@talvo
Copy link
Member Author

talvo commented Jan 6, 2017

The * arg to locate() doesn't do player name lookups without a leading *, despite saying it does everything (should include MAT_PMATCH)

@talvo
Copy link
Member Author

talvo commented Jun 11, 2017

(Since locate() has probably been that way for a decade or two, I'd actually be inclined to treat that one as a documentation bug, rather than change it and risk breaking stuff.)

shawnw added a commit that referenced this issue Mar 15, 2018
@shawnw
Copy link
Contributor

shawnw commented Mar 15, 2018

Fixed the help for locate() to match actual behavior. Commit aafdc16

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

No branches or pull requests

2 participants