Improve comments and minor bugfix

Author: Max Burnette

app/api/Datasets.scala

@@ -2065,6 +2065,10 @@ class  Datasets @Inject()(
     * @param dataset dataset from which to get teh files
     * @param chunkSize chunk size in memory in which to buffer the stream
     * @param compression java built in compression value. Use 0 for no compression.
+    * @param bagit whether or not to include bagit structures in zip
+    * @param user an optional user to include in metadata
+    * @param fileIDs a list of UUIDs of files in the dataset to include (i.e. marked file downloads)
+    * @param folderId a folder UUID in the dataset to include (i.e. folder download)
     * @return Enumerator to produce array of bytes from a zipped stream containing the bytes of each file
     *         in the dataset
     */
@@ -2440,6 +2444,7 @@ class  Datasets @Inject()(
     }
   }
 
+  // Takes dataset ID and a comma-separated string of file UUIDs in the dataset and streams just those files as a zip
   def downloadPartial(id: UUID, fileList: String) = PermissionAction(Permission.DownloadFiles, Some(ResourceRef(ResourceRef.dataset, id))) { implicit request =>
     implicit val user = request.user
     datasets.get(id) match {
@@ -2479,7 +2484,7 @@ class  Datasets @Inject()(
             // Use a 1MB in memory byte array
             Ok.chunked(enumeratorFromDataset(dataset,1024*1024, -1, bagit, user, None, Some(folderId))).withHeaders(
               CONTENT_TYPE -> "application/zip",
-              CONTENT_DISPOSITION -> (FileUtils.encodeAttachment(dataset.name+ " ( "+fo.name+" Folder).zip", request.headers.get("user-agent").getOrElse("")))
+              CONTENT_DISPOSITION -> (FileUtils.encodeAttachment(dataset.name+ " ("+fo.name+" Folder).zip", request.headers.get("user-agent").getOrElse("")))
             )
           }
           case None => NotFound

app/views/datasets/filesAndFolders.scala.html

@@ -85,10 +85,10 @@
             $('#selected-count-box').html('<div class="dropdown"><a href="#" class="dropdown-toggle" ' +
                     'data-toggle="dropdown" aria-expanded="false"> Marked Files (<b>'+fileUUIDs.length+'</b>)' +
                     '<b class="caret"></b></a><ul class="dropdown-menu">' +
-                    // TODO: Need better solution than cookies to support 'Show Only Marked' request
-                    //'<li><a onclick="showOnlyMarked();"><span class="glyphicon glyphicon-ok"/>&nbsp;Show Only Marked</a></li>' +
-                    //'<li><a onclick="showAllFiles();"><span class="glyphicon glyphicon-ok"/>&nbsp;Show All Files</a></li>' +
-                    // TODO: Support for downloading by folder and marked list will be implemented in the future
+                    @* TODO: Need better solution than cookies to support 'Show Only Marked' request
+                    '<li><a onclick="showOnlyMarked();"><span class="glyphicon glyphicon-ok"/>&nbsp;Show Only Marked</a></li>' +
+                    '<li><a onclick="showAllFiles();"><span class="glyphicon glyphicon-ok"/>&nbsp;Show All Files</a></li>' +
+                    *@
                     '<li><a onclick="downloadMarked();"><span class="glyphicon glyphicon-download-alt"/>&nbsp;Download All</a></li>' +
                     '<li><a onclick="confirmTagMarked();"><span class="glyphicon glyphicon-tag"/>&nbsp;Tag All</a></li>' +
                     '<li><a onclick="confirmDeleteMarked();"><span class="glyphicon glyphicon-trash"/>&nbsp;Delete All</a></li>' +
@@ -136,7 +136,6 @@
     }
 
     function deleteAllMarked() {
-        // TODO: Would be nice to have a bulk endpoint for this eventually that takes a string of comma-separated UUIDs
         var selected = $.cookie('[email protected]');
         if (selected) {
             selected = selected.split(',');
@@ -210,7 +209,6 @@ <h4 class="modal-title">Tag All Marked Files</h4>
     }
 
     function tagMarked() {
-        // TODO: Would be nice to have a bulk endpoint for this eventually that takes a string of comma-separated UUIDs
         var tag = $('#bulk-tag-selector')[0].value;
         if (tag.length > 0) {
             var selected = $.cookie('[email protected]');
@@ -265,7 +263,6 @@ <h4 class="modal-title">Tag All Marked Files</h4>
     }
 
     function addFileToMarked(file_id) {
-        // TODO: Use Play framework cookies instead of jquery flavor for better security
         var selected = $.cookie('[email protected]');
         if (selected) {
             if (selected.indexOf(file_id) == -1)