Error message when using OneDrive for NetbeansProjectFolder

Redande · Redande · commit 225b9c0bb7e8 · 2018-04-18T15:01:42.000+03:00
diff --git a/tmc-plugin/src/fi/helsinki/cs/tmc/actions/CheckForOneDrive.java b/tmc-plugin/src/fi/helsinki/cs/tmc/actions/CheckForOneDrive.java
@@ -0,0 +1,34 @@
+package fi.helsinki.cs.tmc.actions;
+
+import fi.helsinki.cs.tmc.core.configuration.TmcSettings;
+import fi.helsinki.cs.tmc.core.holders.TmcSettingsHolder;
+import fi.helsinki.cs.tmc.ui.ConvenientDialogDisplayer;
+
+public class CheckForOneDrive {
+
+    public static void run() {
+        TmcSettings settings = TmcSettingsHolder.get();
+
+        if (!settings.getTmcProjectDirectory().toAbsolutePath().toString().toLowerCase().contains("onedrive")) {
+            return;
+        }
+
+        final ConvenientDialogDisplayer displayer = ConvenientDialogDisplayer.getDefault();
+
+        StringBuilder htmlBuilder = new StringBuilder();
+        htmlBuilder.append("<html><body>");
+        htmlBuilder.append("<h1>OneDrive does not work with Netbeans</h1>");
+        htmlBuilder.append("<div>Many students have had a problem when they have their Netbeans project folder in OneDrive, which results in submissions failing on the server.<br>The problem can be solved by moving the project folder out of OneDrive as follows:</div>");
+        htmlBuilder.append("<ol>");
+        htmlBuilder.append("<li>In Netbeans, close all the exercises by right clicking them on the \"Projects\"-sidebar on the left and selecting \"Close project\".<br>You can select multiple exercises at once by pressing down the shift button on your keyboard and while pressing the button, selecting the first and the last exercise on the list.</li>");
+        htmlBuilder.append("<li>Copy or move the \"NetbeansProjects\" folder to some path that does not contain the \"OneDrive\" folder.</li>");
+        htmlBuilder.append("<li>Now you can change the Netbeans project folder to the path you selected on step 2 by selecting \"TMC\" -> \"Settings\".</li>");
+        htmlBuilder.append("<li>Open the moved exercises by selecting \"File\" -> \"Open Project...\", and selecting the exercises from the new folder.</li>");
+        htmlBuilder.append("</ol>");
+        htmlBuilder.append("</body></html>");
+
+        String errorMsg = htmlBuilder.toString();
+
+        displayer.displayError(errorMsg);
+    }
+}
diff --git a/tmc-plugin/src/fi/helsinki/cs/tmc/actions/TmcModuleInstall.java b/tmc-plugin/src/fi/helsinki/cs/tmc/actions/TmcModuleInstall.java
@@ -46,6 +46,8 @@ public void run() {
 
                 CheckForOutdatedNetbeans.run();
 
+                CheckForOneDrive.run();
+
                 TmcCore.setInstance(new TmcCore());
 
                 CheckForNewExercisesOrUpdates.startTimer();
diff --git a/tmc-plugin/src/fi/helsinki/cs/tmc/ui/PreferencesPanel.java b/tmc-plugin/src/fi/helsinki/cs/tmc/ui/PreferencesPanel.java
@@ -492,8 +492,17 @@ private void folderChooserBtnActionPerformed(java.awt.event.ActionEvent evt) {//
             return;
         }
         File projectDefaultFolder = folderChooser.getSelectedFile();
+        
+        final String projectDefaultFolderPath = projectDefaultFolder.getAbsolutePath();
+
+        final String errorMsg = "OneDrive doesn't work with Netbeans.\nPlease use another folder for your projects.";
+
+        if (projectDefaultFolderPath.toLowerCase().contains("onedrive")) {
+            dialogs.displayError(errorMsg);
+            return;
+        }
 
-        projectFolderTextField.setText(projectDefaultFolder.getAbsolutePath());
+        projectFolderTextField.setText(projectDefaultFolderPath);
     }//GEN-LAST:event_folderChooserBtnActionPerformed
 
     private void changeCourseButtonActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_changeCourseButtonActionPerformed
