Ticket #5850: 5850.patch

_source/plugins/selection/plugin.js

@@ -590 +590 @@
                                                         boundaryInfo = getBoundaryInformation( nativeRange );
                                                         range.setEnd( new CKEDITOR.dom.node( boundaryInfo.container ), boundaryInfo.offset );
 
+                                                        // Correct an invalid IE range case on empty list item. (#5850)
+                                                        if ( range.endContainer.getPosition( range.startContainer ) & CKEDITOR.POSITION_PRECEDING
+                                                                        && range.endOffset <= range.startContainer.getIndex() )
+                                                        {
+                                                                range.collapse();
+                                                        }
+
                                                         return ( cache.ranges = [ range ] );
                                                 }
                                                 else if ( type == CKEDITOR.SELECTION_ELEMENT )
@@ -703 +710 @@
                                                         node = node.getChild( range.startOffset );
 
                                                         if ( !node || node.type != CKEDITOR.NODE_ELEMENT )
-                                                                return range.startContainer;
+                                                                node = range.startContainer;
 
                                                         var child = node.getFirst();
                                                         while (  child && child.type == CKEDITOR.NODE_ELEMENT )
@@ -711 +718 @@
                                                                 node = child;
                                                                 child = child.getFirst();
                                                         }
-
-                                                        return node;
                                                 }
-                                        }
-
-                                        if ( CKEDITOR.env.ie )
-                                        {
-                                                range = sel.createRange();
-                                                range.collapse( true );
-
-                                                node = range.parentElement();
-                                        }
-                                        else
-                                        {
-                                                node = sel.anchorNode;
-
-                                                if ( node && node.nodeType != 1 )
-                                                        node = node.parentNode;
-                                        }
-                        }
+                                                else
+                                                {
+                                                        node = range.startContainer;
+                                                        if ( node.type != CKEDITOR.NODE_ELEMENT )
+                                                                node = node.getParent();
+                                                }
+                                                node = node.$;
+                                        }
+                        }
 
                         return cache.startElement = ( node ? new CKEDITOR.dom.element( node ) : null );
                 },