-
Notifications
You must be signed in to change notification settings - Fork 965
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 6.8.0 release adding missing class alises #1114
Conversation
…for missing class aliases in 6.8
src/autoload.php
Outdated
'\Elasticsearch\Endpoints\Snapshot\Repository\Get' => '\Elasticsearch\Endpoints\Snapshot\GetRepository', | ||
'\Elasticsearch\Endpoints\Snapshot\Repository\Verify' => '\Elasticsearch\Endpoints\Snapshot\VerifyRepository', | ||
'\Elasticsearch\Endpoints\Source\Get' => '\Elasticsearch\Endpoints\GetSource', | ||
'\Elasticsearch\Endpoints\Tasks\TasksList' => '\Elasticsearch\Endpoints\Tasks\ListTasks' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I note the whitespace changes make this a little harder to review (on GitHub in a browser) :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I fixed the indentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@reedy Any chance you could try this on your end if it fixes all the issues?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Our CI stack seems happy with this change :)
* Update Transport.php * Fix 6.8.0 release adding missing class alises (#1114) * Fix for #1112 issue. Adding endpoint aliases + IndicesNamespace::getAliases() * Reverted array alias to class in src/autoload.php + added unit tests for missing class aliases in 6.8 * Fixed indentation in src/ autoload.php * Added TransportTest Co-authored-by: Enrico Zimuel <e.zimuel@gmail.com>
This PR fixes #1112 adding the following missing class aliases for 6.7 BC:
Elasticsearch\Endpoints\Indices\Aliases\Update
Elasticsearch\Endpoints\Indices\Exists\Types
Elasticsearch\Endpoints\Indices\Field\Get
Moreover, it adds a missing function
getAliases()
(proxy ofgetAlias()
) inElasticsearch\Namespaces\IndicesNamespace
.