From ae36ff59b4a169bf1b992baadd5720507da33f29 Mon Sep 17 00:00:00 2001
From: Justin Edmund <justin@jedmund.com>
Date: Wed, 30 Nov 2022 05:17:40 -0800
Subject: [PATCH] Properly gate group conditions

The default group ("All Skills") is technically -1, not null
---
 app/controllers/api/v1/search_controller.rb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/app/controllers/api/v1/search_controller.rb b/app/controllers/api/v1/search_controller.rb
index dcf4013..f8b9f27 100644
--- a/app/controllers/api/v1/search_controller.rb
+++ b/app/controllers/api/v1/search_controller.rb
@@ -88,7 +88,7 @@ class Api::V1::SearchController < Api::V1::ApiController
         if search_params[:filters].present? && search_params[:filters]["group"].present?
             group = search_params[:filters]["group"].to_i
 
-            if (group < 4)
+            if (group >= 0 && group < 4)
                 conditions[:color] = group
                 conditions[:emp] = false
                 conditions[:base] = false