Application Passwords: Add some additional explanation for Multisite instances

[georgestephanis]

To properly set expectations for users, when the WordPress install is a Multisite install, and the current user has roles on multiple blogs, display an extra blurb on the authorization flow and application passwords section of the user profile screen saying something to the effect of:

Application Passwords will also grant the application access the other {$num_blogs} blogs that you have permissions on.

(along with possibly listing them if it's a short list, or a link to a page in the user dashboard or multisite dashboard listing the user's blogs)

As we'd be displaying it in two places, I'm unsure if it would be worth abstracting to a function or just inlining it both places.

For future v2 buildout, as we're looking at adding additional ability to scope application passwords access, being able to limit the sites they are used on may be worth including in that addition.

Props @spacedmonkey for asking multisite related questions that made me think of this.

[johnjamesjacoby]

I support additional clarity here. Suggest to start by linking to the “My Sites” admin page, and iterate if it is determined to be insufficient.

[hellofromTonya]

As discussed in scrub, converting this to task.

[sarahricker]

Per today's bug scrub, this ticket (now a task) will stay in 5.6 (and after Beta 1 if necessary) as it aims to refine a new feature that has already been merged into trunk.

@georgestephanis and @spacedmonkey - Can you add the exact copy for each location it will be added to this ticket to ensure it is good to go?

Thanks!

Add explanation text in multisite environments when the user has multiple blogs clarifying that application passwords will grant access to all their sites.

Trac ticket: https://core.trac.wordpress.org/ticket/51489

[georgestephanis]

@sarahricker Created a github pr for review and discussion. Honestly I'm horrid on verbiage and phrasing, but the context there should be enough for someone better with words for users to make it intelligible. :\

[sarahricker]

Thanks @georgestephanis, that is perfect! Will take a look. Will try to get it in Beta 1, but if not - Beta 2!

[TimothyBlynJacobs]

This will grant access to all 10 blogs in this installation that you have permissions on. and Application passwords grant access to all 10 blogs in this installation that you have permissions on.

@georgestephanis looks like there are linting failures.

[SergeyBiryukov]

A couple of notes:

• The strings should use _n() instead of __() here, many languages have more than one plural form.
• sizeof() should be replaced with count(), see [49193].

Yeah, just fixed.

@SergeyBiryukov's also left comments on the ticket.

[georgestephanis]

Updated addressing _n and count feedback.

[TimothyBlynJacobs]

In 49270:

App Passwords: Explain that apps can access to all sites in a network.

Props georgestephanis, spacedmonkey, johnjamesjacoby, SergeyBiryukov, marybaum.
Fixes #51489.

[johnjamesjacoby]

Is there a compelling reason why the HTML for this feature does not follow the pattern set by every other user profile feature on this page?

This feature would feel much more complete if it did not look as unique and unfamiliar.

Any objections to reopening & continuing to iterate on this during beta?

[georgestephanis]

@johnjamesjacoby Which aspect of the html do you mean?

And I'm fine with reopening this ticket if it's about the new multisite line specifically, but if it's a grander rearchitecting, may be better to do a new ticket.

[SergeyBiryukov]

In 49282:

Coding Standards: Update some alignment in wp-admin/authorize-application.php for consistency.

Save blogs count to a variable for reusing.

Follow-up to [49270].

See #51489.

[SergeyBiryukov]

Replying to johnjamesjacoby:

Is there a compelling reason why the HTML for this feature does not follow the pattern set by every other user profile feature on this page?

This feature would feel much more complete if it did not look as unique and unfamiliar.

Follow-up: #51899