From 0d620a2d85fbead535ee3982317c6a5971de266b Mon Sep 17 00:00:00 2001
From: Ekaterina Tuzova <Ekaterina.Tuzova@jetbrains.com>
Date: Thu, 30 Mar 2017 20:26:11 +0300
Subject: [PATCH] cleanup

---
 .../edu/learning/StudyProjectComponent.java   | 50 ++++++++++---------
 1 file changed, 26 insertions(+), 24 deletions(-)

diff --git a/python/educational-core/src/com/jetbrains/edu/learning/StudyProjectComponent.java b/python/educational-core/src/com/jetbrains/edu/learning/StudyProjectComponent.java
index 9a7b90f2c48b..6a18fc88aff1 100644
--- a/python/educational-core/src/com/jetbrains/edu/learning/StudyProjectComponent.java
+++ b/python/educational-core/src/com/jetbrains/edu/learning/StudyProjectComponent.java
@@ -78,39 +78,18 @@ public class StudyProjectComponent implements ProjectComponent {
     }
 
     if (course != null && !course.isAdaptive() && !course.isUpToDate()) {
-      final Notification notification =
-        new Notification("Update.course", "Course Updates", "Course is ready to <a href=\"update\">update</a>", NotificationType.INFORMATION,
-                         new NotificationListener() {
-                           @Override
-                           public void hyperlinkUpdate(@NotNull Notification notification, @NotNull HyperlinkEvent event) {
-                             FileEditorManagerEx.getInstanceEx(myProject).closeAllFiles();
-
-                             ProgressManager.getInstance().runProcessWithProgressSynchronously(() -> {
-                               ProgressManager.getInstance().getProgressIndicator().setIndeterminate(true);
-                               return execCancelable(() -> {
-                                 updateCourse();
-                                 return true;
-                               });
-                             }, "Updating Course", true, myProject);
-                             EduUtils.synchronize();
-                             course.setUpdated();
-
-                           }
-                         });
-      notification.notify(myProject);
-
+      updateAvailable(course);
     }
 
     StudyUtils.registerStudyToolWindow(course, myProject);
     StartupManager.getInstance(myProject).runWhenProjectIsInitialized(() -> ApplicationManager.getApplication().invokeLater(
       (DumbAwareRunnable)() -> ApplicationManager.getApplication().runWriteAction((DumbAwareRunnable)() -> {
-        Course course1 = StudyTaskManager.getInstance(myProject).getCourse();
-        if (course1 != null) {
+        if (course != null) {
           UISettings instance = UISettings.getInstance();
           instance.setHideToolStripes(false);
           instance.fireUISettingsChanged();
           registerShortcuts();
-          EduUsagesCollector.projectTypeOpened(course1.isAdaptive() ? EduNames.ADAPTIVE : EduNames.STUDY);
+          EduUsagesCollector.projectTypeOpened(course.isAdaptive() ? EduNames.ADAPTIVE : EduNames.STUDY);
         }
       })));
 
@@ -126,6 +105,29 @@ public class StudyProjectComponent implements ProjectComponent {
     });
   }
 
+  private void updateAvailable(Course course) {
+    final Notification notification =
+      new Notification("Update.course", "Course Updates", "Course is ready to <a href=\"update\">update</a>", NotificationType.INFORMATION,
+                       new NotificationListener() {
+                         @Override
+                         public void hyperlinkUpdate(@NotNull Notification notification, @NotNull HyperlinkEvent event) {
+                           FileEditorManagerEx.getInstanceEx(myProject).closeAllFiles();
+
+                           ProgressManager.getInstance().runProcessWithProgressSynchronously(() -> {
+                             ProgressManager.getInstance().getProgressIndicator().setIndeterminate(true);
+                             return execCancelable(() -> {
+                               updateCourse();
+                               return true;
+                             });
+                           }, "Updating Course", true, myProject);
+                           EduUtils.synchronize();
+                           course.setUpdated();
+
+                         }
+                       });
+    notification.notify(myProject);
+  }
+
   private void registerShortcuts() {
     StudyToolWindow window = StudyUtils.getStudyToolWindow(myProject);
     if (window != null) {