Ticket #5481 (closed defect: fixed)

Opened 7 months ago

Last modified 4 months ago

Use is_numeric instead of ctype_digit in wp-admin/includes/image.php

Reported by: josephscott Assigned to: anonymous
Priority: normal Milestone: 2.5
Component: Administration Version: 2.5
Severity: normal Keywords: has-patch
Cc: josephscott

Description

The use of ctype_digit() has brought in as part of r6309 in wp-admin/includes/image.php. This will sound odd, but I've run into PHP installs that don't have ctype_* functions available. Seems like the next best thing is to use is_numeric().

Attachments

image.php.diff (1.1 kB) - added by josephscott on 12/18/07 01:57:10.

Change History

12/18/07 01:57:10 changed by josephscott

  • attachment image.php.diff added.

12/19/07 20:50:51 changed by josephscott

use of the WordPress function absint() would be even better than is_numeric().

03/27/08 16:56:58 changed by ryan

  • status changed from new to closed.
  • resolution set to fixed.

(In [7549]) Use is_numeric instead of ctype_digit. Props josephscott. fixes #5481