Skip to content
Snippets Groups Projects
Commit e9012382 authored by Peter Rotich's avatar Peter Rotich
Browse files

Merge pull request #2293 from greezybacon/issue/null-domain-matchup


org: Fix matching of empty domain name

Reviewed-By: default avatarPeter Rotich <peter@osticket.com>
parents bbc35dcf 57c2cb19
No related branches found
No related tags found
No related merge requests found
...@@ -166,6 +166,8 @@ class Organization extends OrganizationModel { ...@@ -166,6 +166,8 @@ class Organization extends OrganizationModel {
} }
function isMappedToDomain($domain) { function isMappedToDomain($domain) {
if (!$domain || !$this->domain)
return false;
foreach (explode(',', $this->domain) as $d) { foreach (explode(',', $this->domain) as $d) {
$d = trim($d); $d = trim($d);
if ($d[0] == '.') { if ($d[0] == '.') {
...@@ -182,8 +184,12 @@ class Organization extends OrganizationModel { ...@@ -182,8 +184,12 @@ class Organization extends OrganizationModel {
} }
static function forDomain($domain) { static function forDomain($domain) {
foreach (static::objects() if (!$domain)
->filter(array('domain__contains'=>$domain)) as $org) { return null;
foreach (static::objects()->filter(array(
'domain__gt'=>'',
'domain__contains'=>$domain
)) as $org) {
if ($org->isMappedToDomain($domain)) { if ($org->isMappedToDomain($domain)) {
return $org; return $org;
} }
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment