User tests: Successful: Unsuccessful:
Closes #23074
If mail is enabled in Joomla config, but we were unable to send the email, the request is created, but the error message states that the request was not created.
There is no point in creating a request if we cannot send the email - this is why mailing has to be enabled. If your mail settings are wrong, the mail cannot be sent, and therefore we should remove the request and inform the user correctly.
Default 3.9.0 installation with this PR.
Create a "Create Request" Menu item using Privacy -> Create Request menu type
in Joomla Global Config set:
and NO ROW left in #__privacy_requests
(note this error was NOT generated with the above testing instructions, for this one you need to disable mail in php which is harder than the above testing instructions.)
and ROW left in #__privacy_requests even though the error says no request was made.
None
Status | New | ⇒ | Pending |
Category | ⇒ | Front End |
Labels |
Added:
?
|
I don't support this. It was intended to have a "resend email" feature in the admin before I burned out on this work. A request should not be removed simply because the user notification email does not get sent out.
I'm also not comfortable having an "easy to use" PHP API that encourages deleting requests. The workflow is designed so that these should NOT be deleted.
Lastly, this does also make it possible for a request to be invalidated/deleted if for whatever reason the Table API fails to load the request record to be able to build the action log message.
Status | Pending | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-11-16 13:18:44 |
Closed_By | ⇒ | PhilETaylor |
@mbabker please take a look ;-) dont want to ruin your amazing work :)