Ticket #7283 (closed Bug: fixed)

Opened 4 years ago

Last modified 4 years ago

ASP.NET: entered text is not HTML encoded.

Reported by: wwalc Owned by: kaga
Priority: Normal Milestone:
Component: Server : ASP.Net Version: 3.5.2
Keywords: Cc:

Description

Steps to reproduce:

  • Open SubmitData.aspx
  • Paste the following in source mode:
    <p>
    	This is some &lt;tag&gt; in the editor</p>
    
  • Press submit, the result is now:
    <p>
    	This is some <tag>in the editor</tag></p>
    

Another way to reproduce it:

  • In SampleConfig.aspx set the following:
    protected void Page_Load(object sender, EventArgs e)
    {
            CKEditor1.Text = "<p>This is some &lt;tag&gt; in the editor</p>";
    }
    
  • Launch the sample, the result in the editor is:
    <p>
    	This is some <tag>in the editor</tag></p>
    

Attachments

7283.patch (14.6 KB) - added by kaga 4 years ago.
UpdatePanel.aspx (1.3 KB) - added by wwalc 4 years ago.
updatepanel_problem.png (36.7 KB) - added by wwalc 4 years ago.
7283_1.patch (26.5 KB) - added by kaga 4 years ago.
7283_2.patch (26.5 KB) - added by kaga 4 years ago.
7283_3.patch (8.7 KB) - added by wwalc 4 years ago.

Change History

comment:1 Changed 4 years ago by wwalc

  • Status changed from new to confirmed
  • Version set to 3.5.2

Changed 4 years ago by kaga

comment:3 Changed 4 years ago by kaga

  • Status changed from confirmed to review
  • Owner set to kaga

Changed 4 years ago by wwalc

comment:4 Changed 4 years ago by wwalc

  • Status changed from review to review_failed

The attached patch cannot be applied without applying patch for #7285 (otherwise it is impossible to build the dll). Does it really have to depend on the other ticket?

Anyway, when using both patches I found a problem when using the attached sample file (UpdatePanel.aspx) in Firefox.

Steps to reproduce

  1. Load _Samples/UpdatePanel.aspx
  2. Press "Inside UP" button
  3. Press "F5" to refresh the content
  4. The content of the first editor is now broken (see attached screenshot)

Changed 4 years ago by wwalc

Changed 4 years ago by kaga

comment:5 Changed 4 years ago by kaga

  • Status changed from review_failed to review

This patch resolved also ticket #7285

comment:6 Changed 4 years ago by wwalc

  • Status changed from review to review_failed

With the latest patch and I still get the same result as in comment:4

Changed 4 years ago by kaga

comment:7 Changed 4 years ago by kaga

  • Status changed from review_failed to review

comment:8 Changed 4 years ago by wwalc

Another use case to check: http://cksource.com/forums/viewtopic.php?t=21936

I am using ckeditor3.5.2 with asp.net. When I make an update to a page, it displays fine and it saves fine. However, if I push the button to go back to the editor, the text no longer shows itself in wysiwyg mode, but instead is wrapped in html code. If I view the source, it renders all of those extra characters as well. I have to exit the page and get back in for it to show up normally. If I save while in the html code mode, it saves <h1>header</h1><p>stuff here</p> etc to the page. Is there a setting to auto refresh so that it renders correctly when I go back to the editor? Thanks.

comment:9 Changed 4 years ago by wwalc

  • Status changed from review to review_failed

Just like in the comment above: IE8, _Samples/SubmitData.aspx, press submit, then press "Back" button. The content in the editor is now broken.

Changed 4 years ago by wwalc

comment:10 Changed 4 years ago by wwalc

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

The last patch has been committed with [6670].

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