Small code refactoring for condition circuit breaking

Also adds Changelog entry

app/controllers/projects/imports_controller.rb

@@ -23,7 +23,7 @@ class Projects::ImportsController < Projects::ApplicationController
 
   def show
     if @project.import_finished?
-      if continue_params && continue_params[:to]
+      if continue_params&.key?(:to)
         redirect_to continue_params[:to], notice: continue_params[:notice]
       else
         redirect_to project_path(@project), notice: finished_notice

app/models/project_import_state.rb

@@ -73,7 +73,7 @@ class ProjectImportState < ActiveRecord::Base
   alias_method :no_import?, :none?
 
   def in_progress?
-    started? || scheduled?
+    scheduled? || started?
   end
 
   def started?

changelogs/unreleased/50341-cleanup-useless-project-import-attributes.yml

+---
+title: Removes all the irrelevant code and columns that were migrated from the Project
+  table over to the ProjectImportState table
+merge_request: 21497
+author:
+type: performance