Ticket #1744 (closed New Feature: fixed)

Opened 6 years ago

Last modified 6 years ago

FCKeditor.java browser detection enhancement

Reported by: mosipov Owned by:
Priority: Normal Milestone: FCKeditor.Java 2.4
Component: Server : Java Version:
Keywords: Cc:

Description (last modified by alfonsoml) (diff)

Attachments

FCKeditor.java.patch (1.9 KB) - added by mosipov 6 years ago.

Change History

Changed 6 years ago by mosipov

comment:1 Changed 6 years ago by mosipov

FCKeditor.java browser detection enhancement Private: (?) No While trying out the Java/JSP-wrapper to FCKeditor I noticed that it didn't work with my version of Firefox because of it's user agent (Resulted in a ArrayNullPointerException):

Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.8.0.2) Gecko/Debian-1.5.dfsg+1.5.0.2-3 Firefox/1.5.0.2

Anyhow; I reworked FCKeditor.java to use the "rv:" field by a regular expression instead of "get whatever's 6 characters after the beginning of the string 'gecko'".

The patch is tested on

  • Firefox 1.5.0.2 / Mozilla/5.0 (X11; U; Linux i686

(x86_64); en-US; rv:1.8.0.2) Gecko/Debian-1.5.dfsg+1.5.0.2-3 Firefox/1.5.0.2

  • Firefox 1.5.0.2 / Mozilla/5.0 (Windows; U; Windows NT

5.1; en-US; rv:1.8.0.2) Gecko/20060308 Firefox/1.5.0.2

  • Mozilla 1.7.12
  • Epiphany 1.8.3

(Stated in patch...)

Patch is against the file FCKeditor.java, v2.3.

BR. Henrik Johansson

comment:2 Changed 6 years ago by alfonsoml

  • Description modified (diff)

comment:3 Changed 6 years ago by th-schwarz

  • Status changed from new to closed
  • Resolution set to fixed
  • Milestone set to FCKeditor.Java 2.4

Browser detection is completely rewritten and should work as expected.

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