OneToMany 关系在对象的部分更新时丢失

Question

我在更新我的对象时遇到问题。 我的对象有这些属性（还有更多，但问题是一样的）

public String testString;

@OneToMany(cascade = CascadeType.ALL, mappedBy = "user")
public List<Mail> mails;

假设我有一个这种类型的实例，其中 testString 为 NULL，并且邮件列表中有一封电子邮件。这个对象在数据库中得到持久化就好了。

现在尝试在不提供邮件列表的情况下进行更新。

final Map<String, String> values1 = new HashMap<String, String>();
values1.put("id", user.getId().toString());

Form<User> userForm1 = Form.form(User.class);
userForm1 = userForm1.bind(values1).get().update()

（我删除了不必要的代码来检查内容，是的，这是一种空更新，但是当我包含其他字段时也会发生同样的情况） 此次更新后，邮件列表为空......但其他简单字段仍有其值。所以我假设我做对了，并且 play 在“新”对象中看到了一个空字段，而不是用 null 覆盖旧值，只是在 updateobject 中设置的字段。

但这不适用于 OneToMany 关系。这是有意的还是错误的？ 如果是有意的，我应该如何更新具有这种关系的对象？

这是原始对象的 system.out.println

User [username=Timmeey, testString=null, jobs=[], mails=[timmeey@xxx.xxx, isMainMail: true, id: 1], getCreated()=1401647537627, getModified()=1401647537627, getId()=1]

这是来自表格

User [username=null, testString=null, jobs=[], mails=[], getCreated()=null, getModified()=null, getId()=1]

这是更新后的对象

User [username=Timmeey, testString=null, jobs=[], mails=[], getCreated()=1401647537627, getModified()=1401647537727, getId()=1]

我们可以看到，play 并没有覆盖所有的空字段，因为用户名还在，只是邮件字段被覆盖了

当然我知道我可以手动完成并遍历字段并检查它们是否为空，然后设置应该自己更新的字段，但我无法想象这应该是要走的路。

谢谢

TL;DR Play/Ebean 在更新已保存的对象时识别空值。因此它将执行部分更新，仅覆盖具有非空值的字段。这适用于简单的事情，比如字符串用户名。但是当涉及到像@OneToMany 这样的关系时，它会失败，并且总是用新对象的值覆盖存储的值，即使它为空。 我想要的是，关系字段也被视为普通字段，当字段为空时，不应覆盖旧对象上的字段

整个模型类

package models;

import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;
import java.util.regex.Pattern;

import javax.naming.directory.InvalidAttributeValueException;
import javax.persistence.CascadeType;
import javax.persistence.DiscriminatorValue;
import javax.persistence.Entity;
import javax.persistence.ManyToMany;
import javax.persistence.OneToMany;
import javax.persistence.OneToOne;
import javax.persistence.PrePersist;
import javax.persistence.PreUpdate;
import javax.persistence.Table;
import javax.persistence.Transient;
import javax.validation.Valid;
import javax.validation.constraints.Null;

import com.avaje.ebean.Ebean;

import controllers.SettingsController;
import exceptions.InputValidationException;
import exceptions.NotYetInitializedException;
import play.Logger;
import play.data.validation.ValidationError;

@Entity
@Table(name = "userTable")
@DiscriminatorValue("aUser")
// User may be a reserved keyword in some sql databases
public class User extends AbstractSuperModel {
final static Logger.ALogger logger = Logger.of(User.class);
final private static String usernameRegexPattern = "[\\w_-]{3,}";

// This prevents binding of this value from forms
@Null
@javax.persistence.Column(unique = true)
private String username; // Must not be set by forms, only by controllers

// @OneToMany
// private Set<Ticket> responsibleForTickets;
// @ManyToMany
// private Set<Ticket> subscribedTickets;
// @OneToMany
// private Set<Ticket> reportedTickets;
// //public String secondaryEmails;

private String testString;

@ManyToMany(cascade = CascadeType.PERSIST)
private List<Job> jobs;

@Null
@OneToMany(cascade = CascadeType.ALL, mappedBy = "user")
public List<Mail> mails;

public User(String username) throws InputValidationException {
    logger.info("Adding a new User: " + username);
    Pattern regex = Pattern.compile(usernameRegexPattern);
    if (!regex.matcher(username).matches()) {
        logger.error(username
                + " is not a valid username. In case you not just tried to troll the system, consider this as a serious Error and contact the maintainer (Timmeey@xxx.xxx (2014)");
        throw new InputValidationException(username
                + " is not a valid username");

    }

    this.setUsername(username);
    Mail mail = new Mail();
    mail.setMailAddr(username + "@xxx.xxx");
    mail.setIsMainMail(true);
    this.addMail(mail);

}

public void addMail(Mail mail) {
    this.getMails().add(mail);
}

public Mail getMainMail() {
    for (Mail mail : this.getMails()) {
        if (mail != null && mail.getIsMainMail()) {
            return mail;
        }
    }
    return null;
}

/**
 * Removes a Mail from the User.
 * 
 * @param mail
 *            The mail that should get removed
 */
public void removeMail(Mail mail) {
    for (Iterator<Mail> iterator = this.getMails().iterator(); iterator
            .hasNext();) {
        Mail tmpMail = iterator.next();
        if (tmpMail.getMailAddr().equalsIgnoreCase(mail.getMailAddr())) {
            tmpMail.delete();
            return;
        }

    }
}

public void addJob(Job job) {
    this.jobs.add(job);
    this.update();
}

public void removeJob(Job job) {
    this.getJobs().remove(job);
    this.update();
}

public static Finder<Long, User> find = new Finder<Long, User>(Long.class,
        User.class);



public static List<User> findAll() {
    return User.find.all();
}

public static User findById(final Long id) {
    return User.find.byId(id);
}

public static User findByMail(final String address) {
    final Mail mail = Mail.findByAddr(address);
    if (mail != null) {
        return mail.getUser();
    }
    return null;
}

public static User findByName(final String username) {
    User resultUser = null;
    resultUser = User.find.where().eq("username", username).findUnique();
    return resultUser;
}

public static boolean isKnownUser(final String username) {
    return findByName(username) != null;
}

/**
 * Will be executed before a User is saved into the Database.
 */
@PreUpdate
public void processMailAddresses() {
}

public List<ValidationError> validate() {
    List<ValidationError> errors = new ArrayList<ValidationError>();
    if (errors.size() != 0) {
        return errors;
    }
    return null;
}

public List<Job> getJobs() {
    return this.jobs;
}

public List<Mail> getMails() {
    if (this.mails == null) {
        return null;
    }
    return this.mails;
}

public String getTestString() {
    return this.testString;
}

public String getUsername() {
    return this.username;
}

public void setJobs(final List<Job> jobs) {
    this.jobs = jobs;
}

public void setMails(final List<Mail> mails) {
    this.mails = mails;
}

public void setTestString(final String testString) {
    this.testString = testString;
}

public void setUsername(final String username) {
    this.username = username;
}

public boolean hasJob(Job job) {
    if (getJobs() == null) {
        return false;
    }
    return this.getJobs().contains(job);
}

@Override
public String toString() {
    final List<String> jobs = new ArrayList<String>();
    for (Job job : this.getJobs()) {
        jobs.add(job.getJobName());
    }

    final List<String> mails = new ArrayList<String>();
    for (final Mail mail : this.getMails()) {
        mails.add(mail.getMailAddr() + ", isMainMail: "
                + mail.getIsMainMail() + ", id: " + mail.getId());
    }

    return String
            .format("User [username=%s, testString=%s, jobs=%s, mails=%s, getCreated()=%s, getModified()=%s, getId()=%s]",
                    this.username, this.testString, jobs, mails,
                    this.getCreated(), this.getModified(), this.getId());
}


}

还有邮件类

package models;

import java.util.ArrayList;
import java.util.List;

import javax.persistence.Entity;
import javax.persistence.ManyToOne;
import javax.validation.constraints.Null;

import play.Logger;
import play.Logger.ALogger;
import play.data.validation.Constraints.Email;
import play.data.validation.Constraints.Required;
import play.data.validation.ValidationError;

/**
 * Just a container for Emails. Because Play! cannot store List<String> for the
 * Email-Addresses
 * 
 * @author timmeey
 * 
 */
@Entity
public class Mail extends AbstractSuperModel {
private static final ALogger logger = Logger.of(Mail.class);

public static Mail findByAddr(final String address) {
    Mail mail = null;
    mail = find.where().eq("mailAddr", address).findUnique();
    return mail;

}

@Required
@Email
@javax.persistence.Column(unique = true)
private String mailAddr;

@Null
@ManyToOne
private User user;

@Null
Boolean isMainMail;

public static Finder<Long, Mail> find = new Finder<Long, Mail>(Long.class,
        Mail.class);

public static Mail findById(final Long id) {
    if (id == null) {
        return null;
    }
    return find.byId(id);
}

public static List<Mail> findAll() {
    return find.all();
}

public String getMailAddr() {
    return this.mailAddr;
}

public User getUser() {
    return this.user;
}

public void setMailAddr(final String mailAddr) {
    this.mailAddr = mailAddr;
}

public void setUser(final User user) {
    this.user = user;
}

@Override
public String toString() {
    return String
            .format("Mail [id=%s, mailAddr=%s, user=%s, getCreated()=%s, getModified()=%s, getId()=%s]",
                    this.id, this.mailAddr, this.user.getUsername(),
                    this.getCreated(), this.getModified(), this.getId());
}

public Boolean getIsMainMail() {
    return isMainMail;
}

public void setIsMainMail(Boolean isMainMail) {
    this.isMainMail = isMainMail;
}

public List<ValidationError> validate() {
    List<ValidationError> errors = new ArrayList<ValidationError>();

    if (find.where().eq("mailAddr", this.mailAddr).findRowCount() > 0) {
        errors.add(new ValidationError(mailAddr, "Mailaddress already used"));
    }
    if (errors.size() != 0) {
        return errors;
    }
    return null;

}

}

Answer 1

问题实际上不在您的 ORM 注释中。游戏缺乏对在表单中呈现不同对象并获取信息的支持。我假设您想为表单提供一个对象，在视图中呈现结果，然后用户对邮件以外的其他字段进行一些更改，然后更新更改的信息。表单本身自动使用 getter 和 setter 来生成字段，然后从请求中提取信息。在您的情况下，您的邮件列表和 HTML 字段没有正确的映射。我的意思是，如果您有一个带有 getter 和 setter 的字段 String name;，表单将自动创建匹配名称约定的要填充的字段，然后稍后将使用 setter 将此字段中的信息绑定到您的 User 对象，该 setter 具有被命名为setName(String name)。但是，当信息属于不同于基本类型或String 的特定类型时，例如Mail，表单将无法在HTML 中为其创建字段。所以你基本上需要一些手工工作。您必须是否将您的电子邮件保存在 java 代码中，然后将它们添加给必须更新的用户，如果您不愿意更改它们或以适当的形式（如字符串）使用表单将它们提供给视图，然后稍后使用 DynamicForm 从视图中生成的字段中提取信息。

无论您是否使用带有表单参数的渲染方法，视图本身最终都是纯 HTML。因此，您的 java 对象和 HTML 文件之间没有直接映射。 bind 方法将简单地遍历请求的输入字段的所有 id 并尝试找到匹配的设置器（因此设置器的名称很关键）。但是当您使用更具体的对象时，您需要手动完成这项工作。