From: Patrick Canterino <patrick@patrick-canterino.de>
Date: Fri, 14 Jan 2022 19:18:10 +0000 (+0100)
Subject: Created a general function for rotating files
X-Git-Tag: 0.2~2
X-Git-Url: https://git.p6c8.net/psmysqlbackup.git/commitdiff_plain/dc12c9e1658cd18f9dee590b5b47a195b8620060?hp=f6f326704b707f884efceaba1b3a5d0ea31c430e

Created a general function for rotating files

We will use this file later on for rotating logfiles
---

diff --git a/psmysqlbackup.ps1 b/psmysqlbackup.ps1
index 31cfd79..bbfc075 100644
--- a/psmysqlbackup.ps1
+++ b/psmysqlbackup.ps1
@@ -176,23 +176,34 @@ function Create-Backup([String]$database, [String]$target) {
         throw "mysqldump exited with Exit code $LastExitCode"
     }
 }
-
-function Rotate-Backups($backupDir) {
-    if($configRotate -le 0) {
+function Invoke-FileRotation {
+    Param (
+        $Dir,
+        $MaxFiles,
+        [Parameter(Mandatory=$false)]
+        $Pattern,
+        [Parameter(Mandatory=$false)]
+        $LogFile
+    )
+
+    if($MaxFiles -le 0) {
         return
     }
-    
-    $keepBackupsCount = $configRotate
 
-    Get-ChildItem $backupDir -File | Where-Object {($_.Name -match "^backup-.+-\d{8,}-\d{6}\.sql$")} | Sort-Object -Descending |
+    $keepFilesCount = $MaxFiles
+
+    Get-ChildItem $Dir -File | Where-Object {($null -eq $Pattern -or $_.Name -match $Pattern)} | Sort-Object -Descending |
     Foreach-Object {
-        if($keepBackupsCount -ge 0) {
-            $keepBackupsCount--
+        if($keepFilesCount -ge 0) {
+            $keepFilesCount--
         }
 
-        if($keepBackupsCount -eq -1) {
-            Write-Output "Deleting backup $($_.FullName)"
-            #Write-Log "Deleting backup $($_.FullName)"
+        if($keepFilesCount -eq -1) {
+            Write-Output "Deleting file $($_.FullName)"
+            
+            if($null -ne $LogFile) {
+                Write-Log "Deleting file $($_.FullName)" -Path $LogFile
+            }
 
             Remove-Item -Force $_.FullName
         }
@@ -282,7 +293,7 @@ foreach($d in $databasesToBackup) {
     
     try {
         Create-Backup $d $databaseBackupFile
-        Rotate-Backups $databaseBackupDir
+        Invoke-FileRotation -Dir $databaseBackupDir -MaxFiles $configRotate -Pattern "^backup-.+-\d{8,}-\d{6}\.sql$" -LogFile $logFile
     }
     catch {
         Write-Log "Could not backup database $d to $databaseBackupFile" -Path $logFile -Level Error