can set minimum app version. api filters out errors with app-version …

…less than this amount. filtering only happens if both min_app_version and app-version are set.

app/controllers/notices_controller.rb

@@ -11,11 +11,17 @@ def create
     report = ErrorReport.new(notice_params)
 
     if report.valid?
-      report.generate_notice!
-      api_xml = report.notice.to_xml(:only => false, :methods => [:id]) do |xml|
-        xml.url locate_url(report.notice.id, :host => Errbit::Config.host)
+
+      if report.should_keep?
+        report.generate_notice!
+        api_xml = report.notice.to_xml(:only => false, :methods => [:id]) do |xml|
+         xml.url locate_url(report.notice.id, :host => Errbit::Config.host)
+        end
+        render :xml => api_xml
+      else
+        render :nothing => true
       end
-      render :xml => api_xml
+
     else
       render :text => "Your API key is unknown", :status => 422
     end

app/models/app.rb

@@ -9,6 +9,7 @@ class App
   field :bitbucket_repo
   field :asset_host
   field :repository_branch
+  field :min_app_version
   field :resolve_errs_on_deploy, :type => Boolean, :default => false
   field :notify_all_users, :type => Boolean, :default => false
   field :notify_on_errs, :type => Boolean, :default => true

app/models/error_report.rb

@@ -72,6 +72,30 @@ def valid?
     !!app
   end
 
+  def should_keep?
+
+    app_version = server_environment['app-version'] || ''
+
+      if self.app.min_app_version.present?
+
+        if app_version.length > 0
+
+          if Gem::Version.new(app_version) >= Gem::Version.new(self.app.min_app_version)
+            true
+          else
+            false
+          end
+
+        else
+          true
+        end
+
+      else
+        true
+      end
+
+  end
+
   private
 
   def fingerprint

app/views/apps/_fields.html.haml

@@ -22,7 +22,10 @@
   = f.label :asset_host
   %em Used to generate links for JavaScript errors
   = f.text_field :asset_host, :placeholder => "e.g. https://assets.example.com"
-
+%div
+  = f.label :min_app_version
+  %em Use with mobile apps to drop errors from old versions that have already been fixed
+  = f.text_field :min_app_version, :placeholder => "e.g. 2.0.1 from the Bundle Identifier on an iOS app"
 %fieldset
   %legend Notifications
   %div.checkbox

app/views/apps/show.html.haml

@@ -2,6 +2,9 @@
 - content_for :head do
   = auto_discovery_link_tag :atom, app_path(app, User.token_authentication_key => current_user.authentication_token, :format => "atom"), :title => t('.atom_title', :name => app.name, :host => request.host)
 - content_for :meta do
+  - if app.min_app_version.present?
+    %strong="Min App Version:"
+    = app.min_app_version
   %strong=t('.errors_caught')
   = app.problems.count
   %strong=t('.deploy_count')