WordPress.org

Make WordPress Core

Opened 7 years ago

Last modified 3 months ago

#4414 assigned enhancement

Patch to Make Calendars Optionally Category-Specific

Reported by: jacobrose Owned by:
Milestone: Future Release Priority: low
Severity: normal Version: 2.2
Component: Posts, Post Types Keywords: needs-patch get_calendar template, calendar
Focuses: template Cc:

Description

First submission to wordpress...the nice people on wp-hackers directed me here. Here's my post to the list, explaining what this is:

I'm new to WordPress and building a custom template for my new personal site. My design calls for a calendar that stays within a given category, and I couldn't find a way to do that; using "get_calendar()" on my category archive page returned a calendar with all posts in it, therefore providing navigation to pages outside the selected category (which didn't work out).

I have patches to the get_category, get_month_link, and get_day_link functions that let you have in-category calendars (see attached diffs for wp-includes/general-template.php and wp-includes/link-template.php). You can use the modified calendar on the archive page like so:

$cat = array_shift( get_the_category() );
get_calendar( true, $cat->cat_ID );

I'm not sure if that's the canonical way to find the archive page's category id (I kinda doubt it), but it works.

I don't have any other contributors to my site planned, but it wouldn't be hard to make the calendar glue itself to the author in the same way...but is there a standard WordPressian way to accomplish this without modifications like these?

Thanks for any feedback and thanks for WordPress!

Attachments (2)

general-template.diff (2.5 KB) - added by jacobrose 7 years ago.
link-template.diff (1.5 KB) - added by jacobrose 7 years ago.

Download all attachments as: .zip

Change History (10)

jacobrose7 years ago

jacobrose7 years ago

comment:1 Speedboxer7 years ago

  • Keywords get_calendar template calendar added
  • Milestone set to 2.3 (trunk)

comment:2 foolswisdom7 years ago

  • Keywords has-patch added

comment:3 westi7 years ago

  • Keywords needs-patch added; has-patch removed
  • Milestone changed from 2.3 to 2.4 (next)
  • Owner changed from anonymous to westi
  • Status changed from new to assigned

This sounds like an interesting feature.

It maybe plugin material but if the changes to core are minimal to support calendars that follow the current cat/author for the relavent archive pages then it is probably useful

The template tag should be able to detect the cat/author automatically though.

Maybe we need get_calendar varient - get_archive_calendar that gives you the relavent calendar for the type of archive being displayed - post (same as current) / cat / author (both get limited calendars).

Template authors could then use those.

Push this enhancement to 2.4 and adding needs-patch as current patches do not apply.

comment:4 westi7 years ago

  • Severity changed from trivial to normal

This is next of my todo list.

comment:5 westi6 years ago

  • Milestone changed from 2.5 to 2.6

Move to 2.6 as I haven't got anywhere with completing this yet :-(

comment:6 Denis-de-Bernardy5 years ago

  • Keywords changed from get_calendar, template, calendar, needs-patch to needs-patch get_calendar, template, calendar
  • Milestone changed from 2.9 to Future Release

comment:7 westi2 years ago

  • Keywords changed from needs-patch get_calendar, template, calendar to needs-patch get_calendar template, calendar
  • Owner westi deleted
  • Status changed from accepted to assigned

I never got to doing this, it would be a nice thing to have but isn't high priority.

comment:8 nacin3 months ago

  • Component changed from Template to Posts, Post Types
  • Focuses template added
Note: See TracTickets for help on using tickets.