/*
 * Copyright 2014 Michael J. Simons.
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *      http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package ac.simons.biking2.persistence.entities;

import java.io.Serializable;
import java.math.BigDecimal;
import java.util.Calendar;
import java.util.Objects;
import javax.persistence.Column;
import javax.persistence.Entity;
import javax.persistence.GeneratedValue;
import javax.persistence.GenerationType;
import javax.persistence.Id;
import javax.persistence.NamedQueries;
import javax.persistence.NamedQuery;
import javax.persistence.Table;
import javax.persistence.Temporal;
import javax.persistence.TemporalType;
import javax.validation.constraints.NotNull;

/**
 * @author Michael J. Simons, 2014-02-08
 */
@Entity
@Table(name = "assorted_trips")
@NamedQueries({
    @NamedQuery(
	    name = "AssortedTrip.getTotalDistance",
	    query
	    = "Select coalesce(sum(t.distance), 0) from AssortedTrip t"
    )
})
public class AssortedTrip implements Serializable {

    private static final long serialVersionUID = 3222189732938547117L;

    @Id
    @Column(name = "id")
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Integer id;

    @Column(name = "covered_on", nullable = false)
    @Temporal(TemporalType.DATE)
    @NotNull
    private Calendar coveredOn;

    @Column(name = "distance", nullable = false, precision = 8, scale = 2)
    @NotNull
    private BigDecimal distance;

    protected AssortedTrip() {	
    }
    
    public AssortedTrip(Calendar coveredOn, BigDecimal distance) {
	this.coveredOn = coveredOn;
	this.distance = distance;
    }
    
    public Integer getId() {
	return id;
    }

    public Calendar getCoveredOn() {
	return coveredOn;
    }

    public BigDecimal getDistance() {
	return distance;
    }

    @Override
    public int hashCode() {
	int hash = 5;
	hash = 83 * hash + Objects.hashCode(this.coveredOn);
	hash = 83 * hash + Objects.hashCode(this.distance);
	return hash;
    }

    @Override
    public boolean equals(Object obj) {
	if (obj == null) {
	    return false;
	}
	if (getClass() != obj.getClass()) {
	    return false;
	}
	final AssortedTrip other = (AssortedTrip) obj;
	if (!Objects.equals(this.coveredOn, other.coveredOn)) {
	    return false;
	}
	if (!Objects.equals(this.distance, other.distance)) {
	    return false;
	}
	return true;
    }   
}

@@ -2723,21 +2723,23 @@
 se;%0A%09%7D%0A%09
-if (!
+return 
 Objects.
@@ -2779,46 +2779,8 @@
 nce)
-) %7B%0A%09    return false;%0A%09%7D%0A%09return true
 ;%0A  

Fix redundant if statement