diff options
author | James Lopez <james@jameslopez.es> | 2017-05-04 19:11:28 +0300 |
---|---|---|
committer | James Lopez <james@jameslopez.es> | 2017-05-04 19:11:28 +0300 |
commit | cf002738e766f977bdb0e857759f548a5c65c9bd (patch) | |
tree | 4162281ab28ac4eaf1f711e2f20c46125807397a /app/services | |
parent | 78d059141b5f3345d797992e03680654ce762c76 (diff) |
refactor a few things based on feedback
Diffstat (limited to 'app/services')
-rw-r--r-- | app/services/projects/propagate_service.rb | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/app/services/projects/propagate_service.rb b/app/services/projects/propagate_service.rb index 3c05dcce07c..6e24a67d8b0 100644 --- a/app/services/projects/propagate_service.rb +++ b/app/services/projects/propagate_service.rb @@ -2,15 +2,15 @@ module Projects class PropagateService BATCH_SIZE = 100 - def self.propagate!(*args) - new(*args).propagate! + def self.propagate(*args) + new(*args).propagate end def initialize(template) @template = template end - def propagate! + def propagate return unless @template&.active Rails.logger.info("Propagating services for template #{@template.id}") @@ -28,7 +28,7 @@ module Projects batch.each { |project_id| create_from_template(project_id) } - break if batch.count < BATCH_SIZE + break if batch.size < BATCH_SIZE offset += BATCH_SIZE end |