From 4bc2559e8e91c1d14d5ecf05e6656f8b4181c68b Mon Sep 17 00:00:00 2001 From: rob bygrave Date: Thu, 10 Oct 2019 21:51:31 +1300 Subject: [PATCH] Refactor tidy PersistRequestBean --- src/main/java/io/ebean/event/BeanPersistRequest.java | 5 +++++ .../ebeaninternal/server/core/PersistRequestBean.java | 11 ++--------- 2 files changed, 7 insertions(+), 9 deletions(-) diff --git a/src/main/java/io/ebean/event/BeanPersistRequest.java b/src/main/java/io/ebean/event/BeanPersistRequest.java index b03dcc2c8..c5043fb74 100644 --- a/src/main/java/io/ebean/event/BeanPersistRequest.java +++ b/src/main/java/io/ebean/event/BeanPersistRequest.java @@ -41,6 +41,11 @@ public interface BeanPersistRequest { */ Set getUpdatedProperties(); + /** + * Flags set for dirty properties (used by ElasticSearch integration). + */ + boolean[] getDirtyProperties(); + /** * Return true for an update request if at least one of dirty properties is contained * in the given set of property names. diff --git a/src/main/java/io/ebeaninternal/server/core/PersistRequestBean.java b/src/main/java/io/ebeaninternal/server/core/PersistRequestBean.java index 987fa71c5..45fcc1376 100644 --- a/src/main/java/io/ebeaninternal/server/core/PersistRequestBean.java +++ b/src/main/java/io/ebeaninternal/server/core/PersistRequestBean.java @@ -427,6 +427,7 @@ public final class PersistRequestBean extends PersistRequest implements BeanP /** * Return the dirty properties on this request. */ + @Override public boolean[] getDirtyProperties() { return dirtyProperties; } @@ -485,7 +486,7 @@ public final class PersistRequestBean extends PersistRequest implements BeanP /** * Collect L2 cache changes to be applied after the transaction has successfully committed. */ - public void notifyCache(CacheChangeSet changeSet) { + private void notifyCache(CacheChangeSet changeSet) { if (notifyCache) { switch (type) { case INSERT: @@ -740,14 +741,6 @@ public final class PersistRequestBean extends PersistRequest implements BeanP return parentBean; } - /** - * Return the controller if there is one associated with this type of bean. This returns null if - * there is no controller associated. - */ - public BeanPersistController getBeanController() { - return controller; - } - /** * Return the intercept if there is one. */