version bump

addon.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="UTF-8" standalone="yes"?>
 <addon id="script.xbmcbackup"
-    name="Backup" version="1.6.5" provider-name="robweber">
+    name="Backup" version="1.6.6" provider-name="robweber">
   <requires>
     <import addon="xbmc.python" version="3.0.0"/>
     <import addon="script.module.dateutil" version="2.8.0" />
@@ -89,10 +89,8 @@
 		<screenshot>resources/images/screenshot3.jpg</screenshot>
 		<screenshot>resources/images/screenshot4.jpg</screenshot>
     </assets>
-    <news>Version 1.6.5
- - updated to new settings format with levels
- - added ability to change path where temp zip file is built
- - fixed issues with xbmcgui Dialogs
+    <news>Version 1.6.6
+ - fixed issue with backup rotations not working properly
 </news>
   </extension>
 </addon>

changelog.md

@@ -4,6 +4,13 @@ All notable changes to this project will be documented in this file.
 
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 
+## [Version 1.6.6](https://github.com/robweber/xbmcbackup/compare/matrix-1.6.5...robweber:matrix-1.6.6)
+
+### Fixed
+
+- error when typing the remote path, ```listBackups()``` function was not working if final slash not included in typed directory path name.
+- added ```force=True``` flag to the ```rmdir()``` function. Fixes issue with directories being removed when not empty
+
 ## [Version 1.6.5](https://github.com/robweber/xbmcbackup/compare/matrix-1.6.4...robweber:matrix-1.6.5) - 2021-03-06
 
 ### Added

resources/lib/backup.py

@@ -57,16 +57,15 @@ class XbmcBackup:
 
     def configureRemote(self):
         if(utils.getSetting('remote_selection') == '1'):
-            self.remote_base_path = utils.getSetting('remote_path_2')
             self.remote_vfs = XBMCFileSystem(utils.getSetting('remote_path_2'))
             utils.setSetting("remote_path", "")
         elif(utils.getSetting('remote_selection') == '0'):
-            self.remote_base_path = utils.getSetting('remote_path')
             self.remote_vfs = XBMCFileSystem(utils.getSetting("remote_path"))
         elif(utils.getSetting('remote_selection') == '2'):
-            self.remote_base_path = "/"
             self.remote_vfs = DropboxFileSystem("/")
 
+        self.remote_base_path = self.remote_vfs.root_path
+
     def remoteConfigured(self):
         result = True
 

resources/lib/vfs.py

@@ -73,7 +73,7 @@ class XBMCFileSystem(Vfs):
         return xbmcvfs.copy(xbmcvfs.translatePath(source), xbmcvfs.translatePath(dest))
 
     def rmdir(self, directory):
-        return xbmcvfs.rmdir(directory)
+        return xbmcvfs.rmdir(directory, force=True)  # use force=True to make sure it works recursively
 
     def rmfile(self, aFile):
         return xbmcvfs.delete(aFile)