Opened 9 years ago

Closed 9 years ago

#1945 closed Bug (fixed)

Sanitize{File,Folder}Name does miss control characters

Reported by: mosipov Owned by: wwalc
Priority: Must have (possibly next milestone) Milestone: FCKeditor 2.6
Component: File Browser Version: FCKeditor 2.6 Beta
Keywords: Cc:

Description

functions miss to replace http://en.wikipedia.org/wiki/ASCII#ASCII_control_characters I did in Java with a simple "|[\u0000-\u001F]|\u007F"

Just add to PHP and other connectors

Change History (7)

comment:1 Changed 9 years ago by fredck

  • Milestone set to FCKeditor 2.6

comment:2 Changed 9 years ago by wwalc

  • Owner set to wwalc

To create similar pattern in PHP: \x{001F} utf-8 mode is required, it is available @Windows since 4.2.3 so to be fully compatible with such users I think it's better to simply use cntrl set in regular expression: [:cntrl:].

comment:3 Changed 9 years ago by mosipov

wwalc,

great idea! I adapted it for Java with "\p{Cntrl}" too. I missed to check the api Thx

comment:4 Changed 9 years ago by wwalc

  • Component changed from Server : PHP to File Browser

PHP fixed with [1648], Perl: [1649], Python: [1651], ColdFusion: [1652].

comment:5 Changed 9 years ago by wwalc

It seems that it is impossible to implement such check in Lasso due to language limitations, however I have added just a bascic check of folder and file names with [1655].

Someone could fix this in asp & asp.net, then we can close this ticket.

comment:6 Changed 9 years ago by alfonsoml

Asp fixed with [1672]

comment:7 Changed 9 years ago by fredck

  • Resolution set to fixed
  • Status changed from new to closed

ASP.NET fixed with [1701]. Changelog entry added with [1702].

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