Merge pull request #3436 from LacunaSoftware/fix-description-mismatch

Make description an optional field for backups

Duplicati/Server/Database/Connection.cs

@@ -518,7 +518,7 @@ namespace Duplicati.Server.Database
                            
                             return new object[] {
                                 n.Name,
-                                n.Description,
+                                n.Description == null ? "" : n.Description, // Description is optional but the column is set to NOT NULL, an additional check is welcome
                                 string.Join(",", n.Tags ?? new string[0]),
                                 n.TargetURL,
                                 update ? (object)item.ID : (object)n.DBPath 

Duplicati/Server/Database/Database schema/6. Add Description to Backup.sql

@@ -1 +1 @@
-ALTER TABLE "Backup" ADD COLUMN "Description" TEXT NULL;
+ALTER TABLE "Backup" ADD COLUMN "Description" TEXT NOT NULL DEFAULT '';

Duplicati/Server/webroot/ngax/scripts/controllers/EditBackupController.js

@@ -221,7 +221,6 @@ backupApp.controller('EditBackupController', function ($rootScope, $scope, $rout
             return;
         }
 
-
         if (encryptionEnabled) {
             if ($scope.PassphraseScore === '') {
                 DialogService.dialog(gettextCatalog.getString('Missing passphrase'), gettextCatalog.getString('You must enter a passphrase or disable encryption'));
@@ -487,6 +486,13 @@ backupApp.controller('EditBackupController', function ($rootScope, $scope, $rout
         var filters = $scope.Backup.Filters;
         $scope.Backup.Filters = [];
 
+        // If Description is anything other than a string, we are either creating a new
+        // backup or something went wrong when retrieving an existing one
+        // Either way we should set it to an empty string
+        if (typeof $scope.Backup.Description !== 'string') {
+            $scope.Backup.Description = '';
+        }
+
         $scope.Backup.Sources = $scope.Backup.Sources || [];
 
         for(var ix in filters)

Duplicati/Server/webroot/ngax/templates/addoredit.html

@@ -40,7 +40,7 @@
                         <input type="text" name="name" id="name" ng-model="Backup.Name" placeholder="{{'My Photos' | translate}}" />
                     </div>
                     <div class="input textarea">
-                        <label for="description" translate>Description</label>
+                        <label for="description" translate>Description (optional)</label>
                         <textarea name="description" id="description" ng-model="Backup.Description"></textarea>
                     </div>
                     <div class="input select" ng-hide="SystemInfo.EncryptionModules.length == 0">

Duplicati/Server/webroot/ngax/templates/home.html

@@ -13,10 +13,12 @@
                 </a>
 
                 <dl class="taskmenu" ng-show="$parent.TaskMenuID == item.Backup.ID">
-                    <dt translate>Description:</dt>
-                    <dd>
-                        {{item.Backup.Description}}
-                    </dd>
+                    <span ng-show="item.Backup.Description">
+                        <dt translate>Description:</dt>
+                        <dd>
+                            {{item.Backup.Description}}
+                        </dd>
+                    </span>
 
                     <dt translate>Operations:</dt>
                     <dd>