method Elasticsearch\Namespaces\IndicesNamespace::getAliases does not exists in v7.2.0

[max-kut]

Summary of problem or feature request

in version 7.1.* method getAliases($params) exists, but removed in v7.2. In docpage exists https://github.com/elastic/elasticsearch-php/blob/v7.2.0/docs/build/Elasticsearch/Namespaces/IndicesNamespace.asciidoc#Elasticsearch_Namespaces_IndicesNamespacegetAliases_getAliases

Code snippet of problem

$elasticsearch->indices()->getAliases($params->toArray());
// Call to undefined method Elasticsearch\Namespaces\IndicesNamespace::getAliases()

-->

System details

• Operating System Win10
• PHP Version 7.2, 7.3
• ES-PHP client version 7.2.0
• Elasticsearch version 7.3.0

[ezimuel]

@max-kut the IndicesNamespace::getAliases() has been changed in favor of IndicesNamespace::getAlias(), this because in the API specification is indices.get_alias. I generate the new endpoints using a script from the API specification (see here) and I didn't notice the change. I'll add this BC break in the release note of 7.2.0. Sorry for that.

[ezimuel]

@max-kut I decided to release 7.2.1 adding a proxy function getAliases() to getAlias() to avoid BC breaks. This proxy function will be removed with 8.0. I'll release soon.

[lucasvanlierop]

#943 (comment)

[ezimuel]

I reopened the issue because with 7.2.1 I fixed the issue partially. I forgot to use the $params as optional parameter in getAliases() function, ops! See #946.

[ezimuel]

@lucasvanlierop I just sent #947 to fix the issue. When merged, I will release 7.2.2.

[ezimuel]

@lucasvanlierop I just released 7.2.2, thanks for the report.

[max-kut]

@ezimuel, @lucasvanlierop thanks for the reaction

[lucasvanlierop]

Thanks @ezimuel !