Ticket #3994: 3994.patch

_source/plugins/wysiwygarea/plugin.js

@@ -177 +177 @@
                                           && !( nextElement.getName() in nonExitableElementNames ) )
                                 enterBlock = nextElement;
 
+                        // Not all blocks are editable, e.g. <hr />, further checking it.(#3994)
                         if( ( !count
                                   || ( firstChild = children.getItem( 0 ) ) && firstChild.is && firstChild.is( 'br' ) )
-                                && enterBlock )
+                                && enterBlock
+                                && range.moveToElementEditStart( enterBlock ) )
                         {
                                 fixedBlock.remove();
-                                range.moveToElementEditStart( enterBlock );
                                 range.select();
                         }
                 }

_source/core/dom/range.js

@@ -1602 +1602 @@
                         }
 
                         if ( editableElement )
-                                this.moveToPosition( editableElement, CKEDITOR.POSITION_AFTER_START );
+                        {
+                                this.moveToPosition(editableElement, CKEDITOR.POSITION_AFTER_START);
+                                return true;
+                        }
+                        else
+                                return false;
                 },
 
                 getTouchedStartNode : function()

CHANGES.html

@@ -159 +159 @@
                         left around.</li>
                 <li><a href="http://dev.fckeditor.net/ticket/3992">#3992</a> : Fixed the 
                         ckeditor2.html test case.</li>
+                <li><a href="http://dev.fckeditor.net/ticket/3994">#3994</a> : Insert horizontal line at end of document cause error.</li>
         </ul>
         <h3>
                 CKEditor 3.0 RC</h3>