From: Tom Hughes Date: Tue, 8 Nov 2022 18:32:18 +0000 (+0000) Subject: Merge remote-tracking branch 'upstream/pull/3780' X-Git-Tag: live~2274 X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/8e976fbac165155366923a66a93868b48d665f85?hp=ba888f5c7d5ae453e27973e1e57f3c6e0e4f3711 Merge remote-tracking branch 'upstream/pull/3780' --- diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index 5303fb051..c03cf665c 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -1,6 +1,6 @@ # This configuration was generated by # `rubocop --auto-gen-config` -# on 2022-10-25 17:25:58 UTC using RuboCop version 1.37.1. +# on 2022-11-01 18:20:38 UTC using RuboCop version 1.38.0. # The point is for the user to remove these configuration records # one by one as the offenses are removed from the code base. # Note that changes in the inspected code, or installation of new @@ -46,7 +46,7 @@ Lint/AssignmentInCondition: Metrics/AbcSize: Max: 189 -# Offense count: 72 +# Offense count: 73 # Configuration parameters: CountComments, CountAsOne, ExcludedMethods, AllowedMethods, AllowedPatterns, IgnoredMethods. # AllowedMethods: refine Metrics/BlockLength: @@ -82,7 +82,12 @@ Metrics/ParameterLists: Metrics/PerceivedComplexity: Max: 26 -# Offense count: 559 +# Offense count: 2495 +# This cop supports safe autocorrection (--autocorrect). +Minitest/EmptyLineBeforeAssertionMethods: + Enabled: false + +# Offense count: 560 Minitest/MultipleAssertions: Max: 52 @@ -96,6 +101,17 @@ Naming/PredicateName: Exclude: - 'app/models/user.rb' +# Offense count: 8 +# This cop supports unsafe autocorrection (--autocorrect-all). +Rails/ActionControllerFlashBeforeRender: + Exclude: + - 'app/controllers/confirmations_controller.rb' + - 'app/controllers/friendships_controller.rb' + - 'app/controllers/oauth_controller.rb' + - 'app/controllers/traces_controller.rb' + - 'app/controllers/user_blocks_controller.rb' + - 'app/controllers/users_controller.rb' + # Offense count: 5 # Configuration parameters: Database, Include. # SupportedDatabases: mysql, postgresql diff --git a/Gemfile b/Gemfile index cdbd48d04..020cc9166 100644 --- a/Gemfile +++ b/Gemfile @@ -152,7 +152,7 @@ group :test do gem "rubocop" gem "rubocop-minitest" gem "rubocop-performance" - gem "rubocop-rails", "= 2.16.1" + gem "rubocop-rails" gem "rubocop-rake" gem "selenium-webdriver" gem "simplecov", :require => false diff --git a/Gemfile.lock b/Gemfile.lock index fabead7cd..b3beb4e4c 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -86,17 +86,17 @@ GEM autoprefixer-rails (10.4.7.0) execjs (~> 2) aws-eventstream (1.2.0) - aws-partitions (1.650.0) - aws-sdk-core (3.164.0) + aws-partitions (1.657.0) + aws-sdk-core (3.166.0) aws-eventstream (~> 1, >= 1.0.2) - aws-partitions (~> 1, >= 1.525.0) - aws-sigv4 (~> 1.1) + aws-partitions (~> 1, >= 1.651.0) + aws-sigv4 (~> 1.5) jmespath (~> 1, >= 1.6.1) - aws-sdk-kms (1.58.0) - aws-sdk-core (~> 3, >= 3.127.0) + aws-sdk-kms (1.59.0) + aws-sdk-core (~> 3, >= 3.165.0) aws-sigv4 (~> 1.1) - aws-sdk-s3 (1.116.0) - aws-sdk-core (~> 3, >= 3.127.0) + aws-sdk-s3 (1.117.1) + aws-sdk-core (~> 3, >= 3.165.0) aws-sdk-kms (~> 1) aws-sigv4 (~> 1.4) aws-sigv4 (1.5.2) @@ -132,7 +132,7 @@ GEM cancancan (3.4.0) canonical-rails (0.2.14) rails (>= 4.1, <= 7.1) - capybara (3.37.1) + capybara (3.38.0) addressable matrix mini_mime (>= 0.1.3) @@ -153,7 +153,7 @@ GEM crack (0.4.5) rexml crass (1.0.6) - dalli (3.2.2) + dalli (3.2.3) debug_inspector (1.1.0) deep_merge (1.2.2) delayed_job (4.1.11) @@ -166,43 +166,39 @@ GEM railties (>= 5) doorkeeper-i18n (5.2.5) doorkeeper (>= 5.2) - dry-configurable (0.16.1) - dry-core (~> 0.6) + dry-configurable (1.0.0) + dry-core (~> 1.0, < 2) zeitwerk (~> 2.6) - dry-container (0.11.0) - concurrent-ruby (~> 1.0) - dry-core (0.9.1) + dry-core (1.0.0) concurrent-ruby (~> 1.0) zeitwerk (~> 2.6) - dry-inflector (0.3.0) + dry-inflector (1.0.0) dry-initializer (3.1.1) - dry-logic (1.3.0) + dry-logic (1.4.0) concurrent-ruby (~> 1.0) - dry-core (~> 0.9, >= 0.9) + dry-core (~> 1.0, < 2) zeitwerk (~> 2.6) - dry-schema (1.11.3) + dry-schema (1.12.0) concurrent-ruby (~> 1.0) - dry-configurable (~> 0.16, >= 0.16) - dry-core (~> 0.9, >= 0.9) + dry-configurable (~> 1.0, < 2) + dry-core (~> 1.0, < 2) dry-initializer (~> 3.0) - dry-logic (~> 1.3) - dry-types (~> 1.6) + dry-logic (>= 1.4, < 2) + dry-types (>= 1.7, < 2) zeitwerk (~> 2.6) - dry-types (1.6.1) + dry-types (1.7.0) concurrent-ruby (~> 1.0) - dry-container (~> 0.3) - dry-core (~> 0.9, >= 0.9) - dry-inflector (~> 0.1, >= 0.1.2) - dry-logic (~> 1.3, >= 1.3) + dry-core (~> 1.0, < 2) + dry-inflector (~> 1.0, < 2) + dry-logic (>= 1.4, < 2) zeitwerk (~> 2.6) - dry-validation (1.9.0) + dry-validation (1.10.0) concurrent-ruby (~> 1.0) - dry-container (~> 0.7, >= 0.7.1) - dry-core (~> 0.9, >= 0.9) + dry-core (~> 1.0, < 2) dry-initializer (~> 3.0) - dry-schema (~> 1.11, >= 1.11.0) + dry-schema (>= 1.12, < 2) zeitwerk (~> 2.6) - erb_lint (0.2.0) + erb_lint (0.3.0) activesupport better_html (>= 2.0.1) parser (>= 2.7.1.4) @@ -253,7 +249,7 @@ GEM image_processing (1.12.2) mini_magick (>= 4.9.5, < 5) ruby-vips (>= 2.0.17, < 3) - image_size (3.1.0) + image_size (3.2.0) in_threads (1.6.0) jbuilder (2.11.5) actionview (>= 5.0.0) @@ -268,7 +264,7 @@ GEM kgio (2.11.4) kramdown (2.4.0) rexml - libxml-ruby (3.2.3) + libxml-ruby (3.2.4) listen (3.7.1) rb-fsevent (~> 0.10, >= 0.10.3) rb-inotify (~> 0.9, >= 0.9.10) @@ -297,7 +293,7 @@ GEM net-protocol net-protocol (0.1.3) timeout - net-smtp (0.3.2) + net-smtp (0.3.3) net-protocol nio4r (2.5.8) nokogiri (1.13.9) @@ -395,7 +391,7 @@ GEM nokogiri (>= 1.6) rails-html-sanitizer (1.4.3) loofah (~> 2.3) - rails-i18n (7.0.5) + rails-i18n (7.0.6) i18n (>= 0.7, < 2) railties (>= 6.0.0, < 8) railties (7.0.4) @@ -416,7 +412,7 @@ GEM rexml (3.2.5) rinku (2.0.6) rotp (6.2.0) - rubocop (1.37.1) + rubocop (1.38.0) json (~> 2.3) parallel (~> 1.10) parser (>= 3.1.2.1) @@ -428,12 +424,12 @@ GEM unicode-display_width (>= 1.4.0, < 3.0) rubocop-ast (1.23.0) parser (>= 3.1.1.0) - rubocop-minitest (0.22.2) + rubocop-minitest (0.23.1) rubocop (>= 0.90, < 2.0) rubocop-performance (1.15.0) rubocop (>= 1.7.0, < 2.0) rubocop-ast (>= 0.4.0) - rubocop-rails (2.16.1) + rubocop-rails (2.17.2) activesupport (>= 4.2.0) rack (>= 1.1) rubocop (>= 1.33.0, < 2.0) @@ -457,7 +453,7 @@ GEM sprockets-rails tilt secure_headers (6.5.0) - selenium-webdriver (4.5.0) + selenium-webdriver (4.6.1) childprocess (>= 0.5, < 5.0) rexml (~> 3.2, >= 3.2.5) rubyzip (>= 1.2.2, < 3.0) @@ -483,7 +479,7 @@ GEM actionpack (>= 5.2) activesupport (>= 5.2) sprockets (>= 3.0.0) - strong_migrations (1.3.2) + strong_migrations (1.4.0) activerecord (>= 5.2) terser (1.1.12) execjs (>= 0.3.0, < 3) @@ -507,7 +503,7 @@ GEM websocket-extensions (0.1.5) xpath (3.2.0) nokogiri (~> 1.8) - zeitwerk (2.6.1) + zeitwerk (2.6.6) PLATFORMS ruby @@ -585,7 +581,7 @@ DEPENDENCIES rubocop rubocop-minitest rubocop-performance - rubocop-rails (= 2.16.1) + rubocop-rails rubocop-rake sanitize sassc-rails diff --git a/app/assets/javascripts/index/browse.js b/app/assets/javascripts/index/browse.js index 9d3f9a470..a65934d02 100644 --- a/app/assets/javascripts/index/browse.js +++ b/app/assets/javascripts/index/browse.js @@ -54,6 +54,7 @@ OSM.initializeBrowse = function (map) { .text(I18n.t("browse.start_rjs.load_data")), $("
").append( $("