Uploaded image for project: '{RETIRED} CKEditor Integration'
  1. {RETIRED} CKEditor Integration
  2. CKEDITOR-294

The office server can slow down or block the editor loading

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Major
    • 1.34
    • 1.17
    • None
    • None
    • High
    • Trivial

    Description

      I haven't reproduced this problem myself but I had users reporting this behavior and by looking at the code it seems plausible because when CKEdior is loaded we check if the office import feature is available. We do this by testing two conditions:

      • if the office server is connected
      • if we can import an empty file
      #set ($emptyAttachmentReference = $services.model.resolveAttachment('CKEditor.OfficeImporter@empty.txt'))
      #set ($officeImporterSupported = $services.officemanager.serverState == 'Connected' && $services.wysiwyg.importOfficeAttachment($emptyAttachmentReference, {}).equals(''))
      

      The first condition is mandatory and it should be evaluated instantly. The second condition is optional, and it can slow down or block the loading if the office server is busy (even if the file is empty the conversion goes through the office server which is the whole point of having this condition). The office server can be busy is many users are doing office imports or exports at the same time.

      I think we can remove the second condition since it doesn't bring much. The user will get an error message on the Office import dialog if the server can't do the import which I guess is better than not showing the office import button at all and better than getting the spinning wheel while the editor is loading.

      Attachments

        Issue Links

          Activity

            People

              mflorea Marius Dumitru Florea
              mflorea Marius Dumitru Florea
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: