Make WordPress Core

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#5424 closed defect (bug) (invalid)

wpdb: use function_exists() instead of slower version_compare()

Reported by: dd32's profile DD32 Owned by:
Milestone: Priority: normal
Severity: normal Version: 2.5
Component: Optimization Keywords: has-patch
Focuses: Cc:

Description

Wpdb's escaping code currently compares the PHP version to determine if a function will be available or not.

function_exists('mysql_real_escape_string');

is about twice as fast as the current code(on testing of 1,000 iterations):

version_compare( phpversion(), '4.3.0' );

Attachments (1)

5424.wpdb-version_compare.diff (569 bytes) - added by DD32 17 years ago.

Download all attachments as: .zip

Change History (9)

#1 @ryan
17 years ago

That code isn't even be running since we return on the line before. But, yes, function_exists is better for such things.

#2 @DD32
17 years ago

Now why didnt i notice that? :)

#3 @lloydbudd
17 years ago

  • Milestone changed from 2.4 to 2.6

#5 @matt
17 years ago

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

#6 @DD32
17 years ago

While this has been set to Invalid, Please keep it in mind in relation to #5455 which is suggesting to use real_escape again

#7 @darkdragon
17 years ago

I don't think this should be set to invalid.

#8 @Nazgul
17 years ago

  • Milestone 2.6 deleted
Note: See TracTickets for help on using tickets.