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

Fix Entity dropdown using "multiple" option #12717

Merged
merged 3 commits into from
Sep 26, 2022

Conversation

Aeto-J
Copy link
Contributor

@Aeto-J Aeto-J commented Sep 21, 2022

Added parenthesis to fix error which occurs when creating an Entity multiple choice dropdown: "Uncaught Exception TypeError: array_combine(): Argument 2 ($values) must be of type array, null given in glpi10.0.3/src/Html.php at line 4857"

Added Dropdown::testShow to unit test multiple choice dropdown creation on Entity's itemtype

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets

…ity multiple choice dropdown: "Uncaught Exception TypeError: array_combine(): Argument glpi-project#2 ($values) must be of type array, null given in glpi10.0.3/src/Html.php at line 4857"

Added Dropdown::testShow to unit test multiple choice dropdown creation on Entity's itemtype
@cedric-anne cedric-anne changed the title Added parenthesis to fix error which occurs when creating an Entity multiple choice dropdown Fix Entity dropdown using "multiple" option Sep 21, 2022
src/Dropdown.php Outdated Show resolved Hide resolved
@cedric-anne cedric-anne added this to the 10.0.4 milestone Sep 26, 2022
@trasher trasher merged commit 8de64a8 into glpi-project:10.0/bugfixes Sep 26, 2022
@Aeto-J
Copy link
Contributor Author

Aeto-J commented Sep 26, 2022

Hello,
Thank you for the fix
best regards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants