Sign upLogin

flant/superhosting

View on GitHub
Star
lib/superhosting/controller/mysql/db/index.rb

Summary

Maintainability
A
0 mins
Test Coverage

Use each_key instead of each.
Open

          container_dbs(container_name: container_name).each { |k, _v| self.class.index.delete(k) }
Severity: Minor
Found in lib/superhosting/controller/mysql/db/index.rb by rubocop

This cop checks for uses of each_key and each_value Hash methods.

Note: If you have an array of two-element arrays, you can put parentheses around the block arguments to indicate that you're not working with a hash, and suppress RuboCop offenses.

Example:

# bad
hash.keys.each { |k| p k }
hash.values.each { |v| p v }
hash.each { |k, _v| p k }
hash.each { |_k, v| p v }

# good
hash.each_key { |k| p k }
hash.each_value { |v| p v }

Use each_key instead of keys.each.
Open

          @container_controller.index.keys.each { |container_name| reindex_container(container_name: container_name) }
Severity: Minor
Found in lib/superhosting/controller/mysql/db/index.rb by rubocop

This cop checks for uses of each_key and each_value Hash methods.

Note: If you have an array of two-element arrays, you can put parentheses around the block arguments to indicate that you're not working with a hash, and suppress RuboCop offenses.

Example:

# bad
hash.keys.each { |k| p k }
hash.values.each { |v| p v }
hash.each { |k, _v| p k }
hash.each { |_k, v| p v }

# good
hash.each_key { |k| p k }
hash.each_value { |v| p v }

There are no issues that match your filters.

Category
Status