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

Array returned from Java object not converted to JavaScript array #5501

Closed
aro opened this issue Aug 18, 2017 · 5 comments
Closed

Array returned from Java object not converted to JavaScript array #5501

aro opened this issue Aug 18, 2017 · 5 comments
Assignees
Projects
Milestone

Comments

@aro
Copy link
Contributor

aro commented Aug 18, 2017

When a JavaScript function calls a Java method that returns e.g. an array of String, it is not correctly converted to a JavaScript array of strings.
Research if we can improve the handling for arrays, in toNativeObject function.

@aro aro self-assigned this Aug 18, 2017
@alansemenov
Copy link
Member

@alansemenov
Copy link
Member

@aro @srs Isn't this the same as #5257?

@aro
Copy link
Contributor Author

aro commented Sep 4, 2017

It's not the same issue. This one is related to general conversion of objects from Java to JS.
For #5257 we probably need a custom serializer (mapper) for Event objects in java.

@alansemenov
Copy link
Member

Do we want this in 6.12? How complicated/time-consuming is this?

@aro
Copy link
Contributor Author

aro commented Sep 4, 2017

It's a minor improvement for people creating libs. There are workarounds, by using List instead of array, which is more common in Java anyway.
It's not a big change but I would prefer to make it after 6.12 so we get time to test it and make sure there is no unexpected behavior with the conversion.

@rymsha rymsha added the Bug label Oct 7, 2020
@rymsha rymsha added this to Needs triage in Bugs via automation Oct 7, 2020
@rymsha rymsha moved this from Needs triage to Low priority in Bugs Oct 7, 2020
@rymsha rymsha added this to To do in 7.7.0 via automation Jan 20, 2021
@rymsha rymsha moved this from Low priority to Closed in Bugs Jan 20, 2021
@rymsha rymsha added this to the 7.7.0 milestone Jan 20, 2021
@rymsha rymsha added Improvement and removed Bug labels Jan 20, 2021
@rymsha rymsha removed this from Closed in Bugs Jan 20, 2021
@rymsha rymsha moved this from To do to Done in 7.7.0 Jan 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
7.7.0
Done
Development

No branches or pull requests

4 participants