SPARK-2372: Async chat leave

The ‘leave chat’ event can be executed asynchronously, so that the UI isn’t needlessly waiting for things.

core/src/main/java/org/jivesoftware/resource/ConfigurationRes.java

@@ -15,6 +15,8 @@
  */
 package org.jivesoftware.resource;
 
+import org.jivesoftware.spark.util.log.Log;
+
 import javax.swing.ImageIcon;
 
 import java.io.IOException;
@@ -55,9 +57,19 @@ public class ConfigurationRes {
     }
 
     public static ImageIcon getImageIcon(String imageName) {
+        try {
             final String iconURI = getString(imageName);
             final URL imageURL = cl.getResource(iconURI);
+            if (imageURL != null) {
                 return new ImageIcon(imageURL);
+            } else {
+                Log.warning(imageName + " not found.");
+            }
+        }
+        catch (Exception e) {
+            Log.warning("Unable to load image " + imageName, e);
+        }
+        return null;
     }
 
     public static URL getURL(String propertyName) {

core/src/main/java/org/jivesoftware/resource/Default.java

@@ -208,8 +208,8 @@ public class Default {
             cache.put(imageName, icon);
             return icon;
         }
-        catch (Exception ex) {
+        catch (Exception e) {
-            Log.debug(imageName + " not found.");
+            Log.warning(imageName + " not found.", e);
         }
         return null;
     }

core/src/main/java/org/jivesoftware/resource/SoundsRes.java

@@ -15,6 +15,8 @@
  */
 package org.jivesoftware.resource;
 
+import org.jivesoftware.spark.util.log.Log;
+
 import javax.swing.ImageIcon;
 
 import java.io.IOException;
@@ -52,9 +54,19 @@ public class SoundsRes {
     }
 
     public static ImageIcon getImageIcon(String imageName) {
+        try {
             final String iconURI = getString(imageName);
             final URL imageURL = cl.getResource(iconURI);
+            if (imageURL != null) {
                 return new ImageIcon(imageURL);
+            } else {
+                Log.warning(imageName + " not found.");
+            }
+        }
+        catch (Exception e) {
+            Log.warning("Unable to load image " + imageName, e);
+        }
+        return null;
     }
 
     public static URL getURL(String propertyName) {

core/src/main/java/org/jivesoftware/resource/SparkRes.java

@@ -353,10 +353,16 @@ public class SparkRes {
 
     public static ImageIcon getImageIcon(String imageName) {
         try {
-            final URL imageURL = getURL(imageName);
+            final String iconURI = getString(imageName);
+            final URL imageURL = cl.getResource(iconURI);
+            if (imageURL != null) {
                 return new ImageIcon(imageURL);
-        } catch (Exception ex) {
+            } else {
-            Log.error(imageName + " not found.");
+                Log.warning(imageName + " not found.");
+            }
+        }
+        catch (Exception e) {
+            Log.warning("Unable to load image " + imageName, e);
         }
         return null;
     }

core/src/main/java/org/jivesoftware/spark/ui/ContactInfoWindow.java

@@ -303,8 +303,8 @@ public class ContactInfoWindow extends JPanel {
             }
             avatarLabel.setBorder(BorderFactory.createLineBorder(Color.lightGray, 1, true));
         }
-        catch (MalformedURLException e) {
+        catch (Exception e) {
-            Log.error(e);
+            Log.warning("Unable to update avatar in contact info window", e);
         }
 
         // Get VCard from memory (if available)

core/src/main/java/org/jivesoftware/spark/ui/ContactItem.java

@@ -601,8 +601,8 @@ public class ContactItem extends JPanel {
 					setSideIcon(icon);
 				}
 			}
-		} catch (MalformedURLException e) {
+		} catch (Exception e) {
-			Log.error(e);
+			Log.warning("Unable to update avatar in side icon", e);
 		}
 	}
 

core/src/main/java/org/jivesoftware/spark/ui/VCardPanel.java

@@ -77,12 +77,15 @@ public class VCardPanel extends JPanel {
         add(avatarImage, new GridBagConstraints(0, 0, 1, 3, 0.0, 1.0, GridBagConstraints.NORTHWEST, GridBagConstraints.NONE, new Insets(5, 0, 5, 0), 0, 0));
         buildAvatarHover();
 
+        try {
             Image aImage = SparkRes.getImageIcon(SparkRes.BLANK_24x24).getImage();
             aImage = aImage.getScaledInstance(-1, 64, Image.SCALE_SMOOTH);
             ImageIcon ico = new ImageIcon(aImage);
 
             avatarImage.setIcon(ico);
-
+        } catch (Exception e) {
+            Log.error("Unable to process image in vcard!", e);
+        }
 
         VCard vcard = SparkManager.getVCardManager().getVCard(jid);
 
@@ -105,7 +108,7 @@ public class VCardPanel extends JPanel {
                 icon = new ImageIcon(newImage);
             }
             catch (Exception e) {
-                Log.error(e);
+                Log.error("Unable to fetch image in vcard!", e);
             }
         }
         else {
@@ -153,7 +156,7 @@ public class VCardPanel extends JPanel {
                     icon = new ImageIcon(newImage);
 
                 } catch (Exception e1) {
-                    Log.error(e1);
+                    Log.error("Unable to process vcard avatar", e1);
                 }
 
             } else {

core/src/main/java/org/jivesoftware/spark/ui/VCardViewer.java

@@ -75,24 +75,27 @@ public class VCardViewer extends JPanel {
         avatarImage = new JLabel();
         add(avatarImage, new GridBagConstraints(0, 0, 1, 3, 0.0, 1.0, GridBagConstraints.NORTHWEST, GridBagConstraints.NONE, new Insets(5, 0, 5, 0), 0, 0));
 
-
+        try {
             Image aImage = SparkRes.getImageIcon(SparkRes.BLANK_24x24).getImage();
             aImage = aImage.getScaledInstance(-1, 64, Image.SCALE_SMOOTH);
             ImageIcon ico = new ImageIcon(aImage);
 
             avatarImage.setIcon(ico);
 
-        final SwingWorker vcardLoader = new SwingWorker() {
+            final SwingWorker vcardLoader = new SwingWorker()
+            {
                 VCard vcard = null;
 
                 @Override
-			public Object construct() {
+                public Object construct()
+                {
                     vcard = SparkManager.getVCardManager().getVCard(jid);
                     return vcard;
                 }
 
                 @Override
-			public void finished() {
+                public void finished()
+                {
                     if (vcard == null) {
                         // Do nothing.
                         return;
@@ -107,12 +110,10 @@ public class VCardViewer extends JPanel {
                             Image aImage = icon.getImage();
                             aImage = aImage.getScaledInstance(-1, 48, Image.SCALE_SMOOTH);
                             icon = new ImageIcon(aImage);
+                        } catch (Exception e) {
+                            Log.warning("Unable to get scaled avatar from vcard.", e);
                         }
-                    catch (Exception e) {
+                    } else {
-                        Log.error(e);
-                    }
-                }
-                else {
                         icon = SparkRes.getImageIcon(SparkRes.DEFAULT_AVATAR_32x32_IMAGE);
                     }
 
@@ -125,8 +126,10 @@ public class VCardViewer extends JPanel {
                     buildUI(vcard);
                 }
             };
-
             vcardLoader.start();
+        } catch (Exception e) {
+            Log.warning("Unable to get avatar from vcard.", e);
+        }
     }
 
     private void buildUI(final VCard vcard) {

core/src/main/java/org/jivesoftware/spark/ui/conferences/ConferenceRoomBrowser.java

@@ -920,26 +920,37 @@ public class ConferenceRoomBrowser extends JPanel implements ActionListener,
                 }
 
                 if (isbookmark && ispassword) {
+                    try {
                         Image img = ImageCombiner.combine(bookmarkicon, passwordicon);
+                        if (img != null) {
                             iconLabel.setIcon(new ImageIcon(img));
+                        }
+                    } catch (Exception e) {
+                        Log.warning("Unable to set icon for bookmarked & password-protected room " + jid, e);
+                    }
                 } else if (isbookmark) {
                     iconLabel.setIcon(bookmarkicon);
                 } else if (ispassword) {
+                    try {
+                        if (passwordicon != null) {
                             Image img = ImageCombiner.returnTransparentImage(
                                 passwordicon.getIconWidth(), passwordicon.getIconHeight());
-
+                            Image combined = ImageCombiner.combine(new ImageIcon(img), passwordicon);
-                    Image combined = ImageCombiner.combine(new ImageIcon(img),
+                            if (combined != null) {
-                        passwordicon);
-
                                 iconLabel.setIcon(new ImageIcon(combined));
                             }
+                        }
+                    } catch (Exception e) {
+                        Log.warning("Unable to set icon for password-protected room " + jid, e);
+                    }
+                }
 
                 String occupants = Integer.toString(numberOfOccupants);
                 if (numberOfOccupants == -1) {
                     occupants = "n/a";
                 }
 
-                return new Object[] { iconLabel, roomName.toString(), jid.getLocalpart().toString(), occupants };
+                return new Object[] { iconLabel, roomName == null ? jid.getLocalpart().toString() : roomName.toString(), jid.getLocalpart().toString(), occupants };
             }
             
             @Override

core/src/main/java/org/jivesoftware/spark/ui/rooms/GroupChatRoom.java

@@ -46,6 +46,7 @@ import org.jivesoftware.spark.ui.conferences.ConferenceUtils;
 import org.jivesoftware.spark.ui.conferences.DataFormDialog;
 import org.jivesoftware.spark.ui.conferences.GroupChatParticipantList;
 import org.jivesoftware.spark.util.ModelUtil;
+import org.jivesoftware.spark.util.SwingWorker;
 import org.jivesoftware.spark.util.UIComponentRegistry;
 import org.jivesoftware.spark.util.log.Log;
 import org.jivesoftware.sparkimpl.settings.local.LocalPreferences;
@@ -543,14 +544,13 @@ public class GroupChatRoom extends ChatRoom
         // Update Room Notice To Inform Agent that he has left the chat.
         getTranscriptWindow().insertNotificationMessage( Res.getString( "message.user.left.room", getNickname() ), ChatManager.NOTIFICATION_COLOR );
 
-        try
+        SwingUtilities.invokeLater(() -> {
-        {
+            try {
                 chat.leave();
-        }
+            } catch ( Exception e ) {
-        catch ( Exception e )
-        {
                 Log.error( "Closing Group Chat Room error.", e );
             }
+        });
 
         // Set window as greyed out.
         getTranscriptWindow().showWindowDisabled();

core/src/main/java/org/jivesoftware/spark/ui/status/CustomMessages.java

@@ -345,12 +345,16 @@ public class CustomMessages {
             // Add Types
             for (StatusItem statusItem : statusBar.getStatusList()) {
                 if (!PresenceManager.isOnPhone(statusItem.getPresence())) {
+                    try {
                         ImageIcon icon = (ImageIcon) statusItem.getIcon();
 
                         ImageIcon newIcon = new ImageIcon(icon.getImage());
                         newIcon.setDescription(statusItem.getText());
 
                         typeBox.addItem(newIcon);
+                    } catch (Exception e) {
+                        Log.warning("Unable to update icon on custom status bar", e);
+                    }
                 }
             }
 

core/src/main/java/org/jivesoftware/spark/ui/status/StatusBar.java

@@ -165,6 +165,7 @@ public class StatusBar extends JPanel implements VCardListener {
     }
 
     public void setAvatar(Icon icon) {
+        try {
             if (icon == null) {
                 imageLabel.setIcon(null);
             } else {
@@ -178,6 +179,9 @@ public class StatusBar extends JPanel implements VCardListener {
             imageLabel.setBorder(null);
             revalidate();
             allowProfileEditing();
+        } catch (Exception e) {
+            Log.warning("Unable to set avatar", e);
+        }
     }
 
     public CommandPanel getCommandPanel() {
@@ -506,7 +510,7 @@ public class StatusBar extends JPanel implements VCardListener {
                     imageLabel.validate();
                     imageLabel.repaint();
                 } catch (Exception e) {
-                    // no issue
+                    Log.warning("Unable to update vcard viewer with avatar data", e);
                 }
             } else {
                 imageLabel.setIcon(null);

core/src/main/java/org/jivesoftware/spark/util/GraphicUtils.java

@@ -692,52 +692,16 @@ public final class GraphicUtils {
      * @return byte[]
      */
     public static byte[] getBytesFromImage(File file) {
-        FileInputStream fileInputStream = null;
+        try (FileInputStream fileInputStream = new FileInputStream(file)) {
-        try {
-            fileInputStream = new FileInputStream(file);
             byte[] data = new byte[(int) file.length()];
             fileInputStream.read(data);
-            fileInputStream.close();
             return data;
-        } catch (IOException e) {
+        } catch (Exception e) {
-            if (fileInputStream != null) {
+            Log.warning("Unable to read image", e);
-                try {
-                    fileInputStream.close();
-                } catch (IOException e1) {
-                    Log.error(e1);
-                }
-            }
             return null;
         }
     }
 
-    /**
-     * Returns a scaled down image if the height or width is smaller than the image size.
-     *
-     * @param icon      the image icon.
-     * @param newHeight the preferred height.
-     * @param newWidth  the preferred width.
-     * @return the icon.
-     */
-    public static ImageIcon scaleImageIcon(ImageIcon icon, int newHeight, int newWidth) {
-        int height = icon.getIconHeight();
-        int width = icon.getIconWidth();
-        boolean resize = false;
-        if (height > newHeight) {
-            height = newHeight;
-            resize = true;
-        }
-        if (width > newWidth) {
-            width = newWidth;
-            resize = true;
-        }
-        if (!resize) {
-            return icon;
-        }
-        Image img = icon.getImage().getScaledInstance(width, height, Image.SCALE_SMOOTH);
-        return new ImageIcon(img);
-    }
-
     /**
      * Returns a scaled down image if the height or width is smaller than the
      * image size.
@@ -750,7 +714,43 @@ public final class GraphicUtils {
      *            the preferred width.
      * @return the icon.
      */
-    public static ImageIcon scale(ImageIcon icon, int newHeight, int newWidth) {
+    public static ImageIcon scaleImageIcon(ImageIcon icon, int newHeight, int newWidth)
+    {
+        try {
+            Image img = icon.getImage();
+            int height = icon.getIconHeight();
+            int width = icon.getIconWidth();
+
+            if (height > newHeight) {
+                height = newHeight;
+            }
+
+            if (width > newWidth) {
+                width = newWidth;
+            }
+            img = img.getScaledInstance(width, height, Image.SCALE_SMOOTH);
+            return new ImageIcon(img);
+        } catch (Exception e) {
+            Log.warning("Unable to scale image", e);
+            return null;
+        }
+    }
+
+    /**
+     * Returns a scaled down image if the height or width is smaller than the
+     * image size.
+     * 
+     * @param icon
+     *            the image icon.
+     * @param newHeight
+     *            the preferred height.
+     * @param newWidth
+     *            the preferred width.
+     * @return the icon.
+     */
+    public static ImageIcon scale(ImageIcon icon, int newHeight, int newWidth)
+    {
+        try {
             Image img = icon.getImage();
             int height = icon.getIconHeight();
             int width = icon.getIconWidth();
@@ -782,6 +782,10 @@ public final class GraphicUtils {
             }
             img = img.getScaledInstance(width, height, Image.SCALE_SMOOTH);
             return new ImageIcon(img);
+        } catch (Exception e) {
+            Log.warning("Unable to scale image", e);
+            return null;
+        }
     }
 
     /**
@@ -796,7 +800,7 @@ public final class GraphicUtils {
         try {
             return new JFileChooser().getIcon(file);
         } catch (Exception e) {
-            Log.debug("unable to get icon");
+            Log.warning("unable to get icon", e);
         }
 
 	return SparkRes.getImageIcon(SparkRes.DOCUMENT_INFO_32x32);
@@ -833,6 +837,7 @@ public final class GraphicUtils {
 	 * @return
 	 */
 	public static ImageIcon fitToSquare(ImageIcon icon, int newSize) {
+        try {
             if (newSize <= 0) {
                 return icon;
             }
@@ -854,6 +859,10 @@ public final class GraphicUtils {
                 newHeight, Image.SCALE_SMOOTH);
 
             return new ImageIcon(img);
+        } catch (Exception e) {
+            Log.warning("Unable to fit image to square", e);
+            return null;
+        }
 	}
 
     // public static void centerWindowOnScreen(Runnable runnable) {

core/src/main/java/org/jivesoftware/spark/util/ImageCombiner.java

@@ -15,6 +15,8 @@
  */
 package org.jivesoftware.spark.util;
 
+import org.jivesoftware.spark.util.log.Log;
+
 import java.awt.Graphics2D;
 import java.awt.GraphicsConfiguration;
 import java.awt.GraphicsDevice;
@@ -34,7 +36,6 @@ import javax.swing.JComponent;
  */
 public class ImageCombiner {
     
-    
     /**
      * Combines two images into one
      * 
@@ -59,8 +60,9 @@ public class ImageCombiner {
      * @return combined Image
      */
     public static Image combine(ImageIcon image1, ImageIcon image2) {
-
+        try {
-	ImageObserver comp = new JComponent() {
+            ImageObserver comp = new JComponent()
+            {
                 private static final long serialVersionUID = 1L;
             };
 
@@ -77,13 +79,16 @@ public class ImageCombiner {
             g2.dispose();
 
             return image;
+        } catch (Exception e) {
+            Log.warning("Unable to combine two images", e);
+            return null;
+        }
     }
 
     public static Image returnTransparentImage(int w, int h) {
         return new BufferedImage(w, h, BufferedImage.TYPE_INT_ARGB);
     }
  
- 
     /**
      * Creates an Image from the specified Icon
      * 
@@ -107,5 +112,4 @@ public class ImageCombiner {
             return image;
         }
     }
-    
 }

core/src/main/java/org/jivesoftware/spark/util/SwingWorker.java

@@ -139,13 +139,12 @@ public abstract class SwingWorker {
                 setValue(construct());
             }
             catch (Exception e) {
-                Log.error( e );
+                Log.error("SwingWorker exception while async constructing value", e);
             }
             finally {
                 threadVar.clear();
             }
             SwingUtilities.invokeLater(this::finished);
-
         };
 
         Thread t = new Thread(doConstruct);

plugins/fastpath/src/main/java/org/jivesoftware/fastpath/resources/FastpathRes.java

@@ -16,6 +16,7 @@
 package org.jivesoftware.fastpath.resources;
 
 import org.jivesoftware.fastpath.FastpathPlugin;
+import org.jivesoftware.spark.util.log.Log;
 
 import javax.swing.ImageIcon;
 import javax.swing.JEditorPane;
@@ -265,10 +266,14 @@ public class FastpathRes {
         try {
             final String iconURI = getString(imageName);
             final URL imageURL = cl.getResource(iconURI);
+            if (imageURL != null) {
                 return new ImageIcon(imageURL);
+            } else {
+                Log.warning(imageName + " not found.");
             }
-        catch (Exception ex) {
+        }
-            System.out.println(imageName + " not found.");
+        catch (Exception e) {
+            Log.warning("Unable to load image " + imageName, e);
         }
         return null;
     }

plugins/jingle/src/main/java/org/jivesoftware/sparkplugin/IncomingCallUI.java

@@ -207,7 +207,7 @@ public class IncomingCallUI extends JPanel {
                 avatarLabel.repaint();
             }
             catch (Exception e) {
-                // no issue
+                Log.warning("Unable to generate image from avatar", e);
             }
         }
 

plugins/jingle/src/main/java/org/jivesoftware/sparkplugin/JinglePhoneRes.java

@@ -20,6 +20,8 @@ import java.util.PropertyResourceBundle;
 import java.util.ResourceBundle;
 import java.net.URL;
 
+import org.jivesoftware.spark.util.log.Log;
+
 /**
  * Use for Phone Res Internationalization.
  *
@@ -46,10 +48,14 @@ public class JinglePhoneRes {
         try {
             final String iconURI = JinglePhoneRes.getString(imageName);
             final URL imageURL = JinglePhoneRes.cl.getResource(iconURI);
+            if (imageURL != null) {
                 return new ImageIcon(imageURL);
+            } else {
+                Log.warning(imageName + " not found.");
             }
-        catch (Exception ex) {
+        }
-            System.out.println(imageName + " not found.");
+        catch (Exception e) {
+            Log.warning("Unable to load image " + imageName, e);
         }
         return null;
     }
@@ -57,6 +63,4 @@ public class JinglePhoneRes {
     public static final URL getURL(String propertyName) {
         return JinglePhoneRes.cl.getResource(JinglePhoneRes.getString(propertyName));
     }
-
-
 }

plugins/meet/src/main/java/org/jivesoftware/spark/plugin/ofmeet/SparkMeetPlugin.java

@@ -72,7 +72,7 @@ public class SparkMeetPlugin implements Plugin, ChatRoomListener, GlobalMessageL
 
         if (pluginsettings.exists())
         {
-            Log.warning("ofmeet-info: Properties-file does exist= " + pluginsettings.getPath());
+            Log.debug("ofmeet-info: Properties-file does exist= " + pluginsettings.getPath());
 
             try {
                 props.load(new FileInputStream(pluginsettings));
@@ -439,7 +439,7 @@ public class SparkMeetPlugin implements Plugin, ChatRoomListener, GlobalMessageL
                         Log.debug("Native lib folder created and natives extracted");
                     }
                     else {
-                        Log.warning("Native lib folder already exist.");
+                        Log.debug("Native lib folder already exist.");
                     }
 
 

plugins/otr/src/main/java/org/jivesoftware/spark/otrplug/util/OTRResources.java

@@ -45,9 +45,19 @@ public class OTRResources {
      * @return
      */
     public static ImageIcon getIcon(String fileName) {
+        try {
             final ClassLoader cl = OTRResources.class.getClassLoader();
-        ImageIcon icon = new ImageIcon(cl.getResource(fileName));
+            final URL imageURL = cl.getResource(fileName);
-        return icon;
+            if (imageURL != null) {
+                return new ImageIcon(imageURL);
+            } else {
+                Log.warning(imageName + " not found.");
+            }
+        }
+        catch (Exception e) {
+            Log.warning("Unable to load image " + imageName, e);
+        }
+        return null;
     }
 
     /**

plugins/reversi/src/main/java/org/jivesoftware/game/reversi/ReversiRes.java

@@ -1,6 +1,7 @@
 package org.jivesoftware.game.reversi;
 
 import org.jivesoftware.resource.UTF8Control;
+import org.jivesoftware.spark.util.log.Log;
 
 import java.net.URL;
 import java.util.PropertyResourceBundle;
@@ -20,8 +21,6 @@ public class ReversiRes {
     public static final String REVERSI_YOU="REVERSI_YOU";
     public static final String REVERSI_THEM="REVERSI_THEM";
 
-	   	
-	   	
     private ReversiRes() {
 
     }
@@ -40,10 +39,14 @@ public class ReversiRes {
         try {
             final String iconURI = ReversiRes.getString(imageName);
             final URL imageURL = ReversiRes.cl.getResource(iconURI);
+            if (imageURL != null) {
                 return new ImageIcon(imageURL);
+            } else {
+                Log.warning(imageName + " not found.");
             }
-	        catch (Exception ex) {
+        }
-	            System.out.println(imageName + " not found.");
+        catch (Exception e) {
+            Log.warning("Unable to load image " + imageName, e);
         }
         return null;
     }
@@ -51,5 +54,4 @@ public class ReversiRes {
     public static URL getURL(String propertyName) {
         return ReversiRes.cl.getResource(ReversiRes.getString(propertyName));
     }
-	    
 }

plugins/roar/src/main/java/org/jivesoftware/spark/roar/gui/SparkToaster.java

@@ -643,6 +643,7 @@ public class SparkToaster {
 	}
 
 	public void setIcon(Icon icon) {
+        try {
             if (icon.getIconHeight() > 64 || icon.getIconWidth() > 64) {
                 Image image = ImageCombiner.iconToImage(icon);
                 label.setIcon(new ImageIcon(image.getScaledInstance(-1, 64,
@@ -650,6 +651,9 @@ public class SparkToaster {
             } else{
                 label.setIcon(icon);
             }
+        } catch (Exception e) {
+            Log.warning("Unable to set icon in toaster", e);
+        }
 	}
 
 	public RolloverButton getCloseButton() {

plugins/sip/src/main/java/org/jivesoftware/spark/plugin/phone/resource/PhoneRes.java

@@ -58,8 +58,8 @@ public class PhoneRes {
             final URL imageURL = cl.getResource(iconURI);
             return new ImageIcon(imageURL);
         }
-        catch (Exception ex) {
+        catch (Throwable t) {
-            System.out.println(imageName + " not found.");
+            Log.warn(imageName + " not found.", t);
         }
         return null;
     }
@@ -72,10 +72,14 @@ public class PhoneRes {
         try {
             final String iconURI = getString(imageName);
             final URL imageURL = cl.getResource(iconURI);
-           return new ImageIcon(imageURL).getImage();
+            if (imageURL != null) {
+                return new ImageIcon(imageURL);
+            } else {
+                Log.warning(imageName + " not found.");
             }
-       catch (Exception ex) {
+        }
-           Log.error(imageName + " not found.");
+        catch (Exception e) {
+            Log.warning("Unable to load image " + imageName, e);
         }
         return null;
     }

plugins/sip/src/main/java/org/jivesoftware/sparkplugin/ui/call/IncomingCallUI.java

@@ -221,7 +221,7 @@ public class IncomingCallUI extends JPanel {
                 avatarLabel.repaint();
             }
             catch (Exception e) {
-                // no issue
+                Log.warning("Unable to generate image from avatar", e);
             }
         }
 

plugins/sip/src/main/java/org/jivesoftware/sparkplugin/ui/call/OutgoingCallUI.java

@@ -204,7 +204,7 @@ public class OutgoingCallUI extends JPanel {
                 avatarLabel.repaint();
             }
             catch (Exception e) {
-                // no issue
+                Log.warning("Unable to generate image from avatar", e);
             }
         }