What's wrong with the following OR mapping?

[yantriki]

@Entity
@Table(name="ModuleType")
public class ModuleType implements Serializable {

 Collection<Module> modules_of_type = new HashSet<Module>();
 @Id @GeneratedValue int id;

 @OneToMany(cascade=CascadeType.REMOVE,mappedBy="type")
 public Collection<Module> getModules() {
 return modules_of_type;
 }

 public void setModules(Collection<Module> modules) {
 this.modules_of_type = modules;
 }

 @Column(name="TYPE_ID")
 public int getId() {
 return id;
 }
 public void setId(int id) {
 this.id = id;
 }
}

@Entity
@Table(name="Module")
public class Module implements Serializable{
 int moduleID;
 @Id
 @GeneratedValue
 public int getModuleID() {
 return moduleID;
 }
 public void setModuleID(int moduleID) {
 this.moduleID = moduleID;
 }
 @ManyToOne
 @JoinColumn(name="TYPE_ID",nullable=false)
 public ModuleType getType() {
 return type;
 }

 public void setType(ModuleType type) {
 this.type = type;
 }
}

I am getting the following errors:

Caused by: org.hibernate.MappingException: Could not determine type for: java.util.Collection, for columns: [org.hibernate.mapping.Column(modules_of_type)]

[yantriki]

just want to point out that I have a variable of ModuleType defined in the Module class.

[yantriki]

ok I fixed it by moving the @OneToMany annotation over the field rather than the getter method.

[bill.burke]

You cannot mix and match your mapping declarations between field and getter methods.

You placed the @Id on a field, so the mapper is expecting that all your mapping information is placed on fields

[yantriki]

Bill,
Thanks for the clarification.
Regards
Vivek

EJB Persistence spec section 2.1.1

When annotations are used, the placement of the mapping annotations on either the persistent fields or persistent properties of the entity class specifies the access type as being either field- or property-based access respectively.