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

Popup "File '/usr/NPL/lokal/temp/001/25_HelloWorld.xlsx' could not be opened" of ZDEMO_EXCEL and ZDEMO_EXCEL25 #823

Closed
sandraros opened this issue Oct 3, 2021 · 3 comments · Fixed by #850 or #909

Comments

@sandraros
Copy link
Collaborator

sandraros commented Oct 3, 2021

Run ZDEMO_EXCEL25 or ZDEMO_EXCEL. This popup is shown:
image

Information The file '/usr/NPL/lokal/temp/001/25_HelloWorld.xlsx' is not open

What does it mean?

EDIT: or more recently after PR:
image

@AndreaBorgia-Abo
Copy link
Member

With ZDEMO_EXCEL25 I get no such warning, odd. Anyway, in my demo environment the FILE entry for LOCAL_TEMPORARY_FILES points to /tmp and that's where the file lands, using either SAPGUI Java 7.70rev3 or Windows 7.70pl3fix2

With ZDEMO_EXCEL with the Java GUI the program defaults to /tmp/andrea/SAPGUI/NPL.2fd637a/work as directory and I get the ludicrous message that's clearly out of place
image
then it goes on to dump in ZEXCEL_DEMO28 with RAISE UNKNOWN_DP_ERROR, not unexpected, there is no feature parity between the different GUI versions. If I run it from the Windows version, it just works, the whole set of demos I mean.

@sandraros
Copy link
Collaborator Author

sandraros commented Oct 11, 2021

@AndreaBorgia-Abo @bigld

  • In my 7.52 SP01 Developer Edition, the logical path LOCAL_TEMPORARY_FILES points to /usr/<SYSID>/lokal/temp/<MANDT>/<FILENAME>, that's why... And zdemo_excel calls zdemo_excel25 (NB: I don't understand what is the benefit; I think the goal of zdemo_excel is to generate all Excel files to demonstrate all abap2xlsx features, no good reason to create an Excel file "Hello world", as it's already done by zdemo_excel1)
  • I find it a little bit harsh to not have this logical path as a modifiable parameter.
  • Anyway, what is the interest of this demo program, to show how Logical File/Path works? I don't think it's the goal of a demo program to demonstrate something unrelated to abap2xlsx.
  • Maybe we could give another meaning to this little demo program to demonstrate how easy it is to create an abap2xlsx file to the frontend or to the application server, because maybe people are not experienced enough to debug how zdemo_excel_outputopt_incl works? (and of course, without logical path, no interest)
  • Moreover, we could adapt zdemo_excel to not call zdemo_excel25 anymore?
    What do you think?

@sandraros
Copy link
Collaborator Author

sandraros commented Oct 15, 2021

I think we should reopen the issue. The problem is still here for me. See previous comments.

NB: after the PR, I now have this popup:
image

Information 

File '/usr/NPL/lokal/temp/001/25_HelloWorld.xlsx' 
could not be opened; operating system error: 0 
(Cannot create or open file - Check Tx FILE 
Logical Path 'LOCAL _ TEMPORARY_FILES' )

@sandraros sandraros reopened this Oct 15, 2021
@sandraros sandraros changed the title What is this popup "The file '/usr/NPL/lokal/temp/001/25_HelloWorld.xlsx' is not open" of ZDEMO_EXCEL and ZDEMO_EXCEL25 Popup "File '/usr/NPL/lokal/temp/001/25_HelloWorld.xlsx' could not be opened" of ZDEMO_EXCEL and ZDEMO_EXCEL25 Nov 3, 2021
@sandraros sandraros self-assigned this Dec 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants