Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
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.java2
-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.java8
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

Back to the top