-
Notifications
You must be signed in to change notification settings - Fork 980
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
Invalid type given as required for multiple methods #1425
Comments
enum
given as required for multiple methods
This has been a issue for almost 2 years, but nobody is responding. |
Have you figured out what is used to generate the code? |
I think had found it at some point, but I don't know right now. |
this contains the comment and "type" in all of those json files, so maybe changing here to like list|string instead of list may work. I don't know where it's generated, or how to change it to optional |
elasticsearch-php/src/Traits/ClientEndpointsTrait.php
Line 39 in c9a5c85
enum
is not a valid PHP type, the correct type for a generic enum is\UnitEnum
, you probably just wantstring
.number
is not a valid type, the correct one isnumeric
if you wantint|float|numeric-string
.list
is used for comma-separated lists, this should bestring
, but comment might be misleading as it appears it might accept both, in which casestring|list
would be better.long
is not a valid type, useint
time
is not a valid type, use\DateTime
,int
orstring
The text was updated successfully, but these errors were encountered: