Opened 9 months ago

Last modified 5 months ago

#21734 new enhancement

Completely remove global terms

Reported by: scribu Owned by:
Priority: normal Milestone: Future Release
Component: Taxonomy Version:
Severity: normal Keywords: has-patch needs-testing
Cc: kawauso

Description (last modified by scribu)

It's an obscure WPMU feature that 1) hasn't been maintained 2) hardly anyone uses and 3) slows down development on the regular taxonomy code:

#5809 #18609 #18617

There isn't much code to remove, but the idea is to not have to worry about "global terms" anymore at all.

Attachments (4)

21734.diff​ (14.3 KB) - added by scribu 9 months ago.
remove-global-terms.diff​ (15.8 KB) - added by wonderboymusic 9 months ago.
21734.2.diff​ (15.8 KB) - added by scribu 9 months ago.
remove-global-terms-for-plugin.diff​ (16.1 KB) - added by wonderboymusic 8 months ago.

Download all attachments as: .zip

Change History (15)

scribu — 9 months ago

wonderboymusic — 9 months ago

comment:2   scribu — 9 months ago

  • Keywords has-patch added

21734.diff​ takes care of the raw code removal.

To be completely back-compat, we'd need to automatically install a plugin, when upgrading a WPMU network which had global terms enabled.

comment:3   wonderboymusic — 9 months ago

whoops - I added a patch at basically the exact same time

comment:4   scribu — 9 months ago

  • Description modified (diff)

scribu — 9 months ago

comment:5   scribu — 9 months ago

21734.2.diff​ resurrects global_terms_enabled() and adds TODO for where the code for installing the plugin should go.

wonderboymusic — 8 months ago

comment:6   wonderboymusic — 8 months ago

  • Keywords needs-testing added
  • Milestone changed from Awaiting Review to 3.5

Patch refreshed to accomodate plugin, which is now here: ​http://plugins.svn.wordpress.org/global-terms/trunk/global-terms.php (first rev, needs love / testing)

Last edited 8 months ago by wonderboymusic (previous) (diff)

comment:7 follow-up: ↓ 8   scribu — 8 months ago

Why is the change in get_blog_permalink() needed?

$link = set_url_scheme( get_permalink( $post_id ) , 'http' );

comment:8 in reply to: ↑ 7   SergeyBiryukov — 8 months ago

Replying to scribu:

Why is the change in get_blog_permalink() needed?

That was from ticket:19420:blog-perma-set-url-scheme.diff​ on #19420.

comment:9   nacin — 8 months ago

  • Keywords punt added

Not sure I have the stomach for this right now. It's fairly unobtrusive code that we've not needed to maintain or touch since we disabled global terms by default in 3.0.

comment:10   scribu — 8 months ago

  • Keywords punt removed
  • Milestone changed from 3.5 to Future Release

If it doesn't block #5809, I'm fine with it.

comment:11   kawauso — 5 months ago

  • Cc kawauso added
Note: See TracTickets for help on using tickets.