I am attempting to reverse engineer a database with seam-gen/generate-entities. 
 Everything is built correctly.  But the isDefined() method in the generated 
TransectHome class incorrectly applies a Strings.isEmpty() method (the objects 
are not always Strings).  Is this a bug?

Generated Pojo

@Entity
  | @Table(name = "transect", schema = "public")
  | public class Transect implements java.io.Serializable {
  | 
  |     private TransectId id;
  | 
  |     public Transect() {
  |     }
  | 
  |     public Transect(TransectId id) {
  |             this.id = id;
  |     }
  | 
  |     @EmbeddedId
  |     @AttributeOverrides({
  |                     @AttributeOverride(name = "pkTransect", column = 
@Column(name = "pk_transect", nullable = false)),
  |                     @AttributeOverride(name = "transectId", column = 
@Column(name = "transect_id")),
  |                     @AttributeOverride(name = "transectName", column = 
@Column(name = "transect_name")),
  |                     @AttributeOverride(name = "groupedData", column = 
@Column(name = "grouped_data")),
  |                     @AttributeOverride(name = "plots", column = 
@Column(name = "plots"))})
  |     @NotNull
  |     public TransectId getId() {
  |             return this.id;
  |     }
  | 
  |     public void setId(TransectId id) {
  |             this.id = id;
  |     }
  | 
  | }

Generated Id Class

@Embeddable
  | public class TransectId implements java.io.Serializable {
  | 
  |     private int pkTransect;
  |     private Integer transectId;
  |     private String transectName;
  |     private Integer groupedData;
  |     private String plots;
  | 
  |     public TransectId() {
  |     }
  | 
  |     public TransectId(int pkTransect) {
  |             this.pkTransect = pkTransect;
  |     }
  |     public TransectId(int pkTransect, Integer transectId, String 
transectName,
  |                     Integer groupedData, String plots) {
  |             this.pkTransect = pkTransect;
  |             this.transectId = transectId;
  |             this.transectName = transectName;
  |             this.groupedData = groupedData;
  |             this.plots = plots;
  |     }
  | 
  |     @Column(name = "pk_transect", nullable = false)
  |     @NotNull
  |     public int getPkTransect() {
  |             return this.pkTransect;
  |     }
  | 
  |     public void setPkTransect(int pkTransect) {
  |             this.pkTransect = pkTransect;
  |     }
  | 
  |     @Column(name = "transect_id")
  |     public Integer getTransectId() {
  |             return this.transectId;
  |     }
  | 
  |     public void setTransectId(Integer transectId) {
  |             this.transectId = transectId;
  |     }
  | 
  |     @Column(name = "transect_name")
  |     public String getTransectName() {
  |             return this.transectName;
  |     }
  | 
  |     public void setTransectName(String transectName) {
  |             this.transectName = transectName;
  |     }
  | 
  |     @Column(name = "grouped_data")
  |     public Integer getGroupedData() {
  |             return this.groupedData;
  |     }
  | 
  |     public void setGroupedData(Integer groupedData) {
  |             this.groupedData = groupedData;
  |     }
  | 
  |     @Column(name = "plots")
  |     public String getPlots() {
  |             return this.plots;
  |     }
  | 
  |     public void setPlots(String plots) {
  |             this.plots = plots;
  |     }
  | 
  |     public boolean equals(Object other) {
  |             if ((this == other))
  |                     return true;
  |             if ((other == null))
  |                     return false;
  |             if (!(other instanceof TransectId))
  |                     return false;
  |             TransectId castOther = (TransectId) other;
  | 
  |             return (this.getPkTransect() == castOther.getPkTransect())
  |                             && ((this.getTransectId() == 
castOther.getTransectId()) || (this
  |                                             .getTransectId() != null
  |                                             && castOther.getTransectId() != 
null && this
  |                                             
.getTransectId().equals(castOther.getTransectId())))
  |                             && ((this.getTransectName() == 
castOther.getTransectName()) || (this
  |                                             .getTransectName() != null
  |                                             && castOther.getTransectName() 
!= null && this
  |                                             
.getTransectName().equals(castOther.getTransectName())))
  |                             && ((this.getGroupedData() == 
castOther.getGroupedData()) || (this
  |                                             .getGroupedData() != null
  |                                             && castOther.getGroupedData() 
!= null && this
  |                                             
.getGroupedData().equals(castOther.getGroupedData())))
  |                             && ((this.getPlots() == castOther.getPlots()) 
|| (this
  |                                             .getPlots() != null
  |                                             && castOther.getPlots() != null 
&& this.getPlots()
  |                                             .equals(castOther.getPlots())));
  |     }
  | 
  |     public int hashCode() {
  |             int result = 17;
  | 
  |             result = 37 * result + this.getPkTransect();
  |             result = 37
  |                             * result
  |                             + (getTransectId() == null ? 0 : 
this.getTransectId()
  |                                             .hashCode());
  |             result = 37
  |                             * result
  |                             + (getTransectName() == null ? 0 : 
this.getTransectName()
  |                                             .hashCode());
  |             result = 37
  |                             * result
  |                             + (getGroupedData() == null ? 0 : 
this.getGroupedData()
  |                                             .hashCode());
  |             result = 37 * result
  |                             + (getPlots() == null ? 0 : 
this.getPlots().hashCode());
  |             return result;
  |     }
  | 
  | }

Generated Action Class

@Name("transectHome")
  | public class TransectHome extends EntityHome<Transect> {
  | 
  |     public void setTransectId(TransectId id) {
  |             setId(id);
  |     }
  | 
  |     public TransectId getTransectId() {
  |             return (TransectId) getId();
  |     }
  | 
  |     public TransectHome() {
  |             setTransectId(new TransectId());
  |     }
  | 
  |     @Override
  |     public boolean isIdDefined() {
  |             if (getTransectId().getPkTransect() == 0)
  |                     return false;
  |             if (Strings.isEmpty(getTransectId().getTransectId()))
  |                     return false;
  |             if (Strings.isEmpty(getTransectId().getTransectName()))
  |                     return false;
  |             if (Strings.isEmpty(getTransectId().getGroupedData()))
  |                     return false;
  |             if (Strings.isEmpty(getTransectId().getPlots()))
  |                     return false;
  |             return true;
  |     }
  | 
  |     @Override
  |     protected Transect createInstance() {
  |             Transect transect = new Transect();
  |             transect.setId(new TransectId());
  |             return transect;
  |     }
  | 
  |     public void wire() {
  |     }
  | 
  |     public boolean isWired() {
  |             return true;
  |     }
  | 
  |     public Transect getDefinedInstance() {
  |             return isIdDefined() ? getInstance() : null;
  |     }
  | 
  | }






View the original post : 
http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4102358#4102358

Reply to the post : 
http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4102358
_______________________________________________
jboss-user mailing list
jboss-user@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jboss-user

Reply via email to