Making WordPress.org

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#3008 closed enhancement (wontfix)

Optimized version of get_term_link() added..

Reported by: afzalmultani's profile afzalmultani Owned by:
Milestone: Priority: normal
Component: WordPress.tv Keywords:
Cc:

Description

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.
wpcom_vip_get_term_link()

Attachments (2)

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

Download all attachments as: .zip

Change History (4)

@afzalmultani
7 years ago

Patch added

@afzalmultani
7 years ago

Patch file .diff

#1 @obenland
7 years 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
7 years ago

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

Note: See TracTickets for help on using tickets.