From 566005dc5f2d0787c8bd0089a3d3e5bb0d3f1d75 Mon Sep 17 00:00:00 2001 From: James Moger Date: Thu, 2 May 2013 22:24:16 -0400 Subject: [PATCH] Revert "Make WicketUtils return the component for more fluid usage" This reverts commit 19352c18bd27ea61b9466110c1be874079f60b63. --- .../java/com/gitblit/wicket/WicketUtils.java | 24 +++++++------------ 1 file changed, 8 insertions(+), 16 deletions(-) diff --git a/src/main/java/com/gitblit/wicket/WicketUtils.java b/src/main/java/com/gitblit/wicket/WicketUtils.java index 5492717e..6e03032e 100644 --- a/src/main/java/com/gitblit/wicket/WicketUtils.java +++ b/src/main/java/com/gitblit/wicket/WicketUtils.java @@ -52,34 +52,29 @@ import com.gitblit.utils.TimeUtils; public class WicketUtils { - public static Component setCssClass(Component container, String value) { + public static void setCssClass(Component container, String value) { container.add(new SimpleAttributeModifier("class", value)); - return container; } - public static Component setCssStyle(Component container, String value) { + public static void setCssStyle(Component container, String value) { container.add(new SimpleAttributeModifier("style", value)); - return container; } - public static Component setCssBackground(Component container, String value) { + public static void setCssBackground(Component container, String value) { String background = MessageFormat.format("background-color:{0};", StringUtils.getColor(value)); container.add(new SimpleAttributeModifier("style", background)); - return container; } - public static Component setHtmlTooltip(Component container, String value) { + public static void setHtmlTooltip(Component container, String value) { container.add(new SimpleAttributeModifier("title", value)); - return container; } - public static Component setInputPlaceholder(Component container, String value) { + public static void setInputPlaceholder(Component container, String value) { container.add(new SimpleAttributeModifier("placeholder", value)); - return container; } - public static Component setChangeTypeCssClass(Component container, ChangeType type) { + public static void setChangeTypeCssClass(Component container, ChangeType type) { switch (type) { case ADD: setCssClass(container, "addition"); @@ -95,10 +90,9 @@ public class WicketUtils { setCssClass(container, "modification"); break; } - return container; } - public static Component setTicketCssClass(Component container, String state) { + public static void setTicketCssClass(Component container, String state) { String css = null; if (state.equals("open")) { css = "label label-important"; @@ -112,13 +106,11 @@ public class WicketUtils { if (css != null) { setCssClass(container, css); } - return container; } - public static Component setAlternatingBackground(Component c, int i) { + public static void setAlternatingBackground(Component c, int i) { String clazz = i % 2 == 0 ? "light" : "dark"; setCssClass(c, clazz); - return c; } public static Label createAuthorLabel(String wicketId, String author) { -- 2.39.5