Cancelling ContributionsSelfCheck will prevent indexes from being updated.

Opening boards/libs manager when NotificationPopup is shown will close it

Author: Federico Fissore

app/src/cc/arduino/contributions/ContributionsSelfCheck.java

@@ -26,6 +26,9 @@ public class ContributionsSelfCheck extends TimerTask {
   private final LibraryInstaller libraryInstaller;
   private final ProgressListener progressListener;
 
+  private volatile boolean cancelled;
+  private volatile NotificationPopup notificationPopup;
+
   public ContributionsSelfCheck(Base base, HyperlinkListener hyperlinkListener, ContributionsIndexer contributionsIndexer, ContributionInstaller contributionInstaller, LibrariesIndexer librariesIndexer, LibraryInstaller libraryInstaller) {
     this.base = base;
     this.hyperlinkListener = hyperlinkListener;
@@ -34,6 +37,7 @@ public ContributionsSelfCheck(Base base, HyperlinkListener hyperlinkListener, Co
     this.librariesIndexer = librariesIndexer;
     this.libraryInstaller = libraryInstaller;
     this.progressListener = new NoopProgressListener();
+    this.cancelled = false;
   }
 
   @Override
@@ -62,12 +66,29 @@ public void run() {
       text = I18n.format(_("Some {0}boards{1} and some {2}libraries{3} may be updated"), "<a href=\"http://boardsmanager\">", "</a>", "<a href=\"http://librarymanager\">", "</a>");
     }
 
+    if (cancelled) {
+      return;
+    }
+
     SwingUtilities.invokeLater(() -> {
-      new NotificationPopup(base.getActiveEditor(), hyperlinkListener, _("Updates available"), text).setVisible(true);
+      notificationPopup = new NotificationPopup(base.getActiveEditor(), hyperlinkListener, _("Updates available"), text);
+      notificationPopup.setVisible(true);
     });
   }
 
+  @Override
+  public boolean cancel() {
+    cancelled = true;
+    if (notificationPopup != null) {
+      notificationPopup.close();
+    }
+    return super.cancel();
+  }
+
   private void updateLibrariesIndex() {
+    if (cancelled) {
+      return;
+    }
     try {
       libraryInstaller.updateIndex(progressListener);
     } catch (Exception e) {
@@ -76,6 +97,9 @@ private void updateLibrariesIndex() {
   }
 
   private void updateContributionIndex() {
+    if (cancelled) {
+      return;
+    }
     try {
       contributionInstaller.updateIndex(progressListener);
     } catch (Exception e) {

app/src/cc/arduino/view/NotificationPopup.java

@@ -98,7 +98,7 @@ private void updateLocation(Frame parent) {
     setLocation(parentX + parent.getWidth() - getWidth(), parentY + parent.getHeight() - getHeight());
   }
 
-  private void close() {
+  public void close() {
     if (autoCloseAfterTimeout.isRunning()) {
       autoCloseAfterTimeout.stop();
     }

app/src/processing/app/Base.java

@@ -94,7 +94,7 @@ public boolean test(UserLibrary library) {
   public static Map<String, Object> FIND_DIALOG_STATE = new HashMap<String, Object>();
   private final ContributionInstaller contributionInstaller;
   private final LibraryInstaller libraryInstaller;
-  private Timer selfCheckTimer;
+  private ContributionsSelfCheck contributionsSelfCheck;
 
   // set to true after the first time the menu is built.
   // so that the errors while building don't show up again.
@@ -466,8 +466,8 @@ public Base(String[] args) throws Exception {
 
       new Thread(new BuiltInCoreIsNewerCheck(this)).start();
 
-      selfCheckTimer = new Timer(false);
-      selfCheckTimer.schedule(new ContributionsSelfCheck(this, new UpdatableBoardsLibsFakeURLsHandler(this), BaseNoGui.indexer, contributionInstaller, BaseNoGui.librariesIndexer, libraryInstaller), Constants.BOARDS_LIBS_UPDATABLE_CHECK_START_PERIOD);
+      contributionsSelfCheck = new ContributionsSelfCheck(this, new UpdatableBoardsLibsFakeURLsHandler(this), BaseNoGui.indexer, contributionInstaller, BaseNoGui.librariesIndexer, libraryInstaller);
+      new Timer(false).schedule(contributionsSelfCheck, Constants.BOARDS_LIBS_UPDATABLE_CHECK_START_PERIOD);
 
     } else if (parser.isNoOpMode()) {
       // Do nothing (intended for only changing preferences)
@@ -1228,8 +1228,8 @@ public void onBoardOrPortChange() {
   }
 
   public void openLibraryManager(String dropdownItem) {
-    if (selfCheckTimer != null) {
-      selfCheckTimer.cancel();
+    if (contributionsSelfCheck != null) {
+      contributionsSelfCheck.cancel();
     }
     @SuppressWarnings("serial")
     LibraryManagerUI managerUI = new LibraryManagerUI(activeEditor, BaseNoGui.librariesIndexer, libraryInstaller) {
@@ -1257,8 +1257,8 @@ protected void onIndexesUpdated() throws Exception {
   }
 
   public void openBoardsManager(final String filterText, String dropdownItem) throws Exception {
-    if (selfCheckTimer != null) {
-      selfCheckTimer.cancel();
+    if (contributionsSelfCheck != null) {
+      contributionsSelfCheck.cancel();
     }
     @SuppressWarnings("serial")
     ContributionManagerUI managerUI = new ContributionManagerUI(activeEditor, BaseNoGui.indexer, contributionInstaller) {