Skip to content

Commit 0a9b30e

Browse files
committed
move file count limit from dvObject to dvObjectContainer
1 parent e972f80 commit 0a9b30e

File tree

3 files changed

+14
-17
lines changed

3 files changed

+14
-17
lines changed

src/main/java/edu/harvard/iq/dataverse/DvObject.java

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -140,9 +140,6 @@ public String visit(DataFile df) {
140140

141141
@Column(insertable = false, updatable = false) private String dtype;
142142

143-
@Column( nullable = true )
144-
private Integer datasetFileCountLimit;
145-
146143
@OneToMany(mappedBy="dvobject",fetch = FetchType.LAZY,cascade={CascadeType.REMOVE, CascadeType.MERGE, CascadeType.PERSIST, CascadeType.REFRESH})
147144
private List<DataverseFeaturedItem> dataverseFeaturedItems;
148145

@@ -509,15 +506,6 @@ public StorageQuota getStorageQuota() {
509506
public void setStorageQuota(StorageQuota storageQuota) {
510507
this.storageQuota = storageQuota;
511508
}
512-
513-
public Integer getDatasetFileCountLimit() {
514-
return datasetFileCountLimit;
515-
}
516-
public void setDatasetFileCountLimit(Integer datasetFileCountLimit) {
517-
// Store as -1 if missing or invalid
518-
this.datasetFileCountLimit = datasetFileCountLimit != null && datasetFileCountLimit <= 0 ? Integer.valueOf(-1) : datasetFileCountLimit;
519-
}
520-
521509
/**
522510
*
523511
* @param other

src/main/java/edu/harvard/iq/dataverse/DvObjectContainer.java

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,11 +9,9 @@
99
import edu.harvard.iq.dataverse.util.json.JsonUtil;
1010
import jakarta.json.JsonObject;
1111
import jakarta.json.JsonObjectBuilder;
12-
import jakarta.persistence.CascadeType;
12+
import jakarta.persistence.*;
13+
1314
import java.util.Optional;
14-
import jakarta.persistence.MappedSuperclass;
15-
import jakarta.persistence.OneToOne;
16-
import jakarta.persistence.Transient;
1715

1816
import org.apache.commons.lang3.StringUtils;
1917

@@ -56,6 +54,9 @@ public boolean isEffectivelyPermissionRoot() {
5654

5755
@OneToOne(mappedBy = "dvObjectContainer",cascade={ CascadeType.REMOVE, CascadeType.PERSIST}, orphanRemoval=true)
5856
private StorageUse storageUse;
57+
58+
@Column( nullable = true )
59+
private Integer datasetFileCountLimit;
5960

6061
public String getEffectiveStorageDriverId() {
6162
String id = storageDriver;
@@ -260,6 +261,13 @@ public PidProvider getEffectivePidGenerator() {
260261
}
261262
return pidGenerator;
262263
}
264+
public Integer getDatasetFileCountLimit() {
265+
return datasetFileCountLimit;
266+
}
267+
public void setDatasetFileCountLimit(Integer datasetFileCountLimit) {
268+
// Store as -1 if missing or invalid
269+
this.datasetFileCountLimit = datasetFileCountLimit != null && datasetFileCountLimit <= 0 ? Integer.valueOf(-1) : datasetFileCountLimit;
270+
}
263271

264272
public Integer getEffectiveDatasetFileCountLimit() {
265273
if (isDatasetFileCountLimitNotSet(getDatasetFileCountLimit()) && getOwner() != null) {
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1,2 @@
1-
ALTER TABLE DVOBJECT ADD COLUMN IF NOT EXISTS datasetfilecountlimit bigint;
1+
ALTER TABLE DATASET ADD COLUMN IF NOT EXISTS datasetfilecountlimit bigint;
2+
ALTER TABLE DATAVERSE ADD COLUMN IF NOT EXISTS datasetfilecountlimit bigint;

0 commit comments

Comments
 (0)