Skip to content

Commit 3aeea7e

Browse files
author
Till Brehm
committed
Merge branch '6438-php81-deprecation' into 'develop'
Fix php 8.1 deprecation warnings Closes #6438 See merge request ispconfig/ispconfig3!1682
2 parents a2f2eb3 + 27fea5f commit 3aeea7e

File tree

4 files changed

+11
-11
lines changed

4 files changed

+11
-11
lines changed

install/lib/installer_base.lib.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ private function install_acme() {
5252
}
5353

5454
public function update_acme() {
55-
$acme = explode("\n", shell_exec('which acme.sh /usr/local/ispconfig/server/scripts/acme.sh /root/.acme.sh/acme.sh 2> /dev/null'));
55+
$acme = explode("\n", (string)shell_exec('which acme.sh /usr/local/ispconfig/server/scripts/acme.sh /root/.acme.sh/acme.sh 2> /dev/null'));
5656
$acme = reset($acme);
5757
$val = 0;
5858

server/lib/classes/backup.inc.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1305,13 +1305,13 @@ protected static function backups_garbage_collection($server_id, $backup_type =
13051305
foreach ($domains as $rec) {
13061306
$domain_id = $rec['domain_id'];
13071307
$domain_backup_dir = $backup_dir . '/web' . $domain_id;
1308+
$web_path = $rec['document_root'];
1309+
$backup_download_dir = $web_path . '/backup';
13081310

13091311
// Remove backupdir symlink and create as directory instead
13101312
if (is_link($backup_download_dir) || !is_dir($backup_download_dir)) {
1311-
$web_path = $rec['document_root'];
13121313
$app->system->web_folder_protection($web_path, false);
13131314

1314-
$backup_download_dir = $web_path . '/backup';
13151315
if (is_link($backup_download_dir)) {
13161316
unlink($backup_download_dir);
13171317
}

server/lib/classes/cron.d/200-ftplogfiles.inc.php

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ function parse_ftp_log($line){
7070

7171
function add_ftp_traffic(&$traffic_array, $parsed_line)
7272
{
73-
if(is_array($traffic_array[$parsed_line['date']]) && array_key_exists($parsed_line['domain'], $traffic_array[$parsed_line['date']]))
73+
if(isset($traffic_array[$parsed_line['date']]) && is_array($traffic_array[$parsed_line['date']]) && array_key_exists($parsed_line['domain'], $traffic_array[$parsed_line['date']]))
7474
{
7575
$traffic_array[$parsed_line['date']][$parsed_line['domain']][$parsed_line['direction']] += $parsed_line['size'];
7676
}
@@ -89,13 +89,13 @@ function add_ftp_traffic(&$traffic_array, $parsed_line)
8989
while($line = fgets($fp))
9090
{
9191
$parsed_line = parse_ftp_log($line);
92+
if (!empty($parsed_line)) {
93+
$sql = "SELECT wd.domain FROM ftp_user AS fu INNER JOIN web_domain AS wd ON fu.parent_domain_id = wd.domain_id WHERE fu.username = ? ";
94+
$temp = $app->db->queryOneRecord($sql, $parsed_line['username'] );
95+
$parsed_line['domain'] = $temp['domain'];
96+
add_ftp_traffic($ftp_traffic, $parsed_line);
97+
}
9298

93-
$sql = "SELECT wd.domain FROM ftp_user AS fu INNER JOIN web_domain AS wd ON fu.parent_domain_id = wd.domain_id WHERE fu.username = ? ";
94-
$temp = $app->db->queryOneRecord($sql, $parsed_line['username'] );
95-
96-
$parsed_line['domain'] = $temp['domain'];
97-
98-
add_ftp_traffic($ftp_traffic, $parsed_line);
9999
}
100100

101101
fclose($fp);

server/plugins-available/bind_plugin.inc.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ function soa_update($event_name, $data) {
393393
//}
394394

395395
//* Delete old domain file, if domain name has been changed
396-
if($data['old']['origin'] != $data['new']['origin']) {
396+
if(!empty($data['old']['origin']) && $data['old']['origin'] != $data['new']['origin']) {
397397
$filename = $dns_config['bind_zonefiles_dir'].'/' . $this->zone_file_prefix() . str_replace("/", "_", substr($data['old']['origin'], 0, -1));
398398

399399
if(is_file($filename)) unlink($filename);

0 commit comments

Comments
 (0)