Fix rubocop issues, update app version and changelog for v1.2.0 release

Author: Freika

.app_version

@@ -1 +1 @@
-1.1.0
+1.2.0

CHANGELOG.md

@@ -4,7 +4,7 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
-## [1.1.1] - Unreleased
+## [1.2.0] - 2026-02-15
 
 ### Changed
 
@@ -22,7 +22,6 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 - Scratch map layer is now working again on Map v2.
 - Colored routes on Map v2 are now working correctly. Zoom in closer to see colored segments. #2254
-- Frontend was sped by optimizing loaded assets and lazy-loading parts of the UI.
 - Live mode on Map v2 is now working again.
 
 ## [1.1.0] - 2026-02-08

app/jobs/points/create_job.rb

@@ -7,11 +7,13 @@ def perform(params, user_id)
     data = Points::Params.new(params, user_id).call
 
     data.each_slice(1000) do |location_batch|
+      # rubocop:disable Rails/SkipsModelValidations
       Point.upsert_all(
         location_batch,
         unique_by: %i[lonlat timestamp user_id],
         returning: false
       )
+      # rubocop:enable Rails/SkipsModelValidations
     end
   end
 end

app/services/overland/points_creator.rb

@@ -35,11 +35,13 @@ def upsert_points(locations)
     created_points = []
 
     locations.each_slice(1000) do |batch|
+      # rubocop:disable Rails/SkipsModelValidations
       result = Point.upsert_all(
         batch,
         unique_by: %i[lonlat timestamp user_id],
         returning: Arel.sql(RETURNING_COLUMNS)
       )
+      # rubocop:enable Rails/SkipsModelValidations
       created_points.concat(result) if result
     end
 

app/services/own_tracks/point_creator.rb

@@ -33,11 +33,13 @@ def upsert_points(locations)
     created_points = []
 
     locations.each_slice(1000) do |batch|
+      # rubocop:disable Rails/SkipsModelValidations
       result = Point.upsert_all(
         batch,
         unique_by: %i[lonlat timestamp user_id],
         returning: Arel.sql(RETURNING_COLUMNS)
       )
+      # rubocop:enable Rails/SkipsModelValidations
       created_points.concat(result) if result
     end
 

app/services/reverse_geocoding/places/fetch_data.rb

@@ -153,7 +153,9 @@ def save_places(places_to_create, places_to_update)
         updated_at: Time.current
       }
     end
+    # rubocop:disable Rails/SkipsModelValidations
     Place.upsert_all(update_attributes, unique_by: :id)
+    # rubocop:enable Rails/SkipsModelValidations
   end
 
   def build_point_coordinates(coordinates)

app/services/users/import_data/areas.rb

@@ -16,7 +16,7 @@ def call
     valid_areas = filter_and_prepare_areas
 
     if valid_areas.empty?
-      Rails.logger.info "Areas import completed. Created: 0"
+      Rails.logger.info 'Areas import completed. Created: 0'
       return 0
     end
 
@@ -53,9 +53,7 @@ def filter_and_prepare_areas
       valid_areas << prepared_attributes if prepared_attributes
     end
 
-    if skipped_count > 0
-      Rails.logger.warn "Skipped #{skipped_count} areas with invalid or missing required data"
-    end
+    Rails.logger.warn "Skipped #{skipped_count} areas with invalid or missing required data" if skipped_count > 0
 
     valid_areas
   end
@@ -84,7 +82,7 @@ def filter_existing_areas(areas)
       existing_areas_lookup[key] = true
     end
 
-    filtered_areas = areas.reject do |area|
+    areas.reject do |area|
       key = [area[:name], area[:latitude].to_f, area[:longitude].to_f]
       if existing_areas_lookup[key]
         Rails.logger.debug "Area already exists: #{area[:name]}"
@@ -93,31 +91,28 @@ def filter_existing_areas(areas)
         false
       end
     end
-
-    filtered_areas
   end
 
   def bulk_import_areas(areas)
     total_created = 0
 
     areas.each_slice(BATCH_SIZE) do |batch|
-      begin
-        result = Area.upsert_all(
-          batch,
-          returning: %w[id],
-          on_duplicate: :skip
-        )
-
-        batch_created = result.count
-        total_created += batch_created
-
-        Rails.logger.debug "Processed batch of #{batch.size} areas, created #{batch_created}, total created: #{total_created}"
-
-      rescue StandardError => e
-        Rails.logger.error "Failed to process area batch: #{e.message}"
-        Rails.logger.error "Batch size: #{batch.size}"
-        Rails.logger.error "Backtrace: #{e.backtrace.first(3).join('\n')}"
-      end
+      # rubocop:disable Rails/SkipsModelValidations
+      result = Area.upsert_all(
+        batch,
+        returning: %w[id],
+        on_duplicate: :skip
+      )
+      # rubocop:enable Rails/SkipsModelValidations
+
+      batch_created = result.count
+      total_created += batch_created
+
+      Rails.logger.debug "Processed batch of #{batch.size} areas, created #{batch_created}, total created: #{total_created}"
+    rescue StandardError => e
+      Rails.logger.error "Failed to process area batch: #{e.message}"
+      Rails.logger.error "Batch size: #{batch.size}"
+      Rails.logger.error "Backtrace: #{e.backtrace.first(3).join('\n')}"
     end
 
     total_created

app/services/users/import_data/notifications.rb

@@ -64,9 +64,7 @@ def prepare_notification_attributes(notification_data)
 
     attributes['user_id'] = user.id
 
-    unless attributes['created_at'].present?
-      attributes['created_at'] = Time.current
-    end
+    attributes['created_at'] = Time.current unless attributes['created_at'].present?
 
     attributes['updated_at'] = Time.current
 
@@ -90,7 +88,7 @@ def filter_existing_notifications(notifications)
       existing_notifications_lookup[exact_key] = true
     end
 
-    filtered_notifications = notifications.reject do |notification|
+    notifications.reject do |notification|
       title = notification[:title]&.strip
       content = notification[:content]&.strip
 
@@ -104,8 +102,6 @@ def filter_existing_notifications(notifications)
         false
       end
     end
-
-    filtered_notifications
   end
 
   def normalize_timestamp(timestamp)
@@ -124,23 +120,22 @@ def bulk_import_notifications(notifications)
     total_created = 0
 
     notifications.each_slice(BATCH_SIZE) do |batch|
-      begin
-        result = Notification.upsert_all(
-          batch,
-          returning: %w[id],
-          on_duplicate: :skip
-        )
-
-        batch_created = result.count
-        total_created += batch_created
-
-        Rails.logger.debug "Processed batch of #{batch.size} notifications, created #{batch_created}, total created: #{total_created}"
-
-      rescue StandardError => e
-        Rails.logger.error "Failed to process notification batch: #{e.message}"
-        Rails.logger.error "Batch size: #{batch.size}"
-        Rails.logger.error "Backtrace: #{e.backtrace.first(3).join('\n')}"
-      end
+      # rubocop:disable Rails/SkipsModelValidations
+      result = Notification.upsert_all(
+        batch,
+        returning: %w[id],
+        on_duplicate: :skip
+      )
+      # rubocop:enable Rails/SkipsModelValidations
+
+      batch_created = result.count
+      total_created += batch_created
+
+      Rails.logger.debug "Processed batch of #{batch.size} notifications, created #{batch_created}, total created: #{total_created}"
+    rescue StandardError => e
+      Rails.logger.error "Failed to process notification batch: #{e.message}"
+      Rails.logger.error "Batch size: #{batch.size}"
+      Rails.logger.error "Backtrace: #{e.backtrace.first(3).join('\n')}"
     end
 
     total_created

app/services/users/import_data/points.rb

@@ -86,12 +86,14 @@ def flush_batch
     normalized_batch = normalize_point_keys(@buffer)
 
     begin
+      # rubocop:disable Rails/SkipsModelValidations
       result = Point.upsert_all(
         normalized_batch,
         unique_by: %i[lonlat timestamp user_id],
         returning: %w[id],
         on_duplicate: :skip
       )
+      # rubocop:enable Rails/SkipsModelValidations
 
       batch_created = result&.count.to_i
       @total_created += batch_created

app/services/users/import_data/stats.rb

@@ -16,7 +16,7 @@ def call
     valid_stats = filter_and_prepare_stats
 
     if valid_stats.empty?
-      Rails.logger.info "Stats import completed. Created: 0"
+      Rails.logger.info 'Stats import completed. Created: 0'
       return 0
     end
 
@@ -52,9 +52,7 @@ def filter_and_prepare_stats
       valid_stats << prepared_attributes if prepared_attributes
     end
 
-    if skipped_count > 0
-      Rails.logger.warn "Skipped #{skipped_count} stats with invalid or missing required data"
-    end
+    Rails.logger.warn "Skipped #{skipped_count} stats with invalid or missing required data" if skipped_count > 0
 
     valid_stats
   end
@@ -83,7 +81,7 @@ def filter_existing_stats(stats)
       existing_stats_lookup[key] = true
     end
 
-    filtered_stats = stats.reject do |stat|
+    stats.reject do |stat|
       key = [stat[:year], stat[:month]]
       if existing_stats_lookup[key]
         Rails.logger.debug "Stat already exists: #{stat[:year]}-#{stat[:month]}"
@@ -92,29 +90,26 @@ def filter_existing_stats(stats)
         false
       end
     end
-
-    filtered_stats
   end
 
   def bulk_import_stats(stats)
     total_created = 0
 
     stats.each_slice(BATCH_SIZE) do |batch|
-      begin
-        result = Stat.upsert_all(
-          batch,
-          returning: %w[id],
-          on_duplicate: :skip
-        )
-
-        batch_created = result.count
-        total_created += batch_created
-
-        Rails.logger.debug "Processed batch of #{batch.size} stats, created #{batch_created}, total created: #{total_created}"
-
-      rescue StandardError => e
-        ExceptionReporter.call(e, 'Failed to process stat batch')
-      end
+      # rubocop:disable Rails/SkipsModelValidations
+      result = Stat.upsert_all(
+        batch,
+        returning: %w[id],
+        on_duplicate: :skip
+      )
+      # rubocop:enable Rails/SkipsModelValidations
+
+      batch_created = result.count
+      total_created += batch_created
+
+      Rails.logger.debug "Processed batch of #{batch.size} stats, created #{batch_created}, total created: #{total_created}"
+    rescue StandardError => e
+      ExceptionReporter.call(e, 'Failed to process stat batch')
     end
 
     total_created