No synchronisation between "most used" and "all categories" in link edit

[solarissmoke]

There are issues with the "all categories" and "most used" tabs when editing links:

• Go to edit a link

• Click on "Most used" in the category section. All the listed categories are unchecked - including the ones that are checked in the "All Categories" tab. There is no sync between the two.

• Now check a category in the "most used" tab. Go to the "all categories" tab and you will find that the same category is now checked (as expected). Now uncheck it from the "all categories" tab. Go to "most used" and it is still checked there. The sync only works one way ( "most used" -> "all categories" ).

The first issue is because wp_popular_terms_checklist() assumes that you're using it in the context of a particular post, and sets checkedness based on whether the $post global has that term. If the $post global is empty (as when editing links), nothing is checked. I don't think we should be using wp_popular_terms_checklist() as it's context is for individual posts.

The second issue is because the IDs on the most used elements are wrong compared to what the Javascript is looking for.

Patch to follow.

[hakre]

Thanks for reporting the issue. I could review your patch. I see a small issue with the $number parameter you've added. It's not used at all. Please review.

After changing, please add the has-patch tag again. I've changed it for workflow reasons.

[solarissmoke]

Oops sorry, I think I changed variable names half way through writing the patch. Fixed now.

[helen]

I'm sure it was discussed, but mind leaving a brief comment as to why the wontfix?

[transom]

Sorry about the lack of comment.

This functionality is no longer part of Core. Now part of ​http://wordpress.org/plugins/link-manager/

[helen]

Ah, I see, it's specific to links.