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

Syntax Error on Hana 2023 #85

Open
jani80k opened this issue Mar 4, 2024 · 11 comments
Open

Syntax Error on Hana 2023 #85

jani80k opened this issue Mar 4, 2024 · 11 comments

Comments

@jani80k
Copy link

jani80k commented Mar 4, 2024

SAP_BASIS 758 0001
SAP_ABA 75I 0000
S4CORE 108 0000 - S/4HANA Core Applications 1

grafik

@jani80k
Copy link
Author

jani80k commented Mar 4, 2024

I am in the course of doing a release upgrade from Hana 2020 to Hana 2023.
On the Hana 2020 system, this code is not executed, when I start my report.
On the Hana 2023 system, this code is passed and throws a shortdump.
I downloaded the latest version of ZCL_FALV from github and cannot compile it, unless I comment the line.
My report works when I comment the line but I am not sure if I can leave it like that. That line must be there for a reason.

@fidley
Copy link
Owner

fidley commented Mar 4, 2024

I think it was added in one of the latest changes to omit one dump, when developer was trying to refresh toolbar which was not yet initialized. I'd say it's should be fine to comment it, as I'm still using the version without it on few systems, but could be also an option to check if there is no replacement for that for version 2023. Unfortunatelly I don't have an access to 2023 yet, to check it.

@jani80k
Copy link
Author

jani80k commented Mar 4, 2024

Looks like that is what SAP themselves are doing.

grafik
grafik
grafik

@jani80k
Copy link
Author

jani80k commented Mar 4, 2024

Unfortunately I have not so much knowledge about ALV grids, which is why I am using your nice and easy to use FALV. If you want I could set up a teams call and let you do some tests on my HANA 2023 System.

@fidley
Copy link
Owner

fidley commented Mar 4, 2024

Could be good idea. But as this variable is deleted now, I would comment there for the moment.
I know it was setup in the set_ready_for_input method before, but they must have change a lot inside :)
image

@jani80k
Copy link
Author

jani80k commented Mar 4, 2024

Ok, if you want, you can close the issue... unless you want to keep it open for some reason.

@fidley
Copy link
Owner

fidley commented Mar 4, 2024

I'll leave it open and I'll come back to you for the short session in next weeks :)

@jani80k
Copy link
Author

jani80k commented Mar 4, 2024

Is there a way to DM you my work email?

@fidley
Copy link
Owner

fidley commented Mar 4, 2024

You can use twitter @ abapblog or LinkedIn https://www.linkedin.com/in/lukaszpegiel/, that would be most secure :)

@Hhpy410
Copy link

Hhpy410 commented Apr 12, 2024

Hi ,@fidley
The same thing happens to me.
What's the solution

@fidley
Copy link
Owner

fidley commented Apr 12, 2024

Comment the line for the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants