Bad function

get_completion

  helper_method :get_completion
  def get_completion(user)
    if user.name
      done = {}
      comp = JSON.parse(user.completion)
      if comp.length > 0
        comp.each do |catkey,catval|
          if catkey
            done[catkey] = {}
            if catval
              catval.each do |stepkey,stepval|
                done[catkey][stepkey] = stepval
              end
            end
          end
        end
        return done
      end
    end
  end