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

bug: [CKEDITOR] Error code: invalid-lts-license-key. #7097

Closed
Vijay-Nirmal opened this issue Dec 10, 2023 · 7 comments · Fixed by #7101
Closed

bug: [CKEDITOR] Error code: invalid-lts-license-key. #7097

Vijay-Nirmal opened this issue Dec 10, 2023 · 7 comments · Fixed by #7101
Labels
Bug For Prior Version For a prior OpenEMR version javascript Pull requests that update Javascript code
Milestone

Comments

@Vijay-Nirmal
Copy link

Describe the bug

CKEditor is not loading, its just shows an empty test area without any button and other functionality in the page is also not working like "Question Mark", "New Paragraph" buttons.

image

When I checked the console, I could see "[CKEDITOR] Error code: invalid-lts-license-key." error.

image

To Reproduce

I was also able to reproduce it in a demo instance provided by OpenEMR itself

Steps to reproduce the behavior:

  1. Open/click a field where the data type is "NationNotes"
  2. In the dialog/popup, We can only see a plan text area without CKEditor getting loaded
  3. Check the browser console to see the error message

Expected behavior

CKEditor loading properly and showing the format buttons in UI

Client configuration

Note: I was also able to reproduce it in a demo instance provided by OpenEMR itself

  • Browser: Microsoft Edge (Latest version) and Google chrome
  • OpenEMR version: 7.0.2
  • Operating system: Windows 11 23H2
@sjpadgett
Copy link
Sponsor Member

Unfortunately the last stable release of package is v4.22.1 which was EOL version. The normal updating would follow that v4.23.0 would error on install however the editor project administrators know that the free version update to v5.0 is a major effort so decided to capitalize and charge to support ongoing security updates.

For those of you that are capable of running in the command line you may do a simple fix:

  • Edit the openemr/package.json and change the "ckeditor4": "4.23.0", L-79 to "ckeditor4": "4.22.1",
  • from command line and inside openemr root run npm install
  • Delete your browsers cache.
  • Log in and enjoy:)

Putting out an update in a patch is doable and unsure when.

@Vijay-Nirmal
Copy link
Author

@sjpadgett Thanks, it fixed it, I will leave this issue open till the fix rolls out. But feel free to close the issue if you feel otherwise. Again, thanks a lot.

@sjpadgett sjpadgett added Bug For Prior Version For a prior OpenEMR version javascript Pull requests that update Javascript code labels Dec 10, 2023
@sjpadgett
Copy link
Sponsor Member

@Vijay-Nirmal You're welcome. Thanks for the issue report and we'll leave open for patch reference.

@stephenwaite
Copy link
Sponsor Member

stephenwaite commented Dec 11, 2023

nice @sjpadgett!

here's the zipped tarball we'll need when creating the patch
ckeditor4.22.1.tar.gz

@stephenwaite
Copy link
Sponsor Member

keeping open so we don't forget in patch as Jerry recommended

@bradymiller
Copy link
Sponsor Member

Good place to place "reminder" on needing to join the zip for this with the patch when we build it would be at:
https://www.open-emr.org/wiki/index.php/Ongoing_7.0.2_Patch_Trail

@bradymiller
Copy link
Sponsor Member

btw, a nice way for getting this to work in rel-702 branch (and next patch) without needing to muck with the actual patch would be placing the ckeditor 4.22.1 package in the public/assets/modified/ folder and then redirecting the ckeditor token to it in the /config/config.yaml

@adunsulag adunsulag added this to the 7.0.2.1 milestone Apr 23, 2024
@adunsulag adunsulag changed the title [CKEDITOR] Error code: invalid-lts-license-key. bug: [CKEDITOR] Error code: invalid-lts-license-key. May 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For Prior Version For a prior OpenEMR version javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants