Making WordPress.org

Opened 8 months ago

Closed 2 months ago

Last modified 2 months ago

#3008 closed enhancement (wontfix)

Optimized version of get_term_link() added..

Reported by: afzalmultani Owned by:
Milestone: Priority: normal
Component: WordPress.tv Keywords:


The optimized version of get_term_link that adds caching for slug-based lookups. Returns Permalink for a taxonomy term archive, or a WP_Error object if the term does not exist.

Attachments (2)

3008.diff (4.0 KB) - added by afzalmultani 8 months ago.
Patch added
3008.2.diff (734 bytes) - added by afzalmultani 8 months ago.
Patch file .diff

Download all attachments as: .zip

Change History (4)

8 months ago

Patch added

8 months ago

Patch file .diff

#1 @obenland
2 months ago

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

Hi @afzalmultani, thanks for this suggestion!

Unfortunately wpcom_vip_get_term_link() is not available on WordPress.tv, it's a function that is not shared with the wider wpcom infrastructure. I also don't think that adding another caching wrapper around that function is super helpful, the existing caching infrastructure should be able to handle it pretty well.

Please don't let the resolution to this ticket discourage you from making more optimization suggestions though!

#2 @afzalmultani
2 months ago

Hello @obenland, Understood!! Thank you for the brief.

Note: See TracTickets for help on using tickets.