Author: [log in to unmask]
Date: Wed Apr 22 14:08:45 2015
New Revision: 2788
Log:
Restore Iterable functionality to ConditionsObjectCollection.
Modified:
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/BaseConditionsObject.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/BaseConditionsObjectCollection.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsObject.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsObjectCollection.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsRecord.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/cli/PrintCommand.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/cli/TagCommand.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/database/ConditionsSeriesConverter.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/database/DatabaseConditionsManager.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalChannel.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalConditions.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalConditionsConverter.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalCrystalChannelMap.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/AbstractSvtChannel.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/AbstractSvtConditionsConverter.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtChannel.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtConditions.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtDaqMapping.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtT0Shift.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtChannel.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtConditions.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtDaqMapping.java
java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtT0Shift.java
java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/EngRunConditionsTest.java
java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/api/BaseConditionsObjectTest.java
java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/beam/BeamConditionsTest.java
java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/beam/BeamCurrentTest.java
java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/ecal/EcalLedTest.java
java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/SvtConfigurationTest.java
java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/SvtDaqMappingTest.java
java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/TestRunSvtDaqMappingTest.java
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/BaseConditionsObject.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/BaseConditionsObject.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/BaseConditionsObject.java Wed Apr 22 14:08:45 2015
@@ -33,7 +33,7 @@
protected static String defaultToString(final BaseConditionsObject object) {
final StringBuffer sb = new StringBuffer();
- sb.append("id: " + object.getId() + ", ");
+ sb.append("id: " + object.getRowId() + ", ");
for (final String field : object.getFieldValues().getFieldNames()) {
sb.append(field + "=" + object.getFieldValue(Object.class, field) + ", ");
}
@@ -129,7 +129,7 @@
if (isNew()) {
throw new ConditionsObjectException("Object is not in database and so cannot be deleted.");
}
- final String sql = "DELETE FROM " + this.tableMetaData.getTableName() + " WHERE id=" + this.getId();
+ final String sql = "DELETE FROM " + this.tableMetaData.getTableName() + " WHERE id=" + this.getRowId();
// if (this.verbose) {
// System.out.println(sql);
// }
@@ -162,7 +162,7 @@
}
@Override
- public final int getId() {
+ public final int getRowId() {
return this.id;
}
@@ -231,7 +231,7 @@
@Override
public final boolean isNew() {
- return getId() == UNSET_ID;
+ return getRowId() == UNSET_ID;
}
@Override
@@ -247,7 +247,7 @@
}
sb.setLength(sb.length() - 1);
sb.append(" FROM " + this.tableMetaData.getTableName());
- sb.append(" WHERE id = " + this.getId());
+ sb.append(" WHERE id = " + this.getRowId());
final String sql = sb.toString();
// if (this.verbose) {
// System.out.println(sql);
@@ -336,7 +336,7 @@
}
}
sb.setLength(sb.length() - 2);
- sb.append(" WHERE id=" + this.getId());
+ sb.append(" WHERE id=" + this.getRowId());
final String sql = sb.toString();
// if (this.verbose) {
// System.out.println(sql);
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/BaseConditionsObjectCollection.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/BaseConditionsObjectCollection.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/BaseConditionsObjectCollection.java Wed Apr 22 14:08:45 2015
@@ -6,7 +6,6 @@
import java.sql.SQLException;
import java.sql.Statement;
import java.util.ArrayList;
-import java.util.Collection;
import java.util.Collections;
import java.util.Comparator;
import java.util.Iterator;
@@ -103,12 +102,6 @@
}
@Override
- public Collection<ObjectType> getObjects() {
- // return Collections.unmodifiableCollection(this.objects);
- return this.objects;
- }
-
- @Override
public TableMetaData getTableMetaData() {
return this.tableMetaData;
}
@@ -143,7 +136,7 @@
try {
this.connection.setAutoCommit(false);
insertObjects = this.connection.prepareStatement(updateStatement, Statement.RETURN_GENERATED_KEYS);
- for (final ObjectType object : this.getObjects()) {
+ for (final ObjectType object : this) {
object.setCollectionId(this.collectionId);
for (int fieldIndex = 0; fieldIndex < this.getTableMetaData().getFieldNames().length; fieldIndex++) {
final String fieldName = this.getTableMetaData().getFieldNames()[fieldIndex];
@@ -266,15 +259,15 @@
*/
@Override
public final void sort(final Comparator<ObjectType> comparator) {
- final List<ObjectType> list = new ArrayList<ObjectType>(this.getObjects());
+ final List<ObjectType> list = new ArrayList<ObjectType>(this.objects);
Collections.sort(list, comparator);
- this.getObjects().clear();
- this.getObjects().addAll(list);
+ this.objects.clear();
+ this.objects.addAll(list);
}
@Override
public ConditionsObjectCollection<ObjectType> sorted(final Comparator<ObjectType> comparator) {
- final List<ObjectType> list = new ArrayList<ObjectType>(this.getObjects());
+ final List<ObjectType> list = new ArrayList<ObjectType>(this.objects);
Collections.sort(list, comparator);
BaseConditionsObjectCollection<ObjectType> collection;
try {
@@ -282,17 +275,26 @@
} catch (InstantiationException | IllegalAccessException e) {
throw new RuntimeException("Error creating new collection instance.", e);
}
- collection.getObjects().addAll(list);
+ for (final ObjectType object : list) {
+ collection.add(object);
+ }
return collection;
}
@Override
public boolean contains(final Object object) {
- return getObjects().contains(object);
+ return this.objects.contains(object);
}
@Override
public void addAll(final ConditionsObjectCollection<ObjectType> collection) {
- this.objects.addAll(collection.getObjects());
+ for (final ObjectType object : collection) {
+ this.objects.add(object);
+ }
+ }
+
+ @Override
+ public Iterator<ObjectType> iterator() {
+ return this.objects.iterator();
}
}
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsObject.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsObject.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsObject.java Wed Apr 22 14:08:45 2015
@@ -22,7 +22,7 @@
/**
* @return
*/
- int getId();
+ int getRowId();
/**
* @return
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsObjectCollection.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsObjectCollection.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsObjectCollection.java Wed Apr 22 14:08:45 2015
@@ -2,10 +2,9 @@
import java.sql.Connection;
import java.sql.SQLException;
-import java.util.Collection;
import java.util.Comparator;
-public interface ConditionsObjectCollection<ObjectType extends ConditionsObject> {
+public interface ConditionsObjectCollection<ObjectType extends ConditionsObject> extends Iterable<ObjectType> {
boolean add(final ObjectType object);
@@ -14,8 +13,6 @@
ObjectType get(final int index);
int getCollectionId();
-
- Collection<ObjectType> getObjects();
TableMetaData getTableMetaData();
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsRecord.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsRecord.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/api/ConditionsRecord.java Wed Apr 22 14:08:45 2015
@@ -121,7 +121,7 @@
*/
public final Set<String> getConditionsKeys() {
final Set<String> conditionsKeys = new HashSet<String>();
- for (final ConditionsRecord record : this.getObjects()) {
+ for (final ConditionsRecord record : this) {
conditionsKeys.add(record.getName());
}
return conditionsKeys;
@@ -338,7 +338,7 @@
@Override
public String toString() {
final StringBuffer sb = new StringBuffer();
- sb.append("id: " + getId() + '\n');
+ sb.append("id: " + getRowId() + '\n');
sb.append("name: " + getName() + '\n');
sb.append("runStart: " + getRunStart() + '\n');
sb.append("runEnd: " + getRunEnd() + '\n');
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/cli/PrintCommand.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/cli/PrintCommand.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/cli/PrintCommand.java Wed Apr 22 14:08:45 2015
@@ -176,7 +176,7 @@
*/
private void printCollection(final ConditionsObjectCollection<?> collection) {
final StringBuffer buffer = new StringBuffer();
- for (final Object object : collection.getObjects()) {
+ for (final Object object : collection) {
for (final String columnName : collection.getTableMetaData().getFieldNames()) {
buffer.append(((ConditionsObject) object).getFieldValue(columnName));
buffer.append(this.fieldDelimiter);
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/cli/TagCommand.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/cli/TagCommand.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/cli/TagCommand.java Wed Apr 22 14:08:45 2015
@@ -23,6 +23,7 @@
*
* @author <a href="mailto:[log in to unmask]">Jeremy McCormick</a>
*/
+// TODO: Add multiple records strategy (last updated, last created, etc.).
public class TagCommand extends AbstractCommand {
/**
@@ -104,11 +105,8 @@
throw new RuntimeException(e);
}
- // The records from this run.
- final ConditionsRecordCollection records = manager.getConditionsRecords();
-
// The unique conditions keys from this run.
- final Set<String> keys = records.getConditionsKeys();
+ final Set<String> keys = manager.getConditionsRecords().getConditionsKeys();
// Scan through all the unique keys.
for (final String key : keys) {
@@ -116,19 +114,19 @@
// Get the table meta data for the key.
final TableMetaData tableMetaData = manager.findTableMetaData(key);
+ // Get the ConditionsRecord from the collection.
+ final ConditionsRecordCollection records = manager.findConditionsRecords(key);
+ records.sortByUpdated();
+ final ConditionsRecord record = records.get(records.size() - 1);
+
manager.getCachedConditions(tableMetaData.getCollectionClass(), tableMetaData.getTableName())
.getCachedData();
- // Get the ConditionsRecord from the collection.
- // final ConditionsRecord record = collection.getConditionsRecord();
- final ConditionsRecord record = null;
-
// Is this record already part of the new tag?
- if (!addedIds.contains(record.getId())) {
+ if (!addedIds.contains(record.getRowId())) {
// Create a new record copied from the old one.
final ConditionsRecord newRecord = new ConditionsRecord(record);
- // FIXME: This is berken because collection no longer directly links to ConditionsRecord.
// Set the tag value.
newRecord.setFieldValue("tag", newTag);
@@ -136,14 +134,14 @@
tagRecords.add(newRecord);
// Flag the record's ID as used so it is only added once.
- addedIds.add(record.getId());
+ addedIds.add(record.getRowId());
}
}
}
// Print out all the records that were found.
LOGGER.info("found ConditionsRecords for tag " + newTag + " ...");
- for (final ConditionsRecord record : tagRecords.getObjects()) {
+ for (final ConditionsRecord record : tagRecords) {
LOGGER.info(record.toString());
}
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/database/ConditionsSeriesConverter.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/database/ConditionsSeriesConverter.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/database/ConditionsSeriesConverter.java Wed Apr 22 14:08:45 2015
@@ -81,7 +81,7 @@
// Get the ConditionsRecord with the meta-data, which will use the current run number from the manager.
final ConditionsRecordCollection conditionsRecords = conditionsManager.findConditionsRecords(tableName);
- for (final ConditionsRecord conditionsRecord : conditionsRecords.getObjects()) {
+ for (final ConditionsRecord conditionsRecord : conditionsRecords) {
ConditionsObjectCollection<?> collection = null;
try {
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/database/DatabaseConditionsManager.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/database/DatabaseConditionsManager.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/database/DatabaseConditionsManager.java Wed Apr 22 14:08:45 2015
@@ -386,13 +386,13 @@
"conditions").getCachedData();
logger.fine("searching for conditions with name " + name + " in " + runConditionsRecords.size() + " records");
final ConditionsRecordCollection foundConditionsRecords = new ConditionsRecordCollection();
- for (final ConditionsRecord record : runConditionsRecords.getObjects()) {
+ for (final ConditionsRecord record : runConditionsRecords) {
if (record.getName().equals(name)) {
if (matchesTag(record)) {
foundConditionsRecords.add(record);
- logger.finer("found matching conditions record " + record.getId());
+ logger.finer("found matching conditions record " + record.getRowId());
} else {
- logger.finer("conditions record " + record.getId() + " rejected from non-matching tag "
+ logger.finer("conditions record " + record.getRowId() + " rejected from non-matching tag "
+ record.getTag());
}
}
@@ -605,7 +605,7 @@
* @return <code>true</code> if a conditions record exists with the given name
*/
public boolean hasConditionsRecord(final String name) {
- return !findConditionsRecords(name).getObjects().isEmpty();
+ return !(findConditionsRecords(name).size() == 0);
}
/**
@@ -720,12 +720,12 @@
try {
this.connection.setAutoCommit(false);
logger.fine("starting insert transaction");
- final String sql = QueryBuilder.buildPreparedInsert(tableMetaData.getTableName(), collection.getObjects()
- .iterator().next());
+ final String sql = QueryBuilder.buildPreparedInsert(tableMetaData.getTableName(), collection.iterator()
+ .next());
preparedStatement = this.connection.prepareStatement(sql);
logger.fine("using prepared statement: " + sql);
final int collectionId = collection.getCollectionId();
- for (final ConditionsObject object : collection.getObjects()) {
+ for (final ConditionsObject object : collection) {
preparedStatement.setObject(1, collectionId);
int parameterIndex = 2;
if (object instanceof ConditionsRecord) {
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalChannel.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalChannel.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalChannel.java Wed Apr 22 14:08:45 2015
@@ -198,7 +198,7 @@
* @param system the system ID of the subdetector
*/
void buildGeometryMap(final IIdentifierHelper helper, final int system) {
- for (final EcalChannel channel : this.getObjects()) {
+ for (final EcalChannel channel : this) {
final GeometryId geometryId = channel.createGeometryId(helper, system);
this.geometryMap.put(geometryId.encode(), channel);
}
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalConditions.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalConditions.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalConditions.java Wed Apr 22 14:08:45 2015
@@ -42,7 +42,7 @@
/**
* Class constructor.
- *
+ *
* @param subdetector the ECAL subdetector object
*/
EcalConditions(final Subdetector subdetector) {
@@ -54,7 +54,7 @@
/**
* Set the channel map.
- *
+ *
* @param channelCollection the channel map
*/
void setChannelCollection(final EcalChannelCollection channelCollection) {
@@ -67,7 +67,7 @@
/**
* Get the map between database IDs and <code>EcalChannel</code> objects.
- *
+ *
* @return the channel map
*/
public EcalChannelCollection getChannelCollection() {
@@ -76,7 +76,7 @@
/**
* Get the channel information for a geometric crystal.
- *
+ *
* @param crystal the geometric crystal
* @return the channel information or null if does not exist
*/
@@ -87,7 +87,7 @@
/**
* Get the conditions constants for a specific channel. These will be created if they do not exist for the given
* channel, BUT only channels in the channel map are accepted as an argument.
- *
+ *
* @param channel the ECAL channel
* @return the conditions constants for the channel
* @throws IllegalArgumentException if channel does not exist in the channel map
@@ -112,7 +112,7 @@
/**
* Convert this object to a string.
- *
+ *
* @return A string representation of this object.
*/
// FIXME: The print out from this method looks like a mess.
@@ -154,7 +154,7 @@
sb.append('\n');
// Loop over all channels.
- for (final EcalChannel channel : this.channelCollection.getObjects()) {
+ for (final EcalChannel channel : this.channelCollection) {
final EcalChannelConstants constants = getChannelConstants(channel);
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalConditionsConverter.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalConditionsConverter.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalConditionsConverter.java Wed Apr 22 14:08:45 2015
@@ -30,7 +30,7 @@
/**
* Get the default {@link EcalChannel} collection.
- *
+ *
* @param manager the conditions manager
* @return the default ECAL channel object collection
*/
@@ -40,7 +40,7 @@
/**
* Get the default {@link EcalGain} collection.
- *
+ *
* @param manager the conditions manager
* @return the ECAL channel gain collection
*/
@@ -50,7 +50,7 @@
/**
* Get the default collections of {@link EcalBadChannel} objects.
- *
+ *
* @param manager the conditions manager
* @return the collections of ECAL bad channel objects
*/
@@ -61,7 +61,7 @@
/**
* Get the default {@link EcalCalibration} collection.
- *
+ *
* @param manager the conditions manager
* @return the collection of ECAL channel calibration objects
*/
@@ -71,7 +71,7 @@
/**
* Get the default {@link EcalTimeShift} collection.
- *
+ *
* @param manager the conditions manager
* @return the collection of ECAL time shift objects
*/
@@ -81,7 +81,7 @@
/**
* Create combined ECAL conditions object containing all data for the current run.
- *
+ *
* @param manager the conditions manager
* @param name the conditions set name (unused but must satisfy conditions API)
*/
@@ -105,7 +105,7 @@
// Get the ECal gains from the conditions database and add them to the
// conditions set
final EcalGainCollection gains = getEcalGainCollection(databaseConditionsManager);
- for (final EcalGain gain : gains.getObjects()) {
+ for (final EcalGain gain : gains) {
final ChannelId channelId = new ChannelId(new int[] {gain.getChannelId()});
final EcalChannel channel = channels.findChannel(channelId);
conditions.getChannelConstants(channel).setGain(gain);
@@ -115,7 +115,7 @@
// FIXME: How to get EcalBadChannelCollection here instead for the collection type?
// API of ConditionsSeries and ConditionsSeriesConverter needs to be changed!
for (final ConditionsObjectCollection<EcalBadChannel> badChannels : badChannelSeries) {
- for (final EcalBadChannel badChannel : badChannels.getObjects()) {
+ for (final EcalBadChannel badChannel : badChannels) {
final ChannelId channelId = new ChannelId(new int[] {badChannel.getChannelId()});
final EcalChannel channel = channels.findChannel(channelId);
conditions.getChannelConstants(channel).setBadChannel(true);
@@ -125,7 +125,7 @@
// Get the ECal calibrations from the conditions database and add them
// to the conditions set.
final EcalCalibrationCollection calibrations = getEcalCalibrationCollection(databaseConditionsManager);
- for (final EcalCalibration calibration : calibrations.getObjects()) {
+ for (final EcalCalibration calibration : calibrations) {
final ChannelId channelId = new ChannelId(new int[] {calibration.getChannelId()});
final EcalChannel channel = channels.findChannel(channelId);
conditions.getChannelConstants(channel).setCalibration(calibration);
@@ -135,7 +135,7 @@
// the conditions set.
if (databaseConditionsManager.hasConditionsRecord("ecal_time_shifts")) {
final EcalTimeShiftCollection timeShifts = getEcalTimeShiftCollection(databaseConditionsManager);
- for (final EcalTimeShift timeShift : timeShifts.getObjects()) {
+ for (final EcalTimeShift timeShift : timeShifts) {
final ChannelId channelId = new ChannelId(new int[] {timeShift.getChannelId()});
final EcalChannel channel = channels.findChannel(channelId);
conditions.getChannelConstants(channel).setTimeShift(timeShift);
@@ -150,7 +150,7 @@
/**
* Get the type handled by this converter.
- *
+ *
* @return the type handled by this converter
*/
@Override
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalCrystalChannelMap.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalCrystalChannelMap.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/ecal/EcalCrystalChannelMap.java Wed Apr 22 14:08:45 2015
@@ -27,7 +27,7 @@
/**
* Creates the map between crystals and channels.
- *
+ *
* @param api the ECAL geometry API
* @param channels the list of channels
*/
@@ -43,7 +43,7 @@
}
// Map channels to crystals.
- for (final EcalChannel channel : channels.getObjects()) {
+ for (final EcalChannel channel : channels) {
final EcalCrystal crystal = api.getCrystal(channel.getX(), channel.getY());
if (crystal == null) {
throw new RuntimeException("ECAl crystal was not found for channel X Y: " + channel.getX() + " "
@@ -55,7 +55,7 @@
/**
* Get a channel from a crystal.
- *
+ *
* @param crystal the crystal's geometry object
* @return the channel information or <code>null</code> if does not exist
*/
@@ -65,7 +65,7 @@
/**
* Get a crystal from a channel.
- *
+ *
* @param channel the ECAL channel object
* @return the crystal's geometry object or <code>null</code> if does not exist
*/
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/AbstractSvtChannel.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/AbstractSvtChannel.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/AbstractSvtChannel.java Wed Apr 22 14:08:45 2015
@@ -82,7 +82,7 @@
@Override
public final String toString() {
final StringBuffer buff = new StringBuffer();
- for (final T channel : this.getObjects()) {
+ for (final T channel : this) {
buff.append(channel.toString() + '\n');
}
return buff.toString();
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/AbstractSvtConditionsConverter.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/AbstractSvtConditionsConverter.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/AbstractSvtConditionsConverter.java Wed Apr 22 14:08:45 2015
@@ -34,7 +34,7 @@
/**
* Get the default {@link SvtShapeFitParametersCollection} collection from the manager.
- *
+ *
* @param manager the current conditions manager
* @return the default {@link SvtShapeFitParametersCollection}
*/
@@ -45,7 +45,7 @@
/**
* Get the default {@link SvtBadChannelCollection} collection from the manager.
- *
+ *
* @param manager the current conditions manager
* @return the default {@link SvtBadChannelCollection}
*/
@@ -55,7 +55,7 @@
/**
* Get the default series of {@link SvtBadChannelCollection} collections from the manager.
- *
+ *
* @param manager the current conditions manager
* @return the default series of {@link SvtBadChannelCollection}
*/
@@ -66,7 +66,7 @@
/**
* Get the default {@link SvtCalibrationCollection} collection from the manager.
- *
+ *
* @param manager the current conditions manager
* @return the default {@link SvtCalibrationCollection}
*/
@@ -76,7 +76,7 @@
/**
* Get the default {@link SvtGainCollection} collection from the manager.
- *
+ *
* @param manager the current conditions manager
* @return the default {@link SvtGainCollection}
*/
@@ -98,14 +98,14 @@
// Get the SVT calibrations (baseline, noise) from the conditions database
final SvtCalibrationCollection calibrations = getSvtCalibrationCollection(dbConditionsManager);
- for (final SvtCalibration calibration : calibrations.getObjects()) {
+ for (final SvtCalibration calibration : calibrations) {
final AbstractSvtChannel channel = this.conditions.getChannelMap().findChannel(calibration.getChannelID());
this.conditions.getChannelConstants(channel).setCalibration(calibration);
}
// Get the Channel pulse fit parameters from the conditions database
final SvtShapeFitParametersCollection shapeFitParametersCollection = getSvtShapeFitParametersCollection(dbConditionsManager);
- for (final SvtShapeFitParameters shapeFitParameters : shapeFitParametersCollection.getObjects()) {
+ for (final SvtShapeFitParameters shapeFitParameters : shapeFitParametersCollection) {
final AbstractSvtChannel channel = this.conditions.getChannelMap().findChannel(
shapeFitParameters.getChannelID());
this.conditions.getChannelConstants(channel).setShapeFitParameters(shapeFitParameters);
@@ -116,7 +116,7 @@
try {
final ConditionsSeries<SvtBadChannel, SvtBadChannelCollection> badChannelSeries = getSvtBadChannelSeries(dbConditionsManager);
for (final ConditionsObjectCollection<SvtBadChannel> badChannelCollection : badChannelSeries) {
- for (final SvtBadChannel badChannel : badChannelCollection.getObjects()) {
+ for (final SvtBadChannel badChannel : badChannelCollection) {
final AbstractSvtChannel channel = this.conditions.getChannelMap().findChannel(
badChannel.getChannelId());
this.conditions.getChannelConstants(channel).setBadChannel(true);
@@ -128,7 +128,7 @@
// Get the gains and offsets from the conditions database
final SvtGainCollection channelGains = getSvtGainCollection(dbConditionsManager);
- for (final SvtGain channelGain : channelGains.getObjects()) {
+ for (final SvtGain channelGain : channelGains) {
final int channelId = channelGain.getChannelID();
final AbstractSvtChannel channel = this.conditions.getChannelMap().findChannel(channelId);
this.conditions.getChannelConstants(channel).setGain(channelGain);
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtChannel.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtChannel.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtChannel.java Wed Apr 22 14:08:45 2015
@@ -90,7 +90,7 @@
final List<SvtChannel> channels = new ArrayList<SvtChannel>();
final int febID = pair.getFirstElement();
final int febHybridID = pair.getSecondElement();
- for (final SvtChannel channel : this.getObjects()) {
+ for (final SvtChannel channel : this) {
if (channel.getFebID() == febID && channel.getFebHybridID() == febHybridID) {
channels.add(channel);
}
@@ -108,7 +108,7 @@
* @throws {@link RuntimeException} if the channel ID can't be found
*/
public final int findChannelID(final int febID, final int febHybridID, final int channel) {
- for (final SvtChannel svtChannel : this.getObjects()) {
+ for (final SvtChannel svtChannel : this) {
if (svtChannel.getFebID() == febID && svtChannel.getFebHybridID() == febHybridID
&& svtChannel.getChannel() == channel) {
return svtChannel.getChannelID();
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtConditions.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtConditions.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtConditions.java Wed Apr 22 14:08:45 2015
@@ -114,7 +114,7 @@
}
buff.append('\n');
// Loop over channels.
- for (final SvtChannel channel : this.getChannelMap().getObjects()) {
+ for (final SvtChannel channel : this.getChannelMap()) {
// Get the conditions for the channel.
final ChannelConstants constants = getChannelConstants(channel);
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtDaqMapping.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtDaqMapping.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtDaqMapping.java Wed Apr 22 14:08:45 2015
@@ -33,7 +33,7 @@
public Pair<Integer, Integer> getDaqPair(final HpsSiSensor sensor) {
final String svtHalf = sensor.isTopLayer() ? TOP_HALF : BOTTOM_HALF;
- for (final SvtDaqMapping object : this.getObjects()) {
+ for (final SvtDaqMapping object : this) {
if (svtHalf.equals(object.getSvtHalf()) && object.getLayerNumber() == sensor.getLayerNumber()
&& object.getSide().equals(sensor.getSide())) {
@@ -53,7 +53,7 @@
*/
@Override
public String getOrientation(final Pair<Integer, Integer> daqPair) {
- for (final SvtDaqMapping daqMapping : this.getObjects()) {
+ for (final SvtDaqMapping daqMapping : this) {
if (daqPair.getFirstElement() == daqMapping.getFebID()
&& daqPair.getSecondElement() == daqMapping.getFebHybridID()) {
return daqMapping.getOrientation();
@@ -85,7 +85,7 @@
buff.append('\n');
buff.append("----------------------");
buff.append('\n');
- for (final SvtDaqMapping object : this.getObjects()) {
+ for (final SvtDaqMapping object : this) {
buff.append(object.getFebID());
buff.append(" ");
buff.append(object.getFebHybridID());
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtT0Shift.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtT0Shift.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/SvtT0Shift.java Wed Apr 22 14:08:45 2015
@@ -33,7 +33,7 @@
public SvtT0Shift getT0Shift(final Pair<Integer, Integer> pair) {
final int febID = pair.getFirstElement();
final int febHybridID = pair.getSecondElement();
- for (final SvtT0Shift t0Shift : this.getObjects()) {
+ for (final SvtT0Shift t0Shift : this) {
if (t0Shift.getFebID() == febID && t0Shift.getFebHybridID() == febHybridID) {
return t0Shift;
}
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtChannel.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtChannel.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtChannel.java Wed Apr 22 14:08:45 2015
@@ -37,7 +37,7 @@
final List<TestRunSvtChannel> channels = new ArrayList<TestRunSvtChannel>();
final int fpga = pair.getFirstElement();
final int hybrid = pair.getSecondElement();
- for (final TestRunSvtChannel channel : this.getObjects()) {
+ for (final TestRunSvtChannel channel : this) {
if (channel.getFpgaID() == fpga && channel.getHybridID() == hybrid) {
channels.add(channel);
}
@@ -68,7 +68,7 @@
/**
* Implementation of equals.
- *
+ *
* @param o the other object
* @return <code>true</code> if the object equals this one; false if not.
*/
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtConditions.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtConditions.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtConditions.java Wed Apr 22 14:08:45 2015
@@ -5,6 +5,7 @@
import org.hps.conditions.svt.TestRunSvtChannel.TestRunSvtChannelCollection;
import org.hps.conditions.svt.TestRunSvtDaqMapping.TestRunSvtDaqMappingCollection;
import org.hps.conditions.svt.TestRunSvtT0Shift.TestRunSvtT0ShiftCollection;
+
// TODO: Move all constants to their own class
/**
@@ -119,7 +120,7 @@
}
buff.append('\n');
// Loop over channels.
- for (final TestRunSvtChannel channel : this.getChannelMap().getObjects()) {
+ for (final TestRunSvtChannel channel : this.getChannelMap()) {
System.out.println("Channel: " + channel.toString());
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtDaqMapping.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtDaqMapping.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtDaqMapping.java Wed Apr 22 14:08:45 2015
@@ -32,7 +32,7 @@
public Pair<Integer, Integer> getDaqPair(final HpsSiSensor sensor) {
final String svtHalf = sensor.isTopLayer() ? TOP_HALF : BOTTOM_HALF;
- for (final TestRunSvtDaqMapping daqMapping : this.getObjects()) {
+ for (final TestRunSvtDaqMapping daqMapping : this) {
if (svtHalf.equals(daqMapping.getSvtHalf()) && daqMapping.getLayerNumber() == sensor.getLayerNumber()) {
@@ -51,7 +51,7 @@
*/
@Override
public String getOrientation(final Pair<Integer, Integer> daqPair) {
- for (final TestRunSvtDaqMapping daqMapping : this.getObjects()) {
+ for (final TestRunSvtDaqMapping daqMapping : this) {
if (daqPair.getFirstElement() == daqMapping.getFpgaID()
&& daqPair.getSecondElement() == daqMapping.getHybridID()) {
return daqMapping.getOrientation();
@@ -81,7 +81,7 @@
buffer.append('\n');
buffer.append("----------------------");
buffer.append('\n');
- for (final TestRunSvtDaqMapping daqMapping : this.getObjects()) {
+ for (final TestRunSvtDaqMapping daqMapping : this) {
final TestRunSvtDaqMapping testRunDaqMapping = daqMapping;
buffer.append(testRunDaqMapping.getFpgaID());
buffer.append(" ");
Modified: java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtT0Shift.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtT0Shift.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/main/java/org/hps/conditions/svt/TestRunSvtT0Shift.java Wed Apr 22 14:08:45 2015
@@ -32,7 +32,7 @@
public TestRunSvtT0Shift getT0Shift(final Pair<Integer, Integer> pair) {
final int fpgaID = pair.getFirstElement();
final int hybridID = pair.getSecondElement();
- for (final TestRunSvtT0Shift t0Shift : this.getObjects()) {
+ for (final TestRunSvtT0Shift t0Shift : this) {
if (t0Shift.getFpgaID() == fpgaID && t0Shift.getHybridID() == hybridID) {
return t0Shift;
}
Modified: java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/EngRunConditionsTest.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/EngRunConditionsTest.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/EngRunConditionsTest.java Wed Apr 22 14:08:45 2015
@@ -165,7 +165,7 @@
this.ecalConditions = conditionsManager.getCachedConditions(EcalConditions.class, "ecal_conditions")
.getCachedData();
final Set<EcalChannelConstants> channelConstants = new LinkedHashSet<EcalChannelConstants>();
- for (final EcalChannel channel : this.ecalConditions.getChannelCollection().sorted().getObjects()) {
+ for (final EcalChannel channel : this.ecalConditions.getChannelCollection().sorted()) {
channelConstants.add(this.ecalConditions.getChannelConstants(channel));
}
assertEquals("Wrong number of channel constants.", CHANNEL_COUNT, channelConstants.size());
Modified: java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/api/BaseConditionsObjectTest.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/api/BaseConditionsObjectTest.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/api/BaseConditionsObjectTest.java Wed Apr 22 14:08:45 2015
@@ -43,7 +43,7 @@
newObject.setFieldValue("dummy", 1.0);
final boolean inserted = newObject.insert();
assertTrue("Insert failed.", inserted);
- System.out.println("Inserted new object with id " + newObject.getId());
+ System.out.println("Inserted new object with id " + newObject.getRowId());
// Update the same object.
newObject.setFieldValue("dummy", 2.0);
@@ -57,9 +57,9 @@
// Select into another object using the row ID.
final DummyConditionsObject anotherObject = new DummyConditionsObject(connection, tableMetaData);
- final boolean selected = anotherObject.select(newObject.getId());
+ final boolean selected = anotherObject.select(newObject.getRowId());
assertTrue("Select failed.", selected);
- assertEquals("Select object has wrong row ID.", newObject.getId(), anotherObject.getId());
+ assertEquals("Select object has wrong row ID.", newObject.getRowId(), anotherObject.getRowId());
assertEquals("Select object has wrong collcetion ID.", newObject.getFieldValue(Integer.class, "collection_id"),
anotherObject.getFieldValue(Integer.class, "collection_id"));
assertEquals("Select object has wrong value.", newObject.getFieldValue(Double.class, "dummy"),
Modified: java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/beam/BeamConditionsTest.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/beam/BeamConditionsTest.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/beam/BeamConditionsTest.java Wed Apr 22 14:08:45 2015
@@ -57,7 +57,7 @@
"beam").getCachedData();
final BeamConditions beam = beamCollection.get(0);
System.out.print(run + " ");
- System.out.print(beam.getId() + " ");
+ System.out.print(beam.getRowId() + " ");
System.out.print(beam.getCurrent() + " ");
System.out.print(beam.getPositionX() + " ");
System.out.print(beam.getPositionY() + " ");
Modified: java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/beam/BeamCurrentTest.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/beam/BeamCurrentTest.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/beam/BeamCurrentTest.java Wed Apr 22 14:08:45 2015
@@ -76,7 +76,7 @@
/**
* This method will check the beam current against the answer key for the first event of a new run.
- *
+ *
* @param the LCSim event
*/
@Override
@@ -85,7 +85,7 @@
this.currentRun = event.getRunNumber();
final BeamCurrentCollection collection = DatabaseConditionsManager.getInstance()
.getCachedConditions(BeamCurrentCollection.class, "beam_current").getCachedData();
- final BeamCurrent beamCurrent = collection.getObjects().iterator().next();
+ final BeamCurrent beamCurrent = collection.iterator().next();
System.out.println("Run " + event.getRunNumber() + " has integrated beam current "
+ beamCurrent.getIntegratedBeamCurrent() + " nC.");
assertEquals("Wrong beam current for run.", ANSWER_KEY.get(this.currentRun),
Modified: java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/ecal/EcalLedTest.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/ecal/EcalLedTest.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/ecal/EcalLedTest.java Wed Apr 22 14:08:45 2015
@@ -46,14 +46,14 @@
// LED channel information.
final EcalLedCollection leds = conditionsManager.getCachedConditions(EcalLedCollection.class, "ecal_leds")
.getCachedData();
- for (final EcalLed led : leds.getObjects()) {
+ for (final EcalLed led : leds) {
System.out.println(led);
}
// LED calibration data.
final EcalLedCalibrationCollection calibrations = conditionsManager.getCachedConditions(
EcalLedCalibrationCollection.class, "ecal_led_calibrations").getCachedData();
- for (final EcalLedCalibration calibration : calibrations.getObjects()) {
+ for (final EcalLedCalibration calibration : calibrations) {
System.out.println(calibration);
}
}
Modified: java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/SvtConfigurationTest.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/SvtConfigurationTest.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/SvtConfigurationTest.java Wed Apr 22 14:08:45 2015
@@ -23,7 +23,7 @@
final DatabaseConditionsManager manager = DatabaseConditionsManager.getInstance();
final SvtConfigurationCollection collection = manager.getCachedConditions(SvtConfigurationCollection.class,
"svt_configurations").getCachedData();
- for (final SvtConfiguration config : collection.getObjects()) {
+ for (final SvtConfiguration config : collection) {
Document doc = null;
try {
doc = config.createDocument();
Modified: java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/SvtDaqMappingTest.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/SvtDaqMappingTest.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/SvtDaqMappingTest.java Wed Apr 22 14:08:45 2015
@@ -30,7 +30,7 @@
/**
* Load the DAQ map from the database.
- *
+ *
* @throws Exception if there is a test error
*/
public void test() throws Exception {
@@ -41,7 +41,7 @@
int totalSensors = 0;
int febHybridID;
// this.printDebug("");
- for (final SvtDaqMapping daqMapping : daqMappingCollection.getObjects()) {
+ for (final SvtDaqMapping daqMapping : daqMappingCollection) {
// this.printDebug("Sensor: \n" + daqMapping.toString());
// Check that the FEB Hybrid ID is within the allowable limits
febHybridID = daqMapping.getFebHybridID();
Modified: java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/TestRunSvtDaqMappingTest.java
=============================================================================
--- java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/TestRunSvtDaqMappingTest.java (original)
+++ java/branches/conditions-HPSJAVA-488/src/test/java/org/hps/conditions/svt/TestRunSvtDaqMappingTest.java Wed Apr 22 14:08:45 2015
@@ -55,7 +55,7 @@
/**
* Perform checks of SVT DAQ mapping for Test Run.
- *
+ *
* @throws Exception if there is a test or conditions error
*/
public void test() throws Exception {
@@ -69,7 +69,7 @@
int totalSensors = 0;
this.printDebug("");
- for (final TestRunSvtDaqMapping daqMapping : daqMappingCollection.getObjects()) {
+ for (final TestRunSvtDaqMapping daqMapping : daqMappingCollection) {
this.printDebug("Sensor: \n" + daqMapping.toString());
@@ -96,7 +96,7 @@
/**
* Print debug message.
- *
+ *
* @param debugMessage the message
*/
private void printDebug(final String debugMessage) {
|