Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#8995 closed Bug (fixed)

Link gets destroyed in bbcode when you modify it

Reported by: j.swiderski Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.6.5
Component: General Version: 3.6
Keywords: Cc:


To reproduce:

  1. Open bbcode sample and clear all contents
  2. Insert link using link dialog
  3. Now realize that parameter abc should have value txt not xt :). Try to correct it.
  4. Switch to source

Result: [url=][/url]

Ampersand is escaped and URL got doubled.
Each time you switch to source and back new amp; is added:


Reproducible in all browsers from CKEditor 3.6.

Attachments (1)

8995.patch (489 bytes) - added by garry.yao 4 years ago.

Download all attachments as: .zip

Change History (8)

comment:1 Changed 4 years ago by j.swiderski

  • Status changed from new to confirmed

comment:2 Changed 4 years ago by j.swiderski

Here is a screencast showing that the same thing happens when you modify initial content:

It was reported in #8751 but I'm leaving this one open as it is addressing the problem more accurately.

Last edited 4 years ago by j.swiderski (previous) (diff)

Changed 4 years ago by garry.yao

comment:3 Changed 4 years ago by garry.yao

  • Milestone set to CKEditor 3.6.5
  • Owner set to garry.yao
  • Status changed from confirmed to review

comment:4 Changed 4 years ago by garry.yao

While the patch doest not prevent from escaping HTML (which is wrong) on output, it avoid HTML-escaping BBCode from source, which will break a second load for the above case.

comment:5 Changed 4 years ago by fredck

  • Status changed from review to review_passed

comment:6 Changed 4 years ago by garry.yao

  • Resolution set to fixed
  • Status changed from review_passed to closed

Fixed with [7598].

comment:7 Changed 4 years ago by garry.yao

[7598] caused a regression identified by test:


Re-fixing with [7628].

Note: See TracTickets for help on using tickets.
© 2003 – 2016 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy