Opened 7 years ago

Closed 6 years ago

Last modified 4 years ago

#3382 closed enhancement (fixed)

formatting.php - sanitize_file()

Reported by: donncha Owned by: shorty114
Priority: normal Milestone: 2.1
Component: Administration Version:
Severity: normal Keywords: has-patch dev-feedback
Cc:

Description

This patch adds a new function, sanitize_file() to formatting.php, via Matt

Attachments (2)

patch.txt​ (680 bytes) - added by donncha 7 years ago.
patch.2.txt​ (685 bytes) - added by shorty114 6 years ago.
Changed to sanitize_file_name

Download all attachments as: .zip

Change History (8)

donncha — 7 years ago

comment:1 follow-up: ↓ 2   masquerade — 7 years ago

Can we name it sanitize_file_name() or sanitize_filename(), sanitize_file() makes it sound like we are actually doing something to a file.

comment:2 in reply to: ↑ 1   Viper007Bond — 7 years ago

Replying to masquerade:

Can we name it sanitize_file_name() or sanitize_filename(), sanitize_file() makes it sound like we are actually doing something to a file.

Agreed.

shorty114 — 6 years ago

Changed to sanitize_file_name

comment:3   shorty114 — 6 years ago

  • Keywords bg|has-patch bg|dev-feedback added
  • Owner changed from anonymous to shorty114
  • Status changed from new to assigned

comment:4   shorty114 — 6 years ago

  • Keywords has-patch dev-feedback added; bg|has-patch bg|dev-feedback removed

comment:5   ryan — 6 years ago

  • Milestone set to 2.1

comment:6   ryan — 6 years ago

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

(In [4710]) sanitize_file_name(). fixes #3382 #3554

Note: See TracTickets for help on using tickets.