Use `#present?` instead of `defined?` in serializers

8753635c · Grzegorz Bizon · e4cb073b · 8753635c · 8753635c
Commit 8753635c authored Feb 07, 2017 by Grzegorz Bizon
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

app/serializers/environment_serializer.rb app/serializers/environment_serializer.rb +1 -1

app/serializers/pipeline_serializer.rb app/serializers/pipeline_serializer.rb +1 -1

No files found.
--- a/app/serializers/environment_serializer.rb
+++ b/app/serializers/environment_serializer.rb
@@ -16,7 +16,7 @@ class EnvironmentSerializer < BaseSerializer
  end

  def paginated?
-    defined?(@paginator)
+    @paginator.present?
  end

  def represent(resource, opts = {})

--- a/app/serializers/pipeline_serializer.rb
+++ b/app/serializers/pipeline_serializer.rb
@@ -8,7 +8,7 @@ class PipelineSerializer < BaseSerializer
  end

  def paginated?
-    defined?(@paginator)
+    @paginator.present?
  end

  def represent(resource, opts = {})