WordPress.org
Get WordPress

Make WordPress Core

Opened 21 years ago

Closed 20 years ago

Last modified 17 years ago

#1445 closed defect (bug) (fixed)

Add get_sidebar action

Reported by: janX Owned by: ryan
Priority: normal Milestone: 2.2
Component: General Version: 1.5.2
Severity: normal Keywords: has-patch commit
Cc: Focuses:

Description

This is a proposal for new hooks that can be used to develop navigation plug-ins. I can supply an example if there is demand.

I propose to add the hooks template_header, template_footer and template_sidebar to template-functions-general.php.

Attachments (2)

template-functions-general.diff.txt​ (147 bytes ) - added by janX 21 years ago.
diff to release version 1.5.1.2 of the file
1445.diff​ (399 bytes ) - added by rob1n 20 years ago.

Download all attachments as: .zip

Change History (16)

@janX
21 years ago

diff to release version 1.5.1.2 of the file

#1 @janX
21 years ago

  • Keywords bg|has-patch bg|dev-feedback added
  • Severity minor → normal
  • Summary New hooks for navigation plug-ins → Add template_header, template_footer and template_sidebar action hooks [patch]
  • Version 1.5.1.1 → 1.5.2

#2 @davidhouse
21 years ago

Looks good.

#3 @skippy
21 years ago

  • Keywords bg|squashed added
  • Owner changed from anonymous to skippy
  • Status new → assigned

#4 @ryan
21 years ago

  • Milestone 2.0 → 2.1

#5 @janX
21 years ago

Interesting. Is there a special reason for this to be moved to 2.1?

#6 @janX
21 years ago

  • Milestone 2.1 → 2.0.1

#7 @abhay
21 years ago

probably because it's not a hook that's required to fix any bugs. 2.0.1 is supposed to be a bugfix release from 2.0

#8 @ryan
21 years ago

  • Milestone 2.0.1 → 2.1

Yep. We'll look at this for 2.1.

#9 @matt
20 years ago

  • Milestone 2.1 → 2.2

@rob1n
20 years ago

#10 @rob1n
20 years ago

  • Keywords has-patch commit added; bg|has-patch bg|dev-feedback navigation hooks bg|squashed removed
  • Owner changed from skippy to rob1n
  • Status assigned → new

get_header() and get_footer() already had get_header and get_footer actions, the only one that was missing was get_sidebar, which my patch adds in.

#11 @rob1n
20 years ago

  • Status new → assigned

#12 @rob1n
20 years ago

  • Summary Add template_header, template_footer and template_sidebar action hooks [patch] → Add get_sidebar action

#13 @rob1n
20 years ago

  • Owner changed from rob1n to ryan
  • Status assigned → new

Reassigning to ryan to commit.

#14 @rob1n
20 years ago

  • Resolution → fixed
  • Status new → closed

(In [5136]) Add get_sidebar action to get_sidebar(). fixes #1445

Note: See TracTickets for help on using tickets.