Код: Выделить всё
if ($HostingScheme['IsSSLAccess'] == 'yes') { $ISPConfigSSL = 'y'; }else{ $ISPConfigSSL = 'n'; };
все поля начинающиеся с Is* - принудительно приведены к булевым значениям
т.е. можно написать так:
Код: Выделить всё
$ISPConfigSSL = ($HostingScheme['IsSSLAccess'])?'y':'n';
и даже скобки на самом деле не обязательны... просто так читабельней
Отправлено спустя 1 минуту 25 секунд:а... вы её дальше юзаете..
можно было прям так и написать вместо
было бы
Код: Выделить всё
'limit_ssl' => ($HostingScheme['IsSSLAccess'])?'y':'n',
Отправлено спустя 5 минут 4 секунды:Код: Выделить всё
# Запрашиваем ID клиента по его Имени пользователя
юзается три раза
вынесите в отдельную функцию, в конец либы.
читабельность кода повысится, проще будет понимать.
два раза - так что тут смотрите сами.
но я бы вынес... большой кусок повторяющегося кода...
Отправлено спустя 1 минуту 27 секунд:Код: Выделить всё
# Проверяем ответ панели на ошибку запроса
if(Preg_Match('/data_processing_error/',$Response)) {
return new gException('WRONG_ANSWER',$Response);
}
а можете показать содержимое $Response для случая успешного завершения и для ошибки?
просто у меня чувство что вы плохого набрались в библиотеке типа вебнеймса =))
Отправлено спустя 1 минуту 27 секунд:блок
Код: Выделить всё
# Содаем подключение к панели ISPConfig
$SoapLocation = SPrintF('%s://%s:%u/remote/index.php',($Settings['Protocol'] == 'ssl')?'https':'http',$Settings['Address'],$Settings['Port']);
$SoapUri = SPrintF('%s://%s:%u/remote/',($Settings['Protocol'] == 'ssl')?'https':'http',$Settings['Address'],$Settings['Port']);
$client = new SoapClient(null, array('location' => $SoapLocation,
'uri' => $SoapUri,
'trace' => 1,
'exceptions' => 1));
# Открываем сессию на сервере ISPConfig
$session_id = $client->login($Settings['Login'], $Settings['Password']);
тоже можно в отдельную функцию.
из неё вернуть тока $session_id и всё...
Отправлено спустя 1 минуту 59 секунд:а вообще намана. мне нравится.
так что на критику забейте, если лень =)
P.S. если планируете в релиз включать, который в субботу эту будет, надо xml, надо либу в виде файла.
Убей их всех! Бог потом рассортирует...