Skip to main content
summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRyan D. Brooks2017-02-06 17:07:31 +0000
committerDavid W. Miller2017-02-06 17:07:31 +0000
commit7884093a7c36d1c153c813e5d0336273c1b76209 (patch)
treedb4d1a806b595e3ed6fce2e183bc03e76791a067 /plugins/org.eclipse.osee.orcs.db
parent1f84c2daf99fe17eb0c6d07d8b088a1f45cceca3 (diff)
downloadorg.eclipse.osee-7884093a7c36d1c153c813e5d0336273c1b76209.tar.gz
org.eclipse.osee-7884093a7c36d1c153c813e5d0336273c1b76209.tar.xz
org.eclipse.osee-7884093a7c36d1c153c813e5d0336273c1b76209.zip
refactor: Remove superfluous usage of branch.getUuid
Diffstat (limited to 'plugins/org.eclipse.osee.orcs.db')
-rw-r--r--plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/accessor/DatabaseConflictAccessor.java2
-rw-r--r--plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/ArchiveUnarchiveBranchCallable.java6
-rw-r--r--plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/ChangeBranchFieldCallable.java2
-rw-r--r--plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/CommitBranchDatabaseTxCallable.java11
-rw-r--r--plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/PurgeBranchDatabaseCallable.java2
-rw-r--r--plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/exchange/handler/BranchDataSaxHandler.java4
-rw-r--r--plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/search/handlers/BranchAncestorOfSqlHandler.java2
-rw-r--r--plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/search/handlers/BranchChildOfSqlHandler.java2
8 files changed, 15 insertions, 16 deletions
diff --git a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/accessor/DatabaseConflictAccessor.java b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/accessor/DatabaseConflictAccessor.java
index 0db4c40fae5..370d772d082 100644
--- a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/accessor/DatabaseConflictAccessor.java
+++ b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/accessor/DatabaseConflictAccessor.java
@@ -61,7 +61,7 @@ public class DatabaseConflictAccessor {
conflicts.add(
new Conflict(StorageState.LOADED, uniqueId, conflictType, mergeBranch, status, sourceGammaId, destGammaId));
};
- getJdbcClient().runQuery(consumer, SELECT_CONFLICTS, mergeBranch.getUuid());
+ getJdbcClient().runQuery(consumer, SELECT_CONFLICTS, mergeBranch);
}
public void store(Collection<Conflict> conflicts) throws OseeCoreException {
diff --git a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/ArchiveUnarchiveBranchCallable.java b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/ArchiveUnarchiveBranchCallable.java
index 20fa0686ede..5248c7fc1c8 100644
--- a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/ArchiveUnarchiveBranchCallable.java
+++ b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/ArchiveUnarchiveBranchCallable.java
@@ -46,13 +46,13 @@ public class ArchiveUnarchiveBranchCallable extends AbstractDatastoreTxCallable<
String destinationTableName = archive ? "osee_txs_archived" : "osee_txs";
String sql = String.format(INSERT_ADDRESSING, destinationTableName, sourceTableName);
- jdbcClient.runPreparedUpdate(connection, sql, branch.getUuid());
+ jdbcClient.runPreparedUpdate(connection, sql, branch);
sql = String.format(DELETE_ADDRESSING, sourceTableName);
- jdbcClient.runPreparedUpdate(connection, sql, branch.getUuid());
+ jdbcClient.runPreparedUpdate(connection, sql, branch);
BranchArchivedState newState = archive ? BranchArchivedState.ARCHIVED : BranchArchivedState.UNARCHIVED;
- Object[] params = new Object[] {newState.getValue(), branch.getUuid()};
+ Object[] params = new Object[] {newState.getValue(), branch};
jdbcClient.runPreparedUpdate(connection, UPDATE_BRANCH, params);
return null;
}
diff --git a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/ChangeBranchFieldCallable.java b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/ChangeBranchFieldCallable.java
index 1eee3ab9267..a61c5b4a78f 100644
--- a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/ChangeBranchFieldCallable.java
+++ b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/ChangeBranchFieldCallable.java
@@ -44,7 +44,7 @@ public class ChangeBranchFieldCallable extends AbstractDatastoreTxCallable<Void>
protected Void handleTxWork(JdbcConnection connection) throws OseeCoreException {
Conditions.checkNotNull(branch, "branch");
Conditions.checkNotNullOrEmpty(field, "column");
- Object[] params = new Object[] {value, branch.getUuid()};
+ Object[] params = new Object[] {value, branch};
String query = String.format(UPDATE_BRANCH_FIELD, field);
getJdbcClient().runPreparedUpdate(connection, query, params);
return null;
diff --git a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/CommitBranchDatabaseTxCallable.java b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/CommitBranchDatabaseTxCallable.java
index c57d291cf90..4a439d5fffa 100644
--- a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/CommitBranchDatabaseTxCallable.java
+++ b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/CommitBranchDatabaseTxCallable.java
@@ -92,8 +92,7 @@ public class CommitBranchDatabaseTxCallable extends AbstractDatastoreTxCallable<
updatePreviousCurrentsOnDestinationBranch(connection);
insertCommitAddressing(newTx, connection);
- getJdbcClient().runPreparedUpdate(connection, UPDATE_MERGE_COMMIT_TX, newTx, sourceBranch.getUuid(),
- destinationBranch.getUuid());
+ getJdbcClient().runPreparedUpdate(connection, UPDATE_MERGE_COMMIT_TX, newTx, sourceBranch, destinationBranch);
manageBranchStates();
if (mergeBranch.isValid()) {
@@ -154,8 +153,8 @@ public class CommitBranchDatabaseTxCallable extends AbstractDatastoreTxCallable<
String comment = COMMIT_COMMENT + sourceBranch.getName();
getJdbcClient().runPreparedUpdate(connection, INSERT_COMMIT_TRANSACTION,
- TransactionDetailsType.NonBaselined.getId(), destinationBranch.getUuid(), newTransactionNumber, comment,
- timestamp, committer, sourceBranch.getAssociatedArtifact());
+ TransactionDetailsType.NonBaselined.getId(), destinationBranch, newTransactionNumber, comment, timestamp,
+ committer, sourceBranch.getAssociatedArtifact());
return newTransactionNumber;
}
@@ -166,11 +165,11 @@ public class CommitBranchDatabaseTxCallable extends AbstractDatastoreTxCallable<
ApplicabilityToken appToken = change.getNetChange().getApplicabilityToken();
insertData.add(new Object[] {
newTx,
- destinationBranch.getUuid(),
+ destinationBranch,
change.getNetChange().getGammaId(),
modType.getValue(),
TxChange.getCurrent(modType).getValue(),
- appToken.getId()});
+ appToken});
}
getJdbcClient().runBatchUpdate(connection, INSERT_COMMIT_ADDRESSING, insertData);
}
diff --git a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/PurgeBranchDatabaseCallable.java b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/PurgeBranchDatabaseCallable.java
index 899cea16498..ef18411de51 100644
--- a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/PurgeBranchDatabaseCallable.java
+++ b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/callable/PurgeBranchDatabaseCallable.java
@@ -76,7 +76,7 @@ public class PurgeBranchDatabaseCallable extends AbstractDatastoreTxCallable<Voi
private List<Pair<Long, Boolean>> findMergeBranches(JdbcConnection connection) {
List<Pair<Long, Boolean>> toReturn = new LinkedList<>();
JdbcStatement stmt = getJdbcClient().getStatement(connection);
- stmt.runPreparedQuery(SELECT_MERGE_BRANCHES, toDelete.getUuid(), toDelete.getUuid());
+ stmt.runPreparedQuery(SELECT_MERGE_BRANCHES, toDelete, toDelete);
while (stmt.next()) {
Pair<Long, Boolean> toAdd =
new Pair<Long, Boolean>(stmt.getLong("merge_branch_id"), stmt.getBoolean("archived"));
diff --git a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/exchange/handler/BranchDataSaxHandler.java b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/exchange/handler/BranchDataSaxHandler.java
index 2b3108f2f22..8de4b05ebeb 100644
--- a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/exchange/handler/BranchDataSaxHandler.java
+++ b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/exchange/handler/BranchDataSaxHandler.java
@@ -10,7 +10,7 @@
*******************************************************************************/
package org.eclipse.osee.orcs.db.internal.exchange.handler;
-import static org.eclipse.osee.framework.core.enums.CoreBranches.SYSTEM_ROOT_ID;
+import static org.eclipse.osee.framework.core.enums.CoreBranches.SYSTEM_ROOT;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;
@@ -124,7 +124,7 @@ public class BranchDataSaxHandler extends BaseDbSaxHandler {
int index = 0;
for (BranchData branchData : branchesToStore) {
if (!getOptions().getBoolean(ImportOptions.CLEAN_BEFORE_IMPORT.name()) //
- && SYSTEM_ROOT_ID.equals(branchData.getId())) {
+ && SYSTEM_ROOT.equals(branchData.getId())) {
continue;
}
diff --git a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/search/handlers/BranchAncestorOfSqlHandler.java b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/search/handlers/BranchAncestorOfSqlHandler.java
index a6b993ee494..790602e6833 100644
--- a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/search/handlers/BranchAncestorOfSqlHandler.java
+++ b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/search/handlers/BranchAncestorOfSqlHandler.java
@@ -47,7 +47,7 @@ public class BranchAncestorOfSqlHandler extends SqlHandler<CriteriaBranchAncesto
body.append(" SELECT parent_branch_id, branch_level - 1 FROM ").append(withAlias);
body.append(" recurse, osee_branch br");
body.append(" WHERE br.branch_id = recurse.parent_id");
- writer.addParameter(criteria.getChild().getUuid());
+ writer.addParameter(criteria.getChild());
writer.addWithClause(newRecursiveWithClause(withAlias, "(parent_id, branch_level)", body.toString()));
writer.addTable(withAlias);
}
diff --git a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/search/handlers/BranchChildOfSqlHandler.java b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/search/handlers/BranchChildOfSqlHandler.java
index 18595d599f1..a4282c9dee2 100644
--- a/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/search/handlers/BranchChildOfSqlHandler.java
+++ b/plugins/org.eclipse.osee.orcs.db/src/org/eclipse/osee/orcs/db/internal/search/handlers/BranchChildOfSqlHandler.java
@@ -46,7 +46,7 @@ public class BranchChildOfSqlHandler extends SqlHandler<CriteriaBranchChildOf> {
body.append("\n UNION ALL \n");
body.append(" SELECT branch_id, branch_level + 1 FROM ").append(withAlias).append(" recurse, osee_branch br");
body.append(" WHERE recurse.child_id = br.parent_branch_id");
- writer.addParameter(criteria.getParent().getUuid());
+ writer.addParameter(criteria.getParent());
writer.addWithClause(newRecursiveWithClause(withAlias, "(child_id, branch_level)", body.toString()));
writer.addTable(withAlias);
}

Back to the top