-
Notifications
You must be signed in to change notification settings - Fork 15
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
/ui2/cl_json
: add inherit functionality
#831
Conversation
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
/ui2/cl_json
: add inherit functionality
inheritance should work in open-abap, so all good |
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
Co-authored-by: abaplint[bot] <24845621+abaplint[bot]@users.noreply.github.com>
@oblomov-dev if you dont have vscode installed, you can try https://github.com/github/dev, it might be easier |
yes github codespaces (i think it is the same) is my second home after eclipse :) i will check it! |
ok, fixed all the linter issues now, all tests are green. i skipped implementing the call of the "is_compressable" method, means from my understanding: you can inherit /ui2/cl_json now but when you create a unit test based on a change implemented in is_compressable, open-abap will give a different result than the normal abap stack but maybe for the first step, this is enough. |
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.
see comments
Co-authored-by: Lars Hvam <[email protected]>
Co-authored-by: Lars Hvam <[email protected]>
Co-authored-by: Lars Hvam <[email protected]>
see this review and comment function for the first time, nice! code is adjusted. |
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.
thanks 👍
if the compress inheritance and stuff doesnt work, suggest as next step adding a unit test for it in https://github.com/oblomov-dev/open-abap-core/blob/main/src/json/%23ui2%23cl_json.clas.testclasses.abap |
@larshp |
ah ok interesting, i will try it out :) |
open-abap is not working when a class inherits /ui2/json:
https://github.com/abap2UI5/abap2UI5/actions/runs/7583976799/job/20656717865?pr=793
added:
next step / issue:
I have no idea if this inheriting concept is easily implementable in open-abap? before i start making changes here without knowing how this all specifically works, maybe its better you comment on this first.