-
Notifications
You must be signed in to change notification settings - Fork 183
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
Deprecate all Redmine\Api\*::getIdByName()
methods
#417
Comments
Art4
changed the title
Deprecate all
Deprecate all Jul 5, 2024
getIdByName()
methodsRedmine\Api\*::getIdByName()
methods
8 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In #338 we have deprecated all
listing()
methods. These methods are the base for thegetIdByName()
methods.Getting the ID by a name could easily made by using the new
listNames()
methods andarray_search()
.However, some endpoints allow name duplications (e.g.
projects
andversions
), so it might become difficult to decide witch ID should be chosen on a name collision.Instead of editing this methods (to use the
listNames()
methods internally) and made prediction about name collisions I propose to deprecated them and let the user handle this situation.Affected methods
CustomField::getIdByName()
IssueCategory::getIdByName()
IssueStatus::getIdByName()
Project::getIdByName()
TimeEntryActivity::getIdByName()
Tracker::getIdByName()
User::getIdByUsername()
Version::getIdByName()
The text was updated successfully, but these errors were encountered: