Resolves #4739

app/models/product_drive.rb

@@ -69,6 +69,10 @@ def self.search_date_range(dates)
     @search_date_range = { start_date: dates[0], end_date: dates[1] }
   end
 
+  def can_delete?(user)
+    user.has_role?(Role::ORG_ADMIN, organization) && donations.empty?
+  end
+
   # quantities are FILTERED by date then SORTED by name
   #
   # @param date_range [Range]

app/views/product_drives/show.html.erb

@@ -90,7 +90,11 @@
           </p>
           <div class="card-footer clearfix">
             <%= edit_button_to edit_product_drive_path(@product_drive), { text: "Make a correction", size: "md" } %>
-            <%= delete_button_to product_drive_path(@product_drive), { confirm: "Are you sure you want to permanently remove this product drive?", size: "md" } if current_user.has_role?(Role::ORG_ADMIN, current_organization) %>
+            <% if @product_drive.can_delete?(current_user) %>
+              <%= delete_button_to product_drive_path(@product_drive),
+                                   confirm: "Are you sure you want to permanently remove this product drive?",
+                                   size: "md" %>
+            <% end %>
           </div>
           <!-- /.card-footer-->
         </div>

spec/system/product_drive_system_spec.rb

@@ -130,4 +130,34 @@
       expect(page).to have_content 'Endless drive'
     end
   end
+
+  context "when deleting a Product drive" do
+    let(:new_product_drive) { create(:product_drive, name: 'New Test', start_date: 3.weeks.ago, end_date: 3.weeks.from_now, organization: organization) }
+    let(:subject) { product_drive_path(new_product_drive.id) }
+
+    context "when user is a org_admin" do
+      before do
+        user.add_role(:org_admin, organization)
+      end
+
+      it "must delete" do
+        visit subject
+        click_on 'Delete'
+        expect(page).to have_content 'Product drive was successfully destroyed.'
+      end
+
+      it "must not delete if there are donations" do
+        create(:donation, product_drive: new_product_drive)
+        visit subject
+        expect(page).not_to have_button 'Delete'
+      end
+    end
+
+    context "when user is not a org_admin" do
+      it "must not delete" do
+        visit subject
+        expect(page).not_to have_button 'Delete'
+      end
+    end
+  end
 end