Privacy: Don't always link to the menu customizer

[dlh]

Currently, after saving the page for the site's privacy policy, a message displays with "Remember to update your menus!" that links to the menu customizer.

In some situations, it might not make sense to include this message:

1. When the user can't edit_theme_options and thus by default can't reach the Customizer.

2. When the selected page for the privacy policy is unpublished, as unpublished pages can't be searched for by default in the menu customizer.

3. When the current theme doesn't support menus, as the mention of a menu might confuse the user.

The attached patch would skip the menu prompt after saving the setting in these cases.

Of these cases, (3) seems to me like the most debatable, as users can still create menus in the Customizer even if the theme doesn't register a menu location. But (1) and (2) seem to me on firmer ground.

A lighter touch could also be taken than what's in the patch, such as a message like "Remember to update your menus after the page is published!" for case (2).

[desrosj]

Marking privacy bugs as introduced in 4.9.6.

[desrosj]

Moving all tickets in 4.9.7 to 4.9.8.

[desrosj]

@dlh this looks good to me and tests well.

Only nit pick I have is that I believe the rule with multi-line conditionals is one per line leaving the first conditional on the same line as the if(. I couldn't find the rule stated anywhere in WPCS ruleset, or in the handbook, though.

[dlh]

I believe the rule with multi-line conditionals is one per line leaving the first conditional on the same line as the if(.

Intriguing! I'm happy to follow whatever the standard is, although I confess I'm curious about what the advantages are of leaving the first condition alongside the if. Is it possible to get an official ruling?

[westonruter]

The changes make sense to me. At first I thought the customize cap should be checked but it's better to use edit_theme_options since that is the cap for menus in the Customizer, and customize is a meta cap for edit_theme_options.

[westonruter]

In 43343:

Privacy: Only link to menus panel in Customizer if selected privacy page can be accessed there.

Props dlh.
Fixes #44117.

[westonruter]

Re-opening for 4.9.7 consideration.

[SergeyBiryukov]

In 43358:

Privacy: Only link to menus panel in Customizer if selected privacy page can be accessed there.

Props dlh.
Merges [43343] to the 4.9 branch.
Fixes #44117.