From 0908f3aad96f94447e5f5108e790f7d53eb05382 Mon Sep 17 00:00:00 2001 From: Franco Fichtner Date: Mon, 30 Jun 2025 12:22:30 +0200 Subject: [PATCH] sysutils/sftp-backup: replace getCurrentValue() use --- sysutils/sftp-backup/Makefile | 1 + .../src/opnsense/mvc/app/library/OPNsense/Backup/Sftp.php | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/sysutils/sftp-backup/Makefile b/sysutils/sftp-backup/Makefile index 0a740a766..448f13de1 100644 --- a/sysutils/sftp-backup/Makefile +++ b/sysutils/sftp-backup/Makefile @@ -1,5 +1,6 @@ PLUGIN_NAME= sftp-backup PLUGIN_VERSION= 1.1 +PLUGIN_REVISION= 1 PLUGIN_COMMENT= Backup configurations using SFTP PLUGIN_MAINTAINER= ad@opnsense.org PLUGIN_TIER= 2 diff --git a/sysutils/sftp-backup/src/opnsense/mvc/app/library/OPNsense/Backup/Sftp.php b/sysutils/sftp-backup/src/opnsense/mvc/app/library/OPNsense/Backup/Sftp.php index 0a81b101c..a255fc725 100644 --- a/sysutils/sftp-backup/src/opnsense/mvc/app/library/OPNsense/Backup/Sftp.php +++ b/sysutils/sftp-backup/src/opnsense/mvc/app/library/OPNsense/Backup/Sftp.php @@ -262,8 +262,8 @@ class Sftp extends Base implements IBackupProvider /* cleanup only if backup count is > 0*/ if ($this->model->backupcount->asFloat() > 0) { rsort($remote_backups); - if (count($remote_backups) > (int)$this->model->backupcount->getCurrentValue()) { - for ($i = $this->model->backupcount->getCurrentValue(); $i < count($remote_backups); $i++) { + if (count($remote_backups) > (int)$this->model->backupcount->getValue()) { + for ($i = $this->model->backupcount->getValue(); $i < count($remote_backups); $i++) { $this->del($remote_backups[$i]); } $remote_backups = $this->ls(sprintf('%s*.xml', $fileprefix));