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

Highcharts local directory is hardcoded in install-all.sh script #1128

Open
brunobuzzi opened this issue Apr 9, 2020 · 10 comments
Open

Highcharts local directory is hardcoded in install-all.sh script #1128

brunobuzzi opened this issue Apr 9, 2020 · 10 comments
Assignees

Comments

@brunobuzzi
Copy link
Owner

brunobuzzi commented Apr 9, 2020

From Jupiter:
I’ve just done a fresh install of BpmFlow into GS 3.5.1 using the load-all.sh script.

I noticed that the Javascript files from the Highcharts and Highstock file libraries are not loading:

/files/highcharts-6.0.3/js/highcharts-more.js not found
/files/highstock-6.0.3/js/highstock.src.js not found
/files/highstock-6.0.3/modules/exporting.src.js not found
/files/highstock-6.0.3/modules/heatmap.src.js not found

However, .css files from the same file libraries are loading correctly.

Have you seen this? I’ve tried the process a few times now and see the same result.

@brunobuzzi
Copy link
Owner Author

Check line 89 to 96:

GsDeployer deploy: [
    Metacello new
         baseline: 'HighchartsSt';
         filetreeDirectory: '/home/gemstone/GsDevKit_home/shared/repos/HighchartsSt/repository';
         onLock: [:ex | ex honor];
		     onConflictUseLoaded;
         load.
].

The local Highchart directory is hardcoded to my Linux user (gemstone):
filetreeDirectory: '/home/gemstone/GsDevKit_home/shared/repos/HighchartsSt/repository';

@JupiterJones
Copy link

I updated the path in install-all.sh before installing. Highcharts has installed correctly and the file library is present in the image. It's only the javascript files that are not loading. The css files from the libraries are loading correctly.

I've had a quick look and was wondering if it's related to the WADeploymentAwareFileHandler - which is not installed as the 'files' handler.

I'll get a chance over the weekend to look further.

brunobuzzi added a commit that referenced this issue Apr 9, 2020
@brunobuzzi
Copy link
Owner Author

brunobuzzi commented Apr 9, 2020

  • 1) The linux user name in install-all.sh script should be dynamic and not hardcoded

@JupiterJones
Copy link

Perhaps it should simply use $GS_HOME and have a check to ensure $GS_HOME is set.

@brunobuzzi
Copy link
Owner Author

WADeploymentAwareFileHandler is from Boardwalk package and it is a dependency of Highcharts but i do not think that is the problem.

WADeploymentAwareFileHandler is not used in my GS images.

  • Does other JS files are copied correctly ?
  • Does Highstock6DevelopmentMetadataLibrary class has the JS method missing in /files/.... ?
    For example:
    Highstock6DevelopmentMetadataLibrary>>highstocksrcJs
    Highstock6DevelopmentMetadataLibrary>>highchartsmoresrcJs

@brunobuzzi
Copy link
Owner Author

I did a fresh installation and all those files are present.

@JupiterJones
Copy link

Yes, all the javascript files are present in the library, just not being served - not found errors.
Everything else from the library is being served, just not the JS files.

I'll get a chance to look again tomorrow

@brunobuzzi
Copy link
Owner Author

brunobuzzi commented Apr 9, 2020

Did you executed "sh register-application.sh devKit_351" ? See https://bpmflow.gitbook.io/project/installation/bpm-flow

brunobuzzi added a commit that referenced this issue Nov 10, 2020
brunobuzzi added a commit that referenced this issue Nov 10, 2020
brunobuzzi added a commit that referenced this issue Nov 10, 2020
@brunobuzzi
Copy link
Owner Author

This has been fixed. It needs a couple of more installation test then can be closed !

@brunobuzzi
Copy link
Owner Author

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

No branches or pull requests

2 participants