Closed fredericalpers closed 10 months ago
@fredericalpers I need 3 days to fix and test it. Thanks!
@fredericalpers
The form number must be added to e.g. defaultform.php
<form method="post" id="onoffice-form-<?php echo $pForm->getFormNo(); ?>">
This solution will impact backward compatibility If User use "id=onoffice-form" customize in template..
I found an alternative solution that we can select according by the "oo_formno" value in the form.
Do you agree with my solutions?
Note: Now, I saw "spam detected!" notification only show in the contact form. Do you want to add a notification for the "owner form" or "interest form"?
@yeneastgate Please go ahead and implement your suggested solution for backwards compatibility. :)
The "Spam detected" notification should be displayed in all forms that are protected by reCAPTCHA.
@fredericalpers I fixed this bug and added a "Spam detected" notification in all forms that are protected by reCAPTCHA. Please take a look at my video demo and let me know your opinions. Thanks! https://files.fm/u/2x6cc7yfby
@dai-eastgate seems good to me! thank you :) can I label it as "in review"?
@dai-eastgate seems good to me! thank you :) can I label it as "in review"?
Yes, please review this PRs for me. ^^
Current state
Multiple forms on one page with reCAPTCHA cause that when submitting a completed form, empty fields of another form are flagged.
Desired state
If several forms on a page are protected with reCAPTCHA, they should be able to be submitted properly.
Possible Solution
The form number must be added to e.g. defaultform.php
<form method="post" id="onoffice-form-<?php echo $pForm->getFormNo(); ?>">
and at the end the formsubmit.php will be loaded
include(__DIR__.'/formsubmit.php');
Example of the new formsubmit.php: