@C Wenn ein Plugin inkompatibel zu Jameica ist, laesst es sich

jetzt wenigstens noch ueber den Plugin-Manager auf eine aktuelle Version updaten. Bisher hing sich Jameica da komplett auf.
willuhn · Aug 9, 2012 · c65522e · c65522e
1 parent 8063e55
commit c65522e
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 10 deletions.
diff --git a/src/de/willuhn/jameica/gui/internal/parts/PluginDetailPart.java b/src/de/willuhn/jameica/gui/internal/parts/PluginDetailPart.java
@@ -100,6 +100,8 @@ public void paint(Composite parent) throws RemoteException
 
     // Rechte Spalte mit den Eigenschaften
     {
+      Throwable error = Application.getPluginLoader().getInitError(manifest);
+
       // Name
       {
         Label title = new Label(this.comp,SWT.NONE);
@@ -112,8 +114,7 @@ public void paint(Composite parent) throws RemoteException
           title.setForeground(comment);
 
           // Checken, ob wir eine Fehlermeldung haben
-          Throwable t = Application.getPluginLoader().getInitError(manifest);
-          text = i18n.tr("{0} ({1})",text, (t != null ? t.getMessage() : i18n.tr("Neustart erforderlich")));
+          text = i18n.tr("{0} ({1})",text, (error != null ? error.getMessage() : i18n.tr("Neustart erforderlich")));
         }
         title.setText(text);
       }
@@ -171,7 +172,7 @@ public void widgetSelected(SelectionEvent e)
 
       // Update und oeffnen gibt es nicht bei neuen Installationen
       open.setEnabled(manifest.isInstalled());
-      update.setEnabled(manifest.isInstalled());
+      update.setEnabled(manifest.isInstalled() || error != null); // Update auch bei Fehler erlauben
 
       // Checken, ob es installiert/deinstalliert werden kann
       try

diff --git a/src/de/willuhn/jameica/plugin/PluginLoader.java b/src/de/willuhn/jameica/plugin/PluginLoader.java
@@ -186,6 +186,10 @@ else if ((t instanceof OperationCanceledException))
         else
           Logger.error("unable to init plugin " + name, t);
 
+        // Das ist ein bisschen "best guess", um die Fehlermeldung bei inkompatiblen Plugins schoener aussehen zu lassen,
+        if (t instanceof NoSuchMethodError)
+          t = new ApplicationException(Application.getI18n().tr("Inkompatibel mit aktueller Jameica-Version"),t);
+
         this.initErrors.put(mf,t);
         if (!(t instanceof OperationCanceledException)) // nur anzeigen, wenn es kein Abbruch durch das Plugin selbst war
           Application.addWelcomeMessage(Application.getI18n().tr("Plugin \"{0}\" kann nicht initialisiert werden. {1}",name, t.getMessage()));

diff --git a/src/de/willuhn/jameica/services/BeanService.java b/src/de/willuhn/jameica/services/BeanService.java
@@ -161,10 +161,10 @@ else if (lct == Type.SESSION)
     {
       throw je;
     }
-    catch (Exception e)
+    catch (Throwable t)
     {
-      Logger.error("unable to create instance of " + type,e);
-      throw new JameicaException(Application.getI18n().tr("{0} kann nicht erstellt werden: {1}",type.getSimpleName(),e.getMessage()));
+      Logger.error("unable to create instance of " + type,t);
+      throw new JameicaException(Application.getI18n().tr("{0} kann nicht erstellt werden: {1}",type.getSimpleName(),t.getMessage()));
     }
   }
 

diff --git a/src/de/willuhn/jameica/services/ReminderService.java b/src/de/willuhn/jameica/services/ReminderService.java
@@ -206,9 +206,9 @@ public void run()
           }
         }
       }
-      catch (Exception e)
+      catch (Throwable t)
       {
-        Logger.error("unable to send reminder messages for provider " + provider.getClass().getName(),e);
+        Logger.error("unable to send reminder messages for provider " + provider.getClass().getName(),t);
       }
     }
   }

diff --git a/src/de/willuhn/jameica/services/SearchService.java b/src/de/willuhn/jameica/services/SearchService.java
@@ -64,9 +64,9 @@ public void init(BootLoader loader, Bootable caller) throws SkipServiceException
           this.providers.add(p);
           count++;
         }
-        catch (Exception e)
+        catch (Throwable t)
         {
-          Logger.error("unable to load search provider " + providers[i].getName(),e);
+          Logger.error("unable to load search provider " + providers[i].getName(),t);
         }
       }
       Logger.info("loaded " + count + " search providers");