Merge pull request #36 from rage/server_error

Show nicer error message when the credentials are likely wrong.

Author: jamo

tmc-plugin/src/fi/helsinki/cs/tmc/actions/RefreshCoursesAction.java

@@ -24,9 +24,7 @@
  * Refreshes the course list in the background.
  */
 public final class RefreshCoursesAction {
-
     private final static Logger log = Logger.getLogger(RefreshCoursesAction.class.getName());
-
     private ServerAccess serverAccess;
     private CourseDb courseDb;
     private ConvenientDialogDisplayer dialogs;
@@ -35,6 +33,8 @@ public final class RefreshCoursesAction {
 
     private final TmcCore tmcCore;
     private final NbTmcSettings tmcSettings;
+    
+    private static final Logger logger = Logger.getLogger(RefreshCoursesAction.class.getName());
 
     public RefreshCoursesAction() {
         this(NbTmcSettings.getDefault());
@@ -57,7 +57,6 @@ public RefreshCoursesAction(NbTmcSettings settings, TmcCore core) {
         this.courseDb = CourseDb.getInstance();
         this.dialogs = ConvenientDialogDisplayer.getDefault();
         this.listeners = new BgTaskListenerList<List<Course>>();
-
         this.tmcCore = core;
     }
 
@@ -180,8 +179,9 @@ public void bgTaskCancelled() {
         @Override
         public void bgTaskFailed(Throwable ex) {
             if (showDialogOnError) {
-                dialogs.displayError("Course refresh failed.\n" + ServerErrorHelper.getServerExceptionMsg(ex));
+                logger.log(Level.INFO, "Course refresh failed: ", ex);
+                dialogs.displayError("Course refresh failed. Check your server address and credentials\n");
             }
         }
     }
-}
+}