Doladěno UI agendy na definici středisek a komisí.

closes #97

src/main/java/info/bukova/isspst/data/JobMapping.java

@@ -68,4 +68,9 @@ public class JobMapping {
 		return false;
 	}
 
+	@Override
+	public String toString() {
+		return this.getMember() + " - " + this.getRole().getDescription();
+	}
+
 }

src/main/java/info/bukova/isspst/data/Member.java

@@ -26,7 +26,7 @@ public abstract class Member implements DataModel {
 	private boolean valid;
 	
 	public abstract String getFullName();
-	public abstract boolean isHasJob();
+	public abstract boolean isPerson();
 
 	public int getId() {
 		return id;

src/main/java/info/bukova/isspst/data/User.java

@@ -192,7 +192,7 @@ public class User extends Member implements UserDetails, DataModel {
 	}
 
 	@Override
-	public boolean isHasJob() {
+	public boolean isPerson() {
 		return true;
 	}
 

src/main/java/info/bukova/isspst/data/Workgroup.java

@@ -90,11 +90,11 @@ public class Workgroup extends Member implements OwnedDataModel {
 
 	@Override
 	public String getFullName() {
-		return name;
+		return code + " " + name;
 	}
 
 	@Override
-	public boolean isHasJob() {
+	public boolean isPerson() {
 		return false;
 	}
 
@@ -121,5 +121,10 @@ public class Workgroup extends Member implements OwnedDataModel {
 		}
 		return false;
 	}
+
+	@Override
+	public String toString() {
+		return getFullName();
+	}
 	
 }

src/main/java/info/bukova/isspst/filters/WorkgroupFilter.java

@@ -0,0 +1,56 @@
+package info.bukova.isspst.filters;
+
+import static info.bukova.isspst.StringUtils.nullStr;
+import info.bukova.isspst.data.Workgroup;
+
+import org.hamcrest.Description;
+import org.hamcrest.Factory;
+import org.hamcrest.Matcher;
+import org.hamcrest.TypeSafeMatcher;
+
+public class WorkgroupFilter implements Filter<Workgroup> {
+	
+	private Workgroup condWorkgroup;
+	
+	public WorkgroupFilter(Workgroup condWorkgroup) {
+		this.condWorkgroup = condWorkgroup;
+	}
+
+	private static class WorkgroupMatcher extends TypeSafeMatcher<Workgroup> {
+
+		private Workgroup condWorkgroup;
+		
+		public WorkgroupMatcher(Workgroup cond) {
+			this.condWorkgroup = cond;
+		}
+		
+		@Override
+		public void describeTo(Description desc) {
+			desc.appendText("material matches");
+		}
+
+		@Override
+		public boolean matchesSafely(Workgroup item) {
+			return nullStr(item.getCode()).toLowerCase().contains(nullStr(condWorkgroup.getCode()).toLowerCase()) 
+					&& nullStr(item.getName()).toLowerCase().contains(nullStr(condWorkgroup.getName()).toLowerCase())
+					&& item.isCentre() == condWorkgroup.isCentre();
+		}
+		
+		@Factory
+		public static Matcher<Workgroup> matchBuilding(Workgroup material) {
+			return new WorkgroupMatcher(material);
+		}
+	}
+	
+	@Override
+	public WorkgroupMatcher matcher() {
+		return new WorkgroupMatcher(condWorkgroup);
+	}
+
+	@Override
+	public String queryString() {
+		// TODO query string
+		return "";
+	}
+
+}

src/main/java/info/bukova/isspst/services/workgroups/WorkgroupException.java

@@ -12,6 +12,7 @@ public class WorkgroupException extends IsspstException {
 	public final static String NO_WORGROUP = "WorkgroupNotAllowed";
 	public final static String MSUT_HAS_CENTER = "RoleMustHasCentre";
 	public final static String MUST_HAS_WORKGROUP = "WorkgroupMustHasCentre";
+	public final static String CYCLIC_MEMBER = "CyclicMembership";
 	
 	public WorkgroupException(String message) {
 		super(message);

src/main/java/info/bukova/isspst/services/workgroups/WorkgroupServiceImpl.java

@@ -15,7 +15,7 @@ public class WorkgroupServiceImpl extends AbstractOwnedService<Workgroup> implem
 
 	@Override
 	public void addMember(Workgroup workgroup, Member member, Role role) {
-		if (!member.isHasJob() && !role.getAuthority().equals(Constants.ROLE_USER)) {
+		if (!member.isPerson() && !role.getAuthority().equals(Constants.ROLE_USER)) {
 			throw new WorkgroupException(WorkgroupException.NO_WORGROUP);
 		}
 		if (workgroup.isCentre() && !role.isCentre()) {
@@ -25,6 +25,14 @@ public class WorkgroupServiceImpl extends AbstractOwnedService<Workgroup> implem
 			throw new WorkgroupException(WorkgroupException.MUST_HAS_WORKGROUP);
 		}
 		
+		if (!checkGroups(workgroup, member)) {
+			throw new WorkgroupException(WorkgroupException.CYCLIC_MEMBER);
+		}
+		
+		if (!member.isPerson() && !checkGroups((Workgroup)member, workgroup)) {
+			throw new WorkgroupException(WorkgroupException.CYCLIC_MEMBER);
+		}
+		
 		JobMapping mapping = new JobMapping(member, role);
 		if (!workgroup.getMembers().contains(mapping)) {
 			workgroup.getMembers().add(mapping);
@@ -53,5 +61,40 @@ public class WorkgroupServiceImpl extends AbstractOwnedService<Workgroup> implem
 		
 		return false;
 	}
+	
+	private boolean checkGroups(Workgroup workgroup, Member member) {
+		if (member.equals(workgroup)) {
+			return false;
+		}
+		
+		if (member.isPerson()) {
+			return true;
+		}
+		
+		Workgroup toAdd = Workgroup.class.cast(member); 
+		for (JobMapping jm : toAdd.getMembers()) {
+			if (!jm.getMember().isPerson() && !checkGroups(workgroup, jm.getMember())) {
+				return false;
+			}
+		}
+		
+		return true;
+	}
+	
+/*	private boolean noIntersect(Workgroup w1, Workgroup w2) {
+		for (JobMapping jm1 : w1.getMembers()) {
+			for (JobMapping jm2 : w2.getMembers()) {
+				if (jm1.equals(jm2)) {
+					return false;
+				}
+			}
+		}
+		
+		return true;
+	}
+	
+	private boolean checkIntersect() {
+		return true;
+	}*/
 
 }

src/main/java/info/bukova/isspst/ui/BoolConverter.java

@@ -0,0 +1,21 @@
+package info.bukova.isspst.ui;
+
+import info.bukova.isspst.StringUtils;
+
+import org.zkoss.bind.BindContext;
+import org.zkoss.bind.Converter;
+import org.zkoss.zk.ui.Component;
+
+public class BoolConverter implements Converter<String, Boolean, Component> {
+
+	@Override
+	public Boolean coerceToBean(String str, Component component, BindContext cx) {
+		return false;
+	}
+
+	@Override
+	public String coerceToUi(Boolean val, Component component, BindContext cx) {
+		return StringUtils.localize(val.toString());
+	}
+
+}

src/main/java/info/bukova/isspst/ui/workgroups/WorkgroupForm.java

@@ -35,16 +35,23 @@ public class WorkgroupForm extends FormViewModel<Workgroup> {
 	public void init() {
 		users = new ArrayList<Member>();
 		users.addAll(userService.getAll());
-		List<Workgroup> wg = workgroupService.getWorkgroups();
-		
-		for (Workgroup w : wg) {
-			if (!workgroupService.isMember(getDataBean(), w)) {
-				users.add(w);
-			}
-		}
 	}
 	
 	public List<Member> getUsers() {
+		if (getDataBean().isCentre()) {
+			List<Member> listWithWg = new ArrayList<Member>(users);
+			
+			List<Workgroup> wg = workgroupService.getWorkgroups();
+			
+			for (Workgroup w : wg) {
+				if (!workgroupService.isMember(getDataBean(), w)) {
+					listWithWg.add(w);
+				}
+			}
+			
+			return listWithWg;
+		}
+		
 		return users;
 	}
 	
@@ -56,7 +63,7 @@ public class WorkgroupForm extends FormViewModel<Workgroup> {
 		}
 	}
 	
-	@NotifyChange({"workgroupRoles", "centre"})
+	@NotifyChange({"workgroupRoles", "centre", "users"})
 	public void setCentre(boolean centre) {
 		getDataBean().setCentre(centre);
 	}
@@ -106,7 +113,7 @@ public class WorkgroupForm extends FormViewModel<Workgroup> {
 		
 		if (!target.equals("users")) {
 			moveUser(u.getMember(), event);
-		} else if (!u.getMember().isHasJob()) {
+		} else if (!u.getMember().isPerson()) {
 			users.add(u.getMember());
 		}
 	}
@@ -121,7 +128,7 @@ public class WorkgroupForm extends FormViewModel<Workgroup> {
 		
 		workgroupService.addMember(getDataBean(), u, roleService.getRoleByAuthority(target));
 		
-		if (!u.isHasJob()) {
+		if (!u.isPerson()) {
 			users.remove(u);
 		}
 	}

src/main/java/info/bukova/isspst/ui/workgroups/WorkgroupList.java

@@ -1,5 +1,7 @@
 package info.bukova.isspst.ui.workgroups;
 
+import org.zkoss.bind.annotation.Command;
+import org.zkoss.bind.annotation.GlobalCommand;
 import org.zkoss.bind.annotation.Init;
 import org.zkoss.bind.annotation.NotifyChange;
 import org.zkoss.zk.ui.select.annotation.WireVariable;
@@ -8,7 +10,9 @@ import org.zkoss.zul.TreeNode;
 
 import info.bukova.isspst.data.Member;
 import info.bukova.isspst.data.Workgroup;
+import info.bukova.isspst.filters.WorkgroupFilter;
 import info.bukova.isspst.services.workgroups.WorkgroupService;
+import info.bukova.isspst.ui.BoolConverter;
 import info.bukova.isspst.ui.ListViewModel;
 
 public class WorkgroupList extends ListViewModel<Workgroup> {
@@ -16,13 +20,16 @@ public class WorkgroupList extends ListViewModel<Workgroup> {
 	@WireVariable
 	private WorkgroupService workgroupService;
 	private WorkgroupTreeFactory treeFactory;
+	private BoolConverter boolConverter;
 	
 	@Init
 	public void init() {
 		service = workgroupService;
 		dataClass = Workgroup.class;
 		formZul = "workgroupForm.zul";
+		dataFilter = new WorkgroupFilter(getFilterTemplate());
 		treeFactory = new WorkgroupTreeFactory();
+		boolConverter = new BoolConverter();
 	}
 	
 	public TreeModel<TreeNode<Member>> getWorkgroupTreeModel() {
@@ -37,5 +44,23 @@ public class WorkgroupList extends ListViewModel<Workgroup> {
 	public void setDataBean(Workgroup data) {
 		super.setDataBean(data);
 	}
+
+	@Override
+	@GlobalCommand
+	@NotifyChange({ "dataList", "dataBean", "workgroupTreeModel" })
+	public void refresh() {
+		super.refresh();
+	}
+
+	@Override
+	@Command
+	@NotifyChange({ "filter", "dataList", "dataBean", "workgroupTreeModel" })
+	public void filter() {
+		super.filter();
+	}
+	
+	public BoolConverter getBoolConverter() {
+		return boolConverter;
+	}
 	
 }

src/main/java/info/bukova/isspst/ui/workgroups/WorkgroupTreeFactory.java

@@ -25,7 +25,7 @@ public class WorkgroupTreeFactory {
 		Workgroup workgroup = Workgroup.class.cast(member);
 		
 		for (JobMapping jm : workgroup.getMembers()) {
-			if (jm.getRole().getAuthority().equals(Constants.ROLE_USER) && jm.getMember().isHasJob()) {
+			if (jm.getRole().getAuthority().equals(Constants.ROLE_USER) && jm.getMember().isPerson()) {
 				children.add(new WorkgroupTreeNode(jm.getMember()));
 			} else if (jm.getRole().getAuthority().equals(Constants.ROLE_USER)) {
 				children.add(new WorkgroupTreeNode(jm.getMember(), generateChildren(jm.getMember())));