Merge pull request #10 from mahmoudakl/master

[nextcloud-backup-restore.git] / NextcloudBackup.sh

diff --git a/NextcloudBackup.sh b/NextcloudBackup.sh
index 980d15781d88c10707b2f91d19c7fc2d117a4a65..6f09159f9d43e13fdcbcc7d28a13ea4cfe842dd6 100644 (file)
--- a/NextcloudBackup.sh
+++ b/NextcloudBackup.sh
@@ -17,14 +17,14 @@
 currentDate=$(date +"%Y%m%d_%H%M%S")
 # TODO: The directory where you store the Nextcloud backups
 backupMainDir="/mnt/Share/NextcloudBackups/"
-# The actual directory of the current backup - this is is subdirectory of the main directory above with a timestamp
+# The actual directory of the current backup - this is a subdirectory of the main directory above with a timestamp
 backupdir="${backupMainDir}/${currentDate}/"
 # TODO: The directory of your Nextcloud installation (this is a directory under your web root)
 nextcloudFileDir="/var/www/nextcloud"
 # TODO: The directory of your Nextcloud data directory (outside the Nextcloud file directory)
 # If your data directory is located under Nextcloud's file directory (somewhere in the web root), the data directory should not be a separate part of the backup
 nextcloudDataDir="/var/nextcloud_data"
-# TODO: The service name of the webserver. Used to start/stop webserver (e.g. 'service <webserverServiceName> start')
+# TODO: The service name of the web server. Used to start/stop web server (e.g. 'service <webserverServiceName> start')
 webserverServiceName="nginx"
 # TODO: Your Nextcloud database name
 nextcloudDatabase="nextcloud_db"
@@ -32,7 +32,7 @@ nextcloudDatabase="nextcloud_db"
 dbUser="nextcloud_db_user"
 # TODO: The password of the Nextcloud database user
 dbPassword="mYpAsSw0rd"
-# TODO: Your webserver user
+# TODO: Your web server user
 webserverUser="www-data"
 # TODO: The maximum number of backups to keep (when set to 0, all backups are kept)
 maxNrOfBackups=0
@@ -77,9 +77,9 @@ echo "Done"
 echo
 
 #
-# Stop webserver
+# Stop web server
 #
-echo "Stopping webserver..."
+echo "Stopping web server..."
 service "${webserverServiceName}" stop
 echo "Done"
 echo
@@ -106,9 +106,9 @@ echo "Done"
 echo
 
 #
-# Start webserver
+# Start web server
 #
-echo "Starting webserver..."
+echo "Starting web server..."
 service "${webserverServiceName}" start
 echo "Done"
 echo
@@ -126,11 +126,11 @@ echo
 #
 # Delete old backups
 #
-if [ ${maxNrOfBackups} != 0 ]
+if (( ${maxNrOfBackups} != 0 ))
 then   
        nrOfBackups=$(ls -l ${backupMainDir} | grep -c ^d)
        
-       if [  ${maxNrOfBackups} > ${nrOfBackups} ]
+       if (( ${nrOfBackups} > ${maxNrOfBackups} ))
        then
                echo "Removing old backups..."
                ls -t ${backupMainDir} | tail -$(( nrOfBackups - maxNrOfBackups )) | while read dirToRemove; do