[CONTEDIT-242] Investigate: Cannot add magnoliaLink to stories text block Created: 27/May/20  Updated: 09/Feb/21  Resolved: 18/Dec/20

Status: Closed
Project: Content Editor
Component/s: None
Affects Version/s: 1.3.2
Fix Version/s: 2.0

Type: Bug Priority: Neutral
Reporter: Dai Ha Assignee: Jorge Franco
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: 0d
Time Spent: 3.5h
Original Estimate: Not Specified

Attachments: PNG File Screen Shot 2020-06-02 at 9.12.24 AM.png     PNG File image-2020-05-27-14-16-28-108.png    
Issue Links:
Relates
relates to CONTEDIT-237 Migration of LinkChooser & Dialog sys... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)
Bug DoR:
[ ]* Steps to reproduce, expected, and actual results filled
[ ]* Affected version filled
Date of First Response:
Epic Link: Stories App Migration
Sprint: CM & OC 12, CM & OC 13, CM & OC 17, CM & OC 18
Story Points: 3

 Description   

Reproduce steps:

 * Open Stories app

 * Create story

 * Add a magnoliaLink

--> After closing chooser dialog, nothing appear in the field



 Comments   
Comment by Dai Ha [ 27/May/20 ]

First observation during a quick investigation, in `info.magnolia.editor.gwt.client.body.TextBlockWidget#initializeInlineEditor`, the getParent return unidentified (null?) object and the call stack broken there.

Comment by Dai Ha [ 02/Jun/20 ]

Observe js exception after fix the getParent above:

Uncaught TypeError: Cannot read property 'checkReadOnly' of undefined at b.insertElementIntoRange (VM36915 ckeditor.js:379) at b.insertElement (VM36915 ckeditor.js:377) at b.<anonymous> (VM36915 ckeditor.js:387) at a.p (VM36915 ckeditor.js:10) at a.<anonymous> (VM36915 ckeditor.js:12) at a.CKEDITOR.editor.CKEDITOR.editor.fire (VM36915 ckeditor.js:13) at a.insertElement (VM36915 ckeditor.js:287) at a.<anonymous> (VM36974 plugin.js:75) at a.p (VM36915 ckeditor.js:10) at a.<anonymous> (VM36915 ckeditor.js:12)

Also, there are multiple ckeditor version:

Pages app 4.6.2 from CDN:

magnolia-ui-vaadin-common-widgets/src/main/resources/VAADIN/js/ckeditor/ckeditor.js --> 4.14.0

http://localhost:8180/magnoliaAuthor/VAADIN/widgetsets/info.magnolia.widgetset.MagnoliaProWidgetSet/ckeditor/ckeditor.js:formatted --> 4.11.3

Generated at Mon Feb 12 00:18:28 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.