Filters validation added

app/controllers/dmsf_controller.rb

@@ -69,18 +69,30 @@ class DmsfController < ApplicationController
       render_404
       return
     end
-    respond_to do |format|
+    if @query.valid?
-      format.html {
+      respond_to do |format|
-        @dmsf_count = @query.dmsf_count
+        format.html {
-        @dmsf_pages = Paginator.new @dmsf_count, per_page_option, params['page']
+          @dmsf_count = @query.dmsf_count
-        render layout: !request.xhr?
+          @dmsf_pages = Paginator.new @dmsf_count, per_page_option, params['page']
-      }
+          render layout: !request.xhr?
-      format.api {
+        }
-        @offset, @limit = api_offset_and_limit
+        format.api {
-      }
+          @offset, @limit = api_offset_and_limit
-      format.csv  {
+        }
-        send_data query_to_csv(@query.dmsf_nodes, @query), type: 'text/csv; header=present', filename: 'dmsf.csv'
+        format.csv  {
-      }
+          send_data query_to_csv(@query.dmsf_nodes, @query), type: 'text/csv; header=present', filename: 'dmsf.csv'
+        }
+      end
+    else
+      respond_to do |format|
+        format.html {
+          @dmsf_count = 0
+          @dmsf_pages = Paginator.new @dmsf_count, per_page_option, params['page']
+          render layout: !request.xhr?
+        }
+        format.any(:atom, :csv, :pdf) { head 422 }
+        format.api { render_validation_errors(@query) }
+      end
     end
   end
 

app/models/dmsf_query.rb

@@ -132,6 +132,12 @@ class DmsfQuery < Query
     @statement
   end
 
+  def validate_query_filters
+    # Skip validation for empty title (default filter)
+    filters.delete('title')
+    super
+  end
+
   ######################################################################################################################
   # New
 

app/views/dmsf/show.html.erb

@@ -75,8 +75,14 @@
   <%= hidden_field_tag('folder_id', @folder.id) if @folder %>
   <%= render partial: 'queries/query_form' %>
 <% end %>
-<%= render partial: 'query_list', locals: { query: @query, dmsf_pages: @dmsf_pages } %>
+<% if @query.valid? %>
-<span class="pagination"><%= pagination_links_full @dmsf_pages, @dmsf_count %></span>
+  <% if @dmsf_count == 0 %>
+    <p class="nodata"><%= l(:label_no_data) %></p>
+  <% else %>
+    <%= render partial: 'query_list', locals: { query: @query, dmsf_pages: @dmsf_pages } %>
+    <span class="pagination"><%= pagination_links_full @dmsf_pages, @dmsf_count %></span>
+  <% end %>
+<% end %>
 
 <%= context_menu %>