require 'peoplefinder'

module Peoplefinder::Concerns::Completion
  extend ActiveSupport::Concern

  included do
    def completion_score_fields
      [
        :given_name,
        :surname,
        :email,
        :primary_phone_number,
        :location,
        :description,
        :groups,
        :image
      ]
    end

    scope :inadequate_profiles,
      lambda {
        where(%{
          COALESCE(image,'') = ''
          OR COALESCE(location,'') = ''
          OR COALESCE(primary_phone_number,'') = ''
        }).order(:email)
      }

    def completion_score
      completed = completion_score_fields.map { |f| send(f).present? }
      (100 * completed.select { |f| f }.length) / completion_score_fields.length
    end

    def incomplete?
      completion_score < 100
    end
  end
end

require 'peoplefinder'

module Peoplefinder::Concerns::Completion
  extend ActiveSupport::Concern

  included do
    COMPLETION_SCORE_FIELDS = [
      :given_name,
      :surname,
      :email,
      :primary_phone_number,
      :location,
      :description,
      :groups,
      :image
    ]

    def completion_score_fields
      COMPLETION_SCORE_FIELDS
    end

    scope :inadequate_profiles,
      lambda {
        where(%{
          COALESCE(image,'') = ''
          OR COALESCE(location,'') = ''
          OR COALESCE(primary_phone_number,'') = ''
        }).order(:email)
      }

    def completion_score
      completed = completion_score_fields.map { |f| send(f).present? }
      (100 * completed.select { |f| f }.length) / completion_score_fields.length
    end

    def incomplete?
      completion_score < 100
    end
  end
end

--- 
+++ 
@@ -4,19 +4,17 @@
   extend ActiveSupport::Concern
 
   included do
-    COMPLETION_SCORE_FIELDS = [
-      :given_name,
-      :surname,
-      :email,
-      :primary_phone_number,
-      :location,
-      :description,
-      :groups,
-      :image
-    ]
-
     def completion_score_fields
-      COMPLETION_SCORE_FIELDS
+      [
+        :given_name,
+        :surname,
+        :email,
+        :primary_phone_number,
+        :location,
+        :description,
+        :groups,
+        :image
+      ]
     end
 
     scope :inadequate_profiles,
Remove constant that isn't used elsewhere

It's shorter and better encapsulated.
