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: in API GET member categories #29424

Open
JonBendtsen opened this issue Apr 20, 2024 · 1 comment · May be fixed by #29441
Open

BUG: in API GET member categories #29424

JonBendtsen opened this issue Apr 20, 2024 · 1 comment · May be fixed by #29441
Labels
Bug This is a bug (something does not work as expected)

Comments

@JonBendtsen
Copy link
Contributor

JonBendtsen commented Apr 20, 2024

Bug

If you in dolibarr version 20-alpha tries to use the API to get member categories it does not respect when you supply a non existing member id, where as in version 18.0.5 it does respect and gives an error.

Dolibarr Version

20.0.0-alpha

Environment PHP

8.2.7

Environment Database

10.11.7-MariaDB-1:10.11.7+maria~ubu2204

Steps to reproduce the behavior and expected behavior

create some member categories and assign some members to them. use the API to get the member categories for a non existing member

expected is an error better than the one in 18.0.5, because in 18.0.5 it says that the category does not exist, but it is the member that does not exist.

Attached files

v20 ignores non existing members

get_member_categories_v20_ignores_nonexisting_member

@JonBendtsen JonBendtsen added the Bug This is a bug (something does not work as expected) label Apr 20, 2024
@JonBendtsen
Copy link
Contributor Author

JonBendtsen commented Apr 20, 2024

v 18.0.5 complains if member does not exist, but error message could be better than categori does not exists, it should be Member does not exist

get_member_categories_v18 0 5_errors_on_nonexisting_member

@GregoryBlemand GregoryBlemand linked a pull request Apr 23, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug This is a bug (something does not work as expected)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant