Конечно не правильно, во-первых общее - если асинхронный диалог, то я не вижу чтобы вы свой РНР код проверки домена перенесли в него. Во вторых - if($domain = trim($_GET["domain"])) означает, обрезать карйние пробелы суперглобалльной переменной $_GET["domain"] и результат этой операции присвоить переменной $domain так как она может использоваться далее неоднократно. А условие if проверяет истинна ли переменная $domain, не пустая ли, не равна 0 (если бы это были числа), и если нет, то выполняется то, что в фигурных скобках.
Зачем же же еще и этот никчемный сыр-бор:
$domain==null
{
alert ("Нет указан домен!");
return;
}
if ($domain=='')
{
alert ("Нет указан домен!");
return;
}
да еще никоим образом к РНР отношения не имеющий? Вы пытаетесь скрестить два языка, РНР и JS?
Вы открыли тегом РНР php код, кусочек что я написал, а все далее шпарите от Javascript.
Здесь скачиваем толмуты по РНР и изучаем, синтаксис, конструкции, функции, по крайней мере те, что используете. Иначе за вас никто ничего делать не будет, здесь же не кафедра со студентами, здесь форум помогающий в конкретном коде, а вас то учить еще надо.