Project

General

Profile

« Previous | Next » 

Revision bb3572ff

Added by Daniel Lobato Garcia over 8 years ago

Refs #3809 - Stabby lambda syntax for oneliners

I started by spotting log.rb which didn't have a lambda wrapping its
default scope, as needed by Rails 4. Since the style guide and most
Rails 4 documents used the stabby lambda, I turned on the cop so that we
use it everywhere there's a oneliner lambda.

View differences:

app/views/api/v1/import_puppetclasses/show.json.rabl
attributes :name
node(:actions) do |environment|
actions = []
actions << 'new' if @changed['new'][environment.name].present?
actions << 'updated' if @changed['updated'][environment.name].present?
actions << 'obsolete' if @changed['obsolete'][environment.name].present?
actions.as_json
actions = []
actions << 'new' if @changed['new'][environment.name].present?
actions << 'updated' if @changed['updated'][environment.name].present?
actions << 'obsolete' if @changed['obsolete'][environment.name].present?
actions.as_json
end
node(:new_puppetclasses, :if => lambda { |environment| @changed['new'][environment.name].present? }) do |environment|
node(:new_puppetclasses, :if => ->(environment) { @changed['new'][environment.name].present? }) do |environment|
JSON.parse(@changed['new'][environment.name]).keys
end
node(:updated_puppetclasses, :if => lambda { |environment| @changed['updated'][environment.name].present? }) do |environment|
node(:updated_puppetclasses, :if => ->(environment) { @changed['updated'][environment.name].present? }) do |environment|
JSON.parse(@changed['updated'][environment.name]).keys
end
node(:obsolete_puppetclasses, :if => lambda { |environment| @changed['obsolete'][environment.name].present? && !@changed['obsolete'][environment.name].match(/_destroy_/) }) do |environment|
node(:obsolete_puppetclasses, :if => ->(environment) { @changed['obsolete'][environment.name].present? && !@changed['obsolete'][environment.name].match(/_destroy_/) }) do |environment|
JSON.parse(@changed['obsolete'][environment.name])
end
node(:removed_environment, :if => lambda { |environment| @changed['obsolete'][environment.name].present? && @changed['obsolete'][environment.name].match(/_destroy_/) }) do |environment|
node(:removed_environment, :if => ->(environment) { @changed['obsolete'][environment.name].present? && @changed['obsolete'][environment.name].match(/_destroy_/) }) do |environment|
environment.name
end

Also available in: Unified diff