consul/grecia
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Add and apply Lint/RedundantSafeNavigation RuboCop rule

Browse Source 
We've tested the updated behavior in RuboCop 1.69, found a new offense, and corrected it.
This commit is contained in:
taitus
2025-02-13 16:06:05 +01:00
parent 4c7fe8a5d9
commit f4ffee3700
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
.rubocop.yml
View File

@@ -311,6 +311,9 @@ Lint/NonAtomicFileOperation:
Lint/ParenthesesAsGroupedExpression: Lint/ParenthesesAsGroupedExpression:
Enabled: true Enabled: true
Lint/RedundantSafeNavigation:
Enabled: true
Lint/RedundantStringCoercion: Lint/RedundantStringCoercion:
Enabled: true Enabled: true

2
app/controllers/legislation/annotations_controller.rb
View File

@@ -101,7 +101,7 @@ class Legislation::AnnotationsController < Legislation::BaseController
def convert_ranges_parameters def convert_ranges_parameters
annotation = params[:legislation_annotation] annotation = params[:legislation_annotation]
if annotation && annotation[:ranges]&.is_a?(String) if annotation && annotation[:ranges].is_a?(String)
params[:legislation_annotation][:ranges] = JSON.parse(annotation[:ranges]) params[:legislation_annotation][:ranges] = JSON.parse(annotation[:ranges])
end end
rescue JSON::ParserError rescue JSON::ParserError