From 6171fbdb0302cf524f1e17517b27f2a16294306b Mon Sep 17 00:00:00 2001
From: Alexandre Aubin <alex.aubin@mailoo.org>
Date: Wed, 30 Nov 2016 00:04:17 -0500
Subject: [PATCH] Keep track of backups with custom directory using symlinks

---
 locales/en.json        |  1 +
 src/yunohost/backup.py | 27 +++++++++++++++++++++++++--
 2 files changed, 26 insertions(+), 2 deletions(-)

diff --git a/locales/en.json b/locales/en.json
index efeb66e69..c50f3d3de 100644
--- a/locales/en.json
+++ b/locales/en.json
@@ -43,6 +43,7 @@
     "backup_action_required": "You must specify something to save",
     "backup_app_failed": "Unable to back up the app '{app:s}'",
     "backup_archive_app_not_found": "App '{app:s}' not found in the backup archive",
+    "backup_archive_broken_link": "Unable to access backup archive (broken link to {path:s})",
     "backup_archive_hook_not_exec": "Hook '{hook:s}' not executed in this backup",
     "backup_archive_name_exists": "The backup's archive name already exists",
     "backup_archive_name_unknown": "Unknown local backup archive named '{name:s}'",
diff --git a/src/yunohost/backup.py b/src/yunohost/backup.py
index 7d2e9d445..0e991e8fc 100644
--- a/src/yunohost/backup.py
+++ b/src/yunohost/backup.py
@@ -290,7 +290,7 @@ def backup_create(name=None, description=None, output_directory=None,
             raise MoulinetteError(errno.EIO,
                                   m18n.n('backup_archive_open_failed'))
 
-        # Add files to the arvhice
+        # Add files to the archive
         try:
             tar.add(tmp_dir, arcname='')
             tar.close()
@@ -300,10 +300,22 @@ def backup_create(name=None, description=None, output_directory=None,
             raise MoulinetteError(errno.EIO,
                                   m18n.n('backup_creation_failed'))
 
+        # FIXME : it looks weird that the "move info file" is not enabled if
+        # user activated "no_compress" ... or does it really means
+        # "dont_keep_track_of_this_backup_in_history" ?
+
         # Move info file
         shutil.move(tmp_dir + '/info.json',
                   '{:s}/{:s}.info.json'.format(archives_path, name))
 
+        # If backuped to a non-default location, keep a symlink of the archive
+        # to that location
+        if output_directory != archives_path:
+            link = "%s/%s.tar.gz" % (archives_path, name)
+            os.symlink(archive_file, link)
+
+
+
     # Clean temporary directory
     if tmp_dir != output_directory:
         _clean_tmp_dir()
@@ -604,10 +616,21 @@ def backup_info(name, with_details=False, human_readable=False):
 
     """
     archive_file = '%s/%s.tar.gz' % (archives_path, name)
-    if not os.path.isfile(archive_file):
+
+    # Check file exist (even if it's a broken symlink)
+    if not os.path.lexists(archive_file):
         raise MoulinetteError(errno.EIO,
             m18n.n('backup_archive_name_unknown', name=name))
 
+    # If symlink, retrieve the real path
+    if os.path.islink(archive_file) :
+        archive_file = os.path.realpath(archive_file)
+
+        # Raise exception if link is broken (e.g. on unmounted external storage)
+        if (not os.path.exists(archive_file)) :
+            raise MoulinetteError(errno.EIO,
+                m18n.n('backup_archive_broken_link', path=archive_file))
+
     info_file = "%s/%s.info.json" % (archives_path, name)
     try:
         with open(info_file) as f: