Opened 16 years ago

Closed 16 years ago

#1414 closed Bug (fixed)

Entities inside <pre> are converted

Reported by: Alfonso Martínez de Lizarrondo Owned by: Martin Kou
Priority: Normal Milestone: FCKeditor 2.5
Component: General Version: FCKeditor 2.5 Beta
Keywords: Cc:

Description

Using your browser of choice and a current nightly (it works fine in 2.4.3), switch to source mode and paste

<pre>&lt;?php echo "test"; ?&gt;</pre>

switch back to design mode and it looks fine. Go back to source mode and the entities aren't converted:

<pre>
<?php echo "test"; ?></pre>

so the code has been destroyed.

if instead of <pre> the content is inside <p> for example then it works fine.

Reported in http://www.fckeditor.net/forums/viewtopic.php?f=6&t=7246

Attachments (1)

1414.patch (1.1 KB) - added by Frederico Caldeira Knabben 16 years ago.
Proposed changes to the "pre" element processor

Download all attachments as: .zip

Change History (5)

comment:1 Changed 16 years ago by Martin Kou

Owner: set to Martin Kou
Status: newassigned

comment:2 Changed 16 years ago by Martin Kou

Resolution: fixed
Status: assignedclosed

Fixed with [1025] and [1026].

Click here for more info about our SVN system.

Changed 16 years ago by Frederico Caldeira Knabben

Attachment: 1414.patch added

Proposed changes to the "pre" element processor

comment:3 Changed 16 years ago by Frederico Caldeira Knabben

Priority: HighNormal
Resolution: fixed
Status: closedreopened

Martin, it seams that a simpler solution can be found for it, without having to rewrite the code to handle the nodes.

I've attached a patch here. We just need to append a line break to the <pre>, so this is all that patch is doing.

Let us know if you think it is ok, based on your experience with it to fix other tickets.

comment:4 Changed 16 years ago by Martin Kou

Resolution: fixed
Status: reopenedclosed

Yes, your patch does the same thing and it's simpler. I've merged it to the trunk in [1032].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy