fix: lookup email domain directly after split

aawnu · aawnu · commit 1372edd1d835 · 2024-07-02T04:56:01.000+02:00
diff --git a/src/Helper/UserDataHelper.php b/src/Helper/UserDataHelper.php
@@ -31,7 +31,7 @@ public function setEmail(string $email): bool
         ) {
             $x = explode("@", $email, 2);
             // https://support.google.com/mail/thread/125577450/gmail-and-googlemail
-            if (substr($x[1], -mb_strlen("googlemail.com")) == "googlemail.com") {
+            if ($x[1] == "googlemail.com") {
                 $x[1] = "gmail.com";
             }
             // https://gmail.googleblog.com/2008/03/2-hidden-ways-to-get-more-from-your.html

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public function setEmail(string $email): bool`
`31`	`31`	`) {`
`32`	`32`	`$x = explode("@", $email, 2);`
`33`	`33`	`// https://support.google.com/mail/thread/125577450/gmail-and-googlemail`
`34`		`- if (substr($x[1], -mb_strlen("googlemail.com")) == "googlemail.com") {`
	`34`	`+ if ($x[1] == "googlemail.com") {`
`35`	`35`	`$x[1] = "gmail.com";`
`36`	`36`	`}`
`37`	`37`	`// https://gmail.googleblog.com/2008/03/2-hidden-ways-to-get-more-from-your.html`