Merge branch 'issue-28_store-lat/lng-as-strings'

Author: matievisthekat

app/controllers/api/v1/hike_events_controller.rb

@@ -1,6 +1,8 @@
 class Api::V1::HikeEventsController < ApplicationController
   before_action :require_token, only: [:current_user, :create, :destroy, :update, :join, :leave, :confirm]
 
+  include StrUtil
+
   def current_user
     render json: @user.hike_events
   end
@@ -110,8 +112,8 @@ def safe_params (p)
       return
     end
 
-    lat = p[:lat].to_f
-    lng = p[:lng].to_f
+    lat = StrUtil::is_valid_float?(p[:lat])
+    lng = StrUtil::is_valid_float?(p[:lng])
 
     if !lat || !lng
       render json: { error: 'Invalid \'lat\' and \'lng\' params. They failed to parse as floating point numbers' }, status: 400 and return
@@ -126,7 +128,7 @@ def safe_params (p)
       render json: { error: 'Invalid \'difficulty\' param. It failed to parse as an integer or it was out of range (1 <= x <= 10).' }, status: 400 and return
     end
 
-    return { title: p[:title], description: p[:description], duration: duration, lat: lat, lng: lng, difficulty: difficulty }
+    return { title: p[:title], description: p[:description], duration: duration, lat: lat.to_s, lng: lng.to_s, difficulty: difficulty }
   end
 
   def create_params

config/initializers/util.rb

@@ -1,2 +1,3 @@
 require 'rsa_util.rb'
-require 'jwt_util.rb'
+require 'str_util.rb'
+require 'jwt_util.rb'

db/migrate/20220610104307_change_lat_lng_cols_to_string.rb

@@ -0,0 +1,6 @@
+class ChangeLatLngColsToString < ActiveRecord::Migration[7.0]
+  def change
+    change_column :hike_events, :lat, :string
+    change_column :hike_events, :lng, :string
+  end
+end

db/schema.rb

@@ -0,0 +1,5 @@
+module StrUtil
+  def self::is_valid_float? (str)
+    !!Float(str) rescue false
+  end
+end