Opened 8 years ago

Closed 8 years ago

#7518 closed Bug (duplicate)

Getting an error when deleting a row

Reported by: Satya Minnekanti Owned by:
Priority: Normal Milestone:
Component: General Version: 3.0
Keywords: IBM Cc: Damian, James Cunningham, Teresa Monahan

Description

To reproduce the defect:

  1. Open CK Editor and insert a table with default values.
  1. Keep cursor in one of table cells, and open context menu.
  1. Delete the current Row by slecting Row - > Delete Rows.

Expected Result:

Current Table Row is deleted with out showing any errors.

Actual Result:

Current Table Row is deleted but an Error is shown in status bar and the Error is Invalid argument

Attachments (1)

7518.patch (753 bytes) - added by Garry Yao 8 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 8 years ago by Jakub Ś

Status: newconfirmed
Summary: IE: Getting an error when deleting a rowGetting an error when deleting a row
Version: 3.0

This has been true for all versions of IE since CKEditor version 3.0. The error is:

Illegal argument
Line; 1304 
Path: /_source/plugins/selection/plugin.js


This has been also true for Opera 11.01 since CKEditor version 3.5.1 The detailed stack trace form Dragonfly is:

Uncaught exception: Error: BAD_BOUNDARYPOINTS_ERR

Error thrown at line 1390, column 4 in <anonymous function>() in http://192.168.1.100:8080/releases/ckeditor/3.5.2/ckeditor/_source/plugins/selection/plugin.js:
    nativeRange.setStart( startContainer.$, this.startOffset );
called from line 471, column 2 in placeCursorInCell(cell, placeAtEnd) in http://192.168.1.100:8080/releases/ckeditor/3.5.2/ckeditor/_source/plugins/tabletools/plugin.js:
    range.select( true );
called from line 818, column 6 in <anonymous function: exec>(editor) in http://192.168.1.100:8080/releases/ckeditor/3.5.2/ckeditor/_source/plugins/tabletools/plugin.js:
    placeCursorInCell( deleteRows( selection ) );
called from line 52, column 2 in <anonymous function: exec>(data) in http://192.168.1.100:8080/releases/ckeditor/3.5.2/ckeditor/_source/core/command.js:
    return ( commandDefinition.exec.call( this, editor, data ) !== false );
called via Function.prototype.call() from line 581, column 5 in <anonymous function: execCommand>(commandName, data) in http://192.168.1.100:8080/releases/ckeditor/3.5.2/ckeditor/_source/core/editor.js:
    eventData.returnValue = command.exec( eventData.commandData );
called from line 117, column 5 in <anonymous function: onClick>(item) in http://192.168.1.100:8080/releases/ckeditor/3.5.2/ckeditor/_source/plugins/menu/plugin.js:
    this.editor.execCommand( item.command );
called from line 555, column 33 in <anonymous function: bind>() in http://192.168.1.100:8080/releases/ckeditor/3.5.2/ckeditor/_source/core/tools.js:
    return func.apply( obj, arguments );
called via Function.prototype.apply() from line 555, column 33 in <anonymous function: bind>() in http://192.168.1.100:8080/releases/ckeditor/3.5.2/ckeditor/_source/core/tools.js:
    return func.apply( obj, arguments );
called via Function.prototype.apply() from line 304, column 8 in <anonymous function>(index) in http://192.168.1.100:8080/releases/ckeditor/3.5.2/ckeditor/_source/plugins/menu/plugin.js:
    this._.onClick( item );
called from line 641, column 5 in <anonymous function: addFunction>() in http://192.168.1.100:8080/releases/ckeditor/3.5.2/ckeditor/_source/core/tools.js:
    return fn.apply( scope || this, arguments );

comment:2 Changed 8 years ago by Garry Yao

Status: confirmedpending

This can only be confirmed on IE6/7 for me?

Changed 8 years ago by Garry Yao

Attachment: 7518.patch added

comment:3 Changed 8 years ago by alan

Cc: damo,jamescun,tmonahandamo, jamescun, tmonahan

Hi everybody, i have a working copy installed of CKEDITOR 3.5.3 and i was getting that error, applied the patch and the error no longer exists on IE. But the patch introduces a new error on FF, checked on firebug and the error says: Index or size is negative or greater than the allowed amount" code: "1

I just wanted to comment this information i found cause maybe it will help to fix this bug sooner.

comment:4 Changed 8 years ago by Garry Yao

DUP of #7811.

comment:5 Changed 8 years ago by Garry Yao

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