Ticket #4725 (closed Bug: fixed)

Opened 2 years ago

Last modified 2 years ago

hitting return after html comment produces an error "t.getName is not a function"

Reported by: tyler45 Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.1
Component: UI : Enter Key Version: SVN (CKEditor) - OLD
Keywords: Confirmed Review+ Cc:

Description

hi,

when a paragraph ends in an html comment, hitting the enter key with the cursor positioned at the end of this paragraph (to create a new paragraph) throws an error "t.getName is not a function" (line 29 ckeditor.js).

<p>Any old para.<!-- any old comment --></p>

Windows Vista SP2 FF 3.5.5

Attachments

4725.patch Download (461 bytes) - added by garry.yao 2 years ago.
4725_2.patch Download (488 bytes) - added by garry.yao 2 years ago.

Change History

Changed 2 years ago by garry.yao

comment:1 Changed 2 years ago by garry.yao

  • Status changed from new to assigned
  • Component changed from General to UI : Enter Key
  • Owner set to garry.yao
  • Version set to SVN (CKEditor)
  • Milestone set to CKEditor 3.1
  • Keywords Confirmed Review? added

comment:2 Changed 2 years ago by fredck

  • Keywords Review- added; Review? removed

Please use the type property to check the node type. I know it'll add 5 bytes more to the code, but we have already understood that using methods to check for node types is a bad practice.

Changed 2 years ago by garry.yao

comment:3 Changed 2 years ago by garry.yao

  • Keywords Review? added; Review- removed

comment:4 Changed 2 years ago by fredck

  • Keywords Review+ added; Review? removed

comment:5 Changed 2 years ago by garry.yao

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

Fixed with [4596].

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