@@ -11,6 +11,8 @@ class HomeController < ApplicationController
   def index
   	if user_signed_in?
   		name = current_user.name(false)
+# TODO: Investigate if this is even relevant anymore. The name var will never be blank here because the logic in
+#       User says to return the email if the firstname and surname are empty regardless of the flag passed in
   		if name.blank?
   			redirect_to edit_user_registration_path
   		else
class HomeController < ApplicationController
  respond_to :html

  ##
  # Index
  #
  # Currently redirects user to their list of projects
  # UNLESS
  # User's contact name is not filled in
  # Is this the desired behavior?
  def index
  	if user_signed_in?
  		name = current_user.name(false)
  		if name.blank?
  			redirect_to edit_user_registration_path
  		else
  			redirect_to plans_url
  		end
  	end
  end

end

I think you're correct that this is a leftover. Before if you hadn't filled in your first and last names it redirect to edit profile but we use email in place now so this will never happen.