Make WordPress Core

Opened 3 years ago

Closed 20 months ago

#21734 closed enhancement (maybelater)

Completely remove global terms

Reported by: scribu Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Taxonomy Keywords: needs-refresh
Focuses: Cc:

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 3 years ago.
remove-global-terms.diff (15.8 KB) - added by wonderboymusic 3 years ago.
21734.2.diff (15.8 KB) - added by scribu 3 years ago.
remove-global-terms-for-plugin.diff (16.1 KB) - added by wonderboymusic 3 years ago.

Download all attachments as: .zip

Change History (16)

@scribu3 years ago

comment:2 @scribu3 years 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 @wonderboymusic3 years ago

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

comment:4 @scribu3 years ago

  • Description modified (diff)

@scribu3 years ago

comment:5 @scribu3 years ago

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

comment:6 @wonderboymusic3 years ago

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

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

Version 0, edited 3 years ago by wonderboymusic (next)

comment:7 follow-up: @scribu3 years 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 @SergeyBiryukov3 years 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 @nacin3 years 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 @scribu3 years 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 @kawauso2 years ago

  • Cc kawauso added

comment:12 @wonderboymusic20 months ago

  • Keywords needs-refresh added; has-patch needs-testing removed
  • Milestone Future Release deleted
  • Resolution set to maybelater
  • Status changed from new to closed

I don't think this matters right now

Note: See TracTickets for help on using tickets.