Opened 9 years ago

Closed 9 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 9 years ago.
Proposed patch

Download all attachments as: .zip

Change History (3)

Changed 9 years ago by Alfonso Martínez de Lizarrondo

Attachment: 2516.patch added

Proposed patch

comment:1 Changed 9 years ago by Frederico Caldeira Knabben

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

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

Resolution: fixed
Status: newclosed

Fixed with [2570]

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