Privacy: "Retry" action link on "Export Personal Data" page does not work

[davidbaumwald]

During work on #52892, part of the testing process involves forcing an export failure. Upon failure, each row displays "Download failed." and a "Retry" link. However, clicking this link appears to currently do nothing.

Steps to reproduce:

1. Add ​this script as a must-use file.
2. Go to "Tools" > "Export Personal Data" and add a test user in the "Username or email address" field to generate a request.
3. Hover over the email and click on "Download personal data".
4. Once "Download failed" appears, click the "Retry" link, and nothing happens.

Props to @hellofromTonya for helping to confirm this bug.

Fixes export of personal data when the retry on failure button is clicked.
Trac ticket: https://core.trac.wordpress.org/ticket/53032

[NomNom99]

Fixes retry on failure button

[NomNom99]

Add class to HTML

[davidbaumwald]

@NomNom99 Thanks for the patch! The second approach looks better to me. Updating the keywords here to ask for testing of your fix.

[davidbaumwald]

#55168 was marked as a duplicate.

[davidbaumwald]

#55168 was marked as a duplicate.

[rafiahmedd]

Added class in line 142

[SergeyBiryukov]

Just to clarify my comment from #55168, I'm not quite sure if the change on line 142 is needed, it just seemed that way when looking at the code. Would be great to test and confirm whether it's needed or not.

[hasanuzzamanshamim]

I tested with the latest patch, Just adding the missing class export-personal-data-handle on line 63 is solved the personal data retry button issue. And the email retry button is working as expected without the class export-personal-data-handle.
So, I think there is no need to add it on line 142.

you can check the test case here:
​https://drive.google.com/file/d/1yaku_51cjfuvsrMcdFw7wwkeMv1RvMUc/view?usp=sharing

[davidbaumwald]

In 52846:

Privacy: Fix "Retry" action after a personal data export failure.

When exporting personal data for a user request, if the export fails for any reason, a "Retry" link is displayed inline. However, up to this point, clicking this link performed no action. This change adds the export-personal-data-handle CSS class to the link to make sure it's picked up by the same delegated listener as the initial action. Clicking this link will now retry the export.

Props hellofromTonya, nomnom99, rafiahmedd, SergeyBiryukov, hasanuzzamanshamim.
Fixes #53032.

Thanks for the PR! This was merged into core in https://core.trac.wordpress.org/changeset/52846.