Findbugs. CodePro Audit. Checkstyle. Unit test refactoring.

James Moger

2011-05-26 2a7306a1d92522569a8bb6e5a7c0bcdd5cf4cfaa

 src/com/gitblit/wicket/panels/BasePanel.java

@@ -37,7 +37,8 @@
   }



   protected TimeZone getTimeZone() {

      return GitBlit.self().settings().getBoolean(Keys.web.useClientTimezone, false) ? GitBlitWebSession.get().getTimezone() : TimeZone.getDefault();

      return GitBlit.getBoolean(Keys.web.useClientTimezone, false) ? GitBlitWebSession.get()

            .getTimezone() : TimeZone.getDefault();

   }



   protected void setPersonSearchTooltip(Component component, String value, SearchType searchType) {

@@ -48,7 +49,7 @@
      }

   }



   public class JavascriptEventConfirmation extends AttributeModifier {

   public static class JavascriptEventConfirmation extends AttributeModifier {



      private static final long serialVersionUID = 1L;



@@ -57,7 +58,8 @@
      }



      protected String newValue(final String currentValue, final String replacementValue) {

         String prefix = "var conf = confirm('" + replacementValue + "'); " + "if (!conf) return false; ";

         String prefix = "var conf = confirm('" + replacementValue + "'); "

               + "if (!conf) return false; ";

         String result = prefix;

         if (currentValue != null) {

            result = prefix + currentValue;