New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug in validating E-Mailadress #6945

Open
Taste-IT opened this Issue Oct 23, 2018 · 2 comments

Comments

3 participants
@Taste-IT

Taste-IT commented Oct 23, 2018

Hi,
after upgrading to latest Version and do an misstake in entering an E-Mailadress like domain.com, an invalid Mailadress, tine tries to send, but in my opinion has to check befor for an valid Mailadress.
Is this a Bug?

@pschuele pschuele added the Felamimail label Oct 26, 2018

@pschuele

This comment has been minimized.

Member

pschuele commented Oct 26, 2018

yep, seems to be a bug. I'll have a look.

btw. I'm getting this error:

index.php:153666 Uncaught (in promise) TypeError: Cannot read property '1' of null
    at sb.extractMailFromString (index.php:153666)
    at Ext.data.Record.<anonymous> (index.php:153638)
    at Ext.util.MixedCollection.each (ext-all-debug.js:11810)
    at constructor.each (ext-all-debug.js:33751)
    at index.php:153631
    at new Promise (<anonymous>)
    at sb.validateSystemlinkRecipients (index.php:153627)
    at sb.onApplyChanges (index.php:153693)
    at sb.onSaveAndClose (index.php:26547)
    at sb.<anonymous> (index.php:152837)

@pschuele pschuele added the Bug label Oct 26, 2018

@pschuele pschuele added this to To do in Tine 2.0 Development via automation Oct 26, 2018

@pschuele pschuele self-assigned this Oct 26, 2018

@tvannahl

This comment has been minimized.

Contributor

tvannahl commented Nov 2, 2018

This looks to me like it is related to #6870 because example.com looks to me like a valid mail address.

Please consider that the domain part of a mail address is optional, because of this example.com should be the same as example.com@localhost.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment