SAP / openui5

OpenUI5 lets you build enterprise-ready web applications, responsive to all devices, running on almost any browser of your choice.
http://openui5.org
Apache License 2.0
2.95k stars 1.23k forks source link

Richtexteditor: Unknow TinyMCE status #3731

Closed zaunaro closed 1 year ago

zaunaro commented 1 year ago

OpenUI5 version: 1.108.7

Browser/version (+device/version): Chrome 111.0.5563.147

Any other tested browsers/devices(OK/FAIL): FAIL

URL (minimal example if possible):

User/password (if required and possible - do not post any confidential information here):

Steps to reproduce the problem: I want to call the rich text editor ready function (with TinyMCE5) to set the value when the richtexteditor is loaded. I upgraded tinymce4 to tinymce 5.

<r:RichTextEditor editorType="TinyMCE5" id="rteDescriptionEvent" ready=".afterRichTextEditorLoaded" ...>

afterRichTextEditorEventLoaded(oEvent) {
.....
          this.byId("rteDescriptionEvent").setValue(sDescription);
}

What is the expected result? No error

What happens instead?

But what i got is a console printing saining: 2023-04-05 10:43:39.468199 Unknown TinyMCE status: Loaded -

Any other information? (attach screenshot if possible) I think I found the part in the code: loaded

In the Switch case at setvaluetinymce - the loaded status is missing.

In other methods of the richtexteditor this is already done: image

yanaminkova commented 1 year ago

Hello @zaunaro,

Could you please provide a test system, where we could test the described issue or an isolated sample in order to debug it?

Thanks in advance!

Kind Regards, Yana

zaunaro commented 1 year ago

I just debugged it and send you the screenshots - I think I already found the problem. In the switch case you miss the ErrorStatus.Loaded. At line 1564 you must add it.

yanaminkova commented 1 year ago

Hello @zaunaro,

Thank you for sharing!

The RichTextEditor is part of SAPUI5 not of OPENUI5. GitHub is a place for managing issues for OPENUI5 only.

Please, open a ticket in SAP ticket system for managing incidents.

Kind Regards, Yana