Ticket #2550 (closed defect (bug): fixed)

Opened 6 years ago

Last modified 5 years ago

remove smilies uksort

Reported by: random Owned by: markjaquith
Priority: low Milestone:
Component: Optimization Version: 2.0.1
Severity: trivial Keywords: smilies
Cc:

Description

The sort of the smilies array in vars.php is slow and pointless. It's adding ~1.5% overhead to every page load on my stock install.

As far as I can tell, it just rearranges the array on the off chance that a) you happen to display the smilies in a row with (e.g.) wp-grins, and b) you actually care what order they appear in. That isn't most people, and it can be done in a  plugin besides.

See also  #213: even if smilies stay in the core, they don't need to be editable by adding functions to wp-config.php, especially when the result is a slower WordPress for everybody else.

Attachments

vars.php.diff Download (528 bytes) - added by random 6 years ago.
removes uksort & smiliescmp()

Change History

random — 6 years ago

removes uksort & smiliescmp()

comment:1   markjaquith — 6 years ago

  • Owner changed from anonymous to markjaquith
  • Status changed from new to assigned

Agreed. I've never seen anyone print the list out, so that's something a plugin could handle.

+1

comment:2   ryan — 6 years ago

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

(In [3640]) Remove smilies sort. Props random. fixes #2550

comment:3   ryan — 6 years ago

  • Status changed from closed to reopened
  • Resolution fixed deleted
  • Milestone changed from 2.1 to 2.0.3

Reopen for 2.0.3 inclusion.

comment:4   ryan — 6 years ago

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

(In [3741]) Remove smilies sort. Props random. fixes #2550

comment:5   anonymous — 5 years ago

  • Milestone 2.0.3 deleted

Milestone 2.0.3 deleted

Note: See TracTickets for help on using tickets.