Enable PMD for plugins.

Fix assorted PMD errors.

Change-Id: I9af7e36f2aeb10bd658c5825ad2f94161e0bb243
Reviewed-on: http://gerrit.dmdirc.com/3940
Reviewed-by: Greg Holmes <greg@dmdirc.com>
Automatic-Compile: DMDirc Build Manager

build.gradle

 
 subprojects {
     apply plugin: 'java'
+    apply plugin: 'pmd'
 
     configurations {
         bundle
         targetFile << "\n\nversion:\n  number=${version}\n"
     }
 
+    pmd {
+        ruleSets = []
+        ruleSetFiles = files("$rootDir/etc/pmd/full.xml")
+    }
+
     jar {
         from("$buildDir/plugin.config") {
             into 'META-INF'

calc/test/com/dmdirc/addons/calc/EvaluatorTest.java

     private final String input;
     private final float output;
 
-    public EvaluatorTest(String input, String output) {
+    public EvaluatorTest(final String input, final String output) {
         this.input = input;
         this.output = Float.parseFloat(output);
     }

conditional_execute/test/com/dmdirc/addons/conditional_execute/ConditionalExecuteCommandTest.java

         verifyNoCommandExecuted();
     }
 
-    private void execute(String line) {
+    private void execute(final String line) {
         command.execute(container, new CommandArguments(commandController, line), context);
     }
 
-    private void verifyCommandExecuted(String command) {
+    private void verifyCommandExecuted(final String command) {
         verify(commandParser).parseCommand(same(container), eq(command));
     }
 
         verify(commandParser, never()).parseCommand(same(container), anyString());
     }
 
-    private void verifyErrorOutput(String substring) {
+    private void verifyErrorOutput(final String substring) {
         verify(container).addLine(eq("commandError"), contains(substring));
     }
 

dcc/src/com/dmdirc/addons/dcc/DCCManager.java

                     bit = bit.substring(1);
                 }
                 if (bit.endsWith("\"")) {
-                    filenameBits.append(" ").append(bit.substring(0, bit.length() - 1));
+                    filenameBits.append(' ').append(bit.substring(0, bit.length() - 1));
                     break;
                 } else {
-                    filenameBits.append(" ").append(bit);
+                    filenameBits.append(' ').append(bit);
                 }
             }
             tmpFilename = filenameBits.toString().trim();
             return;
         }
 
-        if (DCCTransfer.findByToken(token) == null && !dontAsk) {
-            if (token.isEmpty() || port.equals("0")) {
-                // Make sure this is not a reverse DCC Send that we no longer care about.
-                eventBus.publish(new DccSendRequestEvent(connection, nickname, filename));
-                new SendRequestDialog(mainWindow, this, token, ipLong, portInt, filename, size,
-                        nickname, connection).display();
-            }
+        if (DCCTransfer.findByToken(token) == null && !dontAsk &&
+                (token.isEmpty() || port.equals("0"))) {
+            // Make sure this is not a reverse DCC Send that we no longer care about.
+            eventBus.publish(new DccSendRequestEvent(connection, nickname, filename));
+            new SendRequestDialog(mainWindow, this, token, ipLong, portInt, filename, size,
+                    nickname, connection).display();
         }
     }
 
                     bit = bit.substring(1);
                 }
                 if (bit.endsWith("\"")) {
-                    filenameBits.append(" ")
+                    filenameBits.append(' ')
                             .append(bit.substring(0, bit.length() - 1));
                     break;
                 } else {
-                    filenameBits.append(" ").append(bit);
+                    filenameBits.append(' ').append(bit);
                 }
             }
             filename = filenameBits.toString().trim();

dcc/src/com/dmdirc/addons/dcc/TransferContainer.java

         if (percentageInTitle) {
             final StringBuilder title = new StringBuilder();
             if (dcc.isListenSocket()) {
-                title.append("*");
+                title.append('*');
             }
             title.append(dcc.getType() == DCCTransfer.TransferType.SEND
                     ? "Sending: " : "Receiving: ");

dcc/src/com/dmdirc/addons/dcc/ui/TransferPanel.java

 
         if (hours > 0) {
             result.append(hours);
-            result.append(":");
+            result.append(':');
         }
         result.append(String.format("%0,2d:%0,2d", minutes, seconds));
 

debug/test/com/dmdirc/addons/debug/DebugTest.java

 
     @Mock private CommandArguments arguments;
     @Mock private FrameContainer container;
-    @Mock private DebugPlugin plugin;
     @Mock private CommandController controller;
     @Mock private DebugCommand debugCommand;
     @Mock private CommandContext commandContext;

etc/pmd/full.xml

+<?xml version="1.0"?>
+<ruleset name="DMDirc PMD checks"
+         xmlns="http://pmd.sf.net/ruleset/1.0.0"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:schemaLocation="http://pmd.sf.net/ruleset/1.0.0 http://pmd.sf.net/ruleset_xml_schema.xsd"
+         xsi:noNamespaceSchemaLocation="http://pmd.sf.net/ruleset_xml_schema.xsd">
+        <description>DMDirc minimal PMD checks.</description>
+
+<rule ref="rulesets/java/empty.xml/EmptyIfStmt"><priority>1</priority></rule>
+<rule ref="rulesets/java/empty.xml/EmptyWhileStmt"><priority>1</priority></rule>
+<rule ref="rulesets/java/empty.xml/EmptyTryBlock"><priority>1</priority></rule>
+<rule ref="rulesets/java/empty.xml/EmptyFinallyBlock"><priority>1</priority></rule>
+<rule ref="rulesets/java/empty.xml/EmptySwitchStatements"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/JumbledIncrementer"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/ForLoopShouldBeWhileLoop"><priority>1</priority></rule>
+<rule ref="rulesets/java/unnecessary.xml/UnnecessaryConversionTemporary"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/OverrideBothEqualsAndHashcode"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/DoubleCheckedLocking"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/ReturnFromFinallyBlock"><priority>1</priority></rule>
+<rule ref="rulesets/java/empty.xml/EmptySynchronizedBlock"><priority>1</priority></rule>
+<rule ref="rulesets/java/empty.xml/EmptyStaticInitializer"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/UnconditionalIfStatement"><priority>1</priority></rule>
+<rule ref="rulesets/java/empty.xml/EmptyStatementNotInLoop"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/BooleanInstantiation"><priority>1</priority></rule>
+<rule ref="rulesets/java/unnecessary.xml/UnnecessaryFinalModifier"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/CollapsibleIfStatements"><priority>1</priority></rule>
+<rule ref="rulesets/java/unnecessary.xml/UselessOverridingMethod"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/ClassCastExceptionWithToArray"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/AvoidDecimalLiteralsInBigDecimalConstructor"><priority>1</priority></rule>
+<rule ref="rulesets/java/unnecessary.xml/UselessOperationOnImmutable"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/MisplacedNullCheck"><priority>1</priority></rule>
+<rule ref="rulesets/java/unnecessary.xml/UnusedNullCheckInEquals"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/BrokenNullCheck"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/BigIntegerInstantiation"><priority>1</priority></rule>
+<rule ref="rulesets/java/basic.xml/AvoidMultipleUnaryOperators"><priority>1</priority></rule>
+<rule ref="rulesets/java/empty.xml/EmptyInitializer"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/braces.xml/IfStmtsMustUseBraces"><priority>1</priority></rule>
+<rule ref="rulesets/java/braces.xml/WhileLoopsMustUseBraces"><priority>1</priority></rule>
+<rule ref="rulesets/java/braces.xml/IfElseStmtsMustUseBraces"><priority>1</priority></rule>
+<rule ref="rulesets/java/braces.xml/ForLoopsMustUseBraces"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/unusedcode.xml/UnusedModifier"><priority>1</priority></rule>
+<rule ref="rulesets/java/controversial.xml/DontImportSun"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/design.xml/SimplifyBooleanReturns"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/SimplifyBooleanExpressions"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/AvoidReassigningParameters"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/FinalFieldCouldBeStatic"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/NonStaticInitializer"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/OptimizableToArrayCall"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/BadComparison"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/EqualsNull"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/InstantiationToGetClass"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/IdempotentOperations"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/AssignmentToNonFinalStatic"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/MissingStaticMethodInNonInstantiatableClass"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/UseNotifyAllInsteadOfNotify"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/AvoidInstanceofChecksInCatchClause"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/SimplifyConditional"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/UnnecessaryLocalBeforeReturn"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/NonThreadSafeSingleton"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/AvoidConstantsInterface"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/UnsynchronizedStaticDateFormatter"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/PreserveStackTrace"><priority>1</priority></rule>
+<rule ref="rulesets/java/design.xml/UseCollectionIsEmpty"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/finalizers.xml/EmptyFinalizer"><priority>1</priority></rule>
+<rule ref="rulesets/java/finalizers.xml/FinalizeOnlyCallsSuperFinalize"><priority>1</priority></rule>
+<rule ref="rulesets/java/finalizers.xml/FinalizeOverloaded"><priority>1</priority></rule>
+<rule ref="rulesets/java/finalizers.xml/FinalizeDoesNotCallSuperFinalize"><priority>1</priority></rule>
+<rule ref="rulesets/java/finalizers.xml/FinalizeShouldBeProtected"><priority>1</priority></rule>
+<rule ref="rulesets/java/finalizers.xml/AvoidCallingFinalize"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/imports.xml/DuplicateImports"><priority>1</priority></rule>
+<rule ref="rulesets/java/imports.xml/DontImportJavaLang"><priority>1</priority></rule>
+<rule ref="rulesets/java/imports.xml/UnusedImports"><priority>1</priority></rule>
+<rule ref="rulesets/java/imports.xml/ImportFromSamePackage"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/logging-java.xml/MoreThanOneLogger"><priority>1</priority></rule>
+<rule ref="rulesets/java/logging-java.xml/LoggerIsNotStaticFinal"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/migrating.xml/ReplaceVectorWithList"><priority>1</priority></rule>
+<rule ref="rulesets/java/migrating.xml/ReplaceHashtableWithMap"><priority>1</priority></rule>
+<rule ref="rulesets/java/migrating.xml/ReplaceEnumerationWithIterator"><priority>1</priority></rule>
+<rule ref="rulesets/java/migrating.xml/AvoidEnumAsIdentifier"><priority>1</priority></rule>
+<rule ref="rulesets/java/migrating.xml/AvoidAssertAsIdentifier"><priority>1</priority></rule>
+<rule ref="rulesets/java/migrating.xml/IntegerInstantiation"><priority>1</priority></rule>
+<rule ref="rulesets/java/migrating.xml/ByteInstantiation"><priority>1</priority></rule>
+<rule ref="rulesets/java/migrating.xml/ShortInstantiation"><priority>1</priority></rule>
+<rule ref="rulesets/java/migrating.xml/LongInstantiation"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/naming.xml/VariableNamingConventions"><priority>1</priority></rule>
+<rule ref="rulesets/java/naming.xml/MethodNamingConventions"><priority>1</priority></rule>
+<rule ref="rulesets/java/naming.xml/ClassNamingConventions"><priority>1</priority></rule>
+<rule ref="rulesets/java/naming.xml/AvoidDollarSigns"><priority>1</priority></rule>
+<rule ref="rulesets/java/naming.xml/MethodWithSameNameAsEnclosingClass"><priority>1</priority></rule>
+<rule ref="rulesets/java/naming.xml/SuspiciousHashcodeMethodName"><priority>1</priority></rule>
+<rule ref="rulesets/java/naming.xml/SuspiciousConstantFieldName"><priority>1</priority></rule>
+<rule ref="rulesets/java/naming.xml/SuspiciousEqualsMethodName"><priority>1</priority></rule>
+<rule ref="rulesets/java/naming.xml/NoPackage"><priority>1</priority></rule>
+<rule ref="rulesets/java/naming.xml/PackageCase"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/optimizations.xml/LocalVariableCouldBeFinal"><priority>1</priority><properties><property name="violationSuppressXPath" value="//ForInit|//ForStatement/LocalVariableDeclaration"/></properties></rule>
+<rule ref="rulesets/java/optimizations.xml/MethodArgumentCouldBeFinal"><priority>1</priority></rule>
+<rule ref="rulesets/java/optimizations.xml/UseArrayListInsteadOfVector"><priority>1</priority></rule>
+<rule ref="rulesets/java/optimizations.xml/SimplifyStartsWith"><priority>1</priority></rule>
+<rule ref="rulesets/java/optimizations.xml/UseArraysAsList"><priority>1</priority></rule>
+<rule ref="rulesets/java/optimizations.xml/AvoidArrayLoops"><priority>1</priority></rule>
+<rule ref="rulesets/java/optimizations.xml/UnnecessaryWrapperObjectCreation"><priority>1</priority></rule>
+<rule ref="rulesets/java/optimizations.xml/AddEmptyString"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/strictexception.xml/AvoidCatchingThrowable"><priority>1</priority></rule>
+<rule ref="rulesets/java/strictexception.xml/SignatureDeclareThrowsException"><priority>1</priority></rule>
+<rule ref="rulesets/java/strictexception.xml/ExceptionAsFlowControl"><priority>1</priority></rule>
+<rule ref="rulesets/java/strictexception.xml/AvoidCatchingNPE"><priority>1</priority></rule>
+<rule ref="rulesets/java/strictexception.xml/AvoidThrowingRawExceptionTypes"><priority>1</priority></rule>
+<rule ref="rulesets/java/strictexception.xml/DoNotExtendJavaLangError"><priority>1</priority></rule>
+<rule ref="rulesets/java/strictexception.xml/DoNotThrowExceptionInFinally"><priority>1</priority></rule>
+<rule ref="rulesets/java/strictexception.xml/AvoidThrowingNewInstanceOfSameException"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/strings.xml/StringInstantiation"><priority>1</priority></rule>
+<rule ref="rulesets/java/strings.xml/StringToString"><priority>1</priority></rule>
+<rule ref="rulesets/java/strings.xml/InefficientStringBuffering"><priority>1</priority></rule>
+<rule ref="rulesets/java/strings.xml/UnnecessaryCaseChange"><priority>1</priority></rule>
+<rule ref="rulesets/java/strings.xml/UseStringBufferLength"><priority>1</priority></rule>
+<rule ref="rulesets/java/strings.xml/AppendCharacterWithChar"><priority>1</priority></rule>
+<rule ref="rulesets/java/strings.xml/UseIndexOfChar"><priority>1</priority></rule>
+<rule ref="rulesets/java/strings.xml/InefficientEmptyStringCheck"><priority>1</priority></rule>
+<rule ref="rulesets/java/strings.xml/UselessStringValueOf"><priority>1</priority></rule>
+<rule ref="rulesets/java/strings.xml/StringBufferInstantiationWithChar"><priority>1</priority></rule>
+<rule ref="rulesets/java/strings.xml/UseEqualsToCompareStrings"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/typeresolution.xml/UnusedImports"><priority>1</priority><properties><property name="violationSuppressXPath" value="//ImportDeclaration[@ImportOnDemand='true']"/></properties></rule>
+<rule ref="rulesets/java/typeresolution.xml/SignatureDeclareThrowsException"><priority>1</priority></rule>
+
+<rule ref="rulesets/java/unusedcode.xml/UnusedPrivateField"><priority>1</priority></rule>
+<rule ref="rulesets/java/unusedcode.xml/UnusedLocalVariable"><priority>1</priority></rule>
+<rule ref="rulesets/java/unusedcode.xml/UnusedPrivateMethod"><priority>1</priority></rule>
+<rule ref="rulesets/java/unusedcode.xml/UnusedFormalParameter"><priority>1</priority></rule>
+
+</ruleset>

lagdisplay/src/com/dmdirc/addons/lagdisplay/LagDisplayManager.java

     }
 
     @Handler
-    public void HandleServerPingSent(final ServerPingsentEvent event) {
+    public void handleServerPingSent(final ServerPingsentEvent event) {
         if (!event.getConnection().getWindowModel().getConfigManager().
                 getOptionBool(domain, "usealternate")) {
             return;

logging/src/com/dmdirc/addons/logging/LoggingManager.java

     /** Date format used for "File Opened At" log. */
     private static final DateFormat OPENED_AT_FORMAT = new SimpleDateFormat(
             "EEEE MMMM dd, yyyy - HH:mm:ss");
+    /** Object for synchronising access to the date forma.t */
+    private static final Object FORMAT_LOCK = new Object();
     /** This plugin's plugin info. */
     private final String domain;
     /** Global config. */
             showBackBuffer(event.getQuery(), filename);
         }
 
-        appendLine(filename, "*** Query opened at: %s", OPENED_AT_FORMAT.format(new Date()));
-        appendLine(filename, "*** Query with User: %s", event.getQuery().getHost());
-        appendLine(filename, "");
+        synchronized (FORMAT_LOCK) {
+            appendLine(filename, "*** Query opened at: %s", OPENED_AT_FORMAT.format(new Date()));
+            appendLine(filename, "*** Query with User: %s", event.getQuery().getHost());
+            appendLine(filename, "");
+        }
     }
 
     @Handler
         final Parser parser = event.getQuery().getConnection().getParser();
         final ClientInfo client = parser.getClient(event.getQuery().getHost());
         final String filename = getLogFile(client);
-        appendLine(filename, "*** Query closed at: %s", OPENED_AT_FORMAT.format(new Date()));
+
+        synchronized (FORMAT_LOCK) {
+            appendLine(filename, "*** Query closed at: %s", OPENED_AT_FORMAT.format(new Date()));
+        }
+
         if (openFiles.containsKey(filename)) {
             StreamUtils.close(openFiles.get(filename).writer);
             openFiles.remove(filename);
             showBackBuffer(event.getChannel(), filename);
         }
 
-        appendLine(filename, "*** Channel opened at: %s", OPENED_AT_FORMAT.format(new Date()));
-        appendLine(filename, "");
+        synchronized (FORMAT_LOCK) {
+            appendLine(filename, "*** Channel opened at: %s", OPENED_AT_FORMAT.format(new Date()));
+            appendLine(filename, "");
+        }
     }
 
     @Handler
     public void handleChannelClosed(final ChannelClosedEvent event) {
         final String filename = getLogFile(event.getChannel().getName());
 
-        appendLine(filename, "*** Channel closed at: %s", OPENED_AT_FORMAT.format(new Date()));
+        synchronized (FORMAT_LOCK) {
+            appendLine(filename, "*** Channel closed at: %s", OPENED_AT_FORMAT.format(new Date()));
+        }
+
         if (openFiles.containsKey(filename)) {
             StreamUtils.close(openFiles.get(filename).writer);
             openFiles.remove(filename);

mediasource_dbus/src/com/dmdirc/addons/mediasource_dbus/MPRISSource.java

                 result.append('0');
             }
 
-            result.append(hours).append(":");
+            result.append(hours).append(':');
         }
 
         if (minutes < 10) {
             result.append('0');
         }
 
-        result.append(minutes).append(":");
+        result.append(minutes).append(':');
 
         if (seconds < 10) {
             result.append('0');

mediasource_dcop/src/com/dmdirc/addons/mediasource_dcop/KaffeineSource.java

                 result.append('0');
             }
 
-            result.append(hours).append(":");
+            result.append(hours).append(':');
         }
 
         if (minutes < 10) {
             result.append('0');
         }
 
-        result.append(minutes).append(":");
+        result.append(minutes).append(':');
 
         if (seconds < 10) {
             result.append('0');

mediasource_dcop/src/com/dmdirc/addons/mediasource_dcop/NoatunSource.java

         final long seconds = secondsInput % 60;
 
         if (hours > 0) {
-            result.append(hours).append(":");
+            result.append(hours).append(':');
         }
 
-        result.append(minutes).append(":");
-        result.append(seconds).append(":");
+        result.append(minutes).append(':');
+        result.append(seconds).append(':');
 
         return result.toString();
     }

mediasource_windows/src/com/dmdirc/addons/mediasource_windows/DllSource.java

         final long seconds = (secondsInput % 60);
 
         if (hours > 0) {
-            result.append(hours).append(":");
+            result.append(hours).append(':');
         }
         result.append(String.format("%0,2d:%0,2d", minutes, seconds));
 

ui_swing/src/com/dmdirc/addons/ui_swing/UIUtilities.java

             } catch (InterruptedException ex) {
                 //Ignore
             } catch (InvocationTargetException ex) {
-                throw new RuntimeException(ex);
+                throw new RuntimeException(ex); // NOPMD
             }
         }
     }
                 return task.get();
             }
         } catch (Exception ex) {
-            throw new RuntimeException(ex);
+            throw new RuntimeException(ex); // NOPMD
         }
     }
 

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

 
             for (ChannelClientInfo value : values) {
                 if (builder.length() > 0) {
-                    builder.append("\n");
+                    builder.append('\n');
                 }
 
                 builder.append(value.getClient().getNickname());

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

     private final TextFrame parent;
     /** Colour Manager. */
     private final ColourManager colourManager;
-    /** Config to read from. */
-    private final AggregateConfigProvider config;
     /** Close button. */
     private ImageButton<Object> closeButton;
     /** Next match button. */
         listeners = new ListenerList();
 
         this.parent = newParent;
-        this.config = parent.getContainer().getConfigManager();
         this.colourManager = colourManager;
 
         getInputMap(JComponent.WHEN_FOCUSED).
         final LinePosition result = up ? searcher.searchUp() : searcher.
                 searchDown();
 
-        if (result == null) {
-            //Do nothing
-        } else if ((textPane.getSelectedRange().getEndLine() != 0 || textPane.
-                getSelectedRange().getEndPos() != 0)
-                && ((up && result.getEndLine() > textPane.getSelectedRange().
-                getEndLine())
-                || (!up && result.getStartLine() < textPane.getSelectedRange().
-                getStartLine()))) {
-            wrapIndicator.setVisible(true);
-            textPane.setScrollBarPosition(result.getEndLine());
-            textPane.setSelectedText(result);
-            validator.setValidates(true);
-            searchBox.checkError();
-        } else {
-            //found, select and return found
-            textPane.setScrollBarPosition(result.getEndLine());
-            textPane.setSelectedText(result);
-            validator.setValidates(true);
-            searchBox.checkError();
+        if (result != null) {
+            if ((textPane.getSelectedRange().getEndLine() != 0 || textPane.
+                    getSelectedRange().getEndPos() != 0)
+                    && (up && result.getEndLine() > textPane.getSelectedRange().getEndLine()
+                    || !up && result.getStartLine() < textPane.getSelectedRange().getStartLine())) {
+                wrapIndicator.setVisible(true);
+                textPane.setScrollBarPosition(result.getEndLine());
+                textPane.setSelectedText(result);
+                validator.setValidates(true);
+                searchBox.checkError();
+            } else {
+                //found, select and return found
+                textPane.setScrollBarPosition(result.getEndLine());
+                textPane.setSelectedText(result);
+                validator.setValidates(true);
+                searchBox.checkError();
+            }
         }
     }
 

ui_swing/src/com/dmdirc/addons/ui_swing/components/menubar/MenuBar.java

     /** Normal menu count. */
     private final int menuItemCount;
     /** Stores a list of tokens used to remove added menu items. */
-    private final Map<String, JMenuItem> menuItems;
+    private final Map<String, JMenuItem> menuItems; // NOPMD
 
     /**
      * Instantiates a new menu bar.

ui_swing/src/com/dmdirc/addons/ui_swing/dialogs/actioneditor/ActionConditionDisplayPanel.java

             }
 
             sb.append(condition.getComponent().getName());
-            sb.append(" ");
+            sb.append(' ');
 
             if (condition.getComparison() == null) {
                 sb.append(" ...");
             }
 
             sb.append(condition.getTarget().replace("<", "&lt;"));
-            sb.append("'");
+            sb.append('\'');
             return sb.toString();
         }
     }

ui_swing/test/com/dmdirc/addons/ui_swing/dialogs/profiles/AddNicknameValidatorTest.java

         final ProfileManagerModel model = mock(ProfileManagerModel.class);
         when(model.getNicknames()).thenReturn(Arrays.asList(
                 new String[]{"nickname1", "nickname2",}));
-        AddNicknameValidator instance = new AddNicknameValidator(model);
+        final AddNicknameValidator instance = new AddNicknameValidator(model);
         assertTrue(instance.validate("nickname1").isFailure());
     }
 
         final ProfileManagerModel model = mock(ProfileManagerModel.class);
         when(model.getNicknames()).thenReturn(Arrays.asList(
                 new String[]{"nickname1", "nickname2",}));
-        AddNicknameValidator instance = new AddNicknameValidator(model);
+        final AddNicknameValidator instance = new AddNicknameValidator(model);
         assertFalse(instance.validate("nickname").isFailure());
     }
 }

ui_swing/test/com/dmdirc/addons/ui_swing/dialogs/profiles/EditNicknameValidatorTest.java

         when(model.getSelectedNickname()).thenReturn(null);
         when(model.getNicknames()).thenReturn(Arrays.asList(
                 new String[]{"nickname1", "nickname2",}));
-        EditNicknameValidator instance = new EditNicknameValidator(model);
+        final EditNicknameValidator instance = new EditNicknameValidator(model);
         assertFalse(instance.validate("nickname").isFailure());
     }
 
         when(model.getSelectedNickname()).thenReturn("nickname");
         when(model.getNicknames()).thenReturn(Arrays.asList(
                 new String[]{"nickname1", "nickname2",}));
-        EditNicknameValidator instance = new EditNicknameValidator(model);
+        final EditNicknameValidator instance = new EditNicknameValidator(model);
         assertFalse(instance.validate("nickname").isFailure());
     }
 
         when(model.getSelectedNickname()).thenReturn("nickname");
         when(model.getNicknames()).thenReturn(Arrays.asList(
                 new String[]{"nickname1", "nickname2",}));
-        EditNicknameValidator instance = new EditNicknameValidator(model);
+        final EditNicknameValidator instance = new EditNicknameValidator(model);
         assertFalse(instance.validate("nickname3").isFailure());
     }
 }

ui_swing/test/com/dmdirc/addons/ui_swing/dialogs/profiles/ProfileManagerModelTest.java

         when(configProvider.getOption("profile", "realname")).thenReturn(prefix + "realname");
         when(configProvider.getOption("profile", "ident")).thenReturn(prefix + "ident");
 
-        final Profile profile = new Profile(identityFactory, configProvider);
-        return profile;
+        return new Profile(identityFactory, configProvider);
     }
 
     private ProfileManagerModel createModel() {
     }
 
     @Before
-    public void setup() throws Exception {
+    public void setup() {
         final List<ConfigProvider> identities = Collections.emptyList();
         manager = mock(IdentityController.class);
         when(manager.getProvidersByType("profile")).thenReturn(identities);
         final IdentityController im = mock(IdentityController.class);
         when(im.getProvidersByType("profile")).thenReturn(identities);
 
-        ProfileManagerModel instance = new ProfileManagerModel(im, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(im, identityFactory);
         instance.load();
 
         assertEquals(Arrays.asList(new Profile[]{new Profile(identityFactory, configProvider), }), instance.getProfiles());
     public void testGetAndSetProfiles() {
         final List<Profile> newProfiles = Arrays.asList(
                 new Profile[]{createProfile("2"),});
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertEquals(Collections.emptyList(), instance.getProfiles());
         instance.setProfiles(newProfiles);
         assertEquals(newProfiles, instance.getProfiles());
     @Test
     public void testAddProfiles() {
         final Profile newProfile = createProfile("2");
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         assertEquals(Arrays.asList(new Profile[]{defaultProfile,}), instance.getProfiles());
         final List<Profile> newProfiles = new ArrayList<>();
         newProfiles.add(defaultProfile);
     @Test
     public void testDeleteProfileNullProfile() {
         final Profile first = createProfile("1");
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(first);
         instance.setSelectedProfile(first);
         assertEquals(first, instance.getSelectedProfile());
     public void testDeleteProfileNotSelected() {
         final Profile first = createProfile("1");
         final Profile second = createProfile("2");
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(first);
         instance.addProfile(second);
         instance.setSelectedProfile(second);
     public void testDeleteProfileLastProfile() {
         final Profile first = createProfile("1");
         final Profile second = createProfile("2");
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(first);
         instance.addProfile(second);
         instance.setSelectedProfile(second);
     public void testDeleteProfileFirstProfile() {
         final Profile first = createProfile("1");
         final Profile second = createProfile("2");
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(first);
         instance.addProfile(second);
         instance.setSelectedProfile(first);
         final Profile first = createProfile("1");
         final Profile second = createProfile("2");
         final Profile third = createProfile("3");
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(first);
         instance.addProfile(second);
         instance.addProfile(third);
     @Test
     public void testDeleteProfileOnlyProfile() {
         final Profile first = createProfile("1");
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(first);
         instance.setSelectedProfile(first);
         assertEquals(first, instance.getSelectedProfile());
      */
     @Test
     public void testGetAndSetSelectedProfile() {
-        Profile profile2 = createProfile("2");
-        Profile profile3 = createProfile("3");
-        Profile profile4 = createProfile("4");
-        ProfileManagerModel instance = createModel();
+        final Profile profile2 = createProfile("2");
+        final Profile profile3 = createProfile("3");
+        final Profile profile4 = createProfile("4");
+        final ProfileManagerModel instance = createModel();
         assertEquals(defaultProfile, instance.getSelectedProfile());
         instance.addProfile(profile2);
         assertEquals(profile2, instance.getSelectedProfile());
      */
     @Test
     public void testGetAndSetNicknames() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         final List<String> expResult = defaultProfile.getNicknames();
         List<String> result = instance.getNicknames();
         assertEquals(expResult, result);
      */
     @Test
     public void testAddNickname() {
-        String nickname = "foo";
-        ProfileManagerModel instance = createModel();
+        final String nickname = "foo";
+        final ProfileManagerModel instance = createModel();
         assertEquals(defaultProfile.getNicknames(), instance.getNicknames());
         final List<String> nicknames = new ArrayList<>(defaultProfile.getNicknames());
         instance.addNickname(nickname);
      */
     @Test
     public void testDeleteNicknameObject() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         final List<String> nicknames = new ArrayList<>(instance.getNicknames());
         Object nickname = (Object) "";
         assertEquals(nicknames, instance.getNicknames());
      */
     @Test
     public void testDeleteNicknameNullSelectedProfile() {
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertNull(instance.getSelectedProfile());
         assertTrue(instance.getNicknames().isEmpty());
         instance.deleteNickname("1nickname");
     public void testDeleteNicknameNullNickname() {
         final Profile profile = createProfile("1");
         final String first = "1nickname";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(profile);
         instance.setSelectedProfile(profile);
         instance.setSelectedNickname(first);
         final Profile profile = createProfile("1");
         final String first = "1nickname";
         final String second = "1nickname2";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(profile);
         instance.setSelectedProfile(profile);
         instance.addNickname(second);
         final Profile profile = createProfile("1");
         final String first = "1nickname";
         final String second = "1nickname2";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(profile);
         instance.setSelectedProfile(profile);
         instance.addNickname(second);
         final Profile profile = createProfile("1");
         final String first = "1nickname";
         final String second = "1nickname2";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(profile);
         instance.setSelectedProfile(profile);
         instance.addNickname(second);
         final String first = "1nickname";
         final String second = "1nickname2";
         final String third = "1nickname3";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(profile);
         instance.setSelectedProfile(profile);
         instance.addNickname(second);
     public void testDeleteNicknameOnlyNickname() {
         final Profile profile = createProfile("1");
         final String first = "1nickname";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.addProfile(profile);
         instance.setSelectedProfile(profile);
         instance.setSelectedNickname(first);
     @Test
     public void testGetAndSetSelectedNickname() {
         final String selectedNickname = "1nickname";
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         assertNull(instance.getSelectedNickname());
         instance.setSelectedNickname(selectedNickname);
         assertEquals(selectedNickname, instance.getSelectedNickname());
      */
     @Test
     public void testEditNickname() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         final List<String> nicknames = new ArrayList<>(instance.getNicknames());
         final String nickname = nicknames.get(0);
         assertEquals(nicknames, instance.getNicknames());
      */
     @Test
     public void testGetAndSetName() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         final String name = defaultProfile.getName();
         final String newName = "foo";
         assertEquals(name, instance.getName());
      */
     @Test
     public void testGetAndSetRealname() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         final String name = "1realname";
         final String newName = "foo";
         assertEquals(name, instance.getRealname());
      */
     @Test
     public void testGetAndSetIdent() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         final String name = "1ident";
         final String newName = "foo";
         assertEquals(name, instance.getIdent());
      */
     @Test
     public void testIsManipulateProfileAllowed() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         assertTrue(instance.isManipulateProfileAllowed());
         instance.setProfiles(Arrays.asList(new Profile[]{}));
         assertFalse(instance.isManipulateProfileAllowed());
      */
     @Test
     public void testIsManipulateNicknameAllowed() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         assertFalse(instance.isManipulateNicknameAllowed());
         instance.setSelectedNickname("1nickname");
         assertTrue(instance.isManipulateNicknameAllowed());
         profile.setName("*");
         profile.setRealname("");
         profile.setIdent("*");
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         assertTrue(instance.isChangeProfileAllowed());
         instance.setProfiles(Arrays.asList(new Profile[]{profile,}));
         assertFalse(instance.isChangeProfileAllowed());
         profile.setName("*");
         profile.setRealname("");
         profile.setIdent("*");
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         assertTrue(instance.isOKAllowed());
         instance.setProfiles(Arrays.asList(new Profile[]{}));
         assertFalse(instance.isOKAllowed());
      */
     @Test
     public void testIsNameInValid() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         instance.setProfiles(Arrays.asList(defaultProfile));
         instance.setName("\\");
         assertTrue(instance.isNameValid().isFailure());
      */
     @Test
     public void testIsNameValid() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         instance.setProfiles(Arrays.asList(defaultProfile));
         instance.setName("profile");
         assertFalse(instance.isNameValid().isFailure());
      */
     @Test
     public void testIsNicknamesInValid() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         instance.setProfiles(Arrays.asList(defaultProfile));
         instance.setNicknames(Arrays.asList(new String[]{}));
         assertTrue(instance.isNicknamesValid().isFailure());
      */
     @Test
     public void testIsNicknamesValid() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         instance.setProfiles(Arrays.asList(defaultProfile));
         instance.setNicknames(Arrays.asList(new String[]{"nickname"}));
         assertFalse(instance.isNicknamesValid().isFailure());
      */
     @Test
     public void testIsRealnameInValid() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         instance.setProfiles(Arrays.asList(defaultProfile));
         instance.setRealname("");
         assertTrue(instance.isRealnameValid().isFailure());
      */
     @Test
     public void testIsRealnameValid() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         instance.setProfiles(Arrays.asList(defaultProfile));
         instance.setRealname("realname");
         assertFalse(instance.isRealnameValid().isFailure());
      */
     @Test
     public void testIsIdentInValid() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         instance.setProfiles(Arrays.asList(defaultProfile));
         instance.setIdent("*");
         assertTrue(instance.isIdentValid().isFailure());
      */
     @Test
     public void testIsIdentValid() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         instance.setProfiles(Arrays.asList(defaultProfile));
         instance.setIdent("ident");
         assertFalse(instance.isIdentValid().isFailure());
      */
     @Test
     public void testIsIdentValidEmptyString() {
-        ProfileManagerModel instance = createModel();
+        final ProfileManagerModel instance = createModel();
         instance.setProfiles(Arrays.asList(defaultProfile));
         instance.setIdent("");
         assertFalse(instance.isIdentValid().isFailure());
      */
     @Test
     public void testNullSelectedProfileGetters() {
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertTrue("".equals(instance.getSelectedNickname()));
         assertTrue("".equals(instance.getName()));
         assertTrue("".equals(instance.getRealname()));
      */
     @Test
     public void testNullSelectedProfileValidators() {
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertFalse(instance.isNameValid().isFailure());
         assertFalse(instance.isNicknamesValid().isFailure());
         assertFalse(instance.isRealnameValid().isFailure());
     @Test
     public void testNullSelectedProfileSetSelectedNickname() {
         final String test = "test";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertTrue("".equals(instance.getSelectedNickname()));
         instance.setSelectedNickname(test);
         assertTrue("".equals(instance.getSelectedNickname()));
     @Test
     public void testNullSelectedProfileSetName() {
         final String test = "test";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertTrue("".equals(instance.getName()));
         instance.setName(test);
         assertTrue("".equals(instance.getName()));
     @Test
     public void testNullSelectedProfileSetRealname() {
         final String test = "test";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertTrue("".equals(instance.getRealname()));
         instance.setRealname(test);
         assertTrue("".equals(instance.getRealname()));
     @Test
     public void testNullSelectedProfileSetIdent() {
         final String test = "test";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertTrue("".equals(instance.getIdent()));
         instance.setIdent(test);
         assertTrue("".equals(instance.getIdent()));
     @Test
     public void testNullSelectedProfileSetNicknames() {
         final String test = "test";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertTrue(instance.getNicknames().isEmpty());
         instance.setNicknames(Arrays.asList(new String[]{test, }));
         assertTrue(instance.getNicknames().isEmpty());
     @Test
     public void testNullSelectedProfileAddNickname() {
         final String test = "test";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertTrue(instance.getNicknames().isEmpty());
         instance.addNickname(test);
         assertTrue(instance.getNicknames().isEmpty());
     @Test
     public void testNullSelectedProfileDeleteNickname() {
         final String test = "test";
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         assertTrue(instance.getNicknames().isEmpty());
         instance.deleteNickname(test);
         assertTrue(instance.getNicknames().isEmpty());
         final Profile second = mock(Profile.class);
         when(first.isDeleted()).thenReturn(false);
         when(second.isDeleted()).thenReturn(true);
-        ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
+        final ProfileManagerModel instance = new ProfileManagerModel(manager, identityFactory);
         instance.setProfiles(Arrays.asList(new Profile[]{first, second, }));
         instance.save();
         verify(first).save();

ui_swing/test/com/dmdirc/addons/ui_swing/dialogs/profiles/ProfileNameValidatorTest.java

      */
     @Test
     public void testValidateNoDupes() {
-        ProfileRenameValidator instance = new ProfileRenameValidator(model);
+        final ProfileRenameValidator instance = new ProfileRenameValidator(model);
         assertFalse(instance.validate("Random").isFailure());
     }
 
      */
     @Test
     public void testValidateNonSelectedDupe() {
-        ProfileRenameValidator instance = new ProfileRenameValidator(model);
+        final ProfileRenameValidator instance = new ProfileRenameValidator(model);
         assertTrue(instance.validate("other").isFailure());
     }
 
      */
     @Test
     public void testValidateSelectedDupe() {
-        ProfileRenameValidator instance = new ProfileRenameValidator(model);
+        final ProfileRenameValidator instance = new ProfileRenameValidator(model);
         assertFalse(instance.validate("selected").isFailure());
     }
 }