Code cleanup

Author: javier-godoy

src/main/java/com/flowingcode/addons/applayout/AppDrawer.java

@@ -22,7 +22,7 @@
 
 
 import java.util.ArrayList;
-import java.util.Arrays;
+import java.util.Collections;
 import java.util.List;
 
 import com.flowingcode.addons.applayout.menu.MenuItem;
@@ -43,7 +43,7 @@
 @HtmlImport("bower_components/app-layout/app-drawer/app-drawer.html")
 public class AppDrawer extends Component implements HasComponents {
 
-	private PaperListbox pm = new PaperListbox(Arrays.asList(new Component[] {}));
+	private PaperListbox pm = new PaperListbox(Collections.emptyList());
 	private Component header;
 
     public AppDrawer(String title) {

src/main/java/com/flowingcode/addons/applayout/AppToolbar.java

@@ -75,10 +75,10 @@ public void setToolbarIconButtons(MenuItem[] menuItems) {
 		this.add(menu);
 		if (ctitle!=null) this.add(ctitle);
 		if (divTitle!=null) this.add(divTitle);
-		toolbarIconButtons.forEach(tib->this.add(tib));
+		toolbarIconButtons.forEach(this::add);
 	}
 
-	private List<PaperIconButton> createToolbarIconButtons(MenuItem[] menuItems) {
+	private static List<PaperIconButton> createToolbarIconButtons(MenuItem[] menuItems) {
         List<PaperIconButton> result = new ArrayList<>();
         for (MenuItem menuItem : menuItems) {
                 PaperIconButton paperIconButton = new PaperIconButton(menuItem.getIcon(),menuItem.getCommand());

src/main/java/com/flowingcode/addons/applayout/PaperButton.java

@@ -43,9 +43,7 @@ public PaperButton(String label) {
 
     public PaperButton(String label, Command command) {
     	this.setLabel(label);
-		this.getElement().addEventListener("click", e->{
-			command.execute();
-		});
+		this.getElement().addEventListener("click", e->command.execute());
     }
 
     public void setLabel(String label) {

src/main/java/com/flowingcode/addons/applayout/PaperCard.java

@@ -87,7 +87,7 @@ public void setCardActions(final MenuItem... cardActions) {
             final Div inner = new Div();
             cardActionsDiv.add(inner);
             inner.addClassNames("horizontal", "justified");
-            buttons.forEach(b -> inner.add(b));
+            buttons.forEach(inner::add);
             hasComponentsVersion.add(cardActionsDiv);
         }
     }

src/main/java/com/flowingcode/addons/applayout/PaperIconButton.java

@@ -44,9 +44,7 @@ public PaperIconButton(String icon) {
 
     public PaperIconButton(String icon, Command command) {
     	this.setIcon(icon);
-		this.getElement().addEventListener("click", e->{
-			command.execute();
-		});
+		this.getElement().addEventListener("click", e->command.execute());
     }
 
     public void setIcon(String icon) {

src/main/java/com/flowingcode/addons/applayout/PaperIconItem.java

@@ -20,14 +20,12 @@
 package com.flowingcode.addons.applayout;
 
 
-
 import com.vaadin.flow.component.Component;
 import com.vaadin.flow.component.HasComponents;
 import com.vaadin.flow.component.HasSize;
 import com.vaadin.flow.component.HasText;
 import com.vaadin.flow.component.Tag;
 import com.vaadin.flow.component.Text;
-import com.vaadin.flow.component.UI;
 import com.vaadin.flow.component.dependency.HtmlImport;
 import com.vaadin.flow.server.Command;
 

src/main/java/com/flowingcode/addons/applayout/PaperListbox.java

@@ -40,7 +40,7 @@
 public class PaperListbox extends Component implements HasComponents {
 
 	public PaperListbox(List<Component> components) {
-		components.stream().forEach(c->this.add(c));
+		components.stream().forEach(this::add);
 	}
 
 	public PaperListbox(String label, PaperItem...items) {

src/main/java/com/flowingcode/addons/applayout/menu/MenuItem.java

@@ -115,7 +115,7 @@ public void setSubMenuItems(List<MenuItem> subMenuItems) {
 	 * @return true if this item has sub menu items
 	 */
 	public boolean isSubMenuFolder() {
-		return getSubMenuItems().size() > 0;
+		return !getSubMenuItems().isEmpty();
 	}
 
 	/**