|
From: <id...@us...> - 2009-04-27 11:32:32
|
Revision: 183
http://cse-ip.svn.sourceforge.net/cse-ip/?rev=183&view=rev
Author: idueppe
Date: 2009-04-27 11:31:38 +0000 (Mon, 27 Apr 2009)
Log Message:
-----------
code polishing
Modified Paths:
--------------
trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Account.java
trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Attribute.java
trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Category.java
trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Course.java
trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/State.java
trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/TransientAttribute.java
Modified: trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Account.java
===================================================================
--- trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Account.java 2009-04-27 11:31:17 UTC (rev 182)
+++ trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Account.java 2009-04-27 11:31:38 UTC (rev 183)
@@ -51,7 +51,32 @@
setId(id);
}
+ @XmlElementWrapper(name=Constants.ROLES)
+ @XmlElement(name=Constants.ROLE)
+ public List<Role> getRoles() {
+ return roles;
+ }
+
+ public void setRoles(List<Role> roles) {
+ this.roles = roles;
+ }
+
/**
+ * Convenience method to check if account is member of a group
+ * @param group group to check membership of
+ * @return membership state of account and given group
+ */
+ public boolean isInGroup(Group group){
+ if (group == null){
+ return false;
+ }
+ if (group.getMembers()==null){
+ return false;
+ }
+ return (group.getMembers().contains(this));
+ }
+
+ /**
* Convenience method to add an account to a group
* @param group group account should be added to
* @return success of operation
@@ -77,7 +102,7 @@
group.setMembers(accounts);
return true;
}
-
+
/**
* Convenience method to remove an account from a group
* @param group group to remove account from
@@ -98,23 +123,23 @@
group.setMembers(accounts);
return true;
}
+
+ @XmlElementWrapper(name=Constants.GROUPS)
+ @XmlElement(name=Constants.GROUP)
+ @XmlIDREF
+ public List<Group> getGroups() {
+ return groups;
+ }
+
+ public void setGroups(List<Group> groups) {
+ this.groups = groups;
+ }
- /**
- * Convenience method to check if account is member of a group
- * @param group group to check membership of
- * @return membership state of account and given group
- */
- public boolean isInGroup(Group group){
- if (group == null){
- return false;
- }
- if (group.getMembers()==null){
- return false;
- }
- return (group.getMembers().contains(this));
+ public void accept(Visitor visitor) {
+ visitor.visit(this);
}
-
- @Override
+
+ @Override
public boolean equals(Object obj) {
if (this == obj) {
return true;
@@ -141,29 +166,4 @@
public String toString() {
return new ToStringBuilder(this, ToStringStyle.DEFAULT_STYLE).append(this.getId()).toString();
}
-
- @XmlElementWrapper(name=Constants.ROLES)
- @XmlElement(name=Constants.ROLE)
- public List<Role> getRoles() {
- return roles;
- }
-
- public void setRoles(List<Role> roles) {
- this.roles = roles;
- }
-
- @XmlElementWrapper(name=Constants.GROUPS)
- @XmlElement(name=Constants.GROUP)
- @XmlIDREF
- public List<Group> getGroups() {
- return groups;
- }
-
- public void setGroups(List<Group> groups) {
- this.groups = groups;
- }
-
- public void accept(Visitor visitor) {
- visitor.visit(this);
- }
}
Modified: trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Attribute.java
===================================================================
--- trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Attribute.java 2009-04-27 11:31:17 UTC (rev 182)
+++ trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Attribute.java 2009-04-27 11:31:38 UTC (rev 183)
@@ -6,7 +6,10 @@
import javax.xml.bind.annotation.XmlType;
import javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter;
+import org.apache.commons.lang.builder.ToStringBuilder;
+import org.apache.commons.lang.builder.ToStringStyle;
+
/**
* Attribute entity object
* @author Sebastian Roekens
@@ -55,5 +58,43 @@
public void setValue(String value) {
this.value = value;
}
+
+ @Override
+ public int hashCode() {
+ final int prime = 31;
+ int result = 1;
+ result = prime * result + ((name == null) ? 0 : name.hashCode());
+ result = prime * result + (trans ? 1231 : 1237);
+ result = prime * result + ((value == null) ? 0 : value.hashCode());
+ return result;
+ }
+
+ @Override
+ public boolean equals(Object obj) {
+ if (this == obj)
+ return true;
+ if (obj == null)
+ return false;
+ if (getClass() != obj.getClass())
+ return false;
+ Attribute other = (Attribute) obj;
+ if (name == null) {
+ if (other.name != null)
+ return false;
+ } else if (!name.equals(other.name))
+ return false;
+ if (trans != other.trans)
+ return false;
+ if (value == null) {
+ if (other.value != null)
+ return false;
+ } else if (!value.equals(other.value))
+ return false;
+ return true;
+ }
+ public String toString() {
+ return new ToStringBuilder(this, ToStringStyle.DEFAULT_STYLE).append(name).append(value).toString();
+ }
+
}
Modified: trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Category.java
===================================================================
--- trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Category.java 2009-04-27 11:31:17 UTC (rev 182)
+++ trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Category.java 2009-04-27 11:31:38 UTC (rev 183)
@@ -39,10 +39,10 @@
private Category parent;
@OneToMany(mappedBy = Constants.PARENT, cascade = { CascadeType.REMOVE })
- private List<Category> children = new ArrayList<Category>();
+ private List<Category> children = new ArrayList<Category>(5);
@ManyToMany(mappedBy = Constants.CATEGORIES)
- private List<Course> courses = new ArrayList<Course>();
+ private List<Course> courses = new ArrayList<Course>(5);
public Category() {
}
Modified: trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Course.java
===================================================================
--- trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Course.java 2009-04-27 11:31:17 UTC (rev 182)
+++ trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/Course.java 2009-04-27 11:31:38 UTC (rev 183)
@@ -40,7 +40,7 @@
private Course parent;
@OneToMany(mappedBy=Constants.PARENT, cascade=CascadeType.REMOVE)
- private List<Course> workgroups = new ArrayList<Course>(3);
+ private List<Course> workgroups = new ArrayList<Course>(2);
@ManyToMany(cascade={CascadeType.MERGE, CascadeType.PERSIST, CascadeType.REFRESH})
@JoinTable(
@@ -114,10 +114,6 @@
return true;
}
- public String toString() {
- return new ToStringBuilder(this, ToStringStyle.DEFAULT_STYLE).toString();
- }
-
@XmlElement(name=Constants.PARENT)
@XmlIDREF
public Course getParent() {
@@ -158,10 +154,6 @@
public void setRoles(List<Role> roles) {
this.roles = roles;
}
-
- public void accept(Visitor visitor) {
- visitor.visit(this);
- }
public void addRole(Role role) {
if (role != null) {
@@ -170,5 +162,13 @@
}
}
+ public void accept(Visitor visitor) {
+ visitor.visit(this);
+ }
+
+ public String toString() {
+ return new ToStringBuilder(this, ToStringStyle.DEFAULT_STYLE).toString();
+ }
+
}
Modified: trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/State.java
===================================================================
--- trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/State.java 2009-04-27 11:31:17 UTC (rev 182)
+++ trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/State.java 2009-04-27 11:31:38 UTC (rev 183)
@@ -1,7 +1,31 @@
package de.campussource.cse.cdmm.domain;
-public enum State{
- NOT_EXISTS,
- KNOWN,
- EXISTS,
+
+import javax.xml.bind.annotation.XmlEnum;
+import javax.xml.bind.annotation.XmlEnumValue;
+import javax.xml.bind.annotation.XmlType;
+
+/**
+ * State of the domain objects.
+ *
+ * <ul>
+ * <li><strong>NOT_EXISTS</strong> <br/> An entity that doesn't exists within the database. This is an initial temporary state. </li>
+ * <li><strong>KNOWN</strong> <br/> An entity that is referenced by existing entities, but was not until now explicit created by an event.</li>
+ * <li><strong>EXISTS</strong> <br/> An entity that exists in the database. </li>
+ * <li><strong>DELETED</strong> <br> An entity that is marked as deleted.</li>
+ * <ul>
+ *
+ * @author Ingo Dueppe
+ * @author Sebastian Roekens
+ */
+@XmlEnum
+@XmlType(name="STATE", namespace=Constants.NAMESPACE_DATATYPES)
+public enum State {
+ @XmlEnumValue(value="NOT_EXISTS")
+ NOT_EXISTS,
+ @XmlEnumValue(value="KNOWN")
+ KNOWN,
+ @XmlEnumValue(value="EXISTS")
+ EXISTS,
+ @XmlEnumValue(value="DELETED")
DELETED
}
Modified: trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/TransientAttribute.java
===================================================================
--- trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/TransientAttribute.java 2009-04-27 11:31:17 UTC (rev 182)
+++ trunk/cse-ip/sc-cdmm/src/main/java/de/campussource/cse/cdmm/domain/TransientAttribute.java 2009-04-27 11:31:38 UTC (rev 183)
@@ -4,14 +4,16 @@
/**
* Representation of an attribute object, which should not be saved persistent
+ *
* @author Sebastian Roekens
- *
+ *
*/
@XmlTransient
public class TransientAttribute extends Attribute {
- public TransientAttribute(){}
- public TransientAttribute(String name, String value){
+ public TransientAttribute() {}
+
+ public TransientAttribute(String name, String value) {
setName(name);
setValue(value);
setTransient(true);
This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.
|