Feeling Lucky
?
[#27948] - Email validation not consistent in the system
- Closed
- 5 Apr 2020
- Medium
- Build: staging
- # 27948
Steps to reproduce the issue
- Go to the global configuration
- Set an email like: "foobar@localhost"
- save
- press "send test email" button
Expected result
Either I get an error on saving, that the email is wrong or sending an email works.
Actual result
Sending an email fails, because Joomla! validation uses the Email ruleset (https://github.com/joomla/joomla-cms/blob/staging/libraries/src/Form/Rule/EmailRule.php) while the PHPMailer class ( addRecipient) uses the PHP validation (https://github.com/joomla/joomla-cms/blob/staging/libraries/vendor/phpmailer/phpmailer/class.phpmailer.php#L1157)
Additional comments
| Labels |
Added:
?
|
||
brianteeman
- comment
- 16 Feb 2020
That would be wrong
dgrammatiko
- comment
- 17 Feb 2020
Just add type="email" instead type="text"in the input. Browsers know better how a valid email looks like
jwaisner
- | Status | New | ⇒ | Confirmed |
| Status | Confirmed | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-04-05 09:41:16 |
| Closed_By | ⇒ | joomla-cms-bot |
| Closed_By | joomla-cms-bot | ⇒ | alikon |
joomla-cms-bot
- comment
- 5 Apr 2020
Set to "closed" on behalf of @alikon by The JTracker Application at issues.joomla.org/joomla-cms/27948
alikon
- comment
- 5 Apr 2020
please test #28574
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27948.
brianteeman
- comment
- 5 Apr 2020
For reference the problem was with phpmailer configuration
We could make TLD required in our forms. The validation rule already supports it.