Opened 18 years ago
Closed 18 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><?php echo "test"; ?></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)
Change History (5)
comment:1 Changed 18 years ago by
| Owner: | set to Martin Kou |
|---|---|
| Status: | new → assigned |
comment:2 Changed 18 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Changed 18 years ago by
| Attachment: | 1414.patch added |
|---|
Proposed changes to the "pre" element processor
comment:3 Changed 18 years ago by
| Priority: | High → Normal |
|---|---|
| Resolution: | fixed |
| Status: | closed → reopened |
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 18 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | reopened → closed |
Yes, your patch does the same thing and it's simpler. I've merged it to the trunk in [1032].

Fixed with [1025] and [1026].
Click here for more info about our SVN system.