Merge pull request #388 from marcosddourado/refatoring_repositories_c…

…ontroller

refactor latest method

CHANGELOG.md

@@ -6,6 +6,7 @@ The version numbers below try to follow the conventions at http://semver.org/.
 
 ## Unreleased
 
+- Refactor latest method
 - Remove the repository's instance that has remained in the db
 - Adding translation of the periodicity options in repository helper
 - Fixing wrong configurations translation in portuguese navbar

app/models/concerns/has_owner.rb

@@ -4,6 +4,13 @@ module HasOwner
   extend ActiveSupport::Concern
 
   class_methods do
+    def latest(count = 1)
+      all.sort { |one, another| another.id <=> one.id }.select { |entity|
+        attributes = entity.attributes
+        attributes && attributes.public
+      }.first(count)
+    end
+
     def public_or_owned_by_user(user = nil)
       class_name = name+"Attributes"
       collection = class_name.constantize.where(public: true)

app/models/kalibro_configuration.rb

@@ -8,13 +8,6 @@ def self.public
     self.public_or_owned_by_user
   end
 
-  def self.latest(count = 1)
-    all.sort { |one, another| another.id <=> one.id }.select { |kalibro_configuration|
-      attributes = kalibro_configuration.attributes
-      attributes && attributes.public
-    }.first(count)
-  end
-
   def attributes
     @attributes ||= KalibroConfigurationAttributes.find_by(kalibro_configuration_id: self.id)
   end

app/models/project.rb

@@ -4,13 +4,6 @@ class Project < KalibroClient::Entities::Processor::Project
 
   attr_writer :attributes
 
-  def self.latest(count = 1)
-    all.sort { |one, another| another.id <=> one.id }.select { |project|
-      attributes = project.attributes
-      attributes && attributes.public
-    }.first(count)
-  end
-
   def attributes
     @attributes ||= ProjectAttributes.find_by_project_id(@id)
   end

app/models/repository.rb

@@ -4,10 +4,6 @@ class Repository < KalibroClient::Entities::Processor::Repository
 
   attr_writer :attributes
 
-  def self.latest(count=1)
-    all.sort { |one, another| another.id <=> one.id }.select { |repository| repository.attributes.public }.first(count)
-  end
-
   def attributes
     @attributes ||= RepositoryAttributes.find_by_repository_id(@id)
   end