? ? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
3 Oct 2018

Pull Request for Issue #22066

Summary of Changes

Implement a workaround for the consentbox with com_mailto. Lets softcode the consentbox than :(

Testing Instructions

  • Install 3.9.0beta4
  • enable mailto
  • enable recaptcha (or invisible captcha) (keys for localhost can be found here)
  • go to the mailto page
    image
  • make sure the form can't be submitted when the privacy checkbox is not ticked.
  • make sure the form can be submitted when everything is correct.

Expected result

Form now works in combination with recatcha too.

Actual result

Email is not sent. An error message is displayed at the top with the legend: ‘Field Required: Privacy Note’.

Documentation Changes Required

None

avatar zero-24 zero-24 - open - 3 Oct 2018
avatar zero-24 zero-24 - change - 3 Oct 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Oct 2018
Category Front End com_mailto
avatar zero-24
zero-24 - comment - 5 Oct 2018

@Thomsterdam @sandewt Please test this PR so we can fix it before 3.9.0 stable. Thanks!

avatar alikon
alikon - comment - 5 Oct 2018

I have tested this item successfully on 8b42396


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22481.

avatar alikon alikon - test_item - 5 Oct 2018 - Tested successfully
avatar Quy
Quy - comment - 6 Oct 2018

I have tested this item successfully on 8b42396


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22481.

avatar Quy Quy - test_item - 6 Oct 2018 - Tested successfully
avatar Quy Quy - change - 6 Oct 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 6 Oct 2018

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/22481.

avatar mbabker mbabker - change - 6 Oct 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-10-06 18:48:56
Closed_By mbabker
Labels Added: ? ?
avatar mbabker mbabker - close - 6 Oct 2018
avatar mbabker mbabker - merge - 6 Oct 2018
avatar zero-24
zero-24 - comment - 7 Oct 2018

Thanks ?

Add a Comment

Login with GitHub to post a comment