Skip to content

Commit ac169bb

Browse files
committed
Reduce line length and complexity.
1 parent 1b3a56b commit ac169bb

File tree

1 file changed

+10
-4
lines changed

1 file changed

+10
-4
lines changed

server/lib/classes/letsencrypt.inc.php

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -382,11 +382,17 @@ public function request_certificates($data, $server_type = 'apache') {
382382
$cli_domain_arg = '';
383383
$subdomains = null;
384384
$aliasdomains = null;
385+
$powerdns = (isset($conf['powerdns']['installed']) && $conf['powerdns']['installed'] == false);
385386

386387
//* be sure to have good domain
387-
if(substr($domain,0,4) != 'www.' && ($data['new']['subdomain'] == "www" || ($data['new']['subdomain'] == "*" && (!$use_acme || $global_sites_config['acme_dns_user'] == '' && (!isset($conf['powerdns']['installed']) || isset($conf['powerdns']['installed']) && $conf['powerdns']['installed'] == false))))) {
388+
if (substr($domain, 0, 4) != 'www.'
389+
&& ($data['new']['subdomain'] == "www"
390+
|| ($data['new']['subdomain'] == "*"
391+
&& (!$powerdns && !$use_acme || $global_sites_config['acme_dns_user'] == '')))) {
388392
$temp_domains[] = "www." . $domain;
389-
} elseif ($data['new']['subdomain'] == "*" && ($use_acme && $global_sites_config['acme_dns_user'] != '' && (!isset($conf['powerdns']['installed']) || isset($conf['powerdns']['installed']) && $conf['powerdns']['installed'] == false))) {
393+
}
394+
elseif ($data['new']['subdomain'] == "*"
395+
&& (!$powerdns && $use_acme && $global_sites_config['acme_dns_user'] != '')) {
390396
$temp_domains[] = "*." . $domain;
391397
}
392398

@@ -403,9 +409,9 @@ public function request_certificates($data, $server_type = 'apache') {
403409
if(is_array($aliasdomains)) {
404410
foreach($aliasdomains as $aliasdomain) {
405411
$temp_domains[] = $aliasdomain['domain'];
406-
if (isset($aliasdomain['subdomain']) && substr($aliasdomain['domain'],0,4) != 'www.' && ($aliasdomain['domain']['subdomain'] == "www" || ($aliasdomain['domain']['subdomain'] == "*" && (!$use_acme || $global_sites_config['acme_dns_user'] == '' && (!isset($conf['powerdns']['installed']) || isset($conf['powerdns']['installed']) && $conf['powerdns']['installed'] == false))))) {
412+
if (isset($aliasdomain['subdomain']) && substr($aliasdomain['domain'],0,4) != 'www.' && ($aliasdomain['domain']['subdomain'] == "www" || ($aliasdomain['domain']['subdomain'] == "*" && (!$use_acme || $global_sites_config['acme_dns_user'] == '' && !$powerdns)))) {
407413
$temp_domains[] = "www." . $aliasdomain['domain'];
408-
} elseif ($aliasdomain['domain']['subdomain'] == "*" && ($use_acme && $global_sites_config['acme_dns_user'] != '' && (!isset($conf['powerdns']['installed']) || isset($conf['powerdns']['installed']) && $conf['powerdns']['installed'] == false))) {
414+
} elseif ($aliasdomain['domain']['subdomain'] == "*" && ($use_acme && $global_sites_config['acme_dns_user'] != '' && !$powerdns)) {
409415
$temp_domains[] = "*." . $aliasdomain['domain'];
410416
}
411417
}

0 commit comments

Comments
 (0)