Skip to content

Commit 3c9c97b

Browse files
labkey-matthewblabkey-matthewb
committed
checkpoint - migrate to using long/Long to represent any integer ID value (rowId, objectId, etc).
1 parent 8dabc44 commit 3c9c97b

11 files changed

Lines changed: 28 additions & 20 deletions

File tree

‎Viral_Load_Assay/src/org/labkey/viral_load_assay/assay/ABI7500ImportMethod.java‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
import org.json.JSONException;
2828
import org.json.JSONObject;
2929
import org.labkey.api.collections.CaseInsensitiveHashMap;
30+
import org.labkey.api.collections.IntHashMap;
3031
import org.labkey.api.data.CompareType;
3132
import org.labkey.api.data.Container;
3233
import org.labkey.api.data.ConvertHelper;
@@ -573,7 +574,7 @@ public void doGenerateTemplate(JSONObject json, HttpServletRequest request, Http
573574
response.setHeader("Pragma", "private");
574575
response.setHeader("Cache-Control", "private");
575576

576-
Map<Integer, String[]> rowMap = new HashMap<>();
577+
Map<Integer, String[]> rowMap = new IntHashMap<>();
577578

578579
int rowIdx = 0;
579580
for (JSONObject row : results)

‎ehr/src/org/labkey/ehr/EHRManager.java‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@
2424
import org.labkey.api.cache.CacheManager;
2525
import org.labkey.api.collections.CaseInsensitiveHashMap;
2626
import org.labkey.api.collections.CaseInsensitiveHashSet;
27+
import org.labkey.api.collections.IntHashMap;
2728
import org.labkey.api.data.CompareType;
2829
import org.labkey.api.data.Container;
2930
import org.labkey.api.data.ContainerManager;
@@ -1338,7 +1339,7 @@ public boolean canDiscardTask(Container c, User u, String taskId, List<String> e
13381339
{
13391340
DataEntryForm def = getDataEntryFormForTask(c, u, taskId);
13401341

1341-
Map<Integer, EHRQCState> qcStateMap = new HashMap<>();
1342+
Map<Integer, EHRQCState> qcStateMap = new IntHashMap<>();
13421343
for (EHRQCState qc : EHRManager.get().getQCStates(c))
13431344
{
13441345
qcStateMap.put(qc.getRowId(), qc);

‎ehr/src/org/labkey/ehr/utils/TriggerScriptHelper.java‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
import org.json.JSONArray;
2727
import org.labkey.api.collections.CaseInsensitiveHashMap;
2828
import org.labkey.api.collections.CaseInsensitiveHashSet;
29+
import org.labkey.api.collections.IntHashMap;
2930
import org.labkey.api.data.Aggregate;
3031
import org.labkey.api.data.ColumnInfo;
3132
import org.labkey.api.data.CompareType;
@@ -122,7 +123,7 @@ public class TriggerScriptHelper
122123
protected final static SimpleDateFormat _dateTimeFormat = new SimpleDateFormat("yyyy-MM-dd kk:mm");
123124

124125
//NOTE: consider moving these to SharedCache, to allow them to be shared across scripts, yet reset from admin console
125-
private final Map<Integer, String> _cachedAccounts = new HashMap<>();
126+
private final Map<Integer, String> _cachedAccounts = new IntHashMap<>();
126127
/**
127128
* Options that can be set in modules using EHR trigger scripts to opt in/out of or alter the behavior
128129
* of certain validations and business logic. If there are specific aspects of the core EHR trigger code that

‎ehr_billing/src/org/labkey/ehr_billing/query/EHRBillingTriggerHelper.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@
1515
*/
1616
package org.labkey.ehr_billing.query;
1717

18+
import org.labkey.api.collections.IntHashMap;
1819
import org.labkey.api.data.CompareType;
1920
import org.labkey.api.data.Container;
2021
import org.labkey.api.data.ContainerManager;
@@ -32,7 +33,6 @@
3233
import org.labkey.ehr_billing.EHR_BillingManager;
3334
import org.labkey.ehr_billing.EHR_BillingSchema;
3435

35-
import java.util.HashMap;
3636
import java.util.Map;
3737

3838
/**
@@ -43,7 +43,7 @@ public class EHRBillingTriggerHelper
4343
{
4444
private final Container _container;
4545
private final User _user;
46-
private final Map<Integer, Map<String, Object>> _cachedCharges = new HashMap<>();
46+
private final Map<Integer, Map<String, Object>> _cachedCharges = new IntHashMap<>();
4747

4848
public EHRBillingTriggerHelper(int userId, String containerId)
4949
{

‎snd/api-src/org/labkey/api/snd/Package.java‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
import org.json.JSONArray;
2121
import org.json.JSONObject;
2222
import org.labkey.api.collections.ArrayListMap;
23+
import org.labkey.api.collections.IntHashMap;
2324
import org.labkey.api.data.Container;
2425
import org.labkey.api.gwt.client.model.GWTPropertyDescriptor;
2526
import org.labkey.api.security.User;
@@ -49,7 +50,7 @@ public class Package
4950
private boolean _hasEvent;
5051
private boolean _hasProject;
5152
private Date _modified; // column will be used as a timestamp
52-
private Map<Integer, String> _categories = new HashMap<>();
53+
private Map<Integer, String> _categories = new IntHashMap<>();
5354
private List<GWTPropertyDescriptor> _attributes = new ArrayList<>();
5455
private List<SuperPackage> _subpackages = new ArrayList<>();
5556
private Map<GWTPropertyDescriptor, Object> _extraFields = new HashMap<>();

‎snd/src/org/labkey/snd/PackageUserSchema.java‎

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import org.jetbrains.annotations.Nullable;
77
import org.labkey.api.collections.CaseInsensitiveHashSet;
88
import org.labkey.api.collections.CaseInsensitiveTreeSet;
9+
import org.labkey.api.collections.IntHashMap;
910
import org.labkey.api.data.AbstractTableInfo;
1011
import org.labkey.api.data.BaseColumnInfo;
1112
import org.labkey.api.data.ColumnInfo;
@@ -92,7 +93,7 @@ public Set<String> getTableNames()
9293
PackageTableInfo createPackageTable(String name)
9394
{
9495
// find domain for name
95-
Map<Integer,Package> pkgs = new HashMap<>();
96+
Map<Integer,Package> pkgs = new IntHashMap<>();
9697
visitAll(p -> {
9798
if (name.equalsIgnoreCase(p.description))
9899
pkgs.put(p.packageId, p);
@@ -291,7 +292,7 @@ void initPackages()
291292
List<SuperPkg> supers = new SqlSelector(getDbSchema(), new SQLFragment(
292293
new SQLFragment("SELECT SuperPkgId, ParentSuperPkgId, Pkgs.PkgId, Description FROM snd.Pkgs INNER JOIN snd.SuperPkgs ON Pkgs.PkgId = SuperPkgs.PkgId WHERE Pkgs.Container = ").appendValue(getContainer())
293294
)).getArrayList(SuperPkg.class);
294-
Map<Integer, Package> map = new HashMap<>();
295+
Map<Integer, Package> map = new IntHashMap<>();
295296
supers.forEach(superPkg -> {
296297
var package_ = map.computeIfAbsent(superPkg.pkgId, id ->
297298
{

‎snd/src/org/labkey/snd/SNDController.java‎

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,7 @@
2727
import org.labkey.api.action.SimpleRedirectAction;
2828
import org.labkey.api.action.SimpleViewAction;
2929
import org.labkey.api.action.SpringActionController;
30+
import org.labkey.api.collections.IntHashMap;
3031
import org.labkey.api.gwt.client.DefaultValueType;
3132
import org.labkey.api.gwt.client.model.GWTPropertyDescriptor;
3233
import org.labkey.api.module.Module;
@@ -188,7 +189,7 @@ public ApiResponse execute(SimpleApiJsonForm form, BindException errors)
188189
JSONArray jsonCategories = json.getJSONArray("categories");
189190
if (null != jsonCategories)
190191
{
191-
Map<Integer, String> categories = new HashMap<>();
192+
Map<Integer, String> categories = new IntHashMap<>();
192193
for (int j = 0; j < jsonCategories.length(); j++)
193194
{
194195
categories.put(jsonCategories.getInt(j), null);
@@ -227,7 +228,7 @@ public ApiResponse execute(SimpleApiJsonForm form, BindException errors)
227228
{
228229
// Get super packages
229230
JSONArray jsonSubPackages = json.getJSONArray("subPackages"); // only first-level children (as super package IDs) should be here
230-
Map<Integer, LinkedList<SuperPackageInfo>> superPkgIdToExtraInfoMap = new HashMap<>(); // uses lists because top-level super package IDs might show up multiple times
231+
Map<Integer, LinkedList<SuperPackageInfo>> superPkgIdToExtraInfoMap = new IntHashMap<>(); // uses lists because top-level super package IDs might show up multiple times
231232
List<Integer> uiSubSuperPkgIds = new ArrayList<>();
232233

233234
// create super package for root, if needed
@@ -301,7 +302,7 @@ public ApiResponse execute(SimpleApiJsonForm form, BindException errors)
301302
{
302303
topLevelSuperPkgs = SNDManager.convertToTopLevelSuperPkgs(getContainer(), getUser(), uiSubSuperPkgIds);
303304
List<SuperPackage> uiSuperPkgs = SNDManager.getSuperPkgs(getContainer(), getUser(), uiSubSuperPkgIds);
304-
Map<Integer, Integer> superPkgIdToPkgIdMap = new HashMap<>();
305+
Map<Integer, Integer> superPkgIdToPkgIdMap = new IntHashMap<>();
305306

306307
// need to get proper package IDs from db since they're not coming in from UI
307308
if (uiSuperPkgs != null)

‎snd/src/org/labkey/snd/SNDManager.java‎

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@
3131
import org.labkey.api.cache.CacheManager;
3232
import org.labkey.api.collections.ArrayListMap;
3333
import org.labkey.api.collections.CaseInsensitiveHashMap;
34+
import org.labkey.api.collections.IntHashMap;
3435
import org.labkey.api.data.CompareType;
3536
import org.labkey.api.data.Container;
3637
import org.labkey.api.data.DbScope;
@@ -77,6 +78,7 @@
7778
import org.labkey.api.snd.EventNote;
7879
import org.labkey.api.snd.Package;
7980
import org.labkey.api.snd.PackageDomainKind;
81+
import org.labkey.api.snd.PlainTextNarrativeDisplayColumn;
8082
import org.labkey.api.snd.Project;
8183
import org.labkey.api.snd.ProjectItem;
8284
import org.labkey.api.snd.SNDDomainKind;
@@ -87,7 +89,6 @@
8789
import org.labkey.snd.query.PackagesTable;
8890
import org.labkey.snd.security.QCStateActionEnum;
8991
import org.labkey.snd.security.SNDSecurityManager;
90-
import org.labkey.api.snd.PlainTextNarrativeDisplayColumn;
9192
import org.labkey.snd.trigger.SNDTriggerManager;
9293

9394
import java.sql.SQLException;
@@ -552,7 +553,7 @@ private Map<Integer, String> getPackageCategories(Container c, User u, int pkgId
552553
sql.append(" WHERE PkgId = ?").add(pkgId);
553554
SqlSelector selector = new SqlSelector(schema.getDbSchema(), sql);
554555

555-
Map<Integer, String> categories = new HashMap<>();
556+
Map<Integer, String> categories = new IntHashMap<>();
556557
try (TableResultSet rs = selector.getResultSet())
557558
{
558559
for (Map<String, Object> r : rs)
@@ -592,7 +593,7 @@ private Map<Integer, Map<Integer, String>> getBulkPackageCategories(Container c,
592593
}
593594
SqlSelector selector = new SqlSelector(schema.getDbSchema(), sql);
594595

595-
Map<Integer, Map<Integer, String>> pkgCategoriesByPkgId = new HashMap<>();
596+
Map<Integer, Map<Integer, String>> pkgCategoriesByPkgId = new IntHashMap<>();
596597

597598
try (TableResultSet resultSet = selector.getResultSet())
598599
{
@@ -3364,7 +3365,7 @@ private String generateEventDataNarrative(Container c, User u, Event event, Even
33643365
{
33653366
List<GWTPropertyDescriptor> properties = superPackage.getPkg().getAttributes();
33663367
Map<String, GWTPropertyDescriptor> propsByName = new HashMap<>();
3367-
Map<Integer, GWTPropertyDescriptor> propsById = new HashMap<>();
3368+
Map<Integer, GWTPropertyDescriptor> propsById = new IntHashMap<>();
33683369
for (GWTPropertyDescriptor p : properties)
33693370
{
33703371
propsByName.put(p.getName(), p);
@@ -4210,7 +4211,7 @@ private Map<Integer, Map<Integer, Pair<SuperPackage, Integer>>> getNextLevelEven
42104211
));
42114212

42124213
// Prepare a map for the next level of SuperPackages
4213-
Map<Integer, Map<Integer, Pair<SuperPackage, Integer>>> nextLevelEventDataSuperPkgs = new HashMap<>();
4214+
Map<Integer, Map<Integer, Pair<SuperPackage, Integer>>> nextLevelEventDataSuperPkgs = new IntHashMap<>();
42144215
nextLevelEventDataSuperPkgs.put(eventData.getEventId(), new HashMap<>());
42154216

42164217
// Iterate over child event data and link it to the corresponding child SuperPackage

‎snd/src/org/labkey/snd/pipeline/SNDDataHandler.java‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -347,7 +347,7 @@ public void deleteData(ExpData data, Container container, User user)
347347
}
348348

349349
@Override
350-
public void runMoved(ExpData newData, Container container, Container targetContainer, String oldRunLSID, String newRunLSID, User user, int oldDataRowID)
350+
public void runMoved(ExpData newData, Container container, Container targetContainer, String oldRunLSID, String newRunLSID, User user, long oldDataRowID)
351351
{
352352

353353
}

‎snd/src/org/labkey/snd/security/SNDSecurityManager.java‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717

1818

1919
import org.labkey.api.collections.CaseInsensitiveHashMap;
20+
import org.labkey.api.collections.IntHashMap;
2021
import org.labkey.api.data.CompareType;
2122
import org.labkey.api.data.Container;
2223
import org.labkey.api.data.CoreSchema;
@@ -50,7 +51,6 @@
5051
import java.util.ArrayList;
5152
import java.util.Collections;
5253
import java.util.EnumSet;
53-
import java.util.HashMap;
5454
import java.util.List;
5555
import java.util.Map;
5656
import java.util.Set;
@@ -127,7 +127,7 @@ public void updatePermissions(Container c, User u, Map props)
127127

128128
SecurityManager.getGroups(c.getProject(), true);
129129

130-
Map<Integer, MutableSecurityPolicy> policyMap = new HashMap<>(); // Policy cache
130+
Map<Integer, MutableSecurityPolicy> policyMap = new IntHashMap<>(); // Policy cache
131131
MutableSecurityPolicy policy;
132132
Group group;
133133
Role role;

0 commit comments

Comments
 (0)