diff options
Diffstat (limited to 'plugins/org.eclipse.osee.orcs.core/src')
-rw-r--r-- | plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/OrcsApiImpl.java | 2 | ||||
-rw-r--r-- | plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/CollectDirtyData.java (renamed from plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/CollectAndCopyDirtyData.java) | 23 | ||||
-rw-r--r-- | plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/TxDataHandlerFactoryImpl.java | 8 |
3 files changed, 11 insertions, 22 deletions
diff --git a/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/OrcsApiImpl.java b/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/OrcsApiImpl.java index d3bd978e167..aaa0d1cb072 100644 --- a/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/OrcsApiImpl.java +++ b/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/OrcsApiImpl.java @@ -148,7 +148,7 @@ public class OrcsApiImpl implements OrcsApi { proxyFactory = new ArtifactProxyFactory(artifactFactory); - txUpdateFactory = new TxDataHandlerFactoryImpl(module.getDataFactory()); + txUpdateFactory = new TxDataHandlerFactoryImpl(); ArtifactBuilderFactory builderFactory = new ArtifactBuilderFactoryImpl(logger, proxyFactory, artifactFactory, attributeFactory); diff --git a/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/CollectAndCopyDirtyData.java b/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/CollectDirtyData.java index d0b9736e93f..5c47448f4c7 100644 --- a/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/CollectAndCopyDirtyData.java +++ b/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/CollectDirtyData.java @@ -12,47 +12,40 @@ package org.eclipse.osee.orcs.core.internal.transaction.handler; import java.util.ArrayList; import java.util.List; -import org.eclipse.osee.framework.core.exception.OseeCoreException; -import org.eclipse.osee.orcs.core.ds.ArtifactData; import org.eclipse.osee.orcs.core.ds.ArtifactTransactionData; import org.eclipse.osee.orcs.core.ds.ArtifactTxDataImpl; import org.eclipse.osee.orcs.core.ds.AttributeData; -import org.eclipse.osee.orcs.core.ds.DataFactory; import org.eclipse.osee.orcs.core.internal.artifact.Artifact; import org.eclipse.osee.orcs.core.internal.artifact.ArtifactVisitor; import org.eclipse.osee.orcs.core.internal.attribute.Attribute; /** - * Takes a snapshot of all the dirty internal OrcsData + * Collect all the dirty internal OrcsData * * @author Roberto E. Escobar */ -public class CollectAndCopyDirtyData implements ArtifactVisitor { - private final DataFactory dataFactory; +public class CollectDirtyData implements ArtifactVisitor { + private final List<ArtifactTransactionData> data; private ArtifactTransactionData txData; - public CollectAndCopyDirtyData(DataFactory dataFactory, List<ArtifactTransactionData> data) { - this.dataFactory = dataFactory; + public CollectDirtyData(List<ArtifactTransactionData> data) { this.data = data; } - @SuppressWarnings("unused") @Override - public void visit(Artifact artifact) throws OseeCoreException { + public void visit(Artifact artifact) { if (artifact.isDirty()) { - ArtifactData copy = dataFactory.clone(artifact.getOrcsData()); - txData = new ArtifactTxDataImpl(copy, new ArrayList<AttributeData>()); + txData = new ArtifactTxDataImpl(artifact.getOrcsData(), new ArrayList<AttributeData>()); data.add(txData); } } @Override - public void visit(Attribute<?> attribute) throws OseeCoreException { + public void visit(Attribute<?> attribute) { if (attribute.isDirty()) { - AttributeData copy = dataFactory.clone(attribute.getOrcsData()); - txData.getAttributeData().add(copy); + txData.getAttributeData().add(attribute.getOrcsData()); } } }
\ No newline at end of file diff --git a/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/TxDataHandlerFactoryImpl.java b/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/TxDataHandlerFactoryImpl.java index b6455eee1ba..9e6b4109f74 100644 --- a/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/TxDataHandlerFactoryImpl.java +++ b/plugins/org.eclipse.osee.orcs.core/src/org/eclipse/osee/orcs/core/internal/transaction/handler/TxDataHandlerFactoryImpl.java @@ -13,7 +13,6 @@ package org.eclipse.osee.orcs.core.internal.transaction.handler; import java.util.List; import java.util.Map; import org.eclipse.osee.orcs.core.ds.ArtifactTransactionData; -import org.eclipse.osee.orcs.core.ds.DataFactory; import org.eclipse.osee.orcs.core.ds.OrcsVisitor; import org.eclipse.osee.orcs.core.ds.OrcsVisitorAdapter; import org.eclipse.osee.orcs.core.internal.artifact.ArtifactVisitor; @@ -25,16 +24,13 @@ import org.eclipse.osee.orcs.data.ArtifactWriteable; */ public class TxDataHandlerFactoryImpl implements TxDataHandlerFactory { - private final DataFactory dataFactory; - - public TxDataHandlerFactoryImpl(DataFactory dataFactory) { + public TxDataHandlerFactoryImpl() { super(); - this.dataFactory = dataFactory; } @Override public ArtifactVisitor createOnDirtyHandler(List<ArtifactTransactionData> data) { - return new CollectAndCopyDirtyData(dataFactory, data); + return new CollectDirtyData(data); } @Override |