We are migrating CKEditor issue tracking to GitHub. Please, use GitHub to report any new issues.

The former tracking system (this website) will still be available in the read-only mode. All issues reported in the past will still be available publicly and can be referenced.

Important: we decided not to transfer all the tickets to GitHub, as many of them are not reproducible anymore or simply no longer requested by the community.
If the issue you are interested in, can be still reproduced in the latest version of CKEditor, feel free to report it again on GitHub.
At the same time please note that issues reported on this website are still taken into consideration when picking up candidates for next milestones.

To report a new issue, go to https://github.com/ckeditor/ckeditor-dev/issues and follow the instructions in the issue template.

Context Navigation

← Previous Ticket
Next Ticket →

Opened 16 years ago

Closed 16 years ago

#2516 closed Bug (fixed)

Remove the Array.AddItem method

Reported by:	Alfonso Martínez de Lizarrondo	Owned by:	Alfonso Martínez de Lizarrondo
Priority:	Normal	Milestone:	FCKeditor 2.6.4
Component:	General	Version:	FCKeditor 2.3.2
Keywords:	Review+	Cc:

Description

In fckjscoreextensions.js there's a Array.prototype.AddItem that it's just a custom implementation of the Array.push method, so we should remove it and replace its occurrences with .push.

Attachments (1)

2516.patch (2.3 KB) - added by Alfonso Martínez de Lizarrondo 16 years ago.: Proposed patch

Download all attachments as: .zip

Change History (3)

Changed 16 years ago by Alfonso Martínez de Lizarrondo

Attachment:	2516.patch added

Proposed patch

comment:1 Changed 16 years ago by Frederico Caldeira Knabben

Keywords:	Review+ added; Review? removed
Milestone:	→ FCKeditor 2.6.4

comment:2 Changed 16 years ago by Alfonso Martínez de Lizarrondo

Resolution:	→ fixed
Status:	new → closed

Fixed with [2570]

Note: See TracTickets for help on using tickets.

Download in other formats: