Add id attributes to the plugin activate/deactivate buttons

[shooper]

I'm working on a project that includes browser-based testing of a plugin, including it's installation and activation on the build server.

When trying to activate my plugin, the "Activate" link has no id attribute. Currently you have to drill down from the data-slug in the <tr> element to find the link.

It would be nice to add this with "activate-<slug>" or "deactivate-<slug>" so that you can easily find that link using tools like Selenium.

[roytanck]

Adds id attributes the the activate, deactivate, resume and delete links on the plugin screen in wp-admin.

[mukesh27]

@roytanck, I tested the patch 49916.diff and found one bug.

<a href="%s" aria-label="%s" id-"deactivate-%s">%s</a>

Replace to

<a href="%s" aria-label="%s" id="deactivate-%s">%s</a>

Plz correct it and upload a new patch.

[SergeyBiryukov]

In 47588:

Plugins: Add HTML ID attributes to plugin action links for easier targeting in browser-based testing.

Props roytanck, shooper.
Fixes #49916.

[SergeyBiryukov]

Replying to mukesh27:

I tested the patch 49916.diff and found one bug.

<a href="%s" aria-label="%s" id-"deactivate-%s">%s</a>

Thanks for testing! Just noting this is already corrected in [47588].

Sorry for missing props, haven't refreshed the tab before committing :)

[mukesh27]

New and Old plugin

[mukesh27]

@SergeyBiryukov For the above situation, the new changes will show the same/duplicate id for Activate and Delete button.

Do we need to reopen the ticket?

[SergeyBiryukov]

Replying to mukesh27:

For the above situation, the new changes will show the same/duplicate id for Activate and Delete button.

Good catch, thanks!

[roytanck]

Any ideas how to fix the duplicate id issue? For the reporter's use case, I can imagine adding the plugin's version number to the id attribute may be a good solution? Another option could be to make is a class, but with browser-based testing that would still activate both plugins.

[SergeyBiryukov]

Replying to roytanck:

Any ideas how to fix the duplicate id issue?

Using the plugin's directory and file name would give us unique IDs:

activate-akismet/akismet.php
activate-akismet-old/akismet.php

These are technically valid in HTML5, however, as explained in ​this Stack Overflow answer, the dot might need to be escaped when using these IDs.

For the purpose of fixing this edge case, I think it would be enough to save the IDs previously displayed on the page in a static array, and then add an incrementing suffix if we find a duplicate, same as wp_unique_post_slug() does:

activate-akismet
activate-akismet-2

Something like 49916.2.diff​.

[mukesh27]

Hi there!

Patch 49916.2.diff​ working fine for me with unique ID values.

[roytanck]

Replying to SergeyBiryukov:

Using the plugin's directory and file name would give us unique IDs:

activate-akismet/akismet.php
activate-akismet-old/akismet.php

These are technically valid in HTML5, however, as explained in ​this Stack Overflow answer, the dot might need to be escaped when using these IDs.

I'd probably go for a data attribute instead of an id if we're going to add the path. Like is done on the <tr> element.

For the purpose of fixing this edge case, I think it would be enough to save the IDs previously displayed on the page in a static array, and then add an incrementing suffix if we find a duplicate, same as wp_unique_post_slug() does:

activate-akismet
activate-akismet-2

Something like 49916.2.diff​.

I've tested your patch, and it works perfectly for me. Elegant solution, although at the expense of a little additional code. I prefer this solution over using the path.

[SergeyBiryukov]

In 48374:

Plugins: Make sure the HTML ID attributes in plugin action links are unique.

Follow-up to [47588].

Props mukesh27, roytanck, SergeyBiryukov.
Fixes #49916.