Merge pull request #6553 from pulibrary/user_download

Add downloadable to bulk edit.

app/jobs/bulk_update_job.rb

@@ -30,7 +30,8 @@ def self.supported_attributes
       :refresh_remote_metadata,
       :rights_statement,
       :visibility,
-      :pdf_type
+      :pdf_type,
+      :downloadable
     ]
   end
 

app/views/bulk_edit/resources_edit.html.erb

@@ -78,6 +78,14 @@
               input_html: { class: 'form-control pdf-type' },
               required: false %>
         </div>
+        <div class="row">
+          <div class="col-sm-2"><%= label_tag "downloadable", "User Downloads" %></div>
+          <%= select_tag :downloadable,
+              options_from_collection_for_select(ControlledVocabulary.for(:downloadable).all, "value", "label"),
+              include_blank: "[Do not update]",
+              input_html: { class: 'form-control downloadable' },
+              required: false %>
+        </div>
         <div class="row">
           <div class="col-sm-2"><%= label_tag "embargo_date", "Embargo Date" %></div>
           <%= select_tag :embargo_date_action,

spec/features/bulk_edit_spec.rb

@@ -147,6 +147,7 @@
       page.check("mark_complete")
       page.select collection2.title.first, from: "append_collection_ids", visible: false
       page.select "No PDF", from: "pdf_type", visible: false
+      page.select "None", from: "downloadable", visible: false
       accept_alert do
         click_button("Apply Edits")
       end
@@ -156,6 +157,7 @@
       expect(updated.state).to eq ["complete"]
       expect(updated.member_of_collection_ids).to eq [collection.id, collection2.id]
       expect(updated.pdf_type).to eq ["none"]
+      expect(updated.downloadable).to eq ["none"]
     end
 
     it "doesn't add a collection if one isn't picked" do
@@ -173,6 +175,8 @@
       expect(updated.member_of_collection_ids).to eq [collection.id]
       # Ensure PDF type doesn't change.
       expect(updated.pdf_type).to eq ["color"]
+      # Ensure downloadable doesn't change
+      expect(updated.downloadable).to eq ["public"]
     end
 
     context "with linked collections" do