Make WordPress Core

Opened 14 years ago

Closed 13 years ago

#14816 closed defect (bug) (fixed)

Optimise dynamic_sidebar

Reported by: hakre's profile hakre Owned by:
Milestone: 3.2 Priority: normal
Severity: normal Version: 3.1
Component: Widgets Keywords: has-patch 3.2-early commit
Focuses: Cc:

Description (last modified by westi)

Rasmus Lerdorf did some changes to Wordpress for a hphp compilation / performance presentation.

dynamic_sidebar can get out early if there are no sidebars.

Attachments (1)

14816-widgets.patch (509 bytes) - added by hakre 14 years ago.
Shortcut.

Download all attachments as: .zip

Change History (7)

@hakre
14 years ago

Shortcut.

#1 @hakre
14 years ago

Full diff is here: http://progphp.com/wp/ (More Info), dynamic_sidebar can get out early if there are no sidebars.

Just scanning and providing patches for the places I see fit but as I found later out:

Rest seems to be already fixed.

Last edited 13 years ago by hakre (previous) (diff)

#2 @hakre
14 years ago

And if not, probably already reported, like the constructor stuff from the presentation: #10861

#3 @hakre
14 years ago

Related: #14429
Related: [15639]
Related: [15637]
Related: [13684]

#4 @westi
13 years ago

  • Component changed from General to Widgets
  • Description modified (diff)
  • Keywords has-patch 3.2-early commit added
  • Milestone changed from Awaiting Review to Future Release
  • Summary changed from Diverse Stuff from Rasmus Lerdorfs Patch to Optimise dynamic_sidebar
  • Version set to 3.1

#5 @nacin
13 years ago

  • Milestone changed from Future Release to 3.2

#6 @nacin
13 years ago

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

(In [17545]) Bail early in dynamic_sidebar() if there are no sidebars. fixes #14816.

Note: See TracTickets for help on using tickets.