Make RuboCop happy

ComPlat · Aug 10, 2023 · f6183d6 · f6183d6
1 parent 1efcca5
commit f6183d6
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/app/api/chemotion/reaction_api.rb b/app/api/chemotion/reaction_api.rb
@@ -155,7 +155,7 @@ class ReactionAPI < Grape::API
         optional :duration, type: String
         optional :rxno, type: String
         optional :segments, type: Array
-        optional :variations, type: Array[Hash]
+        optional :variations, type: [Hash]
       end
       route_param :id do
         after_validation do
@@ -221,7 +221,7 @@ class ReactionAPI < Grape::API
         requires :container, type: Hash
         optional :duration, type: String
         optional :rxno, type: String
-        optional :variations, type: Array[Hash]
+        optional :variations, type: [Hash]
       end
 
       post do

diff --git a/app/api/entities/reaction_variation_entity.rb b/app/api/entities/reaction_variation_entity.rb
@@ -13,7 +13,7 @@ class ReactionVariationEntity < ApplicationEntity
     def properties
       {}.tap do |properties|
         properties[:temperature] = ReactionVariationPropertyEntity.represent(object[:properties][:temperature])
-        properties[:duration] =  ReactionVariationPropertyEntity.represent(object[:properties][:duration])
+        properties[:duration] = ReactionVariationPropertyEntity.represent(object[:properties][:duration])
       end
     end
 

diff --git a/lib/reporter/docx/detail_reaction.rb b/lib/reporter/docx/detail_reaction.rb
@@ -44,24 +44,24 @@ def content
           gp_title_html: gp_title_html,
           synthesis_title_html: synthesis_title_html,
           synthesis_html: synthesis_html,
-          variations: variations
+          variations: variations,
         }
       end
 
       private
 
       def variations
-        obj.variations.map do |variation|
+        obj.variations.map do |var|
           {
-            'temperature' => "#{variation['properties']['temperature']['value']} #{variation['properties']['temperature']['unit']}",
-            'duration' => "#{variation['properties']['duration']['value']} #{variation['properties']['duration']['unit']}",
-            'startingMaterials' => variation['startingMaterials'].map do |_, v|
+            'temperature' => "#{var['properties']['temperature']['value']} #{var['properties']['temperature']['unit']}",
+            'duration' => "#{var['properties']['duration']['value']} #{var['properties']['duration']['unit']}",
+            'startingMaterials' => var['startingMaterials'].map do |_, v|
               "#{v['aux']['sumFormula']}:\n#{v['value']} #{v['unit']}" + (v['aux']['isReference'] ? '; Ref' : '')
             end,
-            'reactants' => variation['reactants'].map do |_, v|
+            'reactants' => var['reactants'].map do |_, v|
               "#{v['aux']['sumFormula']}:\n#{v['value']} #{v['unit']}" + (v['aux']['isReference'] ? '; Ref' : '')
             end,
-            'products' => variation['products'].map do |_, v|
+            'products' => var['products'].map do |_, v|
               "#{v['aux']['sumFormula']}:\n#{v['value']} #{v['unit']}; (#{v['aux']['yield']} % yield)"
             end,
           }