Merge pull request #4536 from tomhughes/trace-size-limit Add a limit on the number of points in a GPS trace
Add a limit on the number of points in a GPS trace
Reduce priority of multiple simulataneous trace imports
Move scheduling of trace import and destruction to the model
Use trace instead of gpx_id in queries This makes the queries easier to read.
Use hashes to define where..in sql queries This is preferable to using SQL statements.
Strip leading/trailing whitespace from comma-separated trace tags
Remove support for legacy trace files
Remove redundant presence validation on belongs_to There's no need for us to have it when rails does this for us.
Enable active_record.belongs_to_required_by_default This switches the logic so that belongs_to parent objects must exist by default, and marks the optional ones explicitly. This is reflected in the null/not_null status on the relevant db columns.
Specify inverse_of for trace tags and points Rails can't automatically detect bi-directional relationships when `:foreign_key` is present. Adding the `inverse_of` helps when validating traces and tags that haven't yet been saved.
Add a rake task to migrate trace data to ActiveStorage
Switch traces to use ActiveStorage
Fix new rubocop warning
Prefer keyword arguments when method has optional boolean arguments
Fix new rubocop warnings
Fix the Command Injection warnings from Brakeman
Fix new rubocop warnings
Require open3 before using it
Remove tempfile patch for trace data Effectively reverts c0d2ad40c30e5a0837b6012d7b9067d69ce41dd0 This patch is no longer required, since we only use send_data in combination with Tempfile.read and that all works fine.