Opened 3 years ago

Closed 2 years ago

#12285 closed task (blessed) (fixed)

Migrate Link Categories management page to edit-tags.php

Reported by: dd32 Owned by:
Priority: high Milestone: 3.1
Component: Administration Version: 3.0
Severity: normal Keywords: needs-patch
Cc: sorich87@…

Description

In 3.0 with #11838, I've migrated the Categories page and any custom taxonomies to use the edit-tags.php file to prevent duplication of data.

Link categories has remained untouched during this process, and currently duplicates much of the taxonomy editing pages.

We should transition the Link Categories over too (And allow for taxonomies to register themselves for links as well).

One of the issues is that the taxonomy page is currently setup for post type handling, Which may require a bit of handling the post_type=link as a internal dummy post_type for the purpose of links. Or at least, Thats one way of approaching it.

Attachments (1)

12285.diff (911 bytes) - added by sorich87 3 years ago.

Download all attachments as: .zip

Change History (18)

comment:1   dd323 years ago

See Also: http://core.trac.wordpress.org/ticket/12553#comment:3 which brings up this related issue.

It looks like scribu's massive patch addressed this when he added the default-list-tables. Is there any of the above work left to be done?

comment:3   dd323 years ago

It looks like scribu's massive patch addressed this when he added the default-list-tables.

Looks like it.

I'd like to double-check that the old actions have been carried across however (If someone can check that, Feel free to close this ticket)

  • Milestone changed from Awaiting Triage to 3.1
  • Priority changed from normal to high
  • Cc sorich87@… added
  • Keywords has-patch added; early removed

This patch addresses two small remaining things.

I was not sure if 'edit-link-categories.php' and 'edit-link-category-form.php' should be removed or deprecated.

comment:6   dd323 years ago

I was not sure if 'edit-link-categories.php' and 'edit-link-category-form.php' should be removed or deprecated.

They'll probably just get removed, no real point in having them around I don't feel..

We didn't restore any files we removed in 3.0. These should go too.

comment:8   dd323 years ago

My only thought was related to back-compat for existing links, but I don't think that'll be a huge problem.

I'll remove the files once I get around to comparing the hook firing..

[15788] - My bad, but ultimately affects nothing. Just wanted to document that here.

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

(In [15820]) Get rid of old link category management files. Fixes #12285

(In [15823]) The Hook that never was. Use correct / Revert back-compat hookname changes in [15820]. See #12285

  • Resolution fixed deleted
  • Status changed from closed to reopened

Just noticed that none of the Ajax handlers were modified/removed.

for example: http://core.trac.wordpress.org/browser/trunk/wp-admin/admin-ajax.php?rev=15956#L363

(In [15963]) Remove dead 'delete-link-cat' ajax handler. See #12285

  • Keywords needs-patch added; has-patch removed

The only one left is 'add-link-category', which is called from link_categories_meta_box(), which should be made into a wrapper for the normal category metabox.

  • Type changed from enhancement to task (blessed)

Marking as a task as an extension of #14579.

All 3.1 work done?

  • Resolution set to fixed
  • Status changed from reopened to closed

Yeah, guess we can call it a day.

Note: See TracTickets for help on using tickets.