Opened 17 years ago
Closed 17 years ago
#1945 closed Bug (fixed)
Sanitize{File,Folder}Name does miss control characters
| Reported by: | Michael Osipov | Owned by: | Wiktor Walc |
|---|---|---|---|
| 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 17 years ago by
| Milestone: | → FCKeditor 2.6 |
|---|
comment:2 Changed 17 years ago by
| Owner: | set to Wiktor Walc |
|---|
comment:3 Changed 17 years ago by
wwalc,
great idea! I adapted it for Java with "\p{Cntrl}" too. I missed to check the api Thx
comment:4 Changed 17 years ago by
| Component: | Server : PHP → File Browser |
|---|
comment:5 Changed 17 years ago by
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:7 Changed 17 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
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:].