IBX-8962: Unable to use certain special characters in richtext links

OstafinL · OstafinL · commit bcb6315faf75 · 2025-01-13T11:05:32.000+01:00
diff --git a/src/bundle/Resources/public/js/CKEditor/link/link-ui.js b/src/bundle/Resources/public/js/CKEditor/link/link-ui.js
@@ -59,7 +59,7 @@ class IbexaLinkUI extends Plugin {
 
             this.isNew = false;
 
-            this.editor.execute('insertIbexaLink', { href: url, title, target, ibexaLinkClasses, ibexaLinkAttributes });
+            this.editor.execute('insertIbexaLink', { href: encodeURI(url), title, target, ibexaLinkClasses, ibexaLinkAttributes });
             this.hideForm();
         });
 
diff --git a/src/bundle/Resources/public/js/CKEditor/link/ui/link-form-view.js b/src/bundle/Resources/public/js/CKEditor/link/ui/link-form-view.js
@@ -122,7 +122,7 @@ class IbexaLinkFormView extends View {
     }
 
     setValues({ url, title, target, ibexaLinkClasses, ibexaLinkAttributes = {} }) {
-        this.setStringValue(this.urlInputView, url);
+        this.setStringValue(this.urlInputView, decodeURI(url));
         this.setStringValue(this.titleView, title);
 
         this.targetSwitcherView.fieldView.element.value = !!target;

Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,7 @@ class IbexaLinkFormView extends View {`
`122`	`122`	`}`
`123`	`123`
`124`	`124`	`setValues({ url, title, target, ibexaLinkClasses, ibexaLinkAttributes = {} }) {`
`125`		`- this.setStringValue(this.urlInputView, url);`
	`125`	`+ this.setStringValue(this.urlInputView, decodeURI(url));`
`126`	`126`	`this.setStringValue(this.titleView, title);`
`127`	`127`
`128`	`128`	`this.targetSwitcherView.fieldView.element.value = !!target;`