Fixes issue 3904, removes deprecated method usage

Dont fire link events for thigns without servers

Change-Id: Iea21de87ec2f295b7f3a5de079bfad6d9736b69d
Reviewed-on: http://gerrit.dmdirc.com/1005
Automatic-Compile: DMDirc Local Commits <dmdirc@googlemail.com>
Reviewed-by: Chris Smith <chris@dmdirc.com>

src/com/dmdirc/addons/ui_swing/components/TopicBar.java

@@ -300,7 +300,7 @@ public class TopicBar extends JComponent implements ActionListener,
             } else if (url.contains("://")) {
                 URLHandler.getURLHander().launchApp(e.getDescription());
             } else {
-                channel.getServer().addQuery(url).activateFrame();
+                channel.getServer().getQuery(url).activateFrame();
             }
         }
     }

src/com/dmdirc/addons/ui_swing/components/frames/TextFrame.java

@@ -1,4 +1,4 @@
-    /*
+/*
  * Copyright (c) 2006-2010 Chris Smith, Shane Mc Cormack, Gregory Holmes
  *
  * Permission is hereby granted, free of charge, to any person obtaining a copy
@@ -906,8 +906,10 @@ public abstract class TextFrame extends JInternalFrame implements Window,
                 if (type == MouseClickType.CLICKED) {
                     switch (clickType) {
                         case CHANNEL:
-                            if (ActionManager.processEvent(CoreActionType.
-                                    LINK_CHANNEL_CLICKED, null, this, attribute)) {
+                            if (frameParent.getServer() != null &&
+                                    ActionManager.processEvent(CoreActionType.
+                                    LINK_CHANNEL_CLICKED, null, this,
+                                    attribute)) {
                                 frameParent.getServer().join(
                                         new ChannelJoinRequest(attribute));
                             }
@@ -919,10 +921,12 @@ public abstract class TextFrame extends JInternalFrame implements Window,
                             }
                             break;
                         case NICKNAME:
-                            if (ActionManager.processEvent(CoreActionType.
-                                    LINK_NICKNAME_CLICKED, null, this, attribute)) {
-                                getContainer().getServer().addQuery(attribute).
-                                        activateFrame();
+                            if (frameParent.getServer() != null &&
+                                    ActionManager.processEvent(CoreActionType.
+                                    LINK_NICKNAME_CLICKED, null, this,
+                                    attribute)) {
+                                getContainer().getServer().getQuery(
+                                        attribute).activateFrame();
                             }
                             break;
                         default: