Repository: cayenne Updated Branches: refs/heads/master daee122c5 -> 51a3399d4
CAY-2235 Deprecate Query.getDataMap() method Project: http://git-wip-us.apache.org/repos/asf/cayenne/repo Commit: http://git-wip-us.apache.org/repos/asf/cayenne/commit/51a3399d Tree: http://git-wip-us.apache.org/repos/asf/cayenne/tree/51a3399d Diff: http://git-wip-us.apache.org/repos/asf/cayenne/diff/51a3399d Branch: refs/heads/master Commit: 51a3399d42f8325bc8baa3138923e39879201c47 Parents: daee122 Author: Nikita Timofeev <stari...@gmail.com> Authored: Thu Feb 16 10:58:47 2017 +0300 Committer: Nikita Timofeev <stari...@gmail.com> Committed: Thu Feb 16 10:58:47 2017 +0300 ---------------------------------------------------------------------- .../cayenne/lifecycle/id/StringIdQuery.java | 5 ++ .../DefaultConfigurationNodeParentGetter.java | 6 ++ .../apache/cayenne/access/DataDomainQuery.java | 1 + .../access/ObjectsFromDataRowsQuery.java | 2 + .../org/apache/cayenne/map/QueryDescriptor.java | 3 +- .../org/apache/cayenne/query/AbstractQuery.java | 6 ++ .../org/apache/cayenne/query/BatchQuery.java | 4 + .../org/apache/cayenne/query/EJBQLQuery.java | 5 ++ .../org/apache/cayenne/query/IndirectQuery.java | 3 + .../java/org/apache/cayenne/query/Query.java | 3 + .../org/apache/cayenne/query/QueryChain.java | 7 ++ .../org/apache/cayenne/query/RefreshQuery.java | 3 + .../apache/cayenne/remote/IncrementalQuery.java | 3 + .../cayenne/remote/IncrementalSelectQuery.java | 2 + .../org/apache/cayenne/remote/RangeQuery.java | 2 + .../org/apache/cayenne/query/MockQuery.java | 23 +---- docs/doc/src/main/resources/RELEASE-NOTES.txt | 1 + .../cayenne/modeler/action/CopyAction.java | 3 +- .../cayenne/modeler/action/RemoveAction.java | 91 +++++--------------- 19 files changed, 83 insertions(+), 90 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-lifecycle/src/main/java/org/apache/cayenne/lifecycle/id/StringIdQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-lifecycle/src/main/java/org/apache/cayenne/lifecycle/id/StringIdQuery.java b/cayenne-lifecycle/src/main/java/org/apache/cayenne/lifecycle/id/StringIdQuery.java index a5a37e4..2b28f44 100644 --- a/cayenne-lifecycle/src/main/java/org/apache/cayenne/lifecycle/id/StringIdQuery.java +++ b/cayenne-lifecycle/src/main/java/org/apache/cayenne/lifecycle/id/StringIdQuery.java @@ -68,7 +68,9 @@ public class StringIdQuery implements Query { return Arrays.asList(stringIds); } + @Deprecated protected String name; + @Deprecated protected DataMap dataMap; protected Collection<String> stringIds; @@ -252,6 +254,7 @@ public class StringIdQuery implements Query { "This query was supposed to be replace with a set of SelectQueries during the route phase"); } + @Deprecated public String getName() { return name; } @@ -260,10 +263,12 @@ public class StringIdQuery implements Query { this.name = name; } + @Deprecated public DataMap getDataMap() { return dataMap; } + @Deprecated public void setDataMap(DataMap dataMap) { this.dataMap = dataMap; } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-project/src/main/java/org/apache/cayenne/project/DefaultConfigurationNodeParentGetter.java ---------------------------------------------------------------------- diff --git a/cayenne-project/src/main/java/org/apache/cayenne/project/DefaultConfigurationNodeParentGetter.java b/cayenne-project/src/main/java/org/apache/cayenne/project/DefaultConfigurationNodeParentGetter.java index 669d80b..3458c73 100644 --- a/cayenne-project/src/main/java/org/apache/cayenne/project/DefaultConfigurationNodeParentGetter.java +++ b/cayenne-project/src/main/java/org/apache/cayenne/project/DefaultConfigurationNodeParentGetter.java @@ -33,6 +33,7 @@ import org.apache.cayenne.map.ObjEntity; import org.apache.cayenne.map.ObjRelationship; import org.apache.cayenne.map.Procedure; import org.apache.cayenne.map.ProcedureParameter; +import org.apache.cayenne.map.QueryDescriptor; public class DefaultConfigurationNodeParentGetter implements ConfigurationNodeParentGetter { @@ -107,5 +108,10 @@ public class DefaultConfigurationNodeParentGetter implements ConfigurationNodePa public ConfigurationNode visitProcedureParameter(ProcedureParameter parameter) { return (ConfigurationNode) parameter.getParent(); } + + @Override + public ConfigurationNode visitQuery(QueryDescriptor query) { + return query.getDataMap(); + } } } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/access/DataDomainQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/access/DataDomainQuery.java b/cayenne-server/src/main/java/org/apache/cayenne/access/DataDomainQuery.java index 5077359..0488065 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/access/DataDomainQuery.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/access/DataDomainQuery.java @@ -107,6 +107,7 @@ class DataDomainQuery implements Query, QueryMetadata { return null; } + @Deprecated public DataMap getDataMap() { return null; } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/access/ObjectsFromDataRowsQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/access/ObjectsFromDataRowsQuery.java b/cayenne-server/src/main/java/org/apache/cayenne/access/ObjectsFromDataRowsQuery.java index 40277bd..ee23454 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/access/ObjectsFromDataRowsQuery.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/access/ObjectsFromDataRowsQuery.java @@ -68,10 +68,12 @@ class ObjectsFromDataRowsQuery implements Query, QueryMetadata { return null; } + @Deprecated public String getName() { return null; } + @Deprecated public DataMap getDataMap() { return null; } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/map/QueryDescriptor.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/map/QueryDescriptor.java b/cayenne-server/src/main/java/org/apache/cayenne/map/QueryDescriptor.java index 378eb5f..fe18894 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/map/QueryDescriptor.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/map/QueryDescriptor.java @@ -83,8 +83,7 @@ public class QueryDescriptor implements Serializable, ConfigurationNode, XMLSeri case PROCEDURE_QUERY: return procedureQueryDescriptor(); default: - QueryDescriptor descriptor = new QueryDescriptor(type); - return descriptor; + return new QueryDescriptor(type); } } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/query/AbstractQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/query/AbstractQuery.java b/cayenne-server/src/main/java/org/apache/cayenne/query/AbstractQuery.java index 7cab054..3b1dc21 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/query/AbstractQuery.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/query/AbstractQuery.java @@ -37,12 +37,16 @@ public abstract class AbstractQuery extends CacheableQuery { * DbEntity, depending on the specific query and how it was constructed. */ protected Object root; + + @Deprecated protected String name; + @Deprecated protected DataMap dataMap; /** * @since 3.1 */ + @Deprecated public DataMap getDataMap() { return dataMap; } @@ -50,6 +54,7 @@ public abstract class AbstractQuery extends CacheableQuery { /** * @since 3.1 */ + @Deprecated public void setDataMap(DataMap dataMap) { this.dataMap = dataMap; } @@ -69,6 +74,7 @@ public abstract class AbstractQuery extends CacheableQuery { * * @since 1.1 */ + @Deprecated public void setName(String name) { this.name = name; } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/query/BatchQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/query/BatchQuery.java b/cayenne-server/src/main/java/org/apache/cayenne/query/BatchQuery.java index 7042039..de191a7 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/query/BatchQuery.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/query/BatchQuery.java @@ -41,11 +41,13 @@ public abstract class BatchQuery implements Query { */ protected DbEntity dbEntity; + @Deprecated protected String name; /** * @since 3.1 */ + @Deprecated protected DataMap dataMap; /** @@ -85,6 +87,7 @@ public abstract class BatchQuery implements Query { * @since 3.1 */ @Override + @Deprecated public DataMap getDataMap() { return dataMap; } @@ -92,6 +95,7 @@ public abstract class BatchQuery implements Query { /** * @since 3.1 */ + @Deprecated public void setDataMap(DataMap dataMap) { this.dataMap = dataMap; } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/query/EJBQLQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/query/EJBQLQuery.java b/cayenne-server/src/main/java/org/apache/cayenne/query/EJBQLQuery.java index e778096..14c09c0 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/query/EJBQLQuery.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/query/EJBQLQuery.java @@ -38,7 +38,9 @@ import java.util.Map; */ public class EJBQLQuery extends CacheableQuery implements XMLSerializable { + @Deprecated protected String name; + @Deprecated protected DataMap dataMap; protected String ejbqlStatement; @@ -58,6 +60,7 @@ public class EJBQLQuery extends CacheableQuery implements XMLSerializable { /** * @since 3.1 */ + @Deprecated public DataMap getDataMap() { return dataMap; } @@ -66,6 +69,7 @@ public class EJBQLQuery extends CacheableQuery implements XMLSerializable { /** * @since 3.1 */ + @Deprecated public void setDataMap(DataMap dataMap) { this.dataMap = dataMap; } @@ -138,6 +142,7 @@ public class EJBQLQuery extends CacheableQuery implements XMLSerializable { return name; } + @Deprecated public void setName(String name) { this.name = name; } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/query/IndirectQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/query/IndirectQuery.java b/cayenne-server/src/main/java/org/apache/cayenne/query/IndirectQuery.java index fa7a305..c916972 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/query/IndirectQuery.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/query/IndirectQuery.java @@ -47,6 +47,7 @@ public abstract class IndirectQuery implements Query { * @since 3.1 */ @Override + @Deprecated public DataMap getDataMap() { return dataMap; } @@ -54,6 +55,7 @@ public abstract class IndirectQuery implements Query { /** * @since 3.1 */ + @Deprecated public void setDataMap(DataMap dataMap) { this.dataMap = dataMap; } @@ -72,6 +74,7 @@ public abstract class IndirectQuery implements Query { return name; } + @Deprecated public void setName(String name) { this.name = name; } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/query/Query.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/query/Query.java b/cayenne-server/src/main/java/org/apache/cayenne/query/Query.java index 0dbfc92..d0790b9 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/query/Query.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/query/Query.java @@ -70,10 +70,13 @@ public interface Query extends Serializable { * @since 1.1 * @deprecated {@link QueryDescriptor#getName()} should be used instead */ + @Deprecated String getName(); /** * @since 3.1 + * @deprecated since 4.0, {@link QueryDescriptor#getDataMap()} should be used instead */ + @Deprecated DataMap getDataMap(); } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/query/QueryChain.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/query/QueryChain.java b/cayenne-server/src/main/java/org/apache/cayenne/query/QueryChain.java index f6082ec..272c681 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/query/QueryChain.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/query/QueryChain.java @@ -37,11 +37,14 @@ import java.util.Collection; public class QueryChain implements Query { protected Collection<Query> chain; + + @Deprecated protected String name; /** * @since 3.1 */ + @Deprecated protected DataMap dataMap; /** @@ -112,10 +115,12 @@ public class QueryChain implements Query { + "and should've been split into separate queries during routing phase."); } + @Deprecated public String getName() { return name; } + @Deprecated public void setName(String name) { this.name = name; } @@ -123,6 +128,7 @@ public class QueryChain implements Query { /** * @since 3.1 */ + @Deprecated public DataMap getDataMap() { return dataMap; } @@ -130,6 +136,7 @@ public class QueryChain implements Query { /** * @since 3.1 */ + @Deprecated public void setDataMap(DataMap dataMap) { this.dataMap = dataMap; } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/query/RefreshQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/query/RefreshQuery.java b/cayenne-server/src/main/java/org/apache/cayenne/query/RefreshQuery.java index c2935ae..c31ce92 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/query/RefreshQuery.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/query/RefreshQuery.java @@ -135,6 +135,7 @@ public class RefreshQuery implements Query { return wrappedMd; } + @Deprecated public String getName() { return query.getName(); } @@ -146,12 +147,14 @@ public class RefreshQuery implements Query { query.route(router, resolver, this); } + @Deprecated public DataMap getDataMap() { return query.getDataMap(); } }; } + @Deprecated public DataMap getDataMap() { return null; } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/remote/IncrementalQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/remote/IncrementalQuery.java b/cayenne-server/src/main/java/org/apache/cayenne/remote/IncrementalQuery.java index 4ff953c..9541a42 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/remote/IncrementalQuery.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/remote/IncrementalQuery.java @@ -71,13 +71,16 @@ class IncrementalQuery implements Query { return query.createSQLAction(visitor); } + @Deprecated public String getName() { return query.getName(); } /** * @since 3.1 + * @deprecated since 4.0 */ + @Deprecated public DataMap getDataMap() { return query.getDataMap(); } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/remote/IncrementalSelectQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/remote/IncrementalSelectQuery.java b/cayenne-server/src/main/java/org/apache/cayenne/remote/IncrementalSelectQuery.java index 135bbcb..d869d4c 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/remote/IncrementalSelectQuery.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/remote/IncrementalSelectQuery.java @@ -159,6 +159,7 @@ class IncrementalSelectQuery<T> extends SelectQuery<T> { } @Override + @Deprecated public String getName() { return query.getName(); } @@ -272,6 +273,7 @@ class IncrementalSelectQuery<T> extends SelectQuery<T> { } @Override + @Deprecated public void setName(String name) { query.setName(name); } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/main/java/org/apache/cayenne/remote/RangeQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/main/java/org/apache/cayenne/remote/RangeQuery.java b/cayenne-server/src/main/java/org/apache/cayenne/remote/RangeQuery.java index 5c7467e..970b5a7 100644 --- a/cayenne-server/src/main/java/org/apache/cayenne/remote/RangeQuery.java +++ b/cayenne-server/src/main/java/org/apache/cayenne/remote/RangeQuery.java @@ -159,6 +159,7 @@ class RangeQuery implements Query { throw new UnsupportedOperationException(); } + @Deprecated public String getName() { throw new UnsupportedOperationException(); } @@ -170,6 +171,7 @@ class RangeQuery implements Query { /** * @since 3.1 */ + @Deprecated public DataMap getDataMap() { throw new UnsupportedOperationException(); } http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/cayenne-server/src/test/java/org/apache/cayenne/query/MockQuery.java ---------------------------------------------------------------------- diff --git a/cayenne-server/src/test/java/org/apache/cayenne/query/MockQuery.java b/cayenne-server/src/test/java/org/apache/cayenne/query/MockQuery.java index 1526b13..ba3276b 100644 --- a/cayenne-server/src/test/java/org/apache/cayenne/query/MockQuery.java +++ b/cayenne-server/src/test/java/org/apache/cayenne/query/MockQuery.java @@ -19,14 +19,11 @@ package org.apache.cayenne.query; -import org.apache.cayenne.configuration.ConfigurationNodeVisitor; import org.apache.cayenne.map.DataMap; import org.apache.cayenne.map.EntityResolver; public class MockQuery implements Query { - protected String name; - protected DataMap dataMap; protected boolean selecting; protected boolean routeCalled; @@ -37,18 +34,9 @@ public class MockQuery implements Query { public MockQuery() { } - + @Deprecated public DataMap getDataMap() { - return dataMap; - } - - - public void setDataMap(DataMap dataMap) { - this.dataMap = dataMap; - } - - public MockQuery(String name) { - this.name = name; + return null; } public QueryMetadata getMetaData(EntityResolver resolver) { @@ -63,12 +51,9 @@ public class MockQuery implements Query { return selecting; } + @Deprecated public String getName() { - return name; - } - - public void setName(String name) { - this.name = name; + return null; } public SQLAction createSQLAction(SQLActionVisitor visitor) { http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/docs/doc/src/main/resources/RELEASE-NOTES.txt ---------------------------------------------------------------------- diff --git a/docs/doc/src/main/resources/RELEASE-NOTES.txt b/docs/doc/src/main/resources/RELEASE-NOTES.txt index 86cc57c..56984e8 100644 --- a/docs/doc/src/main/resources/RELEASE-NOTES.txt +++ b/docs/doc/src/main/resources/RELEASE-NOTES.txt @@ -31,6 +31,7 @@ CAY-2212 cdbimport cleanup and configuration schema refactoring CAY-2223 JCacheQueryCache - a query cache provider to plug in JCache implementers CAY-2225 Extensible CacheInvalidationFilter logic CAY-2228 Deprecate multiple cache groups in caching and query API +CAY-2235 Deprecate Query.getDataMap() method Bug Fixes: http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/action/CopyAction.java ---------------------------------------------------------------------- diff --git a/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/action/CopyAction.java b/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/action/CopyAction.java index ce9a82f..f5c8301 100644 --- a/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/action/CopyAction.java +++ b/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/action/CopyAction.java @@ -37,6 +37,7 @@ import org.apache.cayenne.map.ObjEntity; import org.apache.cayenne.map.ObjRelationship; import org.apache.cayenne.map.Procedure; import org.apache.cayenne.map.ProcedureParameter; +import org.apache.cayenne.map.QueryDescriptor; import org.apache.cayenne.modeler.Application; import org.apache.cayenne.modeler.ProjectController; import org.apache.cayenne.modeler.editor.ObjCallbackMethod; @@ -123,7 +124,7 @@ public class CopyAction extends CayenneAction { } if (object instanceof DataMap - || object instanceof Query + || object instanceof QueryDescriptor || object instanceof DbEntity || object instanceof ObjEntity || object instanceof Embeddable http://git-wip-us.apache.org/repos/asf/cayenne/blob/51a3399d/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/action/RemoveAction.java ---------------------------------------------------------------------- diff --git a/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/action/RemoveAction.java b/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/action/RemoveAction.java index 8dcf52f..db1fee6 100644 --- a/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/action/RemoveAction.java +++ b/modeler/cayenne-modeler/src/main/java/org/apache/cayenne/modeler/action/RemoveAction.java @@ -577,43 +577,17 @@ public class RemoveAction extends CayenneAction { */ @Override public boolean enableForPath(ConfigurationNode object) { - if (object == null) { - return false; - } - - if (object instanceof DataChannelDescriptor) { - return true; - } - else if (object instanceof DataMap) { - return true; - } - else if (object instanceof DataNodeDescriptor) { - return true; - } - else if (object instanceof Entity) { - return true; - } - else if (object instanceof Attribute) { - return true; - } - else if (object instanceof Relationship) { - return true; - } - else if (object instanceof Procedure) { - return true; - } - else if (object instanceof ProcedureParameter) { - return true; - } - else if (object instanceof Embeddable) { - return true; - } - else if (object instanceof EmbeddableAttribute) { - return true; - } - else { - return false; - } + return (object instanceof DataChannelDescriptor) + || (object instanceof DataMap) + || (object instanceof DataNodeDescriptor) + || (object instanceof Entity) + || (object instanceof Attribute) + || (object instanceof Relationship) + || (object instanceof Procedure) + || (object instanceof ProcedureParameter) + || (object instanceof QueryDescriptor) + || (object instanceof Embeddable) + || (object instanceof EmbeddableAttribute); } /** @@ -632,42 +606,23 @@ public class RemoveAction extends CayenneAction { undo = new RemoveUndoableEdit(application, (DataMap) object); removeDataMap((DataMap) object); } - } - else if (object instanceof DataNodeDescriptor) { + } else if (object instanceof DataNodeDescriptor) { undo = new RemoveUndoableEdit(application, (DataNodeDescriptor) object); - removeDataNode((DataNodeDescriptor) object); - } - else if (object instanceof DbEntity) { - undo = new RemoveUndoableEdit( - ((DbEntity) object).getDataMap(), - (DbEntity) object); - + } else if (object instanceof DbEntity) { + undo = new RemoveUndoableEdit(((DbEntity) object).getDataMap(), (DbEntity) object); removeDbEntity(((DbEntity) object).getDataMap(), (DbEntity) object); - } - else if (object instanceof ObjEntity) { - undo = new RemoveUndoableEdit( - ((ObjEntity) object).getDataMap(), - (ObjEntity) object); - + } else if (object instanceof ObjEntity) { + undo = new RemoveUndoableEdit(((ObjEntity) object).getDataMap(), (ObjEntity) object); removeObjEntity(((ObjEntity) object).getDataMap(), (ObjEntity) object); - } - else if (object instanceof QueryDescriptor) { - undo = new RemoveUndoableEdit(((Query) object).getDataMap(), (QueryDescriptor) object); - - removeQuery(((Query) object).getDataMap(), (QueryDescriptor) object); - } - else if (object instanceof Procedure) { - undo = new RemoveUndoableEdit( - ((Procedure) object).getDataMap(), - (Procedure) object); - + } else if (object instanceof QueryDescriptor) { + undo = new RemoveUndoableEdit(((QueryDescriptor) object).getDataMap(), (QueryDescriptor) object); + removeQuery(((QueryDescriptor) object).getDataMap(), (QueryDescriptor) object); + } else if (object instanceof Procedure) { + undo = new RemoveUndoableEdit(((Procedure) object).getDataMap(), (Procedure) object); removeProcedure(((Procedure) object).getDataMap(), (Procedure) object); - } - else if (object instanceof Embeddable) { - undo = new RemoveUndoableEdit( - ((Embeddable) object).getDataMap(), - (Embeddable) object); + } else if (object instanceof Embeddable) { + undo = new RemoveUndoableEdit(((Embeddable) object).getDataMap(), (Embeddable) object); removeEmbeddable(((Embeddable) object).getDataMap(), (Embeddable) object); }