[wp-trac] [WordPress Trac] #43967: Admin emails after email confirmation don't work for GDPR requests

[WordPress Trac]

#43967: Admin emails after email confirmation don't work for GDPR requests
---------------------------------------------+-----------------------------
 Reporter:  garrett-eclipse                  |       Owner:  (none)
     Type:  defect (bug)                     |      Status:  new
 Priority:  high                             |   Milestone:  4.9.6
Component:  General                          |     Version:  trunk
 Severity:  normal                           |  Resolution:
 Keywords:  gdpr needs-unit-tests has-patch  |     Focuses:  ui,
                                             |  administration
---------------------------------------------+-----------------------------

Comment (by desrosj):

 @iandunn [attachment:43967.2.diff] looks great. Great improvement.

 I was able to break it in one way, though.

 1. Create a confirmation request.
 2. Click the link to confirm the request.
 3. Admin receives an email.
 4. Click the link to confirm the request again.
 5. Admin receives an email.
 6. Can be repeated.

 [attachment:43967.3.diff] adds a post meta flag when the admin is notified
 to prevent any further notifications. Some considerations:

 - Should this meta value be removed when a request is re-initiated?
 - Should it use the status of the request instead? (I don't think so,
 because it will be completed before the email function runs)

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/43967#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform