Feeling Lucky
bug
PR-5.0-dev
?
[#39259] - [Contact] Send email copy - change field type to checkboxes
- Closed
- 31 Aug 2023
- Medium
- Build: 4.2-dev
- # 39259
- Diff
- drmenzelit:mail-copy-checkboxes
User tests: Successful: Unsuccessful:
Pull Request for Issue #34757
Maybe a better solution than #39252
Summary of Changes
Modified field type to checkboxes
Testing Instructions
See issue #34757
Actual result BEFORE applying this Pull Request
See issue #34757
Expected result AFTER applying this Pull Request
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_contact |
| Title |
|
||||||
HLeithner
- comment
- 2 May 2023
This pull request has been automatically rebased to 4.3-dev.
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-08-31 15:42:11 |
| Closed_By | ⇒ | HLeithner | |
| Labels |
Added:
bug
PR-5.0-dev
?
|
||
HLeithner
- comment
- 31 Aug 2023
This functionally should be removed completely instead of fixing it. I close this PR since allowing the user to send emails to an arbitrary address is a security issue.
In addition to the fact that the checkbox itself, depending on the filling or not, directly says Yes / No (a real tautology), I would vote for the previous solution. In addition, at least one more checkbox can be in contacts with a fairly high frequency - a privacy policy, the visual solution of which corresponds to PR #39252 and does NOT correspond to this PR.