From 2b035459fd6d741676470d6c237e8a04ba9eda9b Mon Sep 17 00:00:00 2001
From: Rob H <oeoeaio@gmail.com>
Date: Thu, 27 Mar 2014 15:13:00 +1100
Subject: [PATCH] BPE: Adding blank filters alerts the user

---
 app/assets/javascripts/admin/bulk_product_update.js.coffee | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/app/assets/javascripts/admin/bulk_product_update.js.coffee b/app/assets/javascripts/admin/bulk_product_update.js.coffee
index 7d5ab0a45f..30d3a18f09 100644
--- a/app/assets/javascripts/admin/bulk_product_update.js.coffee
+++ b/app/assets/javascripts/admin/bulk_product_update.js.coffee
@@ -237,6 +237,8 @@ productEditModule.controller "AdminProductEditCtrl", [
           else if confirm("'#{filter.predicate.name}' filter already exists on column '#{filter.property.name}'. Replace it?")
             $scope.currentFilters[existingfilterIndex] = filter
             $scope.fetchProducts()
+      else
+        alert("Please ensure all filter fields are filled in before adding a filter.")
 
     $scope.removeFilter = (filter) ->
       index = $scope.currentFilters.indexOf(filter)